code-review

コードレビュー

見せかけの返答ではなく、技術的な厳密さ、証拠に基づく主張、検証を重視した適切なコードレビューの実践方法を紹介します。

概要

コードレビューでは、次の3つの異なる実践が必要です。

1. フィードバックを受ける - 見せかけの同意ではなく、技術的な評価を行います
2. レビューを依頼する - 体系的なレビューのプロセスを実施します
3. 検証ゲート - 完了を主張する前に、証拠を確認します

基本原則

社会的な気まずさより、技術的な正確さを優先します。 実装する前に検証します。決めつける前に確認します。主張する前に証拠を確認します。

使用するタイミング

フィードバックを受ける

• どのような相手からであっても、コードレビューのコメントを受け取ったとき
• フィードバックが不明瞭、または技術的に疑わしいとき
• 複数のレビュー項目の優先順位付けが必要なとき
• 外部のレビュアーが十分な背景情報を持っていないとき
• 提案が既存の決定と矛盾するとき

レビューを依頼する

• サブエージェント駆動型開発でタスクを完了したとき（各タスクの完了後）
• 主要な機能やリファクタリングを終えたとき
• main ブランチにマージする前
• 行き詰まり、新たな視点が必要なとき
• 複雑なバグを修正した後

検証ゲート

• テストの合格、ビルドの成功、作業の完了を主張しようとしているとき
• コミット、プッシュ、PR の作成前
• 次のタスクに進むとき
• 成功や完了を示唆する発言をするとき

クイック判断ツリー

SITUATION?
│
├─ Received feedback
│  ├─ Unclear items? → STOP, ask for clarification first
│  ├─ From human partner? → Understand, then implement
│  └─ From external reviewer? → Verify technically before implementing
│
├─ Completed work
│  ├─ Major feature/task? → Request systematic review
│  └─ Before merge? → Request systematic review
│
└─ About to claim status
   ├─ Have fresh verification? → State claim WITH evidence
   └─ No fresh verification? → RUN verification command first

CI 検証

完了を主張したり、コミットしたりする前に、次のことを行います。

• CI チェック（型、テスト、リンター）を実行します
• 利用可能であれば、単一の CI コマンドを優先します
• すべてのチェックが合格したことを確認します
• チェックが失敗した場合は先に進みません

参考資料

詳細な手順については、以下を参照してください。

• references/receiving-feedback.md - コードレビューのフィードバックへの対応方法
• references/requesting-review.md - 体系的なレビューのプロセス
• references/verification-gates.md - 主張する前に証拠を確認する手順