mcpField
C
code-review

code-review

github.com/martinffxテスト・品質4949 回インストール5 日前に更新
リポジトリ

code-review について

code-reviewは、専門分野別のサブエージェントが段階的に議論し、証拠に基づいてコード変更をレビューするスキルです。ブランチ差分のレビューと、指摘への対話形式の対応を行えます。

主な機能

  1. 01トリアージ、批評、議論、裁定の各段階
  2. 02簡潔さの確認を含む並列の専門レビュー
  3. 03証拠に基づく指摘の評価と判断
  4. 04rqによる対象ブランチ指定レビュー
  5. 05rsによる指摘への対話形式の回答

活用シーン

  1. 01mainや別ブランチとの差分レビュー
  2. 02正確性、セキュリティ、設計、簡潔さの評価
  3. 03証拠に基づく異論のある指摘の解決

インストール

コマンドをコピーしてターミナルで実行します。

$npx skills add https://github.com/martinffx/atelier --skill code-review

インストール後、Claude Code で/code-reviewを入力して読み込まれたか確認します。

同梱ファイル

SKILL.md と一緒にインストールされる参考資料やスクリプト。

references/
output.md
personas.md
rq.md
rs.md
references/roles/
author.md
critic.md
judge.md
triage.md

詳細

コードレビューのスキル

意見ではなく、議論です:トリアージ → 批評 → 議論。

  • 批評:焦点を絞った批評者が並行して指摘事項を挙げ、すべてのレビューでシンプルさも確認します。
  • 議論:著者が変更を擁護し、批評者は譲歩するか、主張を維持するか、修正を受け入れます(1ラウンド)。裁定者は、なお争点となっている内容のみを判断します。
  • 証拠によって結論が決まります。引用は機械的に検証され、実行されたチェックは議論より優先され、メインエージェントは各ターンで行われたことを監査します。

根拠と先行作業:docs/research/multi-agent-debate.md。

code-subagents の明示的なサブエージェントディスパッチパターンを使用します。

前提条件

  • 必須:git

引数

コマンドのルーティング
呼び出し動作
(引数なし)main ブランチとの差分をレビュー
rqmain ブランチとの差分をレビュー
rq mainmain ブランチとの差分をレビュー
rq developdevelop ブランチとの差分をレビュー
feat/foofeat/foo との差分をレビュー(ブランチ名のみの場合は rq)
rsレビュー指摘事項に回答(インタビューモード)

エージェントの選択

各ステップをサブエージェントとしてディスパッチし、エージェントを明示的に選択します(atelier-orchestrator の委任を参照)。ステップの作業内容に合わせて選択してください。

ステップ作業の種類
トリアージ読み取り専用の調査:変更ファイルの分析、コンテキストの取得、ペルソナとチェックの選択
批評と反論批判的分析:証拠に基づく批評と障害モードの分析
擁護アーキテクチャと設計のレビュー:各指摘事項をコードと照らし合わせて検証し、変更を擁護
裁定批判的分析:公平で、証拠の優先順位に基づく裁定

批評者、著者、裁定者はプロンプト上のペルソナであり、エージェントの種類ではありません。 Security、Correctness、Architecture、PerformanceOperator などのペルソナ名も、同様にプロンプト上のペルソナです。

Simplicity は、すべてのレビューで必須のペルソナです。他の批評者と並行して1回実行し、トリアージの personas には決して記載しません。残りのペルソナはトリアージで選択します。Architecture は選択肢の1つです。

レビューの優先順位

次の順にレビューします。

  1. ユーザーの目標と従来の動作
  2. 必要性と削除
  3. 正確性とセキュリティ
  4. アーキテクチャ
  5. 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出力形式の仕様

ワークフローのルーティング

  • 引数なし、rq、またはブランチ名のみ → rq.md
  • rs → rs.md

コメント