Skip to content

test: stabilize Windows CI without longer timeouts - #50454

Merged
rekram1-node merged 1 commit into
v2from
windows-test-audit
Sep 21, 2026
Merged

rekram1-node merged 1 commit into
v2from
windows-test-audit

Conversation

@rekram1-node

Copy link
Copy Markdown
Collaborator

What

  • cap unit-test Turbo concurrency at 3 on the 4-vCPU CI runners
  • cancel service-smoke deadline timers when processes exit
  • re-signal the compiled plugin watcher with valid source while retaining the existing 10-second failure bound

Why

#50449 exposed two separate Windows CI problems that should not be addressed by increasing test timeouts.

Turbo defaults to 10 concurrent tasks. That oversubscribes the 4-vCPU runners while each Bun test process performs its own work, stretching codemode tests from milliseconds locally into multi-second CI runs and tripping both Bun's default timeout and an existing performance assertion.

The compiled-service smoke also failed twice on v2 after a single plugin creation event was coalesced. It only polled afterward, so waiting longer could not recover. Rewriting valid plugin source periodically preserves the native-watcher coverage and gives the watcher another event without extending the deadline.

The smoke test also left losing 10-second Bun.sleep timers alive after each process exited. Clearing those timers reduced a local smoke run from about 11.5 seconds to about 1.4 seconds.

No test timeout is increased and the codemode performance assertion remains unchanged.

Testing

  • bun test test/errors.test.ts test/limits.test.ts from packages/codemode
  • bun typecheck from packages/cli
  • compiled Bun service smoke, including 10 consecutive runs
  • forced watcher retry path with the initial plugin write omitted
  • compiled Node service smoke
  • bun run check

@rekram1-node
rekram1-node merged commit 1814dd9 into v2 Sep 21, 2026
10 checks passed
@rekram1-node
rekram1-node deleted the windows-test-audit branch September 21, 2026 23:30
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