From 3915dfda3a8087bf8671d3d58b8e0a976b05182e Mon Sep 17 00:00:00 2001 From: Mikael Hugo Date: Sat, 2 May 2026 11:53:31 +0200 Subject: [PATCH] =?UTF-8?q?chore(vitest):=20bump=20testTimeout=2030s?= =?UTF-8?q?=E2=86=9260s=20to=20absorb=20cold-import=20latency?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cold vitest+esbuild module-graph imports take 16-25s on this repo (dynamic imports of captures.js and friends). The 30s testTimeout was racing the import phase, producing 30s spurious failures across dev-engine-wrapper, ensure-db-open, workflow-mcp, sf-tools, verification-gate, hook-key-parsing, visualizer-overlay, and others — all timing out at exactly ~30s with no real assertion failure. Also bumps hookTimeout symmetrically. Re-running the affected files: 147/147 pass. Co-Authored-By: Claude Opus 4.7 (1M context) --- vitest.config.ts | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/vitest.config.ts b/vitest.config.ts index 88ecb02e0..160dc4ba2 100644 --- a/vitest.config.ts +++ b/vitest.config.ts @@ -166,8 +166,13 @@ export default defineConfig({ ], // ── Timeouts ────────────────────────────────────────────────────────────── - testTimeout: 30_000, - hookTimeout: 30_000, + // Cold vitest+esbuild module-graph import takes 16-25s on this repo + // (dynamic imports of captures.js and friends). 30s tests-timeout was + // racing the import phase, producing spurious 30s failures across + // dev-engine, ensure-db-open, workflow-mcp, sf-tools, verification-gate, + // etc. 60s gives the import time to settle without losing real-bug signal. + testTimeout: 60_000, + hookTimeout: 60_000, // ── Pool: forks = one Node process per test file (best for Node.js tests) ─ pool: "forks",