v1.1.3
CI caught another real bug, this time on ubuntu-latest · node 20. The
v1.1.2 timeout fix unblocked Windows; v1.1.3 fixes the test-file ordering
race that was hidden until the matrix finally had room to expose it.
Fixed
-
tests/cli.test.tsreturned exit 1 instead of 2 onubuntu-latest · node 20. Root cause:tests/cli.test.tsassumeddist/cli/cli.js
was already built (it didspawnSync(node, [CLI, ...])without first
running the build), so it was racing against
tests/integration.server.test.ts'sbeforeAll(execSync('npm run build')). On 5 of 6 matrix cells the Vitest parallel worker picked
the integration file first; on ubuntu + node 20 it picked the CLI
file first, the bin didn't exist,spawnSyncreturned 1 (Cannot
find module), and the "unknown command exits 2" assertion failed.Fix:
tests/cli.test.tsnow has its ownbeforeAllthat runsnpm run buildifdist/cli/cli.jsis missing — same pattern
integration.server.test.tsuses. Test-file order is now irrelevant.
Notes
- 193/193 tests still pass. No change to MCP tools, CLI behaviour,
schemas, or any stable surface. Test infrastructure only — this
bug was always there, just deterministically masked by parallel
file order on every host except one. - This is the second real bug the CI matrix has caught in 24h.
ADR-0017's premise (cross-platform CI catches deterministic
regressions that local dev can't) keeps holding.