コードレビュー

概要

目的: ローカルブランチまたは GitHub PR の変更内容をレビューし、品質・一貫性・潜在的な問題を検出する。

AskUserQuestion ツールを使ってレビュー条件をインタビューし、適切なサブエージェントでレビューを実行する。

フェーズ 1: レビュー対象の確認

AskUserQuestion ツールを使って質問する。

質問内容:

• question: "レビュー対象は？"
• header: "対象"
• options:
  • label: "ローカルブランチ", description: "現在のブランチと main の差分をレビュー"
  • label: "GitHub PR", description: "現在のブランチPRの差分をレビュー"
  • label: "ローカル差分", description: "ローカルにある差分をレビュー"
  • label: "ステージ済みのローカル差分", description: "ローカルにあるステージ済みの差分をレビュー"

フェーズ 2: 破壊的変更チェックの確認

AskUserQuestion ツールを使って質問する。

質問内容:

• question: "破壊的変更チェックは？"
• header: "破壊的変更"
• options:
  • label: "行う", description: "API/スキーマ変更確認"
  • label: "行わない", description: "通常レビューのみ"

フェーズ 3: 変更内容の取得

フェーズ 1 の回答に基づいて変更内容を取得する。

ローカルブランチの場合:

bash
1git diff main...HEAD
2git diff --name-only main...HEAD

GitHub PR の場合:

AskUserQuestion ツールを使って PR 番号を確認する。

その後:

bash
1gh pr view <PR番号> --json title,body,files,additions,deletions
2gh pr diff <PR番号>

フェーズ 4: レビューの実行

全てのサブエージェントを並列で実行する。

サブエージェント

フェーズ 6: 破壊的変更のチェック

「行う」を選択した場合のみ実行する。

フェーズ 7: 結果の出力

レビュー観点（サブエージェント別）

出力形式

markdown
1## コードレビュー結果
2
3### サマリー
4- レビュー対象: ローカルブランチ / PR #XXX
5- 変更ファイル数: X
6- 追加行数: +XXX
7- 削除行数: -XXX
8
9### スコア一覧
10
11| 観点 | スコア |
12|------|--------|
13| 品質 | A/B/C/D |
14| パフォーマンス | A/B/C/D |
15| テスト | A/B/C/D |
16| ドキュメント | A/B/C/D |
17| セキュリティ | A/B/C/D |
18| ガイドライン | A/B/C/D |
19| Design Doc | A/B/C/D（設計書ありの場合） |
20
21### 破壊的変更
22🔴 あり / 🟢 なし
23
24### 発見事項
25
26#### 🔴 Critical（修正必須）
27- [ファイル名:行番号] 問題の説明
28
29#### 🟡 Warning（推奨）
30- [ファイル名:行番号] 改善提案
31
32#### 🟢 Good（良い点）
33- 良かった実装のポイント
34
35### 総評
36全体的な評価とコメント

重要な注意事項

• AskUserQuestion ツールを必ず使用 - 会話形式の質問は不可
• 各フェーズで必ず AskUserQuestion を呼び出してからユーザーの回答を待つ