Skip to content

test(lite): stabilize Windows subprocess exit waits - #678

Merged
zuohuadong merged 7 commits into
mainfrom
codex/fix-lite-cli-top-level-await
Aug 2, 2026
Merged

test(lite): stabilize Windows subprocess exit waits#678
zuohuadong merged 7 commits into
mainfrom
codex/fix-lite-cli-top-level-await

Conversation

@zuohuadong

@zuohuadong zuohuadong commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • keep Bun Windows subprocess exit waits alive on the supported stable runtime
  • reuse one scoped compatibility helper across CLI, package, standalone, and snapshot smoke processes
  • verify snapshot lock release and an immediate second status invocation

Root cause

On Bun 1.3.14 for Windows, a process waiting only on Bun.Subprocess.exited can stop polling the Windows IOCP completion path before the child exit is observed (oven-sh/bun#34478). The child has already completed its work, but the harness can remain pending.

Fix

The smoke/test wait sites keep one interval reference while awaiting the subprocess result, then clear it in finally. This is test and verification infrastructure only; it does not change the Lite production CLI, dependencies, workflow timeouts, or exit semantics.

Verification

  • bun run check — 88 tests passed, build, package smoke, standalone smoke
  • bun x tsc --noEmit -p tsconfig.json
  • git diff --check
  • Windows Lite CI passed on Bun 1.3.14 (run 30751644544)

Ready for merge after final metadata review and independent verification.

@zuohuadong zuohuadong changed the title fix(lite): await CLI lifecycle at module boundary test(lite): stabilize Windows subprocess exit waits Aug 2, 2026
@zuohuadong
zuohuadong marked this pull request as ready for review August 2, 2026 14:22
@zuohuadong
zuohuadong merged commit eb60818 into main Aug 2, 2026
10 checks passed
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