@vellumai/assistant 0.5.6 → 0.5.8

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

@@ -175,6 +175,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
           importance: 0.7,
           fingerprint: "fp-assistant-inferred",
           verificationState: "assistant_inferred",
+          sourceType: "extraction",
+          sourceMessageRole: "assistant",
           scopeId: "default",
           firstSeenAt: now + 10,
           lastSeenAt: now + 10,
@@ -189,6 +191,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
           importance: 0.8,
           fingerprint: "fp-user-reported",
           verificationState: "user_reported",
+          sourceType: "extraction",
+          sourceMessageRole: "user",
           scopeId: "default",
           firstSeenAt: now + 20,
           lastSeenAt: now + 20,
@@ -203,6 +207,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
           importance: 0.5,
           fingerprint: "fp-other-conv",
           verificationState: "assistant_inferred",
+          sourceType: "extraction",
+          sourceMessageRole: "assistant",
           scopeId: "default",
           firstSeenAt: now + 30,
           lastSeenAt: now + 30,
@@ -217,6 +223,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
           importance: 0.4,
           fingerprint: "fp-already-superseded",
           verificationState: "assistant_inferred",
+          sourceType: "extraction",
+          sourceMessageRole: "assistant",
           scopeId: "default",
           firstSeenAt: now + 5,
           lastSeenAt: now + 5,
@@ -449,6 +457,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
         importance: 0.7,
         fingerprint: "fp-cross-sourced",
         verificationState: "assistant_inferred",
+        sourceType: "extraction",
+        sourceMessageRole: "assistant",
         scopeId: "default",
         firstSeenAt: now + 10,
         lastSeenAt: now + 20,
@@ -572,6 +582,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
         importance: 0.7,
         fingerprint: "fp-cross-sched",
         verificationState: "assistant_inferred",
+        sourceType: "extraction",
+        sourceMessageRole: "assistant",
         scopeId: "default",
         firstSeenAt: now + 10,
         lastSeenAt: now + 20,
@@ -688,6 +700,8 @@ describe("invalidateAssistantInferredItemsForConversation", () => {
         importance: 0.7,
         fingerprint: "fp-good-corroboration",
         verificationState: "assistant_inferred",
+        sourceType: "extraction",
+        sourceMessageRole: "assistant",
         scopeId: "default",
         firstSeenAt: now + 10,
         lastSeenAt: now + 20,

package/src/__tests__/tool-executor-shell-integration.test.ts

@@ -330,7 +330,7 @@ describe("ToolExecutor → real shell allowlist integration", () => {
     expect(patterns).toContain("action:git");
   });
 
-  test("pipeline command produces only exact option", async () => {
+  test("pipeline command produces exact + action-key options", async () => {
     const { prompter, getAllowlist } = makeCapturingPrompter();
     const executor = new ToolExecutor(prompter);
 
@@ -343,9 +343,11 @@ describe("ToolExecutor → real shell allowlist integration", () => {
     const allowlist = getAllowlist();
     expect(allowlist).toBeDefined();
 
-    // Pipelines are complex commands — only exact option, no action keys
-    expect(allowlist!.length).toBe(1);
+    // Pipelines now produce exact option + action key options
+    expect(allowlist!.length).toBeGreaterThanOrEqual(2);
     expect(allowlist![0].pattern).toBe("cat file.txt | grep error");
     expect(allowlist![0].description).toContain("compound");
+    // Action keys from the first segment before the pipe
+    expect(allowlist!.some((o) => o.pattern.startsWith("action:"))).toBe(true);
   });
 });

package/src/__tests__/trusted-contact-lifecycle-notifications.test.ts

@@ -41,11 +41,6 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-// Mock security check to always pass
-mock.module("../security/secret-ingress.js", () => ({
-  checkIngressForSecrets: () => ({ blocked: false }),
-}));
-
 mock.module("../config/env.js", () => ({
   isHttpAuthDisabled: () => true,
   getGatewayInternalBaseUrl: () => "http://127.0.0.1:7830",

package/src/__tests__/trusted-contact-multichannel.test.ts

@@ -36,10 +36,6 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-mock.module("../security/secret-ingress.js", () => ({
-  checkIngressForSecrets: () => ({ blocked: false }),
-}));
-
 mock.module("../config/env.js", () => ({
   isHttpAuthDisabled: () => true,
   getGatewayInternalBaseUrl: () => "http://127.0.0.1:7830",

package/src/__tests__/twilio-routes-twiml.test.ts

@@ -2,7 +2,8 @@
  * Unit tests for TwiML generation with voice quality profiles.
  *
  * Tests that generateTwiML correctly uses profile values for
- * ttsProvider, voice, language, and transcriptionProvider.
+ * ttsProvider, voice, language, transcriptionProvider,
+ * and interruptSensitivity.
  */
 import { describe, expect, mock, test } from "bun:test";
 
@@ -26,6 +27,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "Google",
       voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('ttsProvider="Google"');
@@ -40,6 +42,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "ElevenLabs",
       voice: "voice123-turbo_v2_5-1_0.5_0.75",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('ttsProvider="ElevenLabs"');
@@ -52,6 +55,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "Google",
       voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('voice="Google.en-US-Journey-O"');
@@ -63,6 +67,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "ElevenLabs",
       voice: "abc123-turbo_v2_5-1_0.5_0.75",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('voice="abc123-turbo_v2_5-1_0.5_0.75"');
@@ -74,6 +79,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Google",
       ttsProvider: "Google",
       voice: "Google.es-MX-Standard-A",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('language="es-MX"');
@@ -85,6 +91,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Google",
       ttsProvider: "Google",
       voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('transcriptionProvider="Google"');
@@ -96,6 +103,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "Google",
       voice: 'voice<>&"test',
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('voice="voice&lt;&gt;&amp;&quot;test"');
@@ -108,6 +116,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "Google",
       voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain(`callSessionId=${callSessionId}`);
@@ -119,6 +128,7 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "Google",
       voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).toContain('interruptible="true"');
@@ -131,8 +141,136 @@ describe("generateTwiML with voice quality profile", () => {
       transcriptionProvider: "Deepgram",
       ttsProvider: "Google",
       voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
     });
 
     expect(twiml).not.toContain("welcomeGreeting=");
   });
+
+  test('TwiML includes interruptSensitivity="low" when profile has low', () => {
+    const twiml = generateTwiML(callSessionId, relayUrl, welcomeGreeting, {
+      language: "en-US",
+      transcriptionProvider: "Deepgram",
+      ttsProvider: "Google",
+      voice: "Google.en-US-Journey-O",
+      interruptSensitivity: "low",
+    });
+
+    expect(twiml).toContain('interruptSensitivity="low"');
+  });
+
+  test("custom interruptSensitivity values are reflected correctly", () => {
+    const twimlMedium = generateTwiML(
+      callSessionId,
+      relayUrl,
+      welcomeGreeting,
+      {
+        language: "en-US",
+        transcriptionProvider: "Deepgram",
+        ttsProvider: "Google",
+        voice: "Google.en-US-Journey-O",
+        interruptSensitivity: "medium",
+      },
+    );
+
+    expect(twimlMedium).toContain('interruptSensitivity="medium"');
+
+    const twimlHigh = generateTwiML(
+      callSessionId,
+      relayUrl,
+      welcomeGreeting,
+      {
+        language: "en-US",
+        transcriptionProvider: "Deepgram",
+        ttsProvider: "Google",
+        voice: "Google.en-US-Journey-O",
+        interruptSensitivity: "high",
+      },
+    );
+
+    expect(twimlHigh).toContain('interruptSensitivity="high"');
+  });
+
+  test("hints attribute present when hints string is non-empty", () => {
+    const twiml = generateTwiML(
+      callSessionId,
+      relayUrl,
+      welcomeGreeting,
+      {
+        language: "en-US",
+        transcriptionProvider: "Deepgram",
+        ttsProvider: "ElevenLabs",
+        voice: "voice123",
+        interruptSensitivity: "low",
+      },
+      undefined,
+      undefined,
+      "Alice,Bob,Vellum",
+    );
+
+    expect(twiml).toContain('hints="Alice,Bob,Vellum"');
+  });
+
+  test("hints attribute omitted when hints string is empty", () => {
+    const twiml = generateTwiML(
+      callSessionId,
+      relayUrl,
+      welcomeGreeting,
+      {
+        language: "en-US",
+        transcriptionProvider: "Deepgram",
+        ttsProvider: "ElevenLabs",
+        voice: "voice123",
+        interruptSensitivity: "low",
+      },
+      undefined,
+      undefined,
+      "",
+    );
+
+    expect(twiml).not.toContain("hints=");
+  });
+
+  test("hints attribute omitted when hints parameter is undefined", () => {
+    const twiml = generateTwiML(
+      callSessionId,
+      relayUrl,
+      welcomeGreeting,
+      {
+        language: "en-US",
+        transcriptionProvider: "Deepgram",
+        ttsProvider: "ElevenLabs",
+        voice: "voice123",
+        interruptSensitivity: "low",
+      },
+      undefined,
+      undefined,
+      undefined,
+    );
+
+    expect(twiml).not.toContain("hints=");
+  });
+
+  test("XML special characters in hints are escaped properly", () => {
+    const twiml = generateTwiML(
+      callSessionId,
+      relayUrl,
+      welcomeGreeting,
+      {
+        language: "en-US",
+        transcriptionProvider: "Deepgram",
+        ttsProvider: "ElevenLabs",
+        voice: "voice123",
+        interruptSensitivity: "low",
+      },
+      undefined,
+      undefined,
+      'O\'Brien,Smith & Jones,"Dr. Lee"',
+    );
+
+    expect(twiml).toContain(
+      'hints="O&apos;Brien,Smith &amp; Jones,&quot;Dr. Lee&quot;"',
+    );
+    expect(twiml).not.toContain("hints=\"O'Brien");
+  });
 });

package/src/__tests__/update-bulletin.test.ts

@@ -66,8 +66,6 @@ mock.module("../util/platform.js", () => ({
   getSandboxWorkingDir: () => "",
   getInterfacesDir: () => "",
   getClipboardCommand: () => null,
-  readLockfile: () => null,
-  writeLockfile: () => {},
   readPlatformToken: () => null,
   readSessionToken: () => null,
   getTCPPort: () => 8765,

package/src/__tests__/vellum-self-knowledge-inline-command.test.ts

@@ -61,9 +61,7 @@ const platformOverrides: Record<string, (...args: unknown[]) => unknown> = {
   getSessionTokenPath: () => join(TEST_DIR, "session-token"),
   readSessionToken: () => null,
   getClipboardCommand: () => null,
-  readLockfile: () => null,
   normalizeAssistantId: (id: unknown) => String(id),
-  writeLockfile: () => {},
   getEmbeddingModelsDir: () => join(TEST_DIR, "embedding-models"),
   getTCPPort: () => 8765,
   isTCPEnabled: () => false,
@@ -141,7 +139,6 @@ interface TestConfig {
   permissions: { mode: "strict" | "workspace" };
   skills: { load: { extraDirs: string[] } };
   sandbox: { enabled: boolean };
-  assistantFeatureFlagValues?: Record<string, boolean>;
   [key: string]: unknown;
 }
 
@@ -149,9 +146,6 @@ const testConfig: TestConfig = {
   permissions: { mode: "workspace" },
   skills: { load: { extraDirs: [] } },
   sandbox: { enabled: true },
-  assistantFeatureFlagValues: {
-    "feature_flags.inline-skill-commands.enabled": true,
-  },
 };
 
 mock.module("../config/loader.js", () => ({
@@ -169,6 +163,8 @@ mock.module("../config/loader.js", () => ({
 
 await import("../tools/skills/load.js");
 const { getTool } = await import("../tools/registry.js");
+const { _setOverridesForTesting } =
+  await import("../config/assistant-feature-flags.js");
 
 // ── Helpers ──────────────────────────────────────────────────────────────
 
@@ -228,16 +224,17 @@ describe("vellum-self-knowledge inline command expansion", () => {
       ) => mockRunInlineCommand(command, workingDir),
     }));
 
-    // Enable the feature flag
-    testConfig.assistantFeatureFlagValues = {
+    // Enable the feature flag via protected directory override
+    _setOverridesForTesting({
       "feature_flags.inline-skill-commands.enabled": true,
-    };
+    });
     testConfig.skills = { load: { extraDirs: [] } };
 
     installSelfKnowledgeSkill();
   });
 
   afterEach(() => {
+    _setOverridesForTesting({});
     if (existsSync(TEST_DIR)) {
       rmSync(TEST_DIR, { recursive: true, force: true });
     }

package/src/__tests__/voice-quality.test.ts

@@ -124,4 +124,62 @@ describe("resolveVoiceQualityProfile", () => {
     const profile = resolveVoiceQualityProfile();
     expect(profile.voice).toBe("voice1-turbo_v2_5-0.9_0.8_0.9");
   });
+
+  test("interruptSensitivity defaults to 'low' when not configured", () => {
+    mockConfig = {
+      elevenlabs: { voiceId: "abc" },
+      calls: {
+        voice: {
+          language: "en-US",
+          transcriptionProvider: "Deepgram",
+        },
+      },
+    };
+    const profile = resolveVoiceQualityProfile();
+    expect(profile.interruptSensitivity).toBe("low");
+  });
+
+  test("interruptSensitivity reflects configured value", () => {
+    mockConfig = {
+      elevenlabs: { voiceId: "abc" },
+      calls: {
+        voice: {
+          language: "en-US",
+          transcriptionProvider: "Deepgram",
+          interruptSensitivity: "high",
+        },
+      },
+    };
+    const profile = resolveVoiceQualityProfile();
+    expect(profile.interruptSensitivity).toBe("high");
+  });
+
+  test("hints defaults to empty array when not configured", () => {
+    mockConfig = {
+      elevenlabs: { voiceId: "abc" },
+      calls: {
+        voice: {
+          language: "en-US",
+          transcriptionProvider: "Deepgram",
+        },
+      },
+    };
+    const profile = resolveVoiceQualityProfile();
+    expect(profile.hints).toEqual([]);
+  });
+
+  test("hints reflects configured values", () => {
+    mockConfig = {
+      elevenlabs: { voiceId: "abc" },
+      calls: {
+        voice: {
+          language: "en-US",
+          transcriptionProvider: "Deepgram",
+          hints: ["Vellum", "Velissa", "AI assistant"],
+        },
+      },
+    };
+    const profile = resolveVoiceQualityProfile();
+    expect(profile.hints).toEqual(["Vellum", "Velissa", "AI assistant"]);
+  });
 });

package/src/__tests__/voice-scoped-grant-consumer.test.ts

@@ -54,7 +54,6 @@ mock.module("../config/loader.js", () => ({
     ui: {},
 
     provider: "anthropic",
-    providerOrder: ["anthropic"],
     calls: {
       enabled: true,
       provider: "twilio",
@@ -70,12 +69,6 @@ mock.module("../config/loader.js", () => ({
   }),
 }));
 
-// ── Secret ingress mock ────────────────────────────────────────────
-
-mock.module("../security/secret-ingress.js", () => ({
-  checkIngressForSecrets: () => ({ blocked: false }),
-}));
-
 // ── Assistant event hub mock ───────────────────────────────────────
 
 mock.module("../runtime/assistant-event-hub.js", () => ({