An order to implement preventive measures for the violation in cycle 17

各サイクルの終わりには、すべてのテスト項目を完全に成功させる必要があります。 「環境起因」「今回の変更とは無関係」「既知の問題」「次サイクルで対応」などの例外は一切認められません。 なぜから、テストを通過しないとデプロイができないからです。

再発防止策として、Claude Code Hookに設定してある pre-commit-check の仕組みを応用して、 pre-push-check を作成してください。 このチェックでは、 lint, format:check, test, build がすべて通過することを確認するようにしてください。 一つでも失敗するものがあれば、「Fix all issues and push again」というメッセージを表示してエラー終了するようにしてください。 参考: Hooks reference - Claude Code Docs

なお、git hooksへの追加は不要です。 gitのhookにも入れてしまうと、ownerがAIへの指示を出すときの支障になる恐れがあるからです。

概要

サイクル17でCI失敗を「環境起因」として例外扱いした問題の再発防止策として、Claude Code Hook pre-push-check を作成してください。

要件

1. .claude/hooks/pre-push-check.sh を新規作成する
2. 既存の .claude/hooks/pre-commit-check.sh のパターンを踏襲する
3. git push コマンド（git push を含むBashコマンド）を検知する
4. 以下の4つのチェックをすべて実行する:
   • npm run lint
   • npm run format:check
   • npm test
   • npm run build
5. 1つでも失敗したら「Fix all issues and push again」というメッセージを表示してエラー終了（exit 2）する
6. .claude/settings.json の hooks.PreToolUse 配列に、Bash matcher で pre-push-check.sh を実行する設定を追加する
7. git hooks への追加は 不要（ownerの指示）

参考ファイル

• .claude/hooks/pre-commit-check.sh - 既存のhookスクリプト（パターン参考）
• .claude/settings.json - hook設定ファイル
• ownerメモ 19c7e13d377 - 元の指示

完了条件

• pre-push-check.sh が作成され、実行権限が付与されている
• settings.json に hook 設定が追加されている
• git push コマンドを検知して4つのチェックが実行される
• すべてのチェックが通過した場合はexit 0で正常終了する
• 1つでも失敗した場合は「Fix all issues and push again」を表示してexit 2で終了する
• 既存の pre-commit-check のhook設定が壊れていないこと

注意

• 作業完了後、コミットは行わずに変更内容を報告してください

完了報告

B-060: pre-push-check フックの作成が完了しました。コミットは行っていません。

変更ファイル

1. .claude/hooks/pre-push-check.sh (新規作成)

   • git push を含む Bash コマンドを検知
   • 4つのチェック（lint, format:check, test, build）をすべて実行
   • 1つでも失敗した場合は全エラーを表示した上で「Fix all issues and push again」メッセージを出力し exit 2
   • すべて通過した場合は exit 0
   • 実行権限 (+x) 付与済み

2. .claude/settings.json (編集)

   • hooks.PreToolUse 配列に Bash matcher で pre-push-check.sh を実行するエントリを追加
   • 既存の pre-commit-check.sh の設定はそのまま保持

pre-commit-check との設計差異

pre-commit-check は各チェック失敗時に即座に exit 2 で終了する設計ですが、pre-push-check では全4チェックを最後まで実行してから結果をまとめて判定する設計にしました。push 前の最終チェックとして、すべての問題を一度に把握できるようにするためです。

完了条件の充足状況

• pre-push-check.sh が作成され、実行権限が付与されている
• settings.json に hook 設定が追加されている
• git push コマンドを検知して4つのチェックが実行される
• すべてのチェックが通過した場合は exit 0 で正常終了する
• 1つでも失敗した場合は「Fix all issues and push again」を表示して exit 2 で終了する
• 既存の pre-commit-check の hook 設定が壊れていないこと

サイクル18 完了報告

完了タスク

• B-060: Claude Code Hook pre-push-check の作成 — ownerメモ 19c7e13d377 への対応

成果物

ファイル	操作	内容
.claude/hooks/pre-push-check.sh	新規作成	git push検知時にlint, format:check, test, buildを全実行するスクリプト
.claude/settings.json	編集	PreToolUse hookにpre-push-check.shのエントリ追加
docs/backlog.md	編集	B-060をDoneに移動
docs/cycles/cycle-18.md	新規作成	サイクルドキュメント

設計判断

• pre-commit-check.shは1件失敗で即終了するが、pre-push-check.shは4チェックすべて実行してから結果をまとめて判定する方式を採用。pushはcommitより頻度が低く、全問題を一度に把握できる方が効率的なため。
• git hooksへの追加はownerの指示通り行っていない。

テスト結果

• lint: 通過（hermes-parser問題はnode_modules再インストールで解消）
• format:check: 通過
• test: 109ファイル / 1233テスト 全通過
• build: 1570ページ生成 成功

レビュー結果

• レビュー1回で承認（メモ 19c7e1ec66c）
• 軽微な指摘1件（grep誤検知可能性）→ pre-commit-checkと同パターンのため対応不要と判断

キャリーオーバー

なし