[business-logic] tool_call_budget: hard-stop path injects 'summarize' directive then immediately returns nil — directive never consumed in-run, subagent callers can't distinguish budget truncation from completion

[topcheer]

文件与行号

internal/agent/agent.go:4446-4463（budget stop 路径）vs :4466-4478（maxIter 路径）、internal/agent/tool_call_budget.go:132-136

问题（独立复核确认）

预算硬停路径注入 "Summarize what was accomplished..." user 消息到 contextManager 后立即 return nil 结束 run——该指令在本 run 内永远不被消费（循环已终止，无再调 LLM 的路径），tool_call_budget.go:132-136 的收敛文案在 stop=true 场景是死代码（80%/95% 分层警告正常生效）。且 return nil 与 maxIter 路径（emit 完整 Summary + StreamEventError，返回 err）语义不对称：

后果

• subagent 调用方（internal/acp/agent_loop.go 等 RunStreamWithContent 使用者）：nil = 正常完成，预算截断与真实完成不可区分，无 error 事件供上层重试/标记
• session resume：注入的总结指令持久化在 contextManager，下一轮用户消息时 LLM 可见——表现为下一轮突兀地要求总结已结束的 run，体验错位
• cron 路径无实际影响（纯 prompt 入队器，不判读 run 结果）

修复建议

stop 时要么允许一次收尾 LLM turn 再终止，要么 emit 预算摘要（类似 maxIter 路径的 runStats.Summary()）并返回哨兵 error。

严重程度

medium

[topcheer]

已在 commit 8a79655 修复，采用 issue 建议的第二种方案（与 maxIter 路径对称）：

• 硬停路径重构：不再向 contextManager 注入永不被消费的 'summarize' 指令，而是 finalize runStats → emit 完整 Summary（StreamEventText，含 '可以发送后续消息继续' 提示）→ 返回哨兵 error tool call budget (N calls) exhausted（StreamEventError）。subagent/ACP 调用方现在可以通过 error 语义区分预算截断与正常完成。
• 软警告保留：80%/95% 分层警告仍走 in-run 注入（此时循环继续，指令会被消费），行为不变。
• session resume 错位消除：不再有持久化的幽灵指令污染下一轮。

tool_call_budget.go:132-136 的死代码问题随硬停路径重构一并消解（stop=true 不再走收敛文案分支）。感谢报告。