[Fix] reviewer.md の engine=codex 時の二重レビュー構造を修正

[BlueEventHorizon]

概要

/forge:review の reviewer 遅延調査で発見された、engine=codex 時にも reviewer Agent 自身が
全参照文書の精読と自前レビュー（P1→P2→P3評価）を実行し得る二重レビュー構造を修正。

背景

plugins/forge/agents/reviewer.md のワークフローが Phase 1（全文書Read）→ Phase 2/3
（レビュー実行）→ Phase 4（エンジン別の実行）の直列構成で、Phase 2/3 に engine=claude
限定の条件が明記されていなかった。

やったこと

• reviewer.md: Phase 1 の Read 範囲を engine=codex 時は最小限に限定、Phase 2/3 を
  engine=claude（Codex不在時のフォールバック含む）限定と明記
• reviewer.md: Codex不在フォールバック時に実効engineをclaudeに切り替え、Phase 1未読分
  • Phase 2/3 を再開する手順を明記
• DES-015 §Phase4①、DES-029 §4.2 シーケンス図を上記のengine分岐と整合するよう更新
• tests/forge/review/test_codex_launch_path.py に静的回帰テストを追加（TestReviewerEngineScopedPhases、6件）
• 副次的に発見された reviewer.md の孤立記述（対応する実行手順のない dprint fmt 言及）も削除

やらないこと（このプルリクエストのスコープ外とすること）

• DES-029自体の設計変更（既にengine分岐が正しく記述済みのため不要）

レビュー観点

• reviewer.md の Phase 2/3 見出し直後の engine 条件明記が、Phase 4 の既存分岐と矛盾しないか
• DES-015 / DES-029 の記述整合性
• 静的回帰テスト（全1461件 OK, skipped=1）

レビューレベル

• Lv0: まったく見ないでAcceptする
• Lv1: ぱっとみて違和感がないかチェックしてAcceptする
• Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証してAcceptする
• Lv3: 実際に環境で動作確認したうえでAcceptする

Closes #210