Review Swarm

4つの読み取り専用サブエージェントで並列に diff をレビューしてから、メインエージェントが重要な問題だけをフィルタリング、順序付け、および要約します。このスキルはレビューのみです。サブエージェントはファイルを編集せず、メインエージェントはこのワークフローの一部として修正を適用しません。

Step 1: スコープと意図の決定

以下のスコープの優先順位を推奨します:

1. ユーザーが明示的に指定したファイルまたはパス
2. 現在の git の変更
3. ユーザーがリクエストした明示的なブランチ、コミット、または PR diff
4. 最近変更されたトラッキング対象ファイル。ユーザーがレビューをリクエストしていてより明確な diff がない場合のみ

明確なレビュースコープがない場合は、簡潔にそう述べて停止してください。

git の変更を使用する場合、最小限の正しい diff コマンドを選択してください:

• ステージングされていないワーク: git diff
• ステージングされたワーク: git diff --cached
• ステージングされたワークとステージングされていないワークが混在している場合: 両方をレビュー
• 明示的なブランチまたはコミット比較: ユーザーがリクエストしたものと正確に同じものを使用

レビュアーを起動する前に、最も近いローカル指示と、影響を受けるエリアに関する関連プロジェクトドキュメントを読んでください。以下の例があります:

• AGENTS.md
• repo ワークフロードキュメント
• 影響を受けるモジュールのアーキテクチャまたは契約ドキュメント

レビュアー向けの短い意図パケットを構築してください:

1. どのような動作が変わることを意図しているか
2. どのような動作が変わらないままであるべきか
3. 互換性、ロールアウト、セキュリティ、マイグレーション期待値などの明示的または推測される制約

ユーザーが意図を明確に述べていない場合は、diff から推測し、その推測が不完全かもしれないと述べてください。

Step 2: 4つの読み取り専用レビュアーを並列で起動

スコープが並列レビューで役立つほど大きい場合、4つのサブエージェントを起動してください。非常に小さい diff または1つの非常に小さいファイルの場合は、代わりにローカルでレビューすることは受け入れられます。

すべてのサブエージェントについて:

• 同じスコープと同じ意図パケットを与えてください
• サブエージェントが読み取り専用であることを述べてください
• サブエージェントがファイルを編集させたり、apply_patch を実行させたり、変更をステージさせたり、コミットさせたり、または他の状態変化させるアクションを実行させないでください
• 簡潔な発見のみをリクエストしてください
• リクエストしてください: ファイルと行または記号、問題、なぜそれが重要なのか、推奨される後続アクション、および信頼度
• サブエージェントに些細な指摘、スタイルの好み、および具体的な影響のない投機的な懸念を避けるよう伝えてください
• サブエージェントに発見をメインエージェントにのみ送り返すよう伝えてください

これら4つのレビュー役割を使用してください。

サブエージェント1: 意図と退行レビュー

diff が意図した動作変化と一致し、予期しない動作ドリフトを導入していないかレビューしてください。

以下をチェックしてください:

1. 述べられたスコープ外の意図しない動作変化
2. 壊れたエッジケースまたはフォールバックパス
3. 呼び出し元と呼び出し先間の契約ドリフト
4. 一緒に変わるべき隣接フローへの更新の欠落

このサブエージェントは読み取り専用です。ファイルを編集させたり、パッチを適用させたり、またはワークスペースの変更を加えさせてはいけません。

推奨サブエージェント役割: reviewer

サブエージェント2: セキュリティとプライバシーレビュー

diff をセキュリティの退行、プライバシーリスク、および信頼境界の誤りについてレビューしてください。

以下をチェックしてください:

1. 認証または認可チェックの欠落または弱化
2. 安全でないインプットハンドリング、インジェクションリスク、または検証隙間
3. シークレット、トークン、または機密データの暴露
4. リスクのあるデフォルト、権限拡張、または未検証データへの信頼

このサブエージェントは読み取り専用です。ファイルを編集させたり、パッチを適用させたり、またはワークスペースの変更を加えさせてはいけません。

推奨サブエージェント役割: reviewer

サブエージェント3: パフォーマンスと信頼性レビュー

diff を新しいコスト、脆さ、または運用リスクについてレビューしてください。

以下をチェックしてください:

1. 重複した作業、冗長な I/O、または不要な再計算
2. スタートアップ、レンダリング、リクエスト、またはその他のホットパスに追加された作業
3. リーク、欠落したクリーンアップ、リトライストーム、またはサブスクリプションドリフト
4. 変更を脆くする順序、競合状態、または失敗処理の問題

このサブエージェントは読み取り専用です。ファイルを編集させたり、パッチを適用させたり、またはワークスペースの変更を加えさせてはいけません。

推奨サブエージェント役割: reviewer

サブエージェント4: 契約とカバレッジレビュー

diff を互換性ギャップと欠落した安全ネットについてレビューしてください。

以下をチェックしてください:

1. API、スキーマ、型、設定、またはフィーチャーフラグの不一致
2. マイグレーションまたは後方互換性の問題
3. 変更された動作に対する不足または弱いテスト
4. 回帰を検出しづらくするログ、メトリクス、アサーション、またはエラーパスの欠落

このサブエージェントは読み取り専用です。ファイルを編集させたり、パッチを適用させたり、またはワークスペースの変更を加えさせてはいけません。

推奨サブエージェント役割: reviewer

正確性、セキュリティ、プライバシー、信頼性、互換性、または変更への信頼に実質的に影響する問題のみを報告してください。わずかな指摘を逃すより、低価値のノイズでユーザーを埋めない方がいいです。

Step 3: 発見の集約とフィルタリング

メインエージェントが統合を担当します。サブエージェント出力を最終出力ではなく、生のレビュー入力として扱ってください。

4つのレビュアー全体の発見を統合し、積極的にフィルタリングしてください:

• 重複を削除
• 弱いまたは投機的な主張を削除
• 述べられた意図と矛盾する問題を削除
• 実際のバグまたはメンテナンスリスクを隠す場合を除き、軽微なスタイルまたは可読性コメントを削除

生き残った発見をこの形に正規化してください:

1. ファイルと行または最も近い記号
2. カテゴリ: 退行、セキュリティ、信頼性、または契約
3. 重大度: 高、中、または低
4. なぜそれが重要なのか
5. 推奨される修正またはフォローアップ
6. 信頼度: 高、中、または低

レビュアーが正しいかもしれないが意図が不明確な場合は、それを発見ではなくオープンクエスチョンに変えてください。

Step 4: 出力の順序付け

この順序で発見を提示してください:

1. 高重大度、高信頼度の問題
2. マージ前に修正する価値があるそうな中程度の重大度の問題
3. より低い重大度の問題またはマージ後でいい後続処理

レビューを簡潔に保ってください。発見は実行可能で証拠に基づいているべきです。

実質的な問題がない場合は、フィードバックを製造する代わりに、直接そう述べてください。

Step 5: 明確な次のステップを推奨

発見の後、ユーザーに短いパスを与えてください:

• マージ前に修正すべきこと
• 時間があれば改善するべきこと
• 安全に放置できること

役に立つ場合、次のステップのパスをグループ化してください:

• fix now
• fix soon
• optional follow-up

このスキルの一部として修正を実装しないでください。出力は読み取り専用レビューと優先順位付けされた推奨です。