Skip to content

fix(deps): bump @actions/core to 2.0.3 and @actions/github to 8.0.1, rebuild dist - #104

Merged
enomoto-kazuya merged 2 commits into
mainfrom
fix/update-actions-deps-and-rebuild-dist
Sep 28, 2026
Merged

enomoto-kazuya merged 2 commits into
mainfrom
fix/update-actions-deps-and-rebuild-dist

Conversation

@enomoto-kazuya

Copy link
Copy Markdown
Contributor

概要

以下の2点に対応します。

  1. dist/index.js が古いままになっている問題
    chore(deps): bump the other-dependencies group across 1 directory with 2 updates #103 で undici を 7.29.1 に上げましたが、Action の実行時に使われる dist/index.js は再ビルドされていませんでした。このため、利用者の環境では修正前の undici 7.29.0 が動いていました。
  2. @actions/core / @actions/github の中に古い undici が残っている問題
    2つのパッケージが依存する @actions/http-client@2 が undici 5.29.0 を使っていました。npm audit --omit=dev では High 1件、Moderate 2件が報告されていました。

変更内容

パッケージ 変更前 変更後 中で使われる undici
@actions/core 1.11.1 2.0.3 5.29.0 → 6.29.0
@actions/github ^6.0.1 ^8.0.1 5.29.0 → 6.29.0
dist/index.js — npm run build で再生成 —

@actions/github@9 と @actions/core@3 は ESM 専用です。使うにはプロジェクト全体を ESM に移す必要があるため、今回は見送りました。CommonJS のまま使える中で最新の 8.0.1 と 2.0.3 を選んでいます。src/main.ts の変更は必要ありませんでした。

確認したこと

  • npm audit --omit=dev:0件(変更前は High 1件、Moderate 2件)
  • npm audit signatures:すべて検証に成功
  • tsc --noEmit:成功
  • npm run build 後、dist/index.js が Node 20 / 25 で読み込みエラーなく起動する
  • PR Test(pr-test.yml)で DeployGate へのアップロードと PR へのコメントが成功する

対象外

開発用の依存(brace-expansion、js-yaml、minimatch など)に対する npm audit の指摘は、このPRでは扱いません。

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

DeployGate Upload Information

Item Content
🔄 Revision 131
📱 App Details View on DeployGate
🔗 Distribution Page https://deploygate.com/distributions/4e531da5dfd007964eb0340bba094e8fbfbe89f2
📲 Open on Mobile QR Code

@enomoto-kazuya
enomoto-kazuya marked this pull request as ready for review September 28, 2026 03:22
Copilot AI lite review requested due to automatic review settings September 28, 2026 03:22
@enomoto-kazuya
enomoto-kazuya requested review from satsukies and removed request for satsukies September 28, 2026 03:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Resolve the Node.js 20 compatibility issue before approval.

Review effort: Lite
Findings: None

What changed in this PR

Updates GitHub Action dependencies to address vulnerable undici versions and rebuilds the distribution bundle.

Changes:

  • Upgrades @actions/core and @actions/github.
  • Refreshes transitive dependencies and the lockfile.
  • Rebuilds dist/index.js.
File Summary
package.json Updates runtime dependencies.
package-lock.json Contains content-type@3.1.1, which requires Node.js 22+ while the action runs on Node.js 20.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings September 28, 2026 03:32
@enomoto-kazuya

Copy link
Copy Markdown
Contributor Author

Copilot review overview

🟡 Changes recommended

Resolve the Node.js 20 compatibility issue before approval.

Review effort: Lite Findings: None

What changed in this PR
Updates GitHub Action dependencies to address vulnerable undici versions and rebuilds the distribution bundle.

Changes:

  • Upgrades @actions/core and @actions/github.
  • Refreshes transitive dependencies and the lockfile.
  • Rebuilds dist/index.js.

File Summary
package.json Updates runtime dependencies.
package-lock.json Contains content-type@3.1.1, which requires Node.js 22+ while the action runs on Node.js 20.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

content-type@3.1.1 の engines: node >=22 は、ESM専用パッケージであることによる指定と考えられます。コードにNode 22でしか使えないAPIは見当たりませんでした。このActionでは ncc で dist/index.js 1ファイルにまとめており、Node 20.0.0 / 20.20.2 で読み込めること、CI(Node 20)で実際のアップロードとPRへのコメントが成功していることを確認しました。そのため、このPRでは対応しません。node24 への移行は別のPRで検討します。

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The dependency tree includes a Node-22-only package while the action targets Node 20.

Review effort: Lite
Findings: None

@satsukies satsukies left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@enomoto-kazuya
enomoto-kazuya merged commit 6a4ee29 into main Sep 28, 2026
4 checks passed
@enomoto-kazuya
enomoto-kazuya deleted the fix/update-actions-deps-and-rebuild-dist branch September 28, 2026 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants