-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: QA native stop recovery can outwait fixed turn timeout #99655
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PR
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PR
Type
Fields
Priority
None yet
Bug type
Behavior bug (incorrect QA result without a runtime crash)
Beta release blocker
No
Summary
qa/scenarios/channels/native-command-session-target.yamltreats the native/stopacknowledgement as if the aborted turn has already released its session lane. Under QA Smoke CI contention, recovery can wait behind abort cleanup longer than the scenario's fixed 15000ms outbound timeout even though/stoptargeted and aborted the correct conversation.Steps to reproduce
mock-openai/gpt-5.5in the QA Smoke CI profile./stop.Agent was aborted, followed immediately by the recovery inbound turn.Exact evidence:
/stopat 04:54:28.301, acknowledgement at 04:54:29.115, recovery at 04:54:29.301, then an 18578ms session-lane wait and final recovery timeout.Expected behavior
The scenario should keep the native command and abort acknowledgement assertions, then allow the recovery message one full QA turn budget, including time queued behind abort cleanup.
Actual behavior
sessions.list.hasActiveRuncan clear before the command lane is released, so it is not a valid idle signal. The QA flow has no gateway RPC exposing canonical session-lane idle state.Fix and validation
Use the existing
liveTurnTimeoutMs(env, 30000)full-turn timeout policy for the recovery outbound wait. Mock providers preserve the 30000ms fallback; live providers apply their model-specific floors. This adds no sleep and does not weaken the abort or recovery marker assertions.Validation for the narrowed patch:
tbx_01kwn0fe5vm4gevwq0c3vks9r9, exit 0.JSONL partial-tail parsing is unrelated and unchanged.
Impact and severity
Intermittent false-negative QA Smoke CI results for native
/stoprecovery proof. No production native-command regression is indicated.