事象
PR ゲート(test-pull-request.yml)も main のビルド(build-main.yml)も ./gradlew build のみで、JS テスト(npm run test、jig-core/src/test/js に16ファイル)、npm run lint:css、Playwright テストを実行するステップがない。
該当箇所
main (a3ad929) 時点: .github/workflows/test-pull-request.yml:33、.github/workflows/build-main.yml:30
影響
テンプレートの主要ロジックは JS 側にあるため、JS のリグレッションが壊れた PR がグリーンのままマージされる。CLAUDE.md が「*.js のみの変更は npm run test」というポリシーを定めているのに、CI にその安全網がない。
修正案
PR ワークフローに Node のセットアップ + npm ci + npm run test + npm run lint:css を追加する。Playwright はコストと相談のうえ、少なくともスナップショット差分が出る変更の検知手段を検討する。
🤖 この issue は Claude Code のコードレビューにより作成されました。
事象
PR ゲート(test-pull-request.yml)も main のビルド(build-main.yml)も
./gradlew buildのみで、JS テスト(npm run test、jig-core/src/test/js に16ファイル)、npm run lint:css、Playwright テストを実行するステップがない。該当箇所
main (a3ad929) 時点:
.github/workflows/test-pull-request.yml:33、.github/workflows/build-main.yml:30影響
テンプレートの主要ロジックは JS 側にあるため、JS のリグレッションが壊れた PR がグリーンのままマージされる。CLAUDE.md が「*.js のみの変更は
npm run test」というポリシーを定めているのに、CI にその安全網がない。修正案
PR ワークフローに Node のセットアップ +
npm ci+npm run test+npm run lint:cssを追加する。Playwright はコストと相談のうえ、少なくともスナップショット差分が出る変更の検知手段を検討する。🤖 この issue は Claude Code のコードレビューにより作成されました。