@vellumai/assistant 0.5.15 → 0.5.16

package/src/__tests__/skills.test.ts

@@ -10,37 +10,7 @@ import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 
-const TEST_DIR = join(tmpdir(), `vellum-skills-test-${crypto.randomUUID()}`);
-
-// eslint-disable-next-line @typescript-eslint/no-require-imports
-const realPlatform = require("../util/platform.js");
-mock.module("../util/platform.js", () => ({
-  ...realPlatform,
-  getProtectedDir: () => join(TEST_DIR, "protected"),
-  getDataDir: () => TEST_DIR,
-
-  getSandboxRootDir: () => join(TEST_DIR, "sandbox"),
-  getSandboxWorkingDir: () => TEST_DIR,
-  getInterfacesDir: () => join(TEST_DIR, "interfaces"),
-  ensureDataDir: () => {},
-  getPidPath: () => join(TEST_DIR, "vellum.pid"),
-  getDbPath: () => join(TEST_DIR, "data", "assistant.db"),
-  getLogPath: () => join(TEST_DIR, "logs", "vellum.log"),
-  getHistoryPath: () => join(TEST_DIR, "history"),
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPlatformName: () => process.platform,
-  getClipboardCommand: () => null,
-  getWorkspaceConfigPath: () => join(TEST_DIR, "config.json"),
-  getWorkspaceSkillsDir: () => join(TEST_DIR, "skills"),
-  getWorkspaceHooksDir: () => join(TEST_DIR, "hooks"),
-  getHooksDir: () => join(TEST_DIR, "hooks"),
-  getWorkspaceDir: () => TEST_DIR,
-  getWorkspacePromptPath: (file: string) => join(TEST_DIR, file),
-  readSessionToken: () => null,
-  normalizeAssistantId: (id: string) => id,
-}));
+const TEST_DIR = process.env.VELLUM_WORKSPACE_DIR!;
 
 const noopLogger = {
   info: () => {},
@@ -91,9 +61,9 @@ describe("skills catalog loading", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
+    const skillsDir = join(TEST_DIR, "skills");
+    if (existsSync(skillsDir))
+      rmSync(skillsDir, { recursive: true, force: true });
   });
 
   test("parses markdown list path entries from SKILLS.md", () => {
@@ -221,9 +191,12 @@ describe("workspace skills", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
+    const skillsDir = join(TEST_DIR, "skills");
+    if (existsSync(skillsDir))
+      rmSync(skillsDir, { recursive: true, force: true });
+    const outsideDir = join(TEST_DIR, "outside");
+    if (existsSync(outsideDir))
+      rmSync(outsideDir, { recursive: true, force: true });
     if (existsSync(WORKSPACE_DIR)) {
       rmSync(WORKSPACE_DIR, { recursive: true, force: true });
     }
@@ -282,9 +255,9 @@ describe("tool manifest detection", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
+    const skillsDir = join(TEST_DIR, "skills");
+    if (existsSync(skillsDir))
+      rmSync(skillsDir, { recursive: true, force: true });
   });
 
   test("attaches toolManifest metadata when valid TOOLS.json is present", () => {
@@ -455,9 +428,9 @@ describe("includes frontmatter parsing", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
+    const skillsDir = join(TEST_DIR, "skills");
+    if (existsSync(skillsDir))
+      rmSync(skillsDir, { recursive: true, force: true });
   });
 
   function writeSkillWithIncludes(skillId: string, includes: string): void {
@@ -553,9 +526,9 @@ describe("bundled browser skill", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
+    const skillsDir = join(TEST_DIR, "skills");
+    if (existsSync(skillsDir))
+      rmSync(skillsDir, { recursive: true, force: true });
   });
 
   test("browser skill appears in full catalog (including bundled)", () => {
@@ -679,9 +652,9 @@ describe("bundled computer-use skill", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
+    const skillsDir = join(TEST_DIR, "skills");
+    if (existsSync(skillsDir))
+      rmSync(skillsDir, { recursive: true, force: true });
   });
 
   test("computer-use skill appears in full catalog (including bundled)", () => {

package/src/__tests__/slack-channel-config.test.ts

@@ -1,9 +1,8 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
+import { rmSync } from "node:fs";
 import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
 
-const testDir = mkdtempSync(join(tmpdir(), "slack-channel-cfg-test-"));
+const testDir = process.env.VELLUM_WORKSPACE_DIR!;
 const secureStorePath = join(testDir, "keys.enc");
 const metadataPath = join(testDir, "metadata.json");
 const originalVellumDev = process.env.VELLUM_DEV;
@@ -59,19 +58,6 @@ mock.module("../config/loader.js", () => ({
   setNestedValue,
 }));
 
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(testDir, "protected"),
-  getDataDir: () => testDir,
-
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () => ({
     info: () => {},
@@ -182,11 +168,6 @@ afterAll(() => {
   } else {
     process.env.VELLUM_DEV = originalVellumDev;
   }
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
 });
 
 describe("Slack channel config handler", () => {

package/src/__tests__/starter-bundle.test.ts

@@ -2,7 +2,7 @@ import { existsSync, mkdirSync, readFileSync, rmSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 
-// Stub the root dir before importing trust-store so it uses our temp directory
+// Set up a temp directory before importing trust-store
 const TEST_ROOT = join(
   import.meta.dirname ?? __dirname,
   "..",
@@ -11,23 +11,17 @@ const TEST_ROOT = join(
 );
 const TRUST_PATH = join(TEST_ROOT, "protected", "trust.json");
 
-// We need to mock getRootDir before importing trust-store
 import { mock } from "bun:test";
 
 // Point the file-based trust backend at the test temp dir.
 process.env.GATEWAY_SECURITY_DIR = join(TEST_ROOT, "protected");
 
-// Mock the platform module to use our test root
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(TEST_ROOT, "protected"),
-}));
-
 // Mock the skills config module used by defaults.ts
 mock.module("../config/skills.js", () => ({
   getBundledSkillsDir: () => join(TEST_ROOT, "bundled-skills"),
 }));
 
-// Now import trust-store (which uses the mocked platform)
+// Now import trust-store (which uses GATEWAY_SECURITY_DIR)
 import {
   acceptStarterBundle,
   clearCache,

package/src/__tests__/stt-hints.test.ts

@@ -64,7 +64,11 @@ mock.module("pino", () => ({ default: mockPinoLogger }));
 mock.module("pino-pretty", () => ({ default: () => ({}) }));
 
 // Import after mocking
-import { buildSttHints, resolveCallHints, type SttHintsInput } from "../calls/stt-hints.js";
+import {
+  buildSttHints,
+  resolveCallHints,
+  type SttHintsInput,
+} from "../calls/stt-hints.js";
 
 function emptyInput(): SttHintsInput {
   return {
@@ -159,7 +163,8 @@ describe("buildSttHints", () => {
 
   test("proper nouns extracted across sentence boundaries", () => {
     const input = emptyInput();
-    input.taskDescription = "Meet with Alice. Then call Bob! Ask Charlie? Done.";
+    input.taskDescription =
+      "Meet with Alice. Then call Bob! Ask Charlie? Done.";
     const result = buildSttHints(input);
     expect(result).toContain("Alice");
     expect(result).toContain("Bob");

package/src/__tests__/system-prompt.test.ts

@@ -5,45 +5,14 @@ import {
   rmSync,
   writeFileSync,
 } from "node:fs";
-import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 
 // Mock platform to use a temp directory
-const TEST_DIR = join(tmpdir(), `vellum-sysprompt-test-${crypto.randomUUID()}`);
+const TEST_DIR = process.env.VELLUM_WORKSPACE_DIR!;
 
 import { mock } from "bun:test";
 
-// eslint-disable-next-line @typescript-eslint/no-require-imports
-const realPlatform = require("../util/platform.js");
-mock.module("../util/platform.js", () => ({
-  ...realPlatform,
-  getProtectedDir: () => join(TEST_DIR, "protected"),
-  getDataDir: () => TEST_DIR,
-  getWorkspaceDir: () => TEST_DIR,
-  getWorkspaceConfigPath: () => join(TEST_DIR, "config.json"),
-  getWorkspaceSkillsDir: () => join(TEST_DIR, "skills"),
-  getWorkspaceHooksDir: () => join(TEST_DIR, "hooks"),
-  getConversationsDir: () => join(TEST_DIR, "conversations"),
-  getWorkspacePromptPath: (file: string) => join(TEST_DIR, file),
-  ensureDataDir: () => {},
-  getPidPath: () => join(TEST_DIR, "vellum.pid"),
-  getDbPath: () => join(TEST_DIR, "data", "assistant.db"),
-  getLogPath: () => join(TEST_DIR, "logs", "vellum.log"),
-  getHistoryPath: () => join(TEST_DIR, "history"),
-  getHooksDir: () => join(TEST_DIR, "hooks"),
-
-  getSandboxRootDir: () => join(TEST_DIR, "sandbox"),
-  getSandboxWorkingDir: () => TEST_DIR,
-  getInterfacesDir: () => join(TEST_DIR, "interfaces"),
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPlatformName: () => process.platform,
-  getClipboardCommand: () => null,
-  readSessionToken: () => null,
-}));
-
 const noopLogger: Record<string, unknown> = new Proxy(
   {} as Record<string, unknown>,
   {
@@ -123,7 +92,6 @@ function basePrompt(result: string): string {
   for (const heading of [
     "## Configuration",
     "## Skills Catalog",
-    "## Available Skills",
     "## External Communications Identity",
     "## Connected Services",
     "## Dynamic Skill Authoring Workflow",
@@ -144,8 +112,16 @@ describe("buildSystemPrompt", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
+    for (const name of [
+      "IDENTITY.md",
+      "SOUL.md",
+      "USER.md",
+      "BOOTSTRAP.md",
+      "UPDATES.md",
+      "skills",
+    ]) {
+      const p = join(TEST_DIR, name);
+      if (existsSync(p)) rmSync(p, { recursive: true, force: true });
     }
   });
 
@@ -196,7 +172,7 @@ describe("buildSystemPrompt", () => {
     expect(basePrompt(result)).toBe("Be kind");
   });
 
-  test("appends skills catalog when skills are configured", () => {
+  test("does not include skills catalog in system prompt", () => {
     const skillsDir = join(TEST_DIR, "skills");
     mkdirSync(join(skillsDir, "release-checklist"), { recursive: true });
     writeFileSync(
@@ -208,11 +184,11 @@ describe("buildSystemPrompt", () => {
     writeFileSync(join(TEST_DIR, "IDENTITY.md"), "Custom identity");
     const result = buildSystemPrompt();
     expect(result).toContain("Custom identity");
-    expect(result).toContain("## Available Skills");
-    expect(result).toContain("**release-checklist**: Deployment checks");
+    expect(result).not.toContain("## Available Skills");
+    expect(result).not.toContain("**release-checklist**");
   });
 
-  test("keeps SOUL.md and IDENTITY.md additive with skills", () => {
+  test("keeps SOUL.md and IDENTITY.md additive without skills catalog", () => {
     const skillsDir = join(TEST_DIR, "skills");
     mkdirSync(join(skillsDir, "incident-response"), { recursive: true });
     writeFileSync(
@@ -225,10 +201,7 @@ describe("buildSystemPrompt", () => {
 
     const result = buildSystemPrompt();
     expect(result).toContain("Identity content\n\nSoul content");
-    expect(result).toContain("## Available Skills");
-    expect(result.indexOf("Soul content")).toBeLessThan(
-      result.indexOf("## Available Skills"),
-    );
+    expect(result).not.toContain("## Available Skills");
   });
 
   test("includes external service access section", () => {
@@ -492,8 +465,15 @@ describe("ensurePromptFiles", () => {
   });
 
   afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
+    for (const name of [
+      "IDENTITY.md",
+      "SOUL.md",
+      "USER.md",
+      "BOOTSTRAP.md",
+      "conversations",
+    ]) {
+      const p = join(TEST_DIR, name);
+      if (existsSync(p)) rmSync(p, { recursive: true, force: true });
     }
   });
 

package/src/__tests__/task-compiler.test.ts

@@ -1,22 +1,6 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, test } from "bun:test";
 import { mock } from "bun:test";
 
-const testDir = mkdtempSync(join(tmpdir(), "task-compiler-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -49,11 +33,6 @@ initializeDb();
 
 afterAll(() => {
   resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
 });
 
 // ── Helpers ──────────────────────────────────────────────────────────

package/src/__tests__/task-management-tools.test.ts

@@ -1,21 +1,5 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
 
-const testDir = mkdtempSync(join(tmpdir(), "task-mgmt-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -76,11 +60,6 @@ initializeDb();
 
 afterAll(() => {
   resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
 });
 
 const ctx: ToolContext = {

package/src/__tests__/task-memory-cleanup.test.ts

@@ -1,6 +1,3 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import {
   afterAll,
   beforeAll,
@@ -15,19 +12,6 @@ import { eq } from "drizzle-orm";
 
 import { DEFAULT_CONFIG } from "../config/defaults.js";
 
-const testDir = mkdtempSync(join(tmpdir(), "task-memory-cleanup-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -105,11 +89,6 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
 
   afterAll(() => {
     resetDb();
-    try {
-      rmSync(testDir, { recursive: true });
-    } catch {
-      // best effort
-    }
   });
 
   function seedConversations() {

package/src/__tests__/task-runner.test.ts

@@ -1,21 +1,5 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
 
-const testDir = mkdtempSync(join(tmpdir(), "task-runner-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -32,11 +16,6 @@ initializeDb();
 
 afterAll(() => {
   resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
 });
 
 // ── renderTemplate ──────────────────────────────────────────────────

package/src/__tests__/task-scheduler.test.ts

@@ -1,21 +1,5 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
 
-const testDir = mkdtempSync(join(tmpdir(), "task-scheduler-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -51,11 +35,6 @@ function forceScheduleDue(scheduleId: string): void {
 
 afterAll(() => {
   resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
 });
 
 // ── scheduleTask helper ─────────────────────────────────────────────

package/src/__tests__/terminal-tools.test.ts

@@ -1,6 +1,3 @@
-import { mkdtempSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 
 import type { ShellOutputResult } from "../tools/shared/shell-output.js";
@@ -16,20 +13,7 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-const testTmpDir = mkdtempSync(join(tmpdir(), "terminal-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(testTmpDir, "protected"),
-  getDataDir: () => join(testTmpDir, "data"),
-  getSandboxWorkingDir: () => join(testTmpDir, "sandbox"),
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testTmpDir, "test.pid"),
-  getDbPath: () => join(testTmpDir, "test.db"),
-  getLogPath: () => join(testTmpDir, "test.log"),
-  ensureDataDir: () => {},
-}));
+const testTmpDir = process.env.VELLUM_WORKSPACE_DIR!;
 
 mock.module("../config/loader.js", () => ({
   getConfig: () => ({

package/src/__tests__/token-estimator-accuracy.benchmark.test.ts

@@ -216,85 +216,6 @@ function makeSystemPrompt(size: "small" | "production" = "small"): string {
     "Each MCP server entry requires: name, command, args, and optional env.",
   );
 
-  // Dynamic skills catalog (~5K chars)
-  sections.push("", "## Available Skills", "<available_skills>");
-  const skillCategories = [
-    {
-      id: "gmail",
-      name: "Gmail",
-      desc: "Send, search, draft, and manage Gmail messages",
-    },
-    {
-      id: "calendar",
-      name: "Google Calendar",
-      desc: "Create, list, update, and delete calendar events",
-    },
-    {
-      id: "slack",
-      name: "Slack",
-      desc: "Send messages, search channels, manage threads",
-    },
-    { id: "contacts", name: "Contacts", desc: "Search and manage contacts" },
-    {
-      id: "tasks",
-      name: "Tasks",
-      desc: "Create, list, update, and complete tasks",
-    },
-    {
-      id: "browser",
-      name: "Browser",
-      desc: "Navigate web pages, take screenshots, interact with web content",
-    },
-    {
-      id: "schedule",
-      name: "Schedule",
-      desc: "Set reminders and schedule recurring tasks",
-    },
-    {
-      id: "messaging",
-      name: "Messaging",
-      desc: "Send iMessage and SMS messages",
-    },
-    {
-      id: "sequences",
-      name: "Sequences",
-      desc: "Create and manage multi-step automation workflows",
-    },
-    {
-      id: "playbooks",
-      name: "Playbooks",
-      desc: "Execute pre-defined operational playbooks",
-    },
-    {
-      id: "notes",
-      name: "Notes",
-      desc: "Create and manage notes in Apple Notes",
-    },
-    { id: "music", name: "Music", desc: "Control Apple Music playback" },
-    {
-      id: "photos",
-      name: "Photos",
-      desc: "Search and manage photos in Apple Photos",
-    },
-    {
-      id: "maps",
-      name: "Maps",
-      desc: "Search locations, get directions, find nearby places",
-    },
-    {
-      id: "weather",
-      name: "Weather",
-      desc: "Get current weather and forecasts",
-    },
-  ];
-  for (const skill of skillCategories) {
-    sections.push(
-      `  <skill id="${skill.id}" name="${skill.name}" description="${skill.desc}" ` +
-        `credential_setup="oauth" enabled="true" />`,
-    );
-  }
-  sections.push("</available_skills>");
-
   // Attachment handling (~1K chars)
   sections.push(
     "",

package/src/__tests__/tool-approval-handler.test.ts

@@ -1,20 +1,6 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
 import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
 
-const testDir = mkdtempSync(join(tmpdir(), "tool-approval-handler-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
+const testDir = process.env.VELLUM_WORKSPACE_DIR!;
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -72,11 +58,6 @@ function clearTables(): void {
 
 afterAll(() => {
   resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
 });
 
 // ---------------------------------------------------------------------------