Skip to content

Test-only: fix two flakes that ejected #961's merge group - #1019

Merged
ualtinok merged 1 commit into
mainfrom
land/test-subprocess-waits
Oct 27, 2026
Merged

ualtinok merged 1 commit into
mainfrom
land/test-subprocess-waits

Conversation

@aft-alfonso

@aft-alfonso aft-alfonso Bot commented Oct 27, 2026

Copy link
Copy Markdown

Test-only, no product change and no release note.

  • packages/pi-plugin headless-exit.test.ts: the six 'approved session UI binding' tests each spawn a harness process with a 15 s rescue kill, but ran under bun's 5 s default timeout. On a loaded macOS runner 'factory without UI delivers the actual plugin version through session context' timed out at 5001 ms (run 40422064215, job 121918648910). Each now has a 30 s budget, above the rescue kill.
  • packages/opencode-plugin e2e outside-permissions.test.ts: stdoutPath walks the whole engine storage root to find a task's stdout file while the engine's index builder creates and removes search-build.tmp folders there; the walk listed one and failed with ENOENT reading it (job 121918649129). A folder that vanishes between listing and reading is now skipped.

…l; tolerate vanished folders in the outside-permissions storage walk

Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
@ualtinok
ualtinok added this pull request to the merge queue Oct 27, 2026
@ualtinok
ualtinok removed this pull request from the merge queue due to a manual request Oct 27, 2026
@ualtinok
ualtinok added this pull request to the merge queue Oct 27, 2026
Merged via the queue into main with commit 62e5943 Oct 27, 2026
48 checks passed
@ualtinok
ualtinok deleted the land/test-subprocess-waits branch October 27, 2026 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant