@vellumai/assistant 0.5.15 → 0.6.0

package/src/__tests__/managed-store.test.ts

@@ -7,8 +7,6 @@ import {
   rmSync,
   writeFileSync,
 } from "node:fs";
-import { mkdtempSync } from "node:fs";
-import { tmpdir } from "node:os";
 import { join } from "node:path";
 import {
   afterEach,
@@ -22,12 +20,7 @@ import {
 
 import { parse as parseYaml } from "yaml";
 
-let TEST_DIR = "";
-
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(TEST_DIR, "protected"),
-  getWorkspaceSkillsDir: () => join(TEST_DIR, "skills"),
-}));
+const TEST_DIR = process.env.VELLUM_WORKSPACE_DIR!;
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -48,12 +41,11 @@ import {
 } from "../skills/managed-store.js";
 
 beforeEach(() => {
-  TEST_DIR = mkdtempSync(join(tmpdir(), "managed-store-test-"));
   mkdirSync(join(TEST_DIR, "skills"), { recursive: true });
 });
 
 afterEach(() => {
-  rmSync(TEST_DIR, { recursive: true, force: true });
+  rmSync(join(TEST_DIR, "skills"), { recursive: true, force: true });
 });
 
 describe("validateManagedSkillId", () => {
@@ -504,9 +496,9 @@ describe("atomic write safety", () => {
     });
 
     const skillDir = join(TEST_DIR, "skills", "atomic-overwrite");
-    const files = readdirSync(skillDir);
-    // Only SKILL.md should exist — no .tmp-* leftover files
-    expect(files).toEqual(["SKILL.md"]);
+    const files = readdirSync(skillDir).sort();
+    // SKILL.md and install-meta.json should exist — no .tmp-* leftover files
+    expect(files).toEqual(["SKILL.md", "install-meta.json"]);
 
     const content = readFileSync(join(skillDir, "SKILL.md"), "utf-8");
     expect(content).toContain('name: "V2"');
@@ -657,7 +649,7 @@ describe("version metadata", () => {
     expect(readSkillVersion("no-version")).toBeNull();
   });
 
-  test("createManagedSkill writes version.json when version is provided", () => {
+  test("createManagedSkill writes install-meta.json when version is provided", () => {
     createManagedSkill({
       id: "versioned",
       name: "Versioned",
@@ -670,7 +662,7 @@ describe("version metadata", () => {
     expect(version).toBe("v1:abc123");
   });
 
-  test("version.json contains valid JSON with version and installedAt", () => {
+  test("install-meta.json contains valid JSON with origin, version, and installedAt", () => {
     createManagedSkill({
       id: "version-meta",
       name: "Meta",
@@ -679,16 +671,43 @@ describe("version metadata", () => {
       version: "v1:deadbeef",
     });
 
-    const metaPath = join(TEST_DIR, "skills", "version-meta", "version.json");
+    const metaPath = join(
+      TEST_DIR,
+      "skills",
+      "version-meta",
+      "install-meta.json",
+    );
     expect(existsSync(metaPath)).toBe(true);
     const meta = JSON.parse(readFileSync(metaPath, "utf-8"));
+    expect(meta.origin).toBe("custom");
     expect(meta.version).toBe("v1:deadbeef");
     expect(typeof meta.installedAt).toBe("string");
     // installedAt should be a valid ISO date
     expect(new Date(meta.installedAt).toISOString()).toBe(meta.installedAt);
   });
 
-  test("overwrite updates version.json", () => {
+  test("install-meta.json includes installedBy when contactId is provided", () => {
+    createManagedSkill({
+      id: "with-contact",
+      name: "With Contact",
+      description: "Has contactId",
+      bodyMarkdown: "Body.",
+      contactId: "contact-uuid-456",
+    });
+
+    const metaPath = join(
+      TEST_DIR,
+      "skills",
+      "with-contact",
+      "install-meta.json",
+    );
+    expect(existsSync(metaPath)).toBe(true);
+    const meta = JSON.parse(readFileSync(metaPath, "utf-8"));
+    expect(meta.origin).toBe("custom");
+    expect(meta.installedBy).toBe("contact-uuid-456");
+  });
+
+  test("overwrite updates install-meta.json", () => {
     createManagedSkill({
       id: "update-version",
       name: "V1",
@@ -709,21 +728,21 @@ describe("version metadata", () => {
     expect(readSkillVersion("update-version")).toBe("v1:second");
   });
 
-  test("readSkillVersion returns null for corrupted version.json", () => {
+  test("readSkillVersion returns null for corrupted install-meta.json", () => {
     createManagedSkill({
       id: "corrupt-version",
       name: "Corrupt",
-      description: "Will corrupt version file",
+      description: "Will corrupt meta file",
       bodyMarkdown: "Body.",
       version: "v1:valid",
     });
 
-    // Corrupt the version.json
+    // Corrupt the install-meta.json
     const metaPath = join(
       TEST_DIR,
       "skills",
       "corrupt-version",
-      "version.json",
+      "install-meta.json",
     );
     writeFileSync(metaPath, "{invalid json!!!", "utf-8");
 

package/src/__tests__/memory-jobs-worker-backoff.test.ts

@@ -34,15 +34,9 @@ mock.module("../memory/jobs-store.js", () => ({
   deferMemoryJob: () => "deferred",
   failMemoryJob: () => {},
   failStalledJobs: () => 0,
-  enqueueCleanupStaleSupersededItemsJob: () => null,
   enqueuePruneOldConversationsJob: () => null,
 }));
 
-// Mock db.js (rawRun used in sweepStaleItems)
-mock.module("../memory/db.js", () => ({
-  rawRun: () => 0,
-}));
-
 import {
   POLL_INTERVAL_MAX_MS,
   POLL_INTERVAL_MIN_MS,
@@ -90,7 +84,7 @@ describe("memory jobs worker adaptive poll interval", () => {
         timeoutDelays.push(delay);
         pendingCallbacks.push(fn);
       }
-      return 999 as unknown as ReturnType<typeof setTimeout>;
+      return (999 as unknown) as ReturnType<typeof setTimeout>;
     }) as typeof setTimeout;
     globalThis.clearTimeout = (() => {}) as typeof clearTimeout;
 
@@ -129,7 +123,7 @@ describe("memory jobs worker adaptive poll interval", () => {
 
       // Should eventually reach the cap
       expect(timeoutDelays[timeoutDelays.length - 1]).toBe(
-        POLL_INTERVAL_MAX_MS,
+        POLL_INTERVAL_MAX_MS
       );
     } finally {
       globalThis.setTimeout = originalSetTimeout;

package/src/__tests__/memory-recall-log-store.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -18,7 +18,7 @@ mock.module("../config/loader.js", () => ({
   }),
 }));
 
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import {
   backfillMemoryRecallLogMessageId,
   getMemoryRecallLogByMessageIds,
@@ -33,10 +33,6 @@ function resetTables(): void {
   db.delete(memoryRecallLogs).run();
 }
 
-afterAll(() => {
-  resetDb();
-});
-
 describe("memory-recall-log-store", () => {
   beforeEach(() => {
     resetTables();

package/src/__tests__/memory-upsert-concurrency.test.ts

@@ -16,7 +16,7 @@
  * processes and is not tested here.
  */
 
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 import { eq } from "drizzle-orm";
 
@@ -59,27 +59,17 @@ mock.module("../config/loader.js", () => ({
   invalidateConfigCache: () => {},
 }));
 
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { indexMessageNow } from "../memory/indexer.js";
-import {
-  conversations,
-  memoryItems,
-  memorySegments,
-  messages,
-} from "../memory/schema.js";
+import { conversations, memorySegments, messages } from "../memory/schema.js";
 
 // Initialize DB once for the entire file. Each test cleans its own tables.
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-});
-
 function resetTables() {
   const db = getDb();
-  db.run("DELETE FROM memory_item_sources");
   db.run("DELETE FROM memory_embeddings");
-  db.run("DELETE FROM memory_items");
+  db.run("DELETE FROM memory_graph_nodes");
   db.run("DELETE FROM memory_segments");
   db.run("DELETE FROM memory_jobs");
   db.run("DELETE FROM messages");
@@ -559,101 +549,3 @@ describe("memory segment job atomicity under repeated indexer invocations", () =
   });
 });
 
-// ─────────────────────────────────────────────────────────────────────────────
-// Test suite: memory_items fingerprint uniqueness under race conditions
-// ─────────────────────────────────────────────────────────────────────────────
-
-describe("memory_items fingerprint uniqueness under race conditions", () => {
-  beforeEach(() => {
-    resetTables();
-  });
-
-  test("duplicate inserts with identical fingerprints produce exactly one row", () => {
-    // The memory_items table has a unique constraint on (fingerprint, scope_id).
-    // Two sequential inserts for the same fingerprint simulate duplicate extractor
-    // runs.  Only one INSERT must land; the second must be absorbed by ON CONFLICT.
-    const db = getDb();
-    const now = Date.now();
-    const fingerprint = "fp-race-unique-test-concurrency";
-    const scopeId = "default";
-
-    // Use raw SQL to replicate what the items-extractor would do when a second
-    // run tries to INSERT the same fingerprint that already exists.
-    const raw = (db as unknown as { $client: import("bun:sqlite").Database })
-      .$client;
-
-    raw.run(`
-      INSERT INTO memory_items (
-        id, kind, subject, statement, status, confidence, importance,
-        fingerprint, verification_state, scope_id, first_seen_at, last_seen_at
-      ) VALUES (
-        'item-race-1', 'preference', 'code style', 'I prefer tabs over spaces.',
-        'active', 0.8, 0.6, '${fingerprint}', 'user_reported', '${scopeId}',
-        ${now}, ${now}
-      )
-    `);
-
-    // Second "worker" tries to insert the same fingerprint — must not create a
-    // duplicate.  INSERT OR IGNORE / ON CONFLICT DO NOTHING is the expected
-    // behavior for the unique constraint.
-    expect(() => {
-      raw.run(`
-        INSERT OR IGNORE INTO memory_items (
-          id, kind, subject, statement, status, confidence, importance,
-          fingerprint, verification_state, scope_id, first_seen_at, last_seen_at
-        ) VALUES (
-          'item-race-2', 'preference', 'code style', 'I prefer tabs over spaces.',
-          'active', 0.8, 0.6, '${fingerprint}', 'user_reported', '${scopeId}',
-          ${now + 1}, ${now + 1}
-        )
-      `);
-    }).not.toThrow();
-
-    const rows = db
-      .select()
-      .from(memoryItems)
-      .all()
-      .filter((r) => r.fingerprint === fingerprint);
-
-    // Only the first insert must have landed.
-    expect(rows).toHaveLength(1);
-    expect(rows[0].id).toBe("item-race-1");
-  });
-
-  test("bare INSERT without IGNORE throws on duplicate fingerprint+scopeId", () => {
-    // Verify the DB-level unique constraint is actually enforced so that any code
-    // path that accidentally omits ON CONFLICT will fail loudly rather than silently
-    // producing inconsistent state.
-    const db = getDb();
-    const now = Date.now();
-    const fingerprint = "fp-constraint-enforcement-test";
-
-    const raw = (db as unknown as { $client: import("bun:sqlite").Database })
-      .$client;
-
-    raw.run(`
-      INSERT INTO memory_items (
-        id, kind, subject, statement, status, confidence, importance,
-        fingerprint, verification_state, scope_id, first_seen_at, last_seen_at
-      ) VALUES (
-        'item-constraint-a', 'preference', 'editor', 'I use VS Code.',
-        'active', 0.9, 0.7, '${fingerprint}', 'user_reported', 'default',
-        ${now}, ${now}
-      )
-    `);
-
-    // A bare INSERT (no ON CONFLICT) for the same fingerprint+scope_id must throw.
-    expect(() => {
-      raw.run(`
-        INSERT INTO memory_items (
-          id, kind, subject, statement, status, confidence, importance,
-          fingerprint, verification_state, scope_id, first_seen_at, last_seen_at
-        ) VALUES (
-          'item-constraint-b', 'preference', 'editor', 'I use VS Code.',
-          'active', 0.9, 0.7, '${fingerprint}', 'user_reported', 'default',
-          ${now + 1}, ${now + 1}
-        )
-      `);
-    }).toThrow();
-  });
-});

package/src/__tests__/messaging-send-tool.test.ts

@@ -64,16 +64,16 @@ const getConversationMock = mock(
 );
 
 const syncMessageToDiskMock = mock(
-  (
-    _conversationId: string,
-    _messageId: string,
-    _createdAtMs: number,
-  ) => {},
+  (_conversationId: string, _messageId: string, _createdAtMs: number) => {},
 );
 
 const getBindingByChannelChatMock = mock(
   (_sourceChannel: string, _externalChatId: string) =>
-    null as { conversationId: string; sourceChannel: string; externalChatId: string } | null,
+    null as {
+      conversationId: string;
+      sourceChannel: string;
+      externalChatId: string;
+    } | null,
 );
 
 mock.module("../memory/conversation-crud.js", () => ({

package/src/__tests__/migration-cross-version-compatibility.test.ts

@@ -20,19 +20,15 @@
 import { createHash } from "node:crypto";
 import {
   mkdirSync,
-  mkdtempSync,
   readdirSync,
   readFileSync,
-  realpathSync,
   rmSync,
   unlinkSync,
   writeFileSync,
 } from "node:fs";
-import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { gzipSync } from "node:zlib";
 import {
-  afterAll,
   afterEach,
   beforeAll,
   beforeEach,
@@ -50,27 +46,11 @@ function toArrayBuffer(data: Uint8Array): ArrayBuffer {
   ) as ArrayBuffer;
 }
 
-const testDir = realpathSync(
-  mkdtempSync(join(tmpdir(), "migration-cross-version-test-")),
-);
+const testDir = process.env.VELLUM_WORKSPACE_DIR!;
 const testDbDir = join(testDir, "data", "db");
 const testDbPath = join(testDbDir, "assistant.db");
 const testConfigPath = join(testDir, "config.json");
 
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(testDir, "protected"),
-  getDataDir: () => join(testDir, "data"),
-  getWorkspaceDir: () => testDir,
-  getWorkspaceConfigPath: () => testConfigPath,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => testDbPath,
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -137,14 +117,6 @@ beforeAll(() => {
   writeFileSync(testConfigPath, JSON.stringify(EXISTING_CONFIG, null, 2));
 });
 
-afterAll(() => {
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // Restore test files before each test
 beforeEach(() => {
   mkdirSync(testDbDir, { recursive: true });

package/src/__tests__/migration-export-http.test.ts

@@ -12,39 +12,16 @@
  * - Integration: existing routes are unaffected by the new endpoint
  */
 import { createHash } from "node:crypto";
-import {
-  mkdirSync,
-  mkdtempSync,
-  realpathSync,
-  rmSync,
-  writeFileSync,
-} from "node:fs";
-import { tmpdir } from "node:os";
+import { mkdirSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { gunzipSync } from "node:zlib";
-import { afterAll, beforeAll, describe, expect, mock, test } from "bun:test";
+import { beforeAll, describe, expect, mock, test } from "bun:test";
 
-const testDir = realpathSync(
-  mkdtempSync(join(tmpdir(), "migration-export-http-test-")),
-);
+const testDir = process.env.VELLUM_WORKSPACE_DIR!;
 const testDbDir = join(testDir, "data", "db");
 const testDbPath = join(testDbDir, "assistant.db");
 const testConfigPath = join(testDir, "config.json");
 
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(testDir, "protected"),
-  getDataDir: () => join(testDir, "data"),
-  getWorkspaceDir: () => testDir,
-  getWorkspaceConfigPath: () => testConfigPath,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => testDbPath,
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -101,14 +78,6 @@ beforeAll(() => {
   writeFileSync(testConfigPath, JSON.stringify(TEST_CONFIG, null, 2));
 });
 
-afterAll(() => {
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // ---------------------------------------------------------------------------
 // Tar parsing helper (mirrors vbundle-validator's internal parser)
 // ---------------------------------------------------------------------------

package/src/__tests__/migration-import-commit-http.test.ts

@@ -15,19 +15,15 @@ import { createHash } from "node:crypto";
 import {
   existsSync,
   mkdirSync,
-  mkdtempSync,
   readdirSync,
   readFileSync,
-  realpathSync,
   rmSync,
   unlinkSync,
   writeFileSync,
 } from "node:fs";
-import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { gzipSync } from "node:zlib";
 import {
-  afterAll,
   afterEach,
   beforeAll,
   beforeEach,
@@ -45,27 +41,11 @@ function toArrayBuffer(data: Uint8Array): ArrayBuffer {
   ) as ArrayBuffer;
 }
 
-const testDir = realpathSync(
-  mkdtempSync(join(tmpdir(), "migration-import-commit-http-test-")),
-);
+const testDir = process.env.VELLUM_WORKSPACE_DIR!;
 const testDbDir = join(testDir, "data", "db");
 const testDbPath = join(testDbDir, "assistant.db");
 const testConfigPath = join(testDir, "config.json");
 
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(testDir, "protected"),
-  getDataDir: () => join(testDir, "data"),
-  getWorkspaceDir: () => testDir,
-  getWorkspaceConfigPath: () => testConfigPath,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => testDbPath,
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -119,14 +99,6 @@ beforeAll(() => {
   writeFileSync(testConfigPath, JSON.stringify(EXISTING_CONFIG, null, 2));
 });
 
-afterAll(() => {
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // Restore test files before each test so mutations from previous tests
 // do not leak across test cases.
 beforeEach(() => {

package/src/__tests__/migration-import-preflight-http.test.ts

@@ -11,17 +11,10 @@
  * - Integration: existing routes are unaffected by the new endpoint
  */
 import { createHash } from "node:crypto";
-import {
-  mkdirSync,
-  mkdtempSync,
-  realpathSync,
-  rmSync,
-  writeFileSync,
-} from "node:fs";
-import { tmpdir } from "node:os";
+import { mkdirSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { gzipSync } from "node:zlib";
-import { afterAll, beforeAll, describe, expect, mock, test } from "bun:test";
+import { beforeAll, describe, expect, mock, test } from "bun:test";
 
 /** Convert a Uint8Array to an ArrayBuffer for BodyInit compatibility. */
 function toArrayBuffer(data: Uint8Array): ArrayBuffer {
@@ -31,27 +24,11 @@ function toArrayBuffer(data: Uint8Array): ArrayBuffer {
   ) as ArrayBuffer;
 }
 
-const testDir = realpathSync(
-  mkdtempSync(join(tmpdir(), "migration-import-preflight-http-test-")),
-);
+const testDir = process.env.VELLUM_WORKSPACE_DIR!;
 const testDbDir = join(testDir, "data", "db");
 const testDbPath = join(testDbDir, "assistant.db");
 const testConfigPath = join(testDir, "config.json");
 
-mock.module("../util/platform.js", () => ({
-  getProtectedDir: () => join(testDir, "protected"),
-  getDataDir: () => join(testDir, "data"),
-  getWorkspaceDir: () => testDir,
-  getWorkspaceConfigPath: () => testConfigPath,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => testDbPath,
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -107,14 +84,6 @@ beforeAll(() => {
   writeFileSync(testConfigPath, JSON.stringify(EXISTING_CONFIG, null, 2));
 });
 
-afterAll(() => {
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // ---------------------------------------------------------------------------
 // Tar archive builder helpers (mirrors validate test)
 // ---------------------------------------------------------------------------

package/src/__tests__/no-domain-routing-in-prompt-guard.test.ts

@@ -5,7 +5,8 @@ import { describe, expect, test } from "bun:test";
 /**
  * Guard test: domain-specific routing sections must NOT appear in the system
  * prompt source file. Routing cues now live in skill frontmatter
- * (`activation-hints` / `avoid-when`) and are projected into the skills catalog.
+ * (`activation-hints` / `avoid-when`) and are seeded as capability memories
+ * for semantic discovery.
  *
  * If this test fails, you are re-introducing hardcoded routing into the system
  * prompt. Instead, add `activation-hints` to the skill's SKILL.md frontmatter.

package/src/__tests__/non-member-access-request.test.ts

@@ -7,7 +7,7 @@
  * 3. Create a guardian approval request for the access request
  * 4. Deduplicate: don't create duplicate requests for repeated messages
  */
-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
@@ -66,17 +66,13 @@ import {
   listCanonicalGuardianDeliveries,
   listCanonicalGuardianRequests,
 } from "../memory/canonical-guardian-store.js";
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { notifyGuardianOfAccessRequest } from "../runtime/access-request-helper.js";
 import { ensureVellumGuardianBinding } from "../runtime/guardian-vellum-migration.js";
 import { handleChannelInbound } from "../runtime/routes/channel-routes.js";
 
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-});
-
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------

package/src/__tests__/notification-guardian-path.test.ts

@@ -5,7 +5,7 @@
  * pipeline (including conversation-created callbacks) without a custom dispatch path.
  */
 
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 import type { ConversationCreatedInfo } from "../notifications/broadcaster.js";
 import type { NotificationDeliveryResult } from "../notifications/types.js";
@@ -80,7 +80,7 @@ import {
   createPendingQuestion,
 } from "../calls/call-store.js";
 import { dispatchGuardianQuestion } from "../calls/guardian-dispatch.js";
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { conversations } from "../memory/schema.js";
 
 initializeDb();
@@ -134,10 +134,6 @@ describe("ASK_GUARDIAN canonical notification path", () => {
     resetTables();
   });
 
-  afterAll(() => {
-    resetDb();
-  });
-
   test("dispatches through emitNotificationSignal with guardian context metadata", async () => {
     const convId = "conv-guardian-notif-1";
     ensureConversation(convId);