Repository navigation
chore: pnpmとdevEnginesへ移行 - #106
Merged
Merged
Conversation
Co-authored-by: Codex <noreply@openai.com>
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟡 Changes recommended
4つのCIワークフローでルート依存関係をインストールしておらず、valibot・Biome・TypeScript・esbuildなどを解決できないため、synth、check、deploy、Lambdaビルドが失敗します。
4 open issues
CDK synth 前にルート依存関係が未インストール · New 検査・テスト前にルート依存関係が未インストール · New CDK deploy 前にルート依存関係が未インストール · New Lambda ビルド前にルート依存関係が未インストール · New
3 files not reviewed
infra/package-lock.json: Generated fileinfra/pnpm-lock.yaml: Generated filepnpm-lock.yaml: Generated file
What changed in this PR
pnpm と devEngines を中心に、依存関係管理、Node.js バージョン、Lambda、CDK、CI/CD、Renovate の設定を統一する移行です。
Changes:
- root と
infraの npm lockfile を pnpm lockfile へ移行し、パッケージマネージャー設定をdevEnginesに統一 - Node.js 24.x の定義を Lambda のビルド、CDK、RIE テストで共有
- GitHub Actions の依存関係セットアップと Renovate 設定を pnpm/JSON5 対応へ更新
| File | Description |
|---|---|
| tsconfig.json | JSON モジュールの読み込みを有効化しました。 |
| scripts/node-version.ts | devEngines.runtime から Node.js のメジャーバージョンを取得する処理を追加しました。 |
| scripts/build-lambda.ts | 共有された Node.js バージョンを使って Lambda をバンドルする処理に変更しました。 |
| pnpm-workspace.yaml | pnpm のリリース待機、完全固定、依存パッケージのビルド許可を設定しました。 |
| pnpm-lock.yaml | root プロジェクトの依存関係を pnpm lockfile で固定しました。 |
| package.json | Lambda ビルド用スクリプトと pnpm/Node.js の devEngines 設定を追加しました。 |
| infra/pnpm-workspace.yaml | infra 向けの pnpm 設定を追加しました。 |
| infra/pnpm-lock.yaml | infra プロジェクトの依存関係を pnpm lockfile で固定しました。 |
| infra/package.json | infra のパッケージマネージャーを devEngines で指定しました。 |
| infra/package-lock.json | 旧 npm lockfile を削除しました。 |
| infra/lib/times-all-bot-stack.ts | 共有された Node.js 定義から Lambda ランタイムを決定し、pnpm lockfile を参照するよう変更しました。 |
| .npmrc | 旧 npm 設定を削除しました。 |
| .github/workflows/_reusable_lambda_rie_test.yml | pnpm セットアップと共有 Node.js バージョンを使用する Lambda RIE テストへ変更しました。 |
| .github/workflows/_reusable_deploy.yml | pnpm ベースの依存関係セットアップと CDK デプロイへ変更しました。 |
| .github/workflows/_reusable_check.yml | 静的チェック、テスト、型チェックを pnpm ベースで実行するよう変更しました。 |
| .github/workflows/_reusable_cdk_synth.yml | CDK synth を pnpm ベースで実行するよう変更しました。 |
| .github/renovate.json5 | Renovate 設定を JSON5 へ移行し、Node.js 24.x の範囲を維持する更新ルールを追加しました。 |
| .github/renovate.json | 旧 JSON 形式の Renovate 設定を削除しました。 |
Review effort: Lite
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

issue
関連: #94(npmの
EALLOWREMOTEによる依存更新の停止)変更内容
devEngines.packageManagerへ統一devEngines.runtimeの24.x・downloadで管理し、Lambda・ビルド・実行テストも同じ定義を参照pnpm/setupへ統合24.xを完全固定へ変えない例外を追加確認したこと
aws-cdk-lib更新時のEALLOWREMOTEが、pnpmのlockfile更新では発生しない24.xを維持、次のメジャーでは26.xへ変更することを確認。pnpm・通常依存の完全固定とメジャー更新の自動マージ禁止も維持スクリーンショット
対象外(UI変更なし)
補足事項
GitHub Actionsの実行結果はPRのチェックを参照。本番デプロイは未実施。
PR時のセルフチェック