fix(task-loop): cancel scheduler task on round timeout - #553
fix(task-loop): cancel scheduler task on round timeout#553openjiuwen-sync-bot[bot] wants to merge 1 commit into
Conversation
|
|
1 similar comment
|
|
|
head_sha: 变更摘要本 PR 修复了任务循环轮次超时后未真正取消调度任务的问题：此前 主要改动
|
|
head_sha: 代码审查✅ 未发现问题 |
A timed-out task-loop round only returned {"error": "completion_timeout"}
and left the scheduler task running in the background. The answer-stream
adapter drops bare error dicts, so callers saw an empty success while the
LLM work kept going.
- TaskLoopEventHandler tracks the registered scheduler task id per round
and, on timeout, requests a coordinator abort plus
task_scheduler.cancel_task; cleanup is bounded to 1.5s and spills to a
background done-callback so the timeout error still returns promptly.
- Timeout and submit-failure paths now return a standard
output/result_type/error dict so the output layer surfaces the error.
- TaskLoopController.submit_round uses the synchronous EventQueue publish
API so the completion timeout cannot start before the task id is
registered, while preserving event ordering and error wrapping.
- DeepAgentConfig/DeepAgentSpec accept completion_timeout=None to disable
the limit; NativeHarness keeps its non-terminating slow-round warning
contract (None disables the warning task).
Update API docs, S_12/S_18 specs, and add F_80 feature notes.
532c3bf to
202b87f
Compare
2 participants
Paired: GitHub #553 ↔ GitCode !2344
What type of PR is this?
/kind fix
Self-checklist:（请自检，在[ ]内打上x，我们将检视你的完成情况，否则会导致pr无法合入）
Linked Closing Issues: