engineering-conventions

Wrap every awaited operation on the init path in withTimeout and degrade non-fatally on timeout, keeping total latency under the ~400ms repro-704 deadline. Defer non-critical I/O to the post-resolution task queue rather than queueMicrotask, and verify with node scripts/repro-704.mjs.

Every bunSpawn call must pass cwd, stdin: 'ignore', a timeout in milliseconds, bounded stdio, and call proc.kill() in a finally block. An outer withTimeout alone is insufficient because it lets the awaiter proceed without aborting the child process.

Unescaped backticks inside TypeScript template literals silently terminate the string, producing parse errors at seemingly unrelated lines. Escape backticks as \` and ${ as \${, then run bun run build immediately to catch the error.

No, mock.module leaks across files in Bun's shared test-runner process. Prefer _test_exports for pure functions or _internals dependency-injection seams, reserving mock.module for unavoidable cases with restoration in afterEach.

Capture the baseline before any code changes in a phase using capture_baseline: true with a phase parameter, so subsequent scans only fail on new findings. Capturing after edits silently encodes new bugs as pre-existing and defeats the gate.

The test_runner tool is limited to 50 files (MAX_SAFE_TEST_FILES) and is meant for targeted validation with explicit files lists, not full-repo runs. For whole-repo validation, run the shell commands in contributing.md or TESTING.md directly.