@vellumai/assistant 0.5.15 → 0.6.0

package/src/__tests__/skills-uninstall.test.ts

@@ -60,7 +60,7 @@ describe("assistant skills uninstall", () => {
 
     // GIVEN a skill is installed locally
     installFakeSkill("weather");
-    writeSkillsIndex("- weather\n- google-oauth-app-setup\n");
+    writeSkillsIndex("- weather\n- vellum-self-knowledge\n");
 
     // WHEN we uninstall the skill
     uninstallSkillLocally("weather");
@@ -73,7 +73,7 @@ describe("assistant skills uninstall", () => {
     expect(index).not.toContain("weather");
 
     // AND other skills should remain in the index
-    expect(index).toContain("google-oauth-app-setup");
+    expect(index).toContain("vellum-self-knowledge");
   });
 
   test("errors when skill is not installed", () => {

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)", () => {
@@ -669,7 +642,7 @@ describe("ingress-dependent setup skills declare public-ingress intentionally",
       "utf-8",
     );
     expect(content).toContain("avoid-when:");
-    expect(content.toLowerCase()).toContain("managed/containerized");
+    expect(content.toLowerCase()).toContain("platform-managed");
   });
 });
 
@@ -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__/slack-inbound-verification.test.ts

@@ -8,7 +8,7 @@
  * 4. Notify the guardian of the access attempt
  * 5. When the user replies with the code in the DM, verify and activate
  */
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 // ---------------------------------------------------------------------------
 // Test isolation: in-memory SQLite via temp directory
@@ -56,16 +56,12 @@ mock.module("../runtime/gateway-client.js", () => ({
 }));
 
 import { createGuardianBinding } from "../contacts/contacts-write.js";
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { findActiveSession } from "../runtime/channel-verification-service.js";
 import { handleChannelInbound } from "../runtime/routes/channel-routes.js";
 
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-});
-
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------

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 { 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>, {
@@ -37,7 +21,7 @@ mock.module("./indexer.js", () => ({
 
 import type { Database } from "bun:sqlite";
 
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import {
   compileTaskFromConversation,
   saveCompiledTask,
@@ -47,15 +31,6 @@ import { getTask } from "../tasks/task-store.js";
 
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // ── Helpers ──────────────────────────────────────────────────────────
 
 function getRawDb(): Database {

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

@@ -1,20 +1,4 @@
-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: () => {},
-}));
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -38,7 +22,7 @@ mock.module("../tools/registry.js", () => ({
 
 import type { Database } from "bun:sqlite";
 
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { renderTemplate } from "../tasks/task-runner.js";
 import {
   createTask,
@@ -74,15 +58,6 @@ import {
 
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 const ctx: ToolContext = {
   workingDir: "/tmp",
   conversationId: "test-conversation",