@vellumai/assistant 0.4.17 → 0.4.18

package/src/__tests__/bundled-asset.test.ts

@@ -1,10 +1,9 @@
-import { mkdirSync, rmSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
+import { mkdirSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
-
-import { resolveBundledDir } from '../util/bundled-asset.js';
+import { resolveBundledDir } from "../util/bundled-asset.js";
 
 let tempDir: string;
 
@@ -17,19 +16,27 @@ afterEach(() => {
   rmSync(tempDir, { recursive: true, force: true });
 });
 
-describe('resolveBundledDir', () => {
-  test('source mode: returns join(callerDir, relativePath) when callerDir is a normal path', () => {
-    const result = resolveBundledDir('/some/source/path', 'templates', 'templates');
-    expect(result).toBe(join('/some/source/path', 'templates'));
+describe("resolveBundledDir", () => {
+  test("source mode: returns join(callerDir, relativePath) when callerDir is a normal path", () => {
+    const result = resolveBundledDir(
+      "/some/source/path",
+      "templates",
+      "templates",
+    );
+    expect(result).toBe(join("/some/source/path", "templates"));
   });
 
-  test('source mode: does not check existsSync for the source path', () => {
+  test("source mode: does not check existsSync for the source path", () => {
     // Even if the resolved path does not exist, it returns it as-is
-    const result = resolveBundledDir('/nonexistent/path', 'templates', 'templates');
-    expect(result).toBe(join('/nonexistent/path', 'templates'));
+    const result = resolveBundledDir(
+      "/nonexistent/path",
+      "templates",
+      "templates",
+    );
+    expect(result).toBe(join("/nonexistent/path", "templates"));
   });
 
-  describe('compiled mode (/$bunfs/ prefix)', () => {
+  describe("compiled mode (/$bunfs/ prefix)", () => {
     // In compiled mode, process.execPath determines fallback locations.
     // We simulate by creating real directories at the expected fallback paths.
 
@@ -43,65 +50,85 @@ describe('resolveBundledDir', () => {
       process.execPath = savedExecPath;
     });
 
-    test('prefers Contents/Resources/<bundleName> when it exists', () => {
+    test("prefers Contents/Resources/<bundleName> when it exists", () => {
       // Simulate macOS .app bundle: binary at Contents/MacOS/vellum-daemon
-      const macosDir = join(tempDir, 'Contents', 'MacOS');
-      const resourcesDir = join(tempDir, 'Contents', 'Resources');
+      const macosDir = join(tempDir, "Contents", "MacOS");
+      const resourcesDir = join(tempDir, "Contents", "Resources");
       mkdirSync(macosDir, { recursive: true });
-      mkdirSync(join(resourcesDir, 'templates'), { recursive: true });
+      mkdirSync(join(resourcesDir, "templates"), { recursive: true });
 
-      process.execPath = join(macosDir, 'vellum-daemon');
+      process.execPath = join(macosDir, "vellum-daemon");
 
-      const result = resolveBundledDir('/$bunfs/root/src/config', 'templates', 'templates');
-      expect(result).toBe(join(resourcesDir, 'templates'));
+      const result = resolveBundledDir(
+        "/$bunfs/root/src/config",
+        "templates",
+        "templates",
+      );
+      expect(result).toBe(join(resourcesDir, "templates"));
     });
 
-    test('falls back to <execDir>/<bundleName> when Resources does not exist', () => {
+    test("falls back to <execDir>/<bundleName> when Resources does not exist", () => {
       // Simulate standalone binary deployment (no .app bundle)
-      const binDir = join(tempDir, 'bin');
-      mkdirSync(join(binDir, 'templates'), { recursive: true });
+      const binDir = join(tempDir, "bin");
+      mkdirSync(join(binDir, "templates"), { recursive: true });
 
-      process.execPath = join(binDir, 'vellum-daemon');
+      process.execPath = join(binDir, "vellum-daemon");
 
-      const result = resolveBundledDir('/$bunfs/root/src/config', 'templates', 'templates');
-      expect(result).toBe(join(binDir, 'templates'));
+      const result = resolveBundledDir(
+        "/$bunfs/root/src/config",
+        "templates",
+        "templates",
+      );
+      expect(result).toBe(join(binDir, "templates"));
     });
 
-    test('falls back to source path when neither Resources nor execDir have the asset', () => {
-      const binDir = join(tempDir, 'bin');
+    test("falls back to source path when neither Resources nor execDir have the asset", () => {
+      const binDir = join(tempDir, "bin");
       mkdirSync(binDir, { recursive: true });
       // Don't create any asset directories
 
-      process.execPath = join(binDir, 'vellum-daemon');
+      process.execPath = join(binDir, "vellum-daemon");
 
-      const result = resolveBundledDir('/$bunfs/root/src/config', 'templates', 'templates');
-      expect(result).toBe(join('/$bunfs/root/src/config', 'templates'));
+      const result = resolveBundledDir(
+        "/$bunfs/root/src/config",
+        "templates",
+        "templates",
+      );
+      expect(result).toBe(join("/$bunfs/root/src/config", "templates"));
     });
 
-    test('Resources path takes priority over execDir path when both exist', () => {
-      const macosDir = join(tempDir, 'Contents', 'MacOS');
-      const resourcesDir = join(tempDir, 'Contents', 'Resources');
+    test("Resources path takes priority over execDir path when both exist", () => {
+      const macosDir = join(tempDir, "Contents", "MacOS");
+      const resourcesDir = join(tempDir, "Contents", "Resources");
       mkdirSync(macosDir, { recursive: true });
-      mkdirSync(join(resourcesDir, 'hook-templates'), { recursive: true });
+      mkdirSync(join(resourcesDir, "hook-templates"), { recursive: true });
       // Also create at execDir level
-      mkdirSync(join(macosDir, 'hook-templates'), { recursive: true });
+      mkdirSync(join(macosDir, "hook-templates"), { recursive: true });
 
-      process.execPath = join(macosDir, 'vellum-daemon');
+      process.execPath = join(macosDir, "vellum-daemon");
 
-      const result = resolveBundledDir('/$bunfs/root/src/hooks', '../../hook-templates', 'hook-templates');
-      expect(result).toBe(join(resourcesDir, 'hook-templates'));
+      const result = resolveBundledDir(
+        "/$bunfs/root/src/hooks",
+        "../../hook-templates",
+        "hook-templates",
+      );
+      expect(result).toBe(join(resourcesDir, "hook-templates"));
     });
 
-    test('works with different bundleName values', () => {
-      const macosDir = join(tempDir, 'Contents', 'MacOS');
-      const resourcesDir = join(tempDir, 'Contents', 'Resources');
+    test("works with different bundleName values", () => {
+      const macosDir = join(tempDir, "Contents", "MacOS");
+      const resourcesDir = join(tempDir, "Contents", "Resources");
       mkdirSync(macosDir, { recursive: true });
-      mkdirSync(join(resourcesDir, 'prebuilt'), { recursive: true });
+      mkdirSync(join(resourcesDir, "prebuilt"), { recursive: true });
 
-      process.execPath = join(macosDir, 'vellum-daemon');
+      process.execPath = join(macosDir, "vellum-daemon");
 
-      const result = resolveBundledDir('/$bunfs/root/src/home-base/prebuilt', '.', 'prebuilt');
-      expect(result).toBe(join(resourcesDir, 'prebuilt'));
+      const result = resolveBundledDir(
+        "/$bunfs/root/src/home-base/prebuilt",
+        ".",
+        "prebuilt",
+      );
+      expect(result).toBe(join(resourcesDir, "prebuilt"));
     });
   });
 });

package/src/__tests__/call-constants.test.ts

@@ -1,23 +1,23 @@
-import { describe, expect,test } from 'bun:test';
+import { describe, expect, test } from "bun:test";
 
-import { isDeniedNumber } from '../calls/call-constants.js';
+import { isDeniedNumber } from "../calls/call-constants.js";
 
-describe('isDeniedNumber', () => {
+describe("isDeniedNumber", () => {
   // Numbers that MUST be blocked
   const blocked = [
-    '911',
-    '112',
-    '999',
-    '000',
-    '110',
-    '119',
-    '+112',     // '+' stripped → '112' exact match
-    '+911',     // '+' stripped → '911' exact match
-    '+1911',    // country code 1 + 911
-    '+44999',   // country code 44 + 999
-    '+61000',   // country code 61 + 000
-    '+49110',   // country code 49 + 110
-    '+81119',   // country code 81 + 119
+    "911",
+    "112",
+    "999",
+    "000",
+    "110",
+    "119",
+    "+112", // '+' stripped → '112' exact match
+    "+911", // '+' stripped → '911' exact match
+    "+1911", // country code 1 + 911
+    "+44999", // country code 44 + 999
+    "+61000", // country code 61 + 000
+    "+49110", // country code 49 + 110
+    "+81119", // country code 81 + 119
   ];
 
   for (const num of blocked) {
@@ -28,9 +28,9 @@ describe('isDeniedNumber', () => {
 
   // Numbers that MUST be allowed (legitimate phone numbers)
   const allowed = [
-    '+14155551212',   // US number — digits after any CC split don't match short codes
-    '+442071234567',  // UK number
-    '+15559998888',   // US number
+    "+14155551212", // US number — digits after any CC split don't match short codes
+    "+442071234567", // UK number
+    "+15559998888", // US number
   ];
 
   for (const num of allowed) {

package/src/__tests__/call-controller.test.ts

@@ -1,7 +1,6 @@
 import { mkdtempSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-
 import {
   afterAll,
   beforeEach,

package/src/__tests__/call-conversation-messages.test.ts

@@ -1,67 +1,83 @@
-import { mkdtempSync, rmSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
+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";
 
-import { afterAll, beforeEach, describe, expect, mock,test } from 'bun:test';
+const testDir = mkdtempSync(join(tmpdir(), "call-conversation-messages-test-"));
 
-const testDir = mkdtempSync(join(tmpdir(), 'call-conversation-messages-test-'));
-
-mock.module('../util/platform.js', () => ({
+mock.module("../util/platform.js", () => ({
   getDataDir: () => testDir,
-  isMacOS: () => process.platform === 'darwin',
-  isLinux: () => process.platform === 'linux',
-  isWindows: () => process.platform === 'win32',
-  getSocketPath: () => join(testDir, 'test.sock'),
-  getPidPath: () => join(testDir, 'test.pid'),
-  getDbPath: () => join(testDir, 'test.db'),
-  getLogPath: () => join(testDir, 'test.log'),
+  isMacOS: () => process.platform === "darwin",
+  isLinux: () => process.platform === "linux",
+  isWindows: () => process.platform === "win32",
+  getSocketPath: () => join(testDir, "test.sock"),
+  getPidPath: () => join(testDir, "test.pid"),
+  getDbPath: () => join(testDir, "test.db"),
+  getLogPath: () => join(testDir, "test.log"),
   ensureDataDir: () => {},
 }));
 
-mock.module('../util/logger.js', () => ({
+mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
       get: () => () => {},
     }),
 }));
 
-import { buildCallCompletionMessage, persistCallCompletionMessage } from '../calls/call-conversation-messages.js';
-import { createCallSession, recordCallEvent,updateCallSession } from '../calls/call-store.js';
-import { getMessages } from '../memory/conversation-store.js';
-import { getDb, initializeDb, resetDb } from '../memory/db.js';
-import { conversations } from '../memory/schema.js';
+import {
+  buildCallCompletionMessage,
+  persistCallCompletionMessage,
+} from "../calls/call-conversation-messages.js";
+import {
+  createCallSession,
+  recordCallEvent,
+  updateCallSession,
+} from "../calls/call-store.js";
+import { getMessages } from "../memory/conversation-store.js";
+import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { conversations } from "../memory/schema.js";
 
 initializeDb();
 
 function ensureConversation(id: string): void {
   const db = getDb();
   const now = Date.now();
-  db.insert(conversations).values({
-    id,
-    title: `Conversation ${id}`,
-    createdAt: now,
-    updatedAt: now,
-  }).run();
+  db.insert(conversations)
+    .values({
+      id,
+      title: `Conversation ${id}`,
+      createdAt: now,
+      updatedAt: now,
+    })
+    .run();
 }
 
 function resetTables(): void {
   const db = getDb();
-  db.run('DELETE FROM call_events');
-  db.run('DELETE FROM call_pending_questions');
-  db.run('DELETE FROM call_sessions');
-  db.run('DELETE FROM messages');
-  db.run('DELETE FROM conversations');
+  db.run("DELETE FROM call_events");
+  db.run("DELETE FROM call_pending_questions");
+  db.run("DELETE FROM call_sessions");
+  db.run("DELETE FROM messages");
+  db.run("DELETE FROM conversations");
 }
 
 function getLatestAssistantText(conversationId: string): string {
-  const rows = getMessages(conversationId).filter((m) => m.role === 'assistant');
+  const rows = getMessages(conversationId).filter(
+    (m) => m.role === "assistant",
+  );
   expect(rows.length).toBeGreaterThan(0);
   const latest = rows[rows.length - 1];
-  const parsed = JSON.parse(latest.content) as Array<{ type: string; text?: string }>;
-  return parsed.filter((b) => b.type === 'text').map((b) => b.text ?? '').join('');
+  const parsed = JSON.parse(latest.content) as Array<{
+    type: string;
+    text?: string;
+  }>;
+  return parsed
+    .filter((b) => b.type === "text")
+    .map((b) => b.text ?? "")
+    .join("");
 }
 
-describe('call-conversation-messages', () => {
+describe("call-conversation-messages", () => {
   beforeEach(() => {
     resetTables();
   });
@@ -75,57 +91,66 @@ describe('call-conversation-messages', () => {
     }
   });
 
-  test('buildCallCompletionMessage labels failed calls correctly', () => {
-    const conversationId = 'conv-call-msg-failed';
+  test("buildCallCompletionMessage labels failed calls correctly", () => {
+    const conversationId = "conv-call-msg-failed";
     ensureConversation(conversationId);
     const session = createCallSession({
       conversationId,
-      provider: 'twilio',
-      fromNumber: '+15550001111',
-      toNumber: '+15550002222',
+      provider: "twilio",
+      fromNumber: "+15550001111",
+      toNumber: "+15550002222",
     });
 
-    updateCallSession(session.id, { status: 'in_progress', startedAt: 1_000 });
-    updateCallSession(session.id, { status: 'failed', endedAt: 6_000 });
-    recordCallEvent(session.id, 'call_connected');
-    recordCallEvent(session.id, 'call_failed');
+    updateCallSession(session.id, { status: "in_progress", startedAt: 1_000 });
+    updateCallSession(session.id, { status: "failed", endedAt: 6_000 });
+    recordCallEvent(session.id, "call_connected");
+    recordCallEvent(session.id, "call_failed");
 
-    expect(buildCallCompletionMessage(session.id)).toBe('**Call failed** (5s). 2 event(s) recorded.');
+    expect(buildCallCompletionMessage(session.id)).toBe(
+      "**Call failed** (5s). 2 event(s) recorded.",
+    );
   });
 
-  test('buildCallCompletionMessage labels cancelled calls correctly', () => {
-    const conversationId = 'conv-call-msg-cancelled';
+  test("buildCallCompletionMessage labels cancelled calls correctly", () => {
+    const conversationId = "conv-call-msg-cancelled";
     ensureConversation(conversationId);
     const session = createCallSession({
       conversationId,
-      provider: 'twilio',
-      fromNumber: '+15550001111',
-      toNumber: '+15550002222',
+      provider: "twilio",
+      fromNumber: "+15550001111",
+      toNumber: "+15550002222",
     });
 
-    updateCallSession(session.id, { status: 'in_progress', startedAt: 1_000 });
-    updateCallSession(session.id, { status: 'cancelled', endedAt: 4_000 });
-    recordCallEvent(session.id, 'call_connected');
-    recordCallEvent(session.id, 'call_ended');
+    updateCallSession(session.id, { status: "in_progress", startedAt: 1_000 });
+    updateCallSession(session.id, { status: "cancelled", endedAt: 4_000 });
+    recordCallEvent(session.id, "call_connected");
+    recordCallEvent(session.id, "call_ended");
 
-    expect(buildCallCompletionMessage(session.id)).toBe('**Call cancelled** (3s). 2 event(s) recorded.');
+    expect(buildCallCompletionMessage(session.id)).toBe(
+      "**Call cancelled** (3s). 2 event(s) recorded.",
+    );
   });
 
-  test('persistCallCompletionMessage keeps completed label when status is completed', async () => {
-    const conversationId = 'conv-call-msg-completed';
+  test("persistCallCompletionMessage keeps completed label when status is completed", async () => {
+    const conversationId = "conv-call-msg-completed";
     ensureConversation(conversationId);
     const session = createCallSession({
       conversationId,
-      provider: 'twilio',
-      fromNumber: '+15550001111',
-      toNumber: '+15550002222',
+      provider: "twilio",
+      fromNumber: "+15550001111",
+      toNumber: "+15550002222",
     });
 
-    updateCallSession(session.id, { status: 'completed' });
-    recordCallEvent(session.id, 'call_ended');
+    updateCallSession(session.id, { status: "completed" });
+    recordCallEvent(session.id, "call_ended");
 
-    const summary = await persistCallCompletionMessage(conversationId, session.id);
-    expect(summary).toBe('**Call completed**. 1 event(s) recorded.');
-    expect(getLatestAssistantText(conversationId)).toBe('**Call completed**. 1 event(s) recorded.');
+    const summary = await persistCallCompletionMessage(
+      conversationId,
+      session.id,
+    );
+    expect(summary).toBe("**Call completed**. 1 event(s) recorded.");
+    expect(getLatestAssistantText(conversationId)).toBe(
+      "**Call completed**. 1 event(s) recorded.",
+    );
   });
 });