[Fix] review orchestrator の Agent 起動に run_in_background: false を明示指定

[BlueEventHorizon]

概要

• /forge:review orchestrator (plugins/forge/skills/review/SKILL.md) が reviewer / evaluator / fixer / 汎用 Agent (関連コード探索) を起動する全5箇所に run_in_background: false を明示指定
• 対応する起動契約の正本 DES-029 にも同期実行契約を反映

Closes #208

背景

Claude Code v2.1.198 以降、Agent ツールで起動した subagent は既定でバックグラウンド実行される。review/SKILL.md の Agent 起動箇所にはこの明示指定が無く、完了待機の手段（allowed-tools に Monitor/TaskOutput 無し）も存在しないため、Issue #200（1層下の reviewer Agent → codex exec 境界、PR #206 で対応済み）と同型の問題が、1層上の orchestrator → Agent ツール境界に残存していた。

やったこと

• plugins/forge/skills/review/SKILL.md に「Agent 起動時の同期実行 [MANDATORY]」節を新規追加（1箇所に集約）し、reviewer起動×2 / evaluator起動 / fixer起動 / 汎用Agent起動（関連コード探索）の全5箇所の起動コードブロックに run_in_background: false を明示追加
• docs/specs/forge/design/DES-029_skill_agent_launch_contract_design.md のシーケンス図 (§4.2/§4.3) を run_in_background: false に更新し、§6.7 として同期実行契約を追記（実装レビューで設計SSOTとの乖離を指摘され対応）
• 回帰防止テスト tests/forge/review/test_orchestrator_agent_sync_execution.py を新規追加（7テスト）: 同期実行ルールの明示・各Agent起動ブロックへの run_in_background: false 指定を静的検証

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

• allowed-tools への Monitor / TaskOutput の追加（DES-029 の設計方針、reviewer/evaluator/fixer は Monitor を持たないことと整合させるため、同期実行の明示指定のみで対処する）
• review 以外の SKILL の Agent 起動箇所の点検（別Issue候補として Issue [Bug] /forge:review orchestrator が reviewer/evaluator Agent をバックグラウンド既定のまま起動し完了待機手段がない（Issue #200 の同型） #208 コメントに記録済み）

レビュー観点

• python3 -m unittest discover -s tests 1448件 pass
• dprint check pass
• 実装レビューで「設計SSOTが実装に追随していない」指摘を受け、DES-029 を追加コミットで対応済み

レビューレベル

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