code-review

コードレビュー タスクカード

マージ準備状況のレビューには、この明示的なオプトインを使用してください。共有の運用上の不変条件は templates/AGENTS.md にあります。このカードではレビュー固有の動作のみを定義します。

使用する場合

• ユーザーがコードレビューや品質／セキュリティ評価を依頼したとき。
• 変更がマージ前のレビュー準備ができている場合、または大規模な機能に独立したレビューが必要な場合。
• 実装、広範な計画、自動クリーンアップの場合は、このカードを有効化しないでください。

入力

• スコープ: 要求されたファイル、コミット、PR、または差分全体。
• 要件／仕様、受け入れ基準、および関連するテスト／CI の証拠。
• 既存のレビュー成果物と既知のリスク（ある場合）。
• ユーザーが continue と指示した場合は、探索をやり直すのではなく、現在検証済みのレビューステップを進めてください。

まず、スコープを記録してください:

git status --short
git diff --stat
git diff -- <scope>

実行

1. 変更されたファイルとレビューの境界を特定してください。スコープを黙って広げないでください。
2. code-reviewer と architect のエージェントを並列に起動してください。両レーンは、明示的なスコープと成果物を持つクリーンなコンテキストで並列に実行されます。いずれかのレーンを起動できない、または証拠を返さない場合は、independent review unavailable と報告してください。現在の（作成者側の）レーンで代用しないでください。また、レビューを承認したりマージ可能とマークしたりしないでください。
3. ユーザーの現在のモデルと推論／エフォートの選択を尊重してください。レビューレーンの呼び出しでは model や reasoning_effort の上書きを渡さないでください。

task(
  agent_type="code-reviewer",
  prompt="CODE REVIEW TASK

Review the supplied scope for spec compliance, security, quality, performance, and maintainability.
Return files reviewed, severity-rated findings with file:line evidence and concrete fixes,
and a recommendation: APPROVE / REQUEST CHANGES / COMMENT. Do not review architecture.
Scope: [scope and artifacts]"
)

task(
  agent_type="architect",
  prompt="ARCHITECTURE / DEVIL'S-ADVOCATE REVIEW TASK

Review the same scope for boundaries, interfaces, hidden coupling, long-term tradeoffs,
and the strongest counterargument against approval. Return file:line evidence,
recommendations, and Architectural Status: CLEAR / WATCH / BLOCK.
Scope: [scope and artifacts]"
)

レビューの分類

• code-reviewer は Security、Code Quality、Performance、Best Practices、Maintainability を確認します。
• 各指摘を評価してください: CRITICAL（セキュリティまたはデータ損失のブロッカー）、HIGH（バグ／主要な臭い）、MEDIUM（重要な改善）、LOW（スタイル／提案）。
• architect は、明示的な境界／インターフェース、隠れた結合、長期的なトレードオフ、および反対意見（devil's advocate）の懸念を確認します。ステータスは CLEAR、WATCH（非ブロッキングの懸念）、または BLOCK（マージのブロッカー）です。
• すべての指摘には file:line、問題、リスク、具体的な修正案を明記し、事実と提案を区別してください。

State/HUD フェーズ契約

• スタンドアロンの $code-review は、フックが管理する skill-active-state.json（skill:"code-review"、phase:"planning"）に依存します。code-review-state.json を作成しないでください。
• Autopilot 内では、mode:"autopilot" を current_phase:"code-review" / skill-active の phase:"code-review" でアクティブに保ち、同列のワークフローを有効化しないでください。
• クリーンなレビューの場合は、ultraqa に進む前に Autopilot の handoff_artifacts.code_review に成果物を保存してください。クリーンでないレビューの場合は、指摘事項を保存し、適切に rework または ralplan を使用してください。

omx state write --input '{"mode":"autopilot","active":true,"current_phase":"code-review"}' --json

最終統合とゲート

アーキテクチャステータス契約

code-reviewer の推奨事項と architect のステータスを組み合わせてください。承認には両方の独立したレーンからの明示的な証拠が必要です。委任が欠落または失敗している場合は、承認のフォールバックではなく、ブロッキングとなる「レビュー利用不可」状態です。最終報告では、architect のブロッカーが見逃されないようにしてください。

• architect のステータスが BLOCK の場合、最終推奨は REQUEST CHANGES です。
• それ以外で、code-reviewer の推奨が REQUEST CHANGES の場合、最終推奨は REQUEST CHANGES です。
• それ以外で、architect のステータスが WATCH の場合、最終推奨は COMMENT です。
• それ以外の場合、最終推奨は code-reviewer のレーンに従います。

承認基準: code-reviewer が APPROVE を返し、architect のステータスが CLEAR であり、両方の独立したレーンが証拠を返した場合にのみ APPROVE です。ブロッカー、未解決の high/critical 指摘、または利用不可のレーンがある場合は REQUEST CHANGES です。COMMENT には非ブロッキングの指摘を記録できます。

フォールバックとしての自己レビューは行わないでください。code-reviewer または architect の経路が存在しない、利用不可、スキップされた、または失敗した場合は、独立したレーンの証拠が得られるまで承認をブロックしてください。サポートされている omx ralph CLI の互換性経路では、指摘事項により追加の許可プロンプトなしで自動修正のフォローアップが行われる場合があります。ただし、通常の code-review 自体は読み取り専用のままであり、自動修正を保証するものではありません。

エビデンス／出力の契約

次の内容を含む簡潔なレポートを返してください：

CODE REVIEW REPORT
Files Reviewed: <count>
Total Issues: 0
Architectural Status: CLEAR | WATCH | BLOCK
CRITICAL (0) | HIGH (0) | MEDIUM (0) | LOW (0)
Findings: file:line -> issue, risk, concrete fix (or none)
ARCHITECTURE WATCHLIST: concern, status, recommendation (or none)
- code-reviewer recommendation: COMMENT
- architect status: WATCH
- final recommendation: COMMENT
RECOMMENDATION: COMMENT

例示のカウントと判定を、観測された値に置き換えてください。スコープ、レーンのエビデンスおよびアーティファクトへの参照、未解決のリスク、検証のギャップを含めてください。

終了条件

スコープ内の差分に独立した2つのレーンの結果と、決定論的な最終推奨が揃った時点で停止してください。APPROVE は承認基準を満たす場合にのみ報告してください。それ以外の場合は、範囲が明確な REQUEST CHANGES、COMMENT、またはレビュー不可の結果を残してください。必要なエビデンスがない限り、マージ可能であると主張しないでください。