[Bug] reviewer の run_review_engine.sh 同期実行必須化 (#200) が Bash デフォルトタイムアウト 120 秒と衝突し完了済み codex レビューが破棄される

[BlueEventHorizon]

背景 / コンテキスト

/forge:review の reviewer 動作が著しく遅い（「レビュー結果を書き終わっているのに agent が終了しない」ように見える）という報告を受けた調査（2026-07-15）で発見した。Issue #200 の修正（PR #206 / b950364）は forge:reviewer Agent に run_review_engine.sh の同期フォアグラウンド実行を義務付けたが、この修正が Claude Code Bash ツールのデフォルトタイムアウトと衝突している。

現象（実際の動作）

plugins/forge/agents/reviewer.md の「同期実行必須 [MANDATORY]（Issue #200）」節は run_review_engine.sh を Bash ツールで run_in_background なしのフォアグラウンド実行し exit code 受領まで待つことを要求するが、Bash ツールの timeout パラメータ指定の指示がない。

Claude Code Bash ツールのタイムアウトは公式仕様でデフォルト 120 秒（timeout パラメータで最大 600 秒まで延長可、BASH_DEFAULT_TIMEOUT_MS / BASH_MAX_TIMEOUT_MS でオーバーライド可。根拠: https://code.claude.com/docs/en/tools-reference ）。本プロジェクトでは BASH_DEFAULT_TIMEOUT_MS は未設定。

一方 codex exec によるレビューは数分かかる場合がある（Issue #200 本文にも明記）。その結果、レビューが 120 秒を超えると:

1. Bash ツールが run_review_engine.sh をプロセスごと kill する
2. codex はレビュー本文を {output}.stdout にストリーム済み（＝結果は「書き終わっている」ように見える）だが、run_review_engine.sh 内の救済コード（rc != 0 でも extract_codex_output.py で部分出力を抽出する経路）はスクリプト自体が kill されるため実行されず、完了済みレビューが丸ごと破棄される
3. reviewer（sonnet）はエラーを受けて codex を再実行、または Claude エンジンへフォールバックし、同じレビューをもう一周する

これにより所要時間が倍々に増え、「書き終わっているのに終了しない／無駄に待っている」という体感遅延になる。

期待動作

codex exec のレビューが数分かかっても Bash ツールのタイムアウトで kill されず、完了後に extract_codex_output.py → convert_codex_findings.py の後工程へ進むこと。仮にタイムアウトが発生しても、ストリーム済みの部分出力（.stdout）が破棄されず救済されること。

再現手順

1. BASH_DEFAULT_TIMEOUT_MS 未設定の環境で /forge:review code --files <レビューに 2 分超かかる規模の対象> を実行する（engine=codex がデフォルト）
2. Phase 4 で forge:reviewer Agent が run_review_engine.sh をフォアグラウンド実行する（reviewer.md の MANDATORY 指示通り、timeout パラメータ未指定）
3. codex exec のレビューが 120 秒を超える
4. Bash ツールがタイムアウトでスクリプトを kill し、{session_dir}/review_<種別>.md が生成されないまま reviewer がエラー処理（再実行 / フォールバック）に入ることを確認する

環境情報

• macOS (Darwin 25.5.0) / Claude Code（Bash ツール default timeout 120000ms・max 600000ms、公式ドキュメントで確認済み）
• codex-cli 0.144.4（codex exec --sandbox read-only）
• forge:reviewer agent 定義: model: sonnet
• 参考実測: 最小プロンプトでの codex exec 実行は約 4.2 秒で完了（起動・終了オーバーヘッドは小さく、遅延の本体はレビュー推論時間）

重大度

機能に支障あり

証跡

なし（codex exec 最小実行の time 計測は調査セッション内でのみ実施）

原因

Issue #200 の修正で追加された reviewer.md の同期実行必須節に、Bash ツールの timeout パラメータ（最大 600000ms）を指定する指示がなく、デフォルト 120 秒が適用されるため。また run_review_engine.sh の部分出力救済コードはスクリプト内部にあり、ツールレベルの kill（スクリプトごと停止）では到達できない構造になっているため。

実装計画

関連ドキュメント

参考 PR（実装パターン）

PR	参考ポイント
#206 [Fix] forge:reviewer Agent が run_review_engine.sh を同期実行するよう明記	reviewer.md の MANDATORY 節への指示追記パターン

実装スコープ

レイヤー	実装内容	ファイルパス
Agent 定義	同期実行必須節に Bash ツールの timeout: 600000 明示指定を追記	plugins/forge/agents/reviewer.md
テスト	timeout 記載の静的回帰テストを追加	tests/forge/review/test_codex_launch_path.py

スコープ外

項目	理由	担当
タイムアウト発生時の部分出力(.stdout)救済機構	SIGTERM/SIGKILL 挙動の技術的不確実性があり別スコープと判断（triage判定）	別 Issue

TODO

着手前確認

• 仕様書の確認完了（triage 引き継ぎ）
• 実装ルールの確認完了（triage 引き継ぎ）
• 参考 PR のパターン確認完了（[Fix] forge:reviewer Agent が run_review_engine.sh を同期実行するよう明記 #206）

実装する

• reviewer.md の同期実行必須節に timeout パラメータ指示を追記する
• test_codex_launch_path.py に timeout 記載の検証を追加する

テストを書く

• test_codex_launch_path.py: reviewer.md の Bash 起動コードブロックに timeout パラメータの明示指定があることを検証する

確認・レビュー準備を行う

• テストを実行する（python3 -m unittest tests.forge.review.test_codex_launch_path -v）

完了する

• PR を作成する
• レビューに対応する

[BlueEventHorizon]

トリアージ結果: 軽量実装

要件の確定性: 確定 — Issue 本文「期待動作」に「codex exec のレビューが数分かかっても Bash ツールのタイムアウトで kill されず、完了後に後工程へ進むこと」と具体的な受け入れ基準が明記されている。
設計判断の残余: なし — 原因は plugins/forge/agents/reviewer.md:158 の同期実行 MANDATORY 節に Bash ツールの timeout パラメータ指定（公式仕様上限 600000ms）が欠落している点であり、既存の公式ツールパラメータを追記するだけの修正（アーキテクチャ変更・契約変更・新概念導入なし）。
リスクゲート: 該当なし — (1) データ破壊/セキュリティ/配布物破損のリスクなし (2) 既存の静的回帰テスト test_codex_launch_path.py パターンで検知可能 (3) 実修正対象は plugins/forge/agents/reviewer.md 1 ファイル（+ テスト追加で最大2ファイル）。

補足: Issue 本文にある「タイムアウト発生時でも部分出力(.stdout)を破棄せず救済する」という副次要求は、Bash ツールのプロセス kill シグナル（SIGTERM/SIGKILL）挙動に技術的不確実性があるため、timeout パラメータ追記の主修正とは別に impl-issue 側で実現可能性を調査しスコープ判断すること（本トリアージでは強制的に SDD には倒さない）。

調査結果（impl-issue が引き継ぐ。4カテゴリとも省略しない）:

• 関連仕様書:
  • docs/specs/forge/design/DES-029_skill_agent_launch_contract_design.md (kind: local_path) — reviewer Agent 化・起動契約の根拠設計
  • docs/specs/forge/design/DES-015_review_workflow_design.md (kind: local_path)
  • docs/specs/forge/requirements/REQ-005_skill_agent_launch_contract.md (kind: local_path)
  • docs/specs/forge-msg/requirements/REQ-006_codex_review_messaging.md (kind: local_path)
  • docs/specs/forge/design/DES-028_review_policy_design.md (kind: local_path)
  • docs/specs/forge/design/ADR-033_structured_finding_output.md (kind: local_path)
  • いずれも Bash ツールの timeout パラメータには言及なし（本 Issue の原因が既存仕様の欠落であることを裏付け）
• ルール文書:
  • docs/rules/implementation_guidelines.md (kind: local_path)
  • docs/rules/skill_authoring_notes.md (kind: local_path)
  • docs/rules/skill_launch_paths_definitions.md (kind: local_path)
  • docs/rules/document_writing_rules.md (kind: local_path)
• 類似 PR:
  • [Fix] forge:reviewer Agent が run_review_engine.sh を同期実行するよう明記 #206（Issue [Bug] forge:reviewer Agent が run_review_engine.sh をバックグラウンド実行し完了を待たずに完了報告する #200 修正: reviewer.md に同期実行 MANDATORY 節を追記。今回の直接の原因コミット b950364）
  • [Fix] extract_review_findings が規定外セクションを幻の finding として plan.yaml に登録する問題を修正 (#151) #152（extract_review_findings のパーサー挙動修正。reviewer パイプライン単体ファイル修正の類似スコープ）
  • コミット 8a00b68 / 4382cca（run_review_engine.sh 自体への過去の単発修正。PR番号なしで直接マージされた同スコープの修正パターン）
• 既存コード:
  • plugins/forge/agents/reviewer.md:158（修正対象。同期実行 MANDATORY 節に timeout パラメータ指定を追記する）
  • plugins/forge/skills/review/scripts/run_review_engine.sh（rescue コードは L64-69。スクリプト自体が kill されると到達不能という Issue の指摘を裏付け。今回の主修正では変更しない）
  • tests/forge/review/test_codex_launch_path.py（reviewer.md の記述内容を静的検証する既存回帰テストパターン。timeout 記載の検証テスト追加に流用可能）