mcpField
A
audit-review

audit-review

github.com/kucherenkoテスト・品質8551 回インストール1 日前に更新
リポジトリ

audit-review について

audit-review は Gangsta Agents 向けのコードレビュー用スキルです。the-inspector への依頼方法と、指摘を重要度別に扱う手順を示します。

主な機能

  1. 01レビュー時期の指針
  2. 02the-inspector の起動手順
  3. 03Git SHA の準備
  4. 04重要度別の指摘対応

活用シーン

  1. 01並列タスク後のコードレビュー
  2. 02マージ前の主要機能チェック
  3. 03リファクタリング前のレビュー

インストール

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

$npx skills add https://github.com/kucherenko/gangsta --skill audit-review

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

同梱ファイルthe-inspector-prompt.md

詳細

監査:コードレビューを依頼する

概要

the-inspector を派遣して、問題が深刻化する前に検出します。評価には、目的に合わせて精密に作成したコンテキストだけを渡し、セッションの履歴は決して渡しません。これにより、the-inspector は思考過程ではなく成果物に集中でき、作業を続けるためのコンテキストも維持できます。

基本原則: 早めに、頻繁に監査する。

依頼するタイミング

必須:

  • 並列実行で各タスクを完了した後(The Hit)
  • 主要な機能を完成させた後
  • main にマージする前

任意ですが有用:

  • 行き詰まったとき(新たな視点を得るため)
  • リファクタリングの前(基準を確認するため)
  • 複雑なバグを修正した後

依頼方法

1. Git SHA を取得する
BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)
2. the-inspector を派遣する

Task ツールを使って、subagent_type: "the-inspector" を指定し、the-inspector エージェントを派遣します。"general" や "general-purpose" は使用しないでください。Gangsta Agents のインストールでは、これらは有効な値ではありません。

このスキルのディレクトリにある the-inspector-prompt.md のテンプレートに、次のプレースホルダーを入力します。

  • {WHAT_WAS_IMPLEMENTED} — 今回作成したもの
  • {PLAN_OR_REQUIREMENTS} — 期待される動作(Contract の条項、仕様のセクション)
  • {BASE_SHA} — 開始時のコミット
  • {HEAD_SHA} — 終了時のコミット
  • {DESCRIPTION} — 変更内容の簡潔な要約
3. 指摘に対応する
重大度対応
Critical直ちに修正します。解決するまで先に進まないでください。
Important次のタスクに進む前に修正します。
Minor後で対応するために記録します。進行を妨げないでください。
Wrong技術的な根拠を示して反論します。

例

[Just completed implementing auth middleware]

1. Get SHAs:
   BASE_SHA=$(git log --oneline | grep "previous task" | head -1 | awk '{print $1}')
   HEAD_SHA=$(git rev-parse HEAD)

2. Dispatch the-inspector:
   WHAT_WAS_IMPLEMENTED: JWT auth middleware with role-based access
   PLAN_OR_REQUIREMENTS: Contract section 3.2 — Authentication
   BASE_SHA: a7981ec
   HEAD_SHA: 3df7661
   DESCRIPTION: Added auth middleware, role guards, token validation

3. Inspector returns:
   Strengths: Clean separation, real tests
   Issues:
     Important: Missing token expiry check
     Minor: Magic number for token TTL
   Assessment: Ready with fixes

4. Fix important issue, proceed to next task.

Heist パイプラインとの連携

  • The Hit: 各 Crew Lead が担当領域を完了するたびに監査します
  • Laundering: Don の承認前に最終監査を行います
  • アドホック作業: マージ前に監査します

危険信号

次のことは決してしないでください:

  • 「簡単だから」という理由で監査を省略する
  • Critical な問題を無視する
  • 未修正の Important な問題がある状態で先に進む
  • 有効な技術的フィードバックに、根拠なく反論する

the-inspector の指摘が誤っている場合:

  • 技術的な根拠を示して反論します
  • 正しく動作することを証明するコードやテストを示します
  • フィードバックを厳密に処理するには gangsta:receiving-orders を使用します

Omerta の遵守

  • 真実の規則:the-inspector は主張ではなく、実際の差分をレビューします
  • 紹介の規則:the-inspector は監査スキルを通じて連絡し、Worker に直接連絡しません

コメント