fix(plan): 完成门只拦 executing plan,不拦 approved-only(修死锁)

[1-ztc]

Bug

完成门拦了 approved-但-未-start 的 plan,导致死锁崩 run。

死锁链:

plan approved (0/N, 未 start)
  → 完成门否决(approved 也算"未完成")
    → correction 让模型 plan_step skip 剩余步骤
      → skipStep: plan.status !== executing → 拒(step 必须 in_progress)
        → 模型重试 → tool-loop guard → run is_error:true 崩

发现方式: critic 真 A/B 裸跑时,模型 approve 了 plan 后按指令不执行就 end_turn,触发完成门,模型想 skip 兜底,skip 报错,崩 run。真实模型场景暴露,非单测能覆盖。

Fix

完成门只拦 status === executing,不拦 approved(plan-completion-gate.ts:51)。

理由: 完成门职责是"执行中没做完不许声称完成"。approved-未-start 是"计划已批、还没开始执行",这种状态下 end_turn 不是"执行到一半偷懒",是模型还没开始/选择不执行——完成门拦它越界。且拦了就死锁(skip 在非 executing 拒)。executing(已 start,0/N)仍照拦,不变。

测试

• Case 10(新增):approved-only plan → 放行(修复前红,修复后绿)
• 4 个回归 spec 全过(含 plan-completion-gate-integ 集成测试,它用 approve+start 进 executing,仍被拦,证明 executing 路径没受影响)
• build + typecheck 干净

关联

• 修的是 main 上完成门(PR feat(plan): 计划完成门 — 拦截未完成 plan 的提前 end_turn #19 已合)的缺陷,跟 critic(PR feat(plan): 接通 plan-critic 真实 subagent runner #20)无关。
• critic 裸跑还暴露另一发现:首 chunk 超时 45s 对 critic 子 agent 太短(子 agent 要 24-81s),但那是 PR feat(plan): 接通 plan-critic 真实 subagent runner #20 的事,本 PR 不处理。

🤖 Generated with Claude Code