refactor: migrate test runner from bun:test to vitest (WIP)

Phase 3 of Bun → Node.js migration. Migrates 327 test files from
bun:test to vitest.

Mechanical changes:
- bun:test imports → vitest imports (327 files)
- mock → vi, spyOn → vi.spyOn, mock.module → vi.mock (52 files)
- Bun.spawn/spawnSync → node:child_process in test files (12 files)
- Bun.file/write/sleep/gzip/etc → Node.js equivalents in tests (40 files)
- Bun.serve → node:http createServer in test mock server
- import.meta.dir → import.meta.dirname (16 files)
- BUN_TEST_WORKER_ID → VITEST_POOL_ID
- Added vitest.config.ts with setupFiles, pool: forks, 15s timeout
- Added custom toStartWith/toEndWith matchers in preload.ts
- Removed bun:sqlite fallback from sqlite.ts
- Updated package.json test scripts to use vitest

Known remaining issues (WIP):
- ESM spy limitations (Cannot spy on ESM namespace exports)
- Some vi import gaps in edge cases
- Test API signature changes (vitest 4)
- self is not defined in model-based tests

Author: BYK

.lore.md

@@ -48,6 +48,7 @@
     "typescript": "^5",
     "ultracite": "6.3.10",
     "uuidv7": "^1.1.0",
+    "vitest": "^4.1.7",
     "wrap-ansi": "^10.0.0",
     "zod": "^3.24.0"
   },
@@ -93,10 +94,10 @@
     "lint": "biome check --no-errors-on-unmatched --max-diagnostics=none ./",
     "lint:fix": "biome check --write --no-errors-on-unmatched --max-diagnostics=none ./",
     "test": "bun run test:unit",
-    "test:unit": "bun run generate:docs && bun run generate:sdk && bun test --timeout 15000 --isolate --parallel test/lib test/commands test/types --coverage --coverage-reporter=lcov",
-    "test:changed": "bun run generate:docs && bun run generate:sdk && bun test --timeout 15000 --isolate --changed",
-    "test:e2e": "bun run generate:docs && bun run generate:sdk && bun test --timeout 15000 test/e2e",
-    "test:init-eval": "bun test test/init-eval --timeout 600000 --concurrency 6",
+    "test:unit": "bun run generate:docs && bun run generate:sdk && vitest run test/lib test/commands test/types --coverage",
+    "test:changed": "bun run generate:docs && bun run generate:sdk && vitest run --changed",
+    "test:e2e": "bun run generate:docs && bun run generate:sdk && vitest run test/e2e",
+    "test:init-eval": "vitest run test/init-eval --testTimeout 600000",
     "generate:parser": "bun run script/generate-parser.ts",
     "generate:sdk": "bun run script/generate-sdk.ts",
     "generate:skill": "bun run script/generate-skill.ts",

package.json

@@ -5,9 +5,7 @@
  * codebase. It provides a `.query(sql).get()` / `.all()` / `.run()`
  * interface and a manual `transaction()` wrapper.
  *
- * Uses `node:sqlite` (Node 22+) as the backing implementation. Falls back
- * to `bun:sqlite` when `node:sqlite` is unavailable (Bun runtime) — this
- * fallback will be removed once the test runner migrates off Bun.
+ * Uses `node:sqlite` (Node 22+) as the backing implementation.
  */
 
 import { logger } from "../logger.js";
@@ -58,29 +56,8 @@ function wrapStatement(stmt: any): StatementWrapper {
   }) as StatementWrapper;
 }
 
-/**
- * Resolve the underlying SQLite database constructor.
- *
- * Prefers `node:sqlite` (Node 22+). Falls back to `bun:sqlite` when
- * `node:sqlite` is unavailable (Bun runtime). The fallback will be
- * removed once the test runner migrates off Bun.
- */
-function getSqliteConstructor(): new (
-  path: string
-) => {
-  exec(sql: string): void;
-  close(): void;
-} {
-  try {
-    return require("node:sqlite").DatabaseSync;
-  } catch (error) {
-    log.debug("node:sqlite unavailable, falling back to bun:sqlite", error);
-    return require("bun:sqlite").Database;
-  }
-}
-
-// biome-ignore lint/suspicious/noExplicitAny: resolved dynamically
-const SqliteImpl: any = getSqliteConstructor();
+// biome-ignore lint/suspicious/noExplicitAny: node:sqlite types loaded lazily
+const { DatabaseSync: SqliteImpl } = require("node:sqlite") as any;
 
 /**
  * SQLite database wrapper.

pnpm-lock.yaml

@@ -5,7 +5,6 @@
  * that are difficult to exhaustively test with example-based tests.
  */
 
-import { describe, expect, test } from "bun:test";
 import {
   array,
   asyncProperty,
@@ -21,6 +20,7 @@ import {
   tuple,
   uniqueArray,
 } from "fast-check";
+import { describe, expect, test } from "vitest";
 import {
   buildFromFields,
   extractJsonBody,

src/lib/db/sqlite.ts

@@ -5,8 +5,9 @@
  * Tests for parsing functions in the api command.
  */
 
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+import { writeFile } from "node:fs/promises";
 import { Readable } from "node:stream";
+import { afterEach, beforeEach, describe, expect, test } from "vitest";
 import {
   buildBodyFromFields,
   buildBodyFromInput,
@@ -1010,7 +1011,7 @@ describe("buildBodyFromInput", () => {
     );
     const testDir = await createTestConfigDir("test-api-file-");
     const tempFile = `${testDir}/test-input.json`;
-    await Bun.write(tempFile, JSON.stringify({ key: "value" }));
+    await writeFile(tempFile, JSON.stringify({ key: "value" }));
 
     try {
       const mockStdin = createMockStdin("");
@@ -1028,7 +1029,7 @@ describe("buildBodyFromInput", () => {
     );
     const testDir = await createTestConfigDir("test-api-file-");
     const tempFile = `${testDir}/test-input.txt`;
-    await Bun.write(tempFile, "plain text from file");
+    await writeFile(tempFile, "plain text from file");
 
     try {
       const mockStdin = createMockStdin("");

test/commands/api.property.test.ts

@@ -6,7 +6,7 @@
  * db/user, and interactive-login to cover all branches without real HTTP
  * calls or database access.
  *
- * The interactive TTY prompt tests use mock.module() at the top of this file
+ * The interactive TTY prompt tests use vi.mock() at the top of this file
  * to stub node:tty (so isatty(0) returns true) and the logger module (so
  * `.prompt()` is controllable).
  */
@@ -16,56 +16,63 @@ import {
   beforeEach,
   describe,
   expect,
-  mock,
-  spyOn,
   test,
-} from "bun:test";
+  vi,
+} from "vitest";
 
 // Mock isatty to simulate interactive terminal for the re-auth prompt path.
 // Bun's ESM wrapper for CJS built-ins exposes `default` + `ReadStream` +
 // `WriteStream` — all must be present.
-const mockIsatty = mock(() => false);
-class FakeReadStream {}
-class FakeWriteStream {}
-const ttyExports = {
-  isatty: mockIsatty,
-  ReadStream: FakeReadStream,
-  WriteStream: FakeWriteStream,
-};
-mock.module("node:tty", () => ({
+const { mockIsatty, FakeReadStream, FakeWriteStream, ttyExports, noop, mockPrompt, fakeLog } = vi.hoisted(() => {
+  const mockIsatty = vi.fn(() => false);
+  class FakeReadStream {}
+  class FakeWriteStream {}
+  const ttyExports = {
+    isatty: mockIsatty,
+    ReadStream: FakeReadStream,
+    WriteStream: FakeWriteStream,
+  };
+
+  /** No-op placeholder for unused logger methods. */
+  function noop() {
+    // intentional no-op
+  }
+
+  // Mock the logger module to intercept the .prompt() call made by the
+  // module-scoped `log = logger.withTag("auth.login")` in login.ts.
+  const mockPrompt = vi.fn(
+    (): Promise<boolean | symbol> => Promise.resolve(true)
+  );
+  const fakeLog: {
+    prompt: typeof mockPrompt;
+    info: ReturnType<typeof vi.fn>;
+    warn: ReturnType<typeof vi.fn>;
+    error: ReturnType<typeof vi.fn>;
+    debug: ReturnType<typeof vi.fn>;
+    success: ReturnType<typeof vi.fn>;
+    withTag: () => typeof fakeLog;
+  } = {
+    prompt: mockPrompt,
+    info: vi.fn(noop),
+    warn: vi.fn(noop),
+    error: vi.fn(noop),
+    debug: vi.fn(noop),
+    success: vi.fn(noop),
+    withTag: () => fakeLog,
+  };
+
+  return { mockIsatty, FakeReadStream, FakeWriteStream, ttyExports, noop, mockPrompt, fakeLog };
+});
+
+vi.mock("node:tty", () => ({
   ...ttyExports,
   default: ttyExports,
 }));
 
-/** No-op placeholder for unused logger methods. */
-function noop() {
-  // intentional no-op
-}
-
-// Mock the logger module to intercept the .prompt() call made by the
-// module-scoped `log = logger.withTag("auth.login")` in login.ts.
-const mockPrompt = mock((): Promise<boolean | symbol> => Promise.resolve(true));
-const fakeLog: {
-  prompt: typeof mockPrompt;
-  info: ReturnType<typeof mock>;
-  warn: ReturnType<typeof mock>;
-  error: ReturnType<typeof mock>;
-  debug: ReturnType<typeof mock>;
-  success: ReturnType<typeof mock>;
-  withTag: () => typeof fakeLog;
-} = {
-  prompt: mockPrompt,
-  info: mock(noop),
-  warn: mock(noop),
-  error: mock(noop),
-  debug: mock(noop),
-  success: mock(noop),
-  withTag: () => fakeLog,
-};
-mock.module("../../../src/lib/logger.js", () => ({
+vi.mock("../../../src/lib/logger.js", () => ({
   logger: fakeLog,
-  setLogLevel: mock(noop),
-  attachSentryReporter: mock(noop),
+  setLogLevel: vi.fn(noop),
+  attachSentryReporter: vi.fn(noop),
   LOG_LEVEL_NAMES: ["error", "warn", "log", "info", "debug", "trace"],
   LOG_LEVEL_ENV_VAR: "SENTRY_LOG_LEVEL",
   parseLogLevel: (name: string) => {
@@ -76,18 +83,34 @@ mock.module("../../../src/lib/logger.js", () => ({
   getEnvLogLevel: () => null,
 }));
 
-// Dynamic import: must run AFTER mock.module() so login.ts picks up fakeLog.
+// Dynamic import: must run AFTER vi.mock() so login.ts picks up fakeLog.
 const { loginCommand, rcTokenHint } = await import(
   "../../../src/commands/auth/login.js"
 );
 
+vi.mock("../../../src/lib/api-client.js", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../../../src/lib/api-client.js")>();
+  return Object.fromEntries(Object.entries(actual).map(([k, v]) => [k, typeof v === "function" ? vi.fn(v) : v]));
+});
 // biome-ignore lint/performance/noNamespaceImport: needed for spyOn mocking
 import * as apiClient from "../../../src/lib/api-client.js";
+vi.mock("../../../src/lib/db/auth.js", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../../../src/lib/db/auth.js")>();
+  return Object.fromEntries(Object.entries(actual).map(([k, v]) => [k, typeof v === "function" ? vi.fn(v) : v]));
+});
 // biome-ignore lint/performance/noNamespaceImport: needed for spyOn mocking
 import * as dbAuth from "../../../src/lib/db/auth.js";
+vi.mock("../../../src/lib/db/user.js", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../../../src/lib/db/user.js")>();
+  return Object.fromEntries(Object.entries(actual).map(([k, v]) => [k, typeof v === "function" ? vi.fn(v) : v]));
+});
 // biome-ignore lint/performance/noNamespaceImport: needed for spyOn mocking
 import * as dbUser from "../../../src/lib/db/user.js";
 import { AuthError } from "../../../src/lib/errors.js";
+vi.mock("../../../src/lib/interactive-login.js", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../../../src/lib/interactive-login.js")>();
+  return Object.fromEntries(Object.entries(actual).map(([k, v]) => [k, typeof v === "function" ? vi.fn(v) : v]));
+});
 // biome-ignore lint/performance/noNamespaceImport: needed for spyOn mocking
 import * as interactiveLogin from "../../../src/lib/interactive-login.js";
 import type { SentryCliRcConfig } from "../../../src/lib/sentryclirc.js";
@@ -121,12 +144,12 @@ function createContext() {
   const stdoutChunks: string[] = [];
   const context = {
     stdout: {
-      write: mock((s: string) => {
+      write: vi.fn((s: string) => {
         stdoutChunks.push(s);
       }),
     },
     stderr: {
-      write: mock((_s: string) => {
+      write: vi.fn((_s: string) => {
         // unused — diagnostics go through logger
       }),
     },
@@ -165,19 +188,19 @@ describe("loginCommand.func --token path", () => {
   let func: LoginFunc;
 
   beforeEach(async () => {
-    isAuthenticatedSpy = spyOn(dbAuth, "isAuthenticated");
-    isEnvTokenActiveSpy = spyOn(dbAuth, "isEnvTokenActive");
-    setAuthTokenSpy = spyOn(dbAuth, "setAuthToken");
-    getUserRegionsSpy = spyOn(apiClient, "getUserRegions");
-    clearAuthSpy = spyOn(dbAuth, "clearAuth");
-    getCurrentUserSpy = spyOn(apiClient, "getCurrentUser");
-    setUserInfoSpy = spyOn(dbUser, "setUserInfo");
-    runInteractiveLoginSpy = spyOn(interactiveLogin, "runInteractiveLogin");
-    hasStoredAuthCredentialsSpy = spyOn(dbAuth, "hasStoredAuthCredentials");
+    isAuthenticatedSpy = vi.spyOn(dbAuth, "isAuthenticated");
+    isEnvTokenActiveSpy = vi.spyOn(dbAuth, "isEnvTokenActive");
+    setAuthTokenSpy = vi.spyOn(dbAuth, "setAuthToken");
+    getUserRegionsSpy = vi.spyOn(apiClient, "getUserRegions");
+    clearAuthSpy = vi.spyOn(dbAuth, "clearAuth");
+    getCurrentUserSpy = vi.spyOn(apiClient, "getCurrentUser");
+    setUserInfoSpy = vi.spyOn(dbUser, "setUserInfo");
+    runInteractiveLoginSpy = vi.spyOn(interactiveLogin, "runInteractiveLogin");
+    hasStoredAuthCredentialsSpy = vi.spyOn(dbAuth, "hasStoredAuthCredentials");
     // Prevent warmOrgCache() fire-and-forget from hitting real fetch.
     // After successful login, warmOrgCache() calls listOrganizationsUncached()
     // which triggers API calls that leak as "unexpected fetch" warnings.
-    listOrgsUncachedSpy = spyOn(apiClient, "listOrganizationsUncached");
+    listOrgsUncachedSpy = vi.spyOn(apiClient, "listOrganizationsUncached");
     listOrgsUncachedSpy.mockResolvedValue([]);
     isEnvTokenActiveSpy.mockReturnValue(false);
     hasStoredAuthCredentialsSpy.mockReturnValue(false);
@@ -429,7 +452,7 @@ describe("loginCommand.func --token path", () => {
 /**
  * Tests for the interactive TTY re-authentication prompt.
  *
- * Uses the module-level `mock.module()` on node:tty (so `isatty(0)` returns
+ * Uses the module-level `vi.mock()` on node:tty (so `isatty(0)` returns
  * true) and the logger (so `.prompt()` is controllable).
  */
 describe("login re-authentication interactive prompt", () => {
@@ -443,20 +466,20 @@ describe("login re-authentication interactive prompt", () => {
 
   function createPromptContext() {
     return {
-      stdout: { write: mock(() => true) },
-      stderr: { write: mock(() => true) },
+      stdout: { write: vi.fn(() => true) },
+      stderr: { write: vi.fn(() => true) },
       cwd: "/tmp",
     };
   }
 
   beforeEach(async () => {
-    isAuthenticatedSpy = spyOn(dbAuth, "isAuthenticated");
-    isEnvTokenActiveSpy = spyOn(dbAuth, "isEnvTokenActive");
-    clearAuthSpy = spyOn(dbAuth, "clearAuth");
-    runInteractiveLoginSpy = spyOn(interactiveLogin, "runInteractiveLogin");
-    getUserInfoSpy = spyOn(dbUser, "getUserInfo");
+    isAuthenticatedSpy = vi.spyOn(dbAuth, "isAuthenticated");
+    isEnvTokenActiveSpy = vi.spyOn(dbAuth, "isEnvTokenActive");
+    clearAuthSpy = vi.spyOn(dbAuth, "clearAuth");
+    runInteractiveLoginSpy = vi.spyOn(interactiveLogin, "runInteractiveLogin");
+    getUserInfoSpy = vi.spyOn(dbUser, "getUserInfo");
     // Prevent warmOrgCache() fire-and-forget from hitting real fetch.
-    listOrgsUncachedSpy = spyOn(apiClient, "listOrganizationsUncached");
+    listOrgsUncachedSpy = vi.spyOn(apiClient, "listOrganizationsUncached");
     listOrgsUncachedSpy.mockResolvedValue([]);
 
     // Defaults
@@ -567,18 +590,18 @@ describe("login re-authentication interactive prompt", () => {
     getUserInfoSpy.mockReturnValue(undefined);
     mockPrompt.mockResolvedValue(true);
 
-    const setAuthTokenSpy = spyOn(dbAuth, "setAuthToken");
+    const setAuthTokenSpy = vi.spyOn(dbAuth, "setAuthToken");
     setAuthTokenSpy.mockImplementation(noop);
-    const getUserRegionsSpy = spyOn(apiClient, "getUserRegions");
+    const getUserRegionsSpy = vi.spyOn(apiClient, "getUserRegions");
     getUserRegionsSpy.mockResolvedValue([]);
-    const getCurrentUserSpy = spyOn(apiClient, "getCurrentUser");
+    const getCurrentUserSpy = vi.spyOn(apiClient, "getCurrentUser");
     getCurrentUserSpy.mockResolvedValue({
       id: "42",
       name: "Jane",
       username: "jane",
       email: "jane@example.com",
     });
-    const setUserInfoSpy = spyOn(dbUser, "setUserInfo");
+    const setUserInfoSpy = vi.spyOn(dbUser, "setUserInfo");
     setUserInfoSpy.mockReturnValue(undefined);
 
     const context = createPromptContext();