requesting-code-review

コードレビューを依頼する

コードレビュアーエージェントをディスパッチする（または利用可能であれば /review を実行する）ことで、問題が連鎖する前に検出します。レビュアーには、評価に必要な情報だけを正確にまとめて渡します。セッションの履歴は決して渡しません。これにより、レビュアーは思考過程ではなく成果物に集中でき、あなた自身のコンテキストもその後の作業のために保てます。

基本原則： 早めに、頻繁にレビューする。

レビューを依頼するタイミング

必須：

• サブエージェント駆動開発の各タスクの完了後
• 主要な機能の完了後
• main にマージする前

任意ですが有益：

• 行き詰まったとき（新たな視点を得る）
• リファクタリングの前（現状を確認する）
• 複雑なバグの修正後

依頼方法

1. git の SHA を取得する：

BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)

2. コードレビュアーをディスパッチする：

次の情報を添えてコードレビュアーエージェントをディスパッチします（または利用可能であれば /review を実行します）。

入力するプレースホルダー：

• {WHAT_WAS_IMPLEMENTED} - 今実装した内容
• {PLAN_OR_REQUIREMENTS} - 実装が満たすべき内容
• {BASE_SHA} - 開始時のコミット
• {HEAD_SHA} - 終了時のコミット
• {DESCRIPTION} - 簡潔な概要

レビュー依頼のテンプレート：

Please review the code changes between {BASE_SHA} and {HEAD_SHA}.

What was implemented: {WHAT_WAS_IMPLEMENTED}

Requirements / plan reference: {PLAN_OR_REQUIREMENTS}

Summary: {DESCRIPTION}

Focus on:
- Correctness vs. requirements
- Missing edge cases or error handling
- Test quality (are tests testing real behavior or mocks?)
- Security concerns
- Code clarity and maintainability

Use project's code review template if one exists (e.g., check for a REVIEW_TEMPLATE or similar in the project docs).

3. フィードバックに対応する：

• Critical の問題は直ちに修正する
• Important の問題は次に進む前に修正する
• Minor の問題は後で対応するものとして記録する
• レビュアーの指摘が誤っている場合は、理由を示して異議を唱える

例

[Just completed Task 2: Add verification function]

You: Let me request code review before proceeding.

BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)

[Dispatch code-reviewer agent or invoke /review]
  WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
  PLAN_OR_REQUIREMENTS: Task 2 from ai-workspace/plans/deployment-plan.md
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661
  DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types

[Reviewer returns]:
  Strengths: Clean architecture, real tests
  Issues:
    Important: Missing progress indicators
    Minor: Magic number (100) for reporting interval
  Assessment: Ready to proceed

You: [Fix progress indicators]
[Continue to Task 3]

ワークフローとの連携

サブエージェント駆動開発：

• 各タスクの後にレビューする
• 問題が重なる前に検出する
• 次のタスクに進む前に修正する

インラインの計画実行：

• 各バッチ（3 タスク）の後にレビューする
• フィードバックを得て、反映してから続行する

アドホック開発：

• マージ前にレビューする
• 行き詰まったときにレビューする

注意すべき危険信号

絶対にしないこと：

• 「簡単だから」という理由でレビューを省略する
• Critical の問題を無視する
• 未修正の Important の問題があるまま進める
• 妥当な技術的なフィードバックに反論する

レビュアーの指摘が誤っている場合：

• 技術的な根拠を示して異議を唱える
• 正しく動作することを示すコードやテストを提示する
• 説明を求める