コードレビュー
概要
目的: ローカルブランチまたは 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 の回答に基づいて変更内容を取得する。
ローカルブランチの場合:
bash1git diff main...HEAD 2git diff --name-only main...HEAD
GitHub PR の場合:
AskUserQuestion ツールを使って PR 番号を確認する。
<rules> - **AskUserQuestion ツールを必ず使用** - 会話形式の質問は不可 </rules>その後:
bash1gh pr view <PR番号> --json title,body,files,additions,deletions 2gh pr diff <PR番号>
フェーズ 4: レビューの実行
全てのサブエージェントを並列で実行する。
サブエージェント
| サブエージェント | 専門領域 |
|---|---|
review-quality | コード品質（命名、SRP、複雑度、エラー処理） |
review-performance | パフォーマンス（計算効率、メモリ、レンダリング） |
review-security | セキュリティ（XSS、SQLi、入力検証） |
review-guideline | ガイドライン準拠（TypeScript/React/テストルール） |
フェーズ 6: 破壊的変更のチェック
「行う」を選択した場合のみ実行する。
| カテゴリ | 例 |
|---|---|
| API 変更 | 関数の引数追加/削除、戻り値の型変更 |
| スキーマ変更 | DB カラムの削除、型変更、NOT NULL 追加 |
| 設定変更 | 環境変数の追加/削除、デフォルト値の変更 |
| 依存関係 | メジャーバージョンアップ、パッケージの削除 |
フェーズ 7: 結果の出力
レビュー観点（サブエージェント別）
| サブエージェント | チェック内容 |
|---|---|
| review-quality | 命名規則、単一責任原則、関数複雑度、エラーハンドリング |
| review-performance | 計算効率、メモリ管理、レンダリング最適化、N+1問題 |
| review-security | XSS、SQLi、入力検証、認証・認可、OWASP Top 10 |
| review-guideline | TypeScript/React/テストルール準拠、コードスタイル |
出力形式
markdown1## コードレビュー結果 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 を呼び出してからユーザーの回答を待つ
