chore: パッケージマネージャを npm から pnpm へ移行#34
Conversation
- packageManager: pnpm@11.8.0 を固定し、preinstall: only-allow pnpm で強制(既存)。 - package-lock.json を削除し pnpm-lock.yaml を生成。 - CI/self-test: pnpm/action-setup(v6.0.9・SHA ピン)を setup-node より前に置き、cache: pnpm / pnpm install --frozen-lockfile / pnpm run ... に変更。 - pnpm-workspace.yaml に minimumReleaseAge: 4320(3日) を設定(供給網 cooldown)。 これに伴い semver 7.8.5->7.8.4 等、公開3日以上の版に解決し dist を再ビルド。 CI の --frozen-lockfile はこのポリシーも毎回検証する。 - ドキュメント(CLAUDE/CONTRIBUTING/README/PR テンプレ/TODO)を pnpm に更新。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
No issues found across 13 files
Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.
Re-trigger cubic
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (8)
📒 Files selected for processing (5)
📝 WalkthroughSummary by CodeRabbitリリースノート
WalkthroughREADME、CONTRIBUTING.md、CLAUDE.md、PR テンプレート、TODO.md の全ドキュメントで Changesnpm → pnpm ドキュメント更新
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
概要
パッケージマネージャを npm → pnpm へ移行します。
packageManagerフィールドで版を固定し、CI と一致させます(先日の npm 版ズレ事故の再発防止)。変更
package.json:packageManager: pnpm@11.8.0固定、preinstall: only-allow pnpmで pnpm 強制、allスクリプトを pnpm に。package-lock.json削除 →pnpm-lock.yaml生成。pnpm-workspace.yaml(新規):minimumReleaseAge: 4320(3日)。公開3日未満の依存版を使わない供給網 cooldown(SHA ピン・checksum 検証と同じ硬化方針)。CI の--frozen-lockfileは毎回このポリシーも検証する。pnpm/action-setup@v6.0.9(SHA ピン)をsetup-nodeより前に配置、cache: pnpm、pnpm install --frozen-lockfile、pnpm run ...。dist/再ビルド: ポリシーによりsemverが 7.8.5→7.8.4 等に解決され、ncc 再バンドル。CI も同 lockfile + pnpm 11.8.0 で再ビルドし鮮度チェックは一致。テスト
pnpm install --frozen-lockfile(ポリシー検証込み)OKpnpm run allgreen(47 tests)🤖 Generated with Claude Code
Summary by cubic
Migrate package management from npm to pnpm across the repo and CI for consistent installs and supply‑chain hardening. Pin pnpm to 11.8.0, add a 3‑day
minimumReleaseAge, replace the lockfile, and rebuilddist/.Refactors
packageManager: pnpm@11.8.0andpreinstall: only-allow pnpm; replacepackage-lock.jsonwithpnpm-lock.yaml.pnpm-workspace.yamlwithminimumReleaseAge: 4320(3 days).pnpm/action-setup(SHA-pinned) beforeactions/setup-node, enablecache: pnpm, runpnpm install --frozen-lockfile, andpnpm run ....dist/to match new resolutions (e.g.,semver7.8.4); update docs/templates to pnpm.Migration
corepack enableto use the pinned pnpm.pnpm run alllocally; CI should runpnpm install --frozen-lockfile.Written for commit 5fdf3d3. Summary will update on new commits.