Skip to content

fix(ci): use corepack to install npm 11 for publish#103

Merged
miya0001 merged 1 commit into
mainfrom
fix/npm-upgrade-method
Apr 5, 2026
Merged

fix(ci): use corepack to install npm 11 for publish#103
miya0001 merged 1 commit into
mainfrom
fix/npm-upgrade-method

Conversation

@miya0001

@miya0001 miya0001 commented Apr 5, 2026

Copy link
Copy Markdown
Member

Summary

  • npm 11 のインストール方法を npm install -g npm@11 から corepack に変更
  • Node.js 22.22.2 にバンドルされた npm 10 は promise-retry モジュール欠損で npm install -g が動作しない
  • corepack enable npm && corepack install -g npm@11 で npm 10 をバイパスして直接インストール

Test plan

  • CI が通ることを確認
  • マージ後、v0.10.1 タグを作成して publish が成功することを確認

Summary by CodeRabbit

リリースノート

このプルリクエストには、エンドユーザーに影響を与える変更は含まれていません。内部ビルドプロセスの構成に関する技術的な更新です。

@coderabbitai

coderabbitai Bot commented Apr 5, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9fca620b-24e2-48e0-8662-aade24b5dceb

📥 Commits

Reviewing files that changed from the base of the PR and between e9be5e4 and 7f0725f.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml

📝 Walkthrough

Walkthrough

publish ジョブの npm ツールアップグレードステップが、直接グローバルインストール(npm install -g npm@11)から Corepack の有効化を経由したグローバルインストール(corepack enable npm && corepack install -g npm@11)に変更されました。

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/publish.yml
npm アップグレードステップが Corepack を使用した方式に変更。直接インストールから Corepack 経由のインストールへ移行し、OIDC Trusted Publishing 用の npm 11 プロビジョニング方法を更新。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • fix(ci): install npm 11 for OIDC Trusted Publishing #102: GitHub Actions の publish ワークフローの npm アップグレードステップを同様に修正し、OIDC Trusted Publishing 用に npm 11 を使用するよう確保(こちらの PR が直接インストール方式から Corepack 方式に切り替える一方で、関連 PR は直接インストールを追加)。

Poem

🐰 Corepack's gentle hop,
Npm tool takes flight,
Trusty wheels turn right,
Publishing dreams ascend,
Workflow refined at last! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルは、Corepackを使用してnpm 11をインストールするという主な変更を明確に要約しており、変更セット全体に直接関連しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/npm-upgrade-method

Comment @coderabbitai help to get the list of available commands and usage tips.

@miya0001
miya0001 merged commit 2c658f9 into main Apr 5, 2026
11 of 13 checks passed
@miya0001
miya0001 deleted the fix/npm-upgrade-method branch April 5, 2026 16:48
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.

1 participant