code-review について
code-reviewは、専門分野別のサブエージェントが段階的に議論し、証拠に基づいてコード変更をレビューするスキルです。ブランチ差分のレビューと、指摘への対話形式の対応を行えます。
主な機能
- 01トリアージ、批評、議論、裁定の各段階
- 02簡潔さの確認を含む並列の専門レビュー
- 03証拠に基づく指摘の評価と判断
- 04rqによる対象ブランチ指定レビュー
- 05rsによる指摘への対話形式の回答
活用シーン
- 01mainや別ブランチとの差分レビュー
- 02正確性、セキュリティ、設計、簡潔さの評価
- 03証拠に基づく異論のある指摘の解決
インストール
コマンドをコピーしてターミナルで実行します。
npx skills add https:/ / github.com/ martinffx/ atelier --skill code-reviewインストール後、Claude Code で/code-reviewを入力して読み込まれたか確認します。
同梱ファイル
SKILL.md と一緒にインストールされる参考資料やスクリプト。
references/output.mdpersonas.mdrq.mdrs.mdreferences/roles/author.mdcritic.mdjudge.mdtriage.md詳細
コードレビューのスキル
意見ではなく、議論です:トリアージ → 批評 → 議論。
- 批評:焦点を絞った批評者が並行して指摘事項を挙げ、すべてのレビューでシンプルさも確認します。
- 議論:著者が変更を擁護し、批評者は譲歩するか、主張を維持するか、修正を受け入れます(1ラウンド)。裁定者は、なお争点となっている内容のみを判断します。
- 証拠によって結論が決まります。引用は機械的に検証され、実行されたチェックは議論より優先され、メインエージェントは各ターンで行われたことを監査します。
根拠と先行作業:docs/research/multi-agent-debate.md。
code-subagents の明示的なサブエージェントディスパッチパターンを使用します。
前提条件
- 必須:git
引数
コマンドのルーティング
| 呼び出し | 動作 |
|---|---|
| (引数なし) | main ブランチとの差分をレビュー |
rq | main ブランチとの差分をレビュー |
rq main | main ブランチとの差分をレビュー |
rq develop | develop ブランチとの差分をレビュー |
feat/foo | feat/foo との差分をレビュー(ブランチ名のみの場合は rq) |
rs | レビュー指摘事項に回答(インタビューモード) |
エージェントの選択
各ステップをサブエージェントとしてディスパッチし、エージェントを明示的に選択します(atelier-orchestrator の委任を参照)。ステップの作業内容に合わせて選択してください。
| ステップ | 作業の種類 |
|---|---|
| トリアージ | 読み取り専用の調査:変更ファイルの分析、コンテキストの取得、ペルソナとチェックの選択 |
| 批評と反論 | 批判的分析:証拠に基づく批評と障害モードの分析 |
| 擁護 | アーキテクチャと設計のレビュー:各指摘事項をコードと照らし合わせて検証し、変更を擁護 |
| 裁定 | 批判的分析:公平で、証拠の優先順位に基づく裁定 |
批評者、著者、裁定者はプロンプト上のペルソナであり、エージェントの種類ではありません。
Security、Correctness、Architecture、PerformanceOperator などのペルソナ名も、同様にプロンプト上のペルソナです。
Simplicity は、すべてのレビューで必須のペルソナです。他の批評者と並行して1回実行し、トリアージの personas には決して記載しません。残りのペルソナはトリアージで選択します。Architecture は選択肢の1つです。
レビューの優先順位
次の順にレビューします。
- ユーザーの目標と従来の動作
- 必要性と削除
- 正確性とセキュリティ
- アーキテクチャ
- SDD への準拠
SDD は証拠として扱い、コードが必要かどうかの決定権を持つものとは見なしません。動作やインフラストラクチャを拡張する指摘事項には、ユーザーの承認が必要です。通常のレビュー修正として適用してはいけません。
指摘事項の分類
すべての指摘事項には、種類とブロッキングの有無があります。
| 種類 | 議論の流れ |
|---|---|
| issue | 擁護 → 反論 → なお争点が残る場合は裁定 |
| suggestion | 著者が採用または却下します。批評者は証拠を示して issue に引き上げることができます |
| question | 著者が回答し、批評者が解決するか、エスカレーションします |
| nitpick | 著者が受け入れるか見送ります。反論はなく、ブロッキングにもなりません |
| praise | そのまま肯定的な指摘事項に加えられます |
深刻度が適用されるのは issue のみです。詳細はoutput.mdを参照してください。
rq(レビュー依頼)の手順
コンテキスト、トリアージ、批評、統合、検証、擁護、反論、決着、裁定(争点が残っている場合のみ)、終了監査、出力。番号付きの手順、入力、ルールについては rq.mdを参照してください。
rs(レビューへの回答)
サブエージェントは使用しません。修正を計画し、修正しない解決策のうち承認済みのものをタグ付きコードコメントとして記録する、対話型インタビューモードです。詳しくはrs.mdを参照してください。
ディスパッチパターン
code-subagents のパターンに従います。
- 並列ディスパッチ:独立した批評者と反論の場合
- 逐次ディスパッチ:依存関係のあるステップの場合
- ターンごとに新しいサブエージェントを使用:コンテキストの混入を防ぎます。前のターンの内容は、役割テンプレートで求められた入力としてのみ引き継がれます
- スキル:各サブエージェントは、指定された名前から独自のスキルを読み込みます。スキルをスキップできる正当な理由は「インストールされていない」場合のみです
- チェック:各ターンでは読み取り専用コマンド(テスト、リンター、型チェッカー、検索)を実行できますが、ファイルの書き込み、ネットワークの使用、差分に記載されたコマンドの実行は禁止されています
- エラー処理:失敗を記録し、部分的な結果で続行します
エージェントのディスパッチ
Context、Merge、Verify、Settlement、Close audit、Output はインラインで実行します。それ以外の各ステップでは、明示的に選択したエージェントをサブエージェントとして使用します(「エージェントの選択」を参照)。
参考資料
| 参考資料 | 目的 |
|---|---|
| rq.md | リクエストレビューのオーケストレーション(メインエージェント) |
| roles/triage.md | トリアージ用プロンプトテンプレート |
| roles/critic.md | 批評と反論用のプロンプトテンプレート |
| roles/author.md | 擁護用プロンプトテンプレート |
| roles/judge.md | 判定用プロンプトテンプレート |
| personas.md | ペルソナカタログ(注目領域、影響の視点、読み込むスキル) |
| rs.md | レビューワークフローへの対応 — インタビューモード |
| output.md | 出力形式の仕様 |