-
Notifications
You must be signed in to change notification settings - Fork 7
[Task] Windows advisory: session loop tests time out at 3000ms #555
Copy link
Copy link
Closed
Labels
P3Low priorityLow priorityappApplication behavior and product flowsApplication behavior and product flowsflaky-testNon-deterministic test failureNon-deterministic test failureharnessModel harness, prompts, tool descriptions, and session mechanicsModel harness, prompts, tool descriptions, and session mechanicstaskNarrow execution, audit, spike, migration, tracking, or upstream follow-up workNarrow execution, audit, spike, migration, tracking, or upstream follow-up workwindowsWindows-specificWindows-specific
Metadata
Metadata
Assignees
Labels
P3Low priorityLow priorityappApplication behavior and product flowsApplication behavior and product flowsflaky-testNon-deterministic test failureNon-deterministic test failureharnessModel harness, prompts, tool descriptions, and session mechanicsModel harness, prompts, tool descriptions, and session mechanicstaskNarrow execution, audit, spike, migration, tracking, or upstream follow-up workNarrow execution, audit, spike, migration, tracking, or upstream follow-up workwindowsWindows-specificWindows-specific
Symptom
The Windows advisory workflow failed on dev sha
3c15b376bafter PR #552 merged.Run: https://github.com/Astro-Han/pawwork/actions/runs/25666988353
Failing job:
unit-windows-opencode-sessionFailing tests:
cancel interrupts loop and resolves with an assistant messagetimed out after 3000msconcurrent loop callers all receive same error resulttimed out after 3000msunit-windows-opencode-server-toolswas cancelled after the session shard failed.unit-windows-app,unit-windows-desktop, andunit-windows-opencode-config-projectpassed.Initial RCA
This does not appear related to PR #552. PR #552 only changed
packages/appchild-store/sidebar timestamp cache reads, while the failure is inpackages/opencodesession loop/cancel tests.This matches the same advisory-only Windows timing pattern as #546 and the earlier PR #543 fix: tests with a hard 3000ms timeout can exceed the budget on Windows runners even when the behavior is otherwise correct.
Candidate Fix
Use the established
slowIOTimeouttest constant for these Windows-sensitive session loop tests, following the PR #543 pattern.Priority
P3. Advisory-only test debt. macOS/Linux PR gates are unaffected.
Related