audit-review について
audit-review は Gangsta Agents 向けのコードレビュー用スキルです。the-inspector への依頼方法と、指摘を重要度別に扱う手順を示します。
主な機能
- 01レビュー時期の指針
- 02the-inspector の起動手順
- 03Git SHA の準備
- 04重要度別の指摘対応
活用シーン
- 01並列タスク後のコードレビュー
- 02マージ前の主要機能チェック
- 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 に直接連絡しません
コメント
Markdown 対応