From 2780dfb5d02f0b18ae0f4a0445c4e5179eaa5067 Mon Sep 17 00:00:00 2001
From: Tobi Lutke <tobi@shopify.com>
Date: Sun, 15 Feb 2026 21:59:18 -0400
Subject: [PATCH] fix: increase bun test timeout to 30s via CLI flag

The default 5s timeout is too short for CLI subprocess tests in CI.
---
 .github/workflows/ci.yml | 2 +-
 src/test-preload.ts      | 5 +----
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 311159e..8bf578e 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -64,7 +64,7 @@ jobs:
       - run: bun install
 
       - name: Tests
-        run: bun test --preload ./src/test-preload.ts test/
+        run: bun test --timeout 30000 --preload ./src/test-preload.ts test/
         env:
           CI: true
           DYLD_LIBRARY_PATH: /opt/homebrew/opt/sqlite/lib
diff --git a/src/test-preload.ts b/src/test-preload.ts
index 36535ec..afbd81f 100644
--- a/src/test-preload.ts
+++ b/src/test-preload.ts
@@ -4,12 +4,9 @@
  * Uses bun:test afterAll to properly dispose of llama.cpp Metal
  * resources before the process exits, avoiding GGML_ASSERT failures.
  */
-import { afterAll, setDefaultTimeout } from "bun:test";
+import { afterAll } from "bun:test";
 import { disposeDefaultLlamaCpp } from "./llm";
 
-// Match vitest's testTimeout (default 5s is too short for CLI subprocess tests)
-setDefaultTimeout(30_000);
-
 // Global afterAll runs after all test files complete
 afterAll(async () => {
   await disposeDefaultLlamaCpp();