mcpField
C
code-review

code-review

github.com/gabriel-lafranceテスト・品質081 回インストール2 日前に更新
リポジトリ

code-review について

指定された範囲を、提出済みの差分や利用可能な要件に照らして確認するコードレビュー用スキルです。レビュー範囲、証拠基準、規約と仕様の並行確認、指摘の扱い方を定めています。

主な機能

  1. 01レビュー範囲の選択
  2. 02規約と仕様の確認
  3. 03証拠に基づく指摘
  4. 04修正レビュー方式
  5. 05指摘の処置ガイダンス

活用シーン

  1. 01提出済み差分全体のレビュー
  2. 02受け入れ基準に照らした変更確認
  3. 03指定された指摘の修正検証
  4. 04フォローアップ事項や任意の問題の報告

インストール

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

$npx skills add https://github.com/gabriel-lafrance/skills --skill code-review

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

同梱ファイルdoctrine.mdexamples.md

詳細

コードレビュー

必読: ../pack-shared/standards.md。判定を行う前に、このターンで AGENTS.md の Taste と Architecture のセクションを適用してください。省略しないでください。

共有コントラクト: レビュー · 実行コンテキスト

読んでください: doctrine.md · examples.md

Standards では、常に AGENTS.md の Taste と Architecture のセクションを適用します。ワーカーの出力には、review-contract の review output フェンス(Principles、Architecture、Correctness hunt)を使用します。ユーザー向けの指摘には、一般的な言葉を使います(plain-language.md)。

入力

指定された固定時点を確定し、リリース済みの差分を調査してください。

親からすでに成果、完了条件、対象外事項、チケットまたは PR、固定時点、レーン、フェーズ、Active Rules、現在のスライス、修正バックログが提示されている場合は、そのチャットコンテキストを拘束力のある引き継ぎとして扱ってください。隠しファイルから意図を再構築しないでください。

それ以外の場合は、次の順に仕様の軸を導き出してください。

  1. ユーザーが示した成果と受け入れ基準
  2. 名前が挙げられた PR、チケット、および参照可能な関連ディスカッション
  3. 関連するリポジトリのコード、ルール、コミット済みドキュメント

仕様を確認できない場合は、その旨を伝え、要件を捏造せずに Standards を実行してください。すべての指摘、判断、修正メモはチャット内に残してください。

Active Rule を挙げるのは、実際に違反している場合に限ってください。そうでなければ、関連する受け入れ基準を挙げるか、該当するルールがないことを明記してください。

範囲

共有レビューのモードを意図的に選択してください。

  • initial では、リリース済みの差分全体を対象に、Standards と Spec を並行してレビューします。Design 軸も、2 回目の敵対的な検証もありません。
  • remediation では、指摘 ID、修正差分、変更された直接パス、および直接の呼び出し元のみを対象にします。
  • full-rescan では、意味のある変更後にフルレビューの深さで再度レビューすることを明示的に求める必要があります。

修正作業に着手する前に、code-review:axes、code-review:blocker-vs-follow-up、code-review:naming-alignment、review-contract のエビデンス基準、および修正分析を適用してください。Standards ワーカーは review output フェンスを返す必要があります。このターンでは、Standards が AGENTS.md の Taste と Architecture のセクションを適用する必要があります。Design レビューの Task は割り当てないでください。

remediation では、対象範囲内で該当する指摘とリグレッションを検証してください。広範なレビューを再開しないでください。

引き継ぎ

親(このチャット、またはネストされた場合の /task / /just-do-it)が、固定時点の設定、ワーカーの割り当て、Completions のレビュー、受け入れエビデンス、レビューゲートを担当します。Standards と Spec は 必ず並行して Task として実行してください。差分に独立した対象範囲がある場合は、追加の Task を割り当てます。Design 軸も、2 回目の敵対的な検証もありません。実装ワーカーはこれらのゲートを実行せず、修正レビューの対象範囲を広げません。安定した指摘 ID と、Fix now / Follow-up / Optional nit の処置をチャットで報告してください。

親が次の手順をすでに担当している場合

修正作業に着手する前に、選択した Fix now の指摘を、review-remediation モードで /analyze に送ってください。そのメモは安定した指摘 ID に紐づけたままにし、その後、明示的な昇格(または文書化された /just-do-it の例外)が必要です。昇格後のレーンは、該当する指摘と提示された現在のスライスに限定されます。

ユーザーからの単発依頼の場合

処置をチャットで報告してください。ユーザーが次の手順を求めていない限り、親の次の段階を勝手に設定したり、修正を昇格させたりしないでください。

コメント