@vellumai/assistant 0.4.43 → 0.4.45

package/src/__tests__/{request-file-tool.test.ts → ui-file-upload-surface.test.ts}

@@ -6,81 +6,10 @@ import type {
 } from "../daemon/message-protocol.js";
 import { INTERACTIVE_SURFACE_TYPES } from "../daemon/message-protocol.js";
 import {
-  requestFileTool,
+  allUiSurfaceTools,
   uiShowTool,
 } from "../tools/ui-surface/definitions.js";
 
-// ---------------------------------------------------------------------------
-// request_file tool definition
-// ---------------------------------------------------------------------------
-
-describe("requestFileTool definition", () => {
-  test("has the correct name and category", () => {
-    expect(requestFileTool.name).toBe("request_file");
-    expect(requestFileTool.category).toBe("ui-surface");
-    expect(requestFileTool.executionMode).toBe("proxy");
-  });
-
-  test("getDefinition returns correct input_schema", () => {
-    const definition = requestFileTool.getDefinition();
-
-    expect(definition.name).toBe("request_file");
-    expect(definition.description).toContain("file");
-
-    const schema = definition.input_schema as {
-      type: string;
-      properties: Record<string, unknown>;
-      required: string[];
-    };
-
-    expect(schema.type).toBe("object");
-    expect(schema.required).toEqual(["prompt"]);
-    expect(schema.properties).toHaveProperty("prompt");
-    expect(schema.properties).toHaveProperty("accepted_types");
-    expect(schema.properties).toHaveProperty("max_files");
-  });
-
-  test("prompt property is a string type", () => {
-    const definition = requestFileTool.getDefinition();
-    const props = (
-      definition.input_schema as {
-        properties: Record<string, { type: string }>;
-      }
-    ).properties;
-
-    expect(props.prompt.type).toBe("string");
-  });
-
-  test("accepted_types property is an array of strings", () => {
-    const definition = requestFileTool.getDefinition();
-    const props = (
-      definition.input_schema as {
-        properties: Record<string, { type: string; items?: { type: string } }>;
-      }
-    ).properties;
-
-    expect(props.accepted_types.type).toBe("array");
-    expect(props.accepted_types.items).toEqual({ type: "string" });
-  });
-
-  test("max_files property is a number type", () => {
-    const definition = requestFileTool.getDefinition();
-    const props = (
-      definition.input_schema as {
-        properties: Record<string, { type: string }>;
-      }
-    ).properties;
-
-    expect(props.max_files.type).toBe("number");
-  });
-
-  test("execute throws proxy error", () => {
-    expect(() => requestFileTool.execute({}, {} as never)).toThrow(
-      "Proxy tool: execution must be forwarded to the connected client",
-    );
-  });
-});
-
 // ---------------------------------------------------------------------------
 // FileUploadSurfaceData shape
 // ---------------------------------------------------------------------------
@@ -164,3 +93,13 @@ describe("ui_show tool includes file_upload", () => {
     expect(definition.description).toContain("file_upload");
   });
 });
+
+describe("UI surface tool registration", () => {
+  test("registers only the base UI surface tools", () => {
+    expect(allUiSurfaceTools.map((tool) => tool.name)).toEqual([
+      "ui_show",
+      "ui_update",
+      "ui_dismiss",
+    ]);
+  });
+});

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

@@ -55,7 +55,6 @@ mock.module("../util/platform.js", () => ({
   getLogPath: () => "",
   getHistoryPath: () => "",
   getHooksDir: () => "",
-  getSocketPath: () => "",
   getSessionTokenPath: () => "",
   getPlatformTokenPath: () => "",
   getPidPath: () => "",

package/src/__tests__/usage-cache-backfill-migration.test.ts

@@ -16,7 +16,6 @@ mock.module("../util/platform.js", () => ({
   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"),

package/src/__tests__/usage-routes.test.ts

@@ -10,7 +10,6 @@ mock.module("../util/platform.js", () => ({
   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"),

package/src/__tests__/verification-control-plane-policy.test.ts

@@ -737,8 +737,8 @@ describe("ToolExecutor verification control-plane policy gate", () => {
   test("unverified channel actor is blocked from side-effect tools", async () => {
     const executor = new ToolExecutor(makePrompter());
     const result = await executor.execute(
-      "reminder_create",
-      { fire_at: "2026-02-27T12:00:00-05:00", label: "test", message: "hello" },
+      "schedule_create",
+      { name: "test", fire_at: "2026-02-27T12:00:00-05:00", message: "hello" },
       makeContext({ trustClass: "unknown" }),
     );
     expect(result.isError).toBe(true);
@@ -748,8 +748,8 @@ describe("ToolExecutor verification control-plane policy gate", () => {
   test("guardian actor can execute side-effect tools", async () => {
     const executor = new ToolExecutor(makePrompter());
     const result = await executor.execute(
-      "reminder_create",
-      { fire_at: "2026-02-27T12:00:00-05:00", label: "test", message: "hello" },
+      "schedule_create",
+      { name: "test", fire_at: "2026-02-27T12:00:00-05:00", message: "hello" },
       makeContext({ trustClass: "guardian" }),
     );
     expect(result.isError).toBe(false);

package/src/__tests__/voice-invite-redemption.test.ts

@@ -10,7 +10,6 @@ mock.module("../util/platform.js", () => ({
   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"),

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

@@ -33,7 +33,6 @@ mock.module("../util/platform.js", () => ({
   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"),

package/src/__tests__/voice-session-bridge.test.ts

@@ -26,7 +26,6 @@ mock.module("../util/platform.js", () => ({
   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"),
@@ -73,6 +72,7 @@ function makeStreamingSession(events: ServerMessage[]): Session {
     setTurnChannelContext: () => {},
     setVoiceCallControlPrompt: () => {},
     updateClient: () => {},
+    ensureActorScopedHistory: async () => {},
     runAgentLoop: async (
       _content: string,
       _messageId: string,
@@ -212,6 +212,7 @@ describe("voice-session-bridge", () => {
       setTurnChannelContext: () => {},
       setVoiceCallControlPrompt: () => {},
       updateClient: () => {},
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         await new Promise((r) => setTimeout(r, 200));
       },
@@ -263,6 +264,7 @@ describe("voice-session-bridge", () => {
       setTurnChannelContext: () => {},
       setVoiceCallControlPrompt: () => {},
       updateClient: () => {},
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         await new Promise((r) => setTimeout(r, 200));
       },
@@ -640,6 +642,7 @@ describe("voice-session-bridge", () => {
       updateClient: (handler: (msg: ServerMessage) => void) => {
         clientHandler = handler;
       },
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         // Simulate the prompter emitting a confirmation_request via the
         // updateClient callback (this is how the real prompter works).
@@ -724,6 +727,7 @@ describe("voice-session-bridge", () => {
       updateClient: (handler: (msg: ServerMessage) => void) => {
         clientHandler = handler;
       },
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         clientHandler({
           type: "confirmation_request",
@@ -791,6 +795,7 @@ describe("voice-session-bridge", () => {
       updateClient: (handler: (msg: ServerMessage) => void) => {
         clientHandler = handler;
       },
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         clientHandler({
           type: "confirmation_request",
@@ -854,6 +859,7 @@ describe("voice-session-bridge", () => {
       updateClient: (handler: (msg: ServerMessage) => void) => {
         clientHandler = handler;
       },
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         clientHandler({
           type: "confirmation_request",
@@ -926,6 +932,7 @@ describe("voice-session-bridge", () => {
       updateClient: (handler: (msg: ServerMessage) => void) => {
         clientHandler = handler;
       },
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         clientHandler({
           type: "secret_request",
@@ -998,6 +1005,7 @@ describe("voice-session-bridge", () => {
       setTurnChannelContext: () => {},
       setVoiceCallControlPrompt: () => {},
       updateClient: () => {},
+      ensureActorScopedHistory: async () => {},
       runAgentLoop: async () => {
         await new Promise((r) => setTimeout(r, 200));
       },

package/src/__tests__/web-fetch.test.ts

@@ -1605,4 +1605,61 @@ describe("web_fetch tool", () => {
     expect(result.content).toContain("<title>Fake Title</title>");
     expect(result.content).toContain("Regular markdown content.");
   });
+
+  test("suggests JS rendering may be needed when HTML page returns very little text content", async () => {
+    const spaHtml =
+      '<!doctype html><html><head><title>My App</title></head><body><div id="root"></div><script src="/app.js"></script></body></html>';
+    const result = await executeWithMockFetch(
+      { url: "https://example.com/spa" },
+      {
+        resolveHostAddresses: async () => ["93.184.216.34"],
+        requestExecutor: async () =>
+          new Response(spaHtml, {
+            status: 200,
+            headers: { "content-type": "text/html; charset=utf-8" },
+          }),
+      },
+    );
+
+    expect(result.isError).toBe(false);
+    expect(result.content).toContain("Extracted text content is very short");
+    expect(result.content).toContain("JavaScript rendering");
+  });
+
+  test("does not suggest browser skill when HTML page has substantial content", async () => {
+    const richHtml = `<!doctype html><html><head><title>Docs</title></head><body><p>${"Lorem ipsum dolor sit amet. ".repeat(20)}</p></body></html>`;
+    const result = await executeWithMockFetch(
+      { url: "https://example.com/docs" },
+      {
+        resolveHostAddresses: async () => ["93.184.216.34"],
+        requestExecutor: async () =>
+          new Response(richHtml, {
+            status: 200,
+            headers: { "content-type": "text/html; charset=utf-8" },
+          }),
+      },
+    );
+
+    expect(result.isError).toBe(false);
+    expect(result.content).not.toContain("Extracted text content is very short");
+  });
+
+  test("does not suggest JS rendering notice in raw mode even for sparse HTML", async () => {
+    const spaHtml =
+      '<!doctype html><html><head><title>My App</title></head><body><div id="root"></div><script src="/app.js"></script></body></html>';
+    const result = await executeWithMockFetch(
+      { url: "https://example.com/spa", raw: true },
+      {
+        resolveHostAddresses: async () => ["93.184.216.34"],
+        requestExecutor: async () =>
+          new Response(spaHtml, {
+            status: 200,
+            headers: { "content-type": "text/html; charset=utf-8" },
+          }),
+      },
+    );
+
+    expect(result.isError).toBe(false);
+    expect(result.content).not.toContain("Extracted text content is very short");
+  });
 });

package/src/__tests__/workspace-git-service.test.ts

@@ -63,11 +63,9 @@ describe("WorkspaceGitService", () => {
       const content = readFileSync(gitignorePath, "utf-8");
       expect(content).toContain("data/db/");
       expect(content).toContain("data/qdrant/");
-      expect(content).toContain("data/ipc-blobs/");
       expect(content).toContain("*.log");
       expect(content).toContain("*.sock");
       expect(content).toContain("*.pid");
-      expect(content).toContain("vellum.sock");
       expect(content).toContain("session-token");
     });
 
@@ -631,7 +629,6 @@ describe("WorkspaceGitService", () => {
       // Verify .gitignore does NOT have our rules yet
       const contentBefore = readFileSync(join(testDir, ".gitignore"), "utf-8");
       expect(contentBefore).not.toContain("data/db/");
-      expect(contentBefore).not.toContain("vellum.sock");
 
       // Initialize the service — should append rules
       const service = new WorkspaceGitService(testDir);
@@ -641,9 +638,8 @@ describe("WorkspaceGitService", () => {
       expect(contentAfter).toContain("node_modules/"); // original rule preserved
       expect(contentAfter).toContain("data/db/");
       expect(contentAfter).toContain("data/qdrant/");
-      expect(contentAfter).toContain("data/ipc-blobs/");
       expect(contentAfter).toContain("*.log");
-      expect(contentAfter).toContain("vellum.sock");
+      expect(contentAfter).toContain("*.sock");
       expect(contentAfter).toContain("session-token");
     });
 
@@ -655,7 +651,7 @@ describe("WorkspaceGitService", () => {
         cwd: testDir,
       });
       const oldGitignore =
-        "# Runtime state - excluded from git tracking\ndata/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.sock\nvellum.pid\nsession-token\n";
+        "# Runtime state - excluded from git tracking\ndata/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.pid\nsession-token\n";
       writeFileSync(join(testDir, ".gitignore"), oldGitignore);
       writeFileSync(join(testDir, "file.txt"), "content");
       execFileSync("git", ["add", "-A"], { cwd: testDir });
@@ -677,12 +673,11 @@ describe("WorkspaceGitService", () => {
       // New selective rules should be present
       expect(contentAfter).toContain("data/db/");
       expect(contentAfter).toContain("data/qdrant/");
-      expect(contentAfter).toContain("data/ipc-blobs/");
 
       // Other existing rules should be preserved
       expect(contentAfter).toContain("logs/");
       expect(contentAfter).toContain("*.log");
-      expect(contentAfter).toContain("vellum.sock");
+      expect(contentAfter).toContain("*.sock");
     });
 
     test("existing repo gets local identity set on init", async () => {
@@ -725,7 +720,7 @@ describe("WorkspaceGitService", () => {
         cwd: testDir,
       });
       const gitignoreContent =
-        "# Runtime state - excluded from git tracking\ndata/db/\ndata/qdrant/\ndata/ipc-blobs/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.sock\nvellum.pid\nsession-token\n";
+        "# Runtime state - excluded from git tracking\ndata/db/\ndata/qdrant/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.pid\nsession-token\n";
       writeFileSync(join(testDir, ".gitignore"), gitignoreContent);
       writeFileSync(join(testDir, "file.txt"), "content");
       execFileSync("git", ["add", "-A"], { cwd: testDir });
@@ -778,9 +773,6 @@ describe("WorkspaceGitService", () => {
         join(testDir, "data", "qdrant", "index.bin"),
         "qdrant content",
       );
-      mkdirSync(join(testDir, "data", "ipc-blobs"), { recursive: true });
-      writeFileSync(join(testDir, "data", "ipc-blobs", "blob1"), "ipc content");
-
       // Create files in tracked data subdirectories
       mkdirSync(join(testDir, "data", "memory"), { recursive: true });
       writeFileSync(join(testDir, "data", "memory", "index.json"), "{}");
@@ -799,9 +791,8 @@ describe("WorkspaceGitService", () => {
       // Ignored subdirectories should NOT be in the commit
       expect(committedFiles).not.toContain("data/db/");
       expect(committedFiles).not.toContain("data/qdrant/");
-      expect(committedFiles).not.toContain("data/ipc-blobs/");
 
-      // Tracked subdirectories SHOULD be in the commit
+      // Non-ignored data subdirectories SHOULD be in the commit
       expect(committedFiles).toContain("data/memory/index.json");
       expect(committedFiles).toContain("data/apps/state.json");
     });

package/src/__tests__/workspace-policy.test.ts

@@ -257,7 +257,6 @@ describe("isWorkspaceScopedInvocation", () => {
   describe("always-scoped tools", () => {
     const safeTools = [
       "skill_load",
-      "view_image",
       "memory_recall",
       "ui_update",
       "ui_dismiss",

package/src/agent/loop.ts

@@ -27,7 +27,6 @@ export interface AgentLoopConfig {
     | { type: "auto" }
     | { type: "any" }
     | { type: "tool"; name: string };
-  maxToolUseTurns?: number;
   /** Minimum interval (ms) between consecutive LLM calls to prevent spin when tools return instantly */
   minTurnIntervalMs?: number;
 }
@@ -65,7 +64,14 @@ export type AgentEvent =
       status?: string;
       contentBlocks?: ContentBlock[];
     }
-  | { type: "input_json_delta"; toolName: string; accumulatedJson: string }
+  | { type: "tool_use_preview_start"; toolUseId: string; toolName: string }
+  | {
+      type: "input_json_delta";
+      toolName: string;
+      toolUseId: string;
+      accumulatedJson: string;
+    }
+  | { type: "server_tool_start"; name: string; toolUseId: string }
   | { type: "error"; error: Error }
   | {
       type: "usage";
@@ -82,7 +88,6 @@ export type AgentEvent =
 const DEFAULT_CONFIG: AgentLoopConfig = {
   maxTokens: 16000,
   effort: "high",
-  maxToolUseTurns: 40,
   minTurnIntervalMs: 150,
 };
 
@@ -90,11 +95,6 @@ const PROGRESS_CHECK_INTERVAL = 5;
 const PROGRESS_CHECK_REMINDER =
   "You have been using tools for several turns. Check whether you are making meaningful progress toward the user's goal. If you are stuck in a loop or not making progress, summarize what you have tried and ask the user for guidance instead of continuing.";
 
-// Warn the model N turns before the hard limit so it can wrap up gracefully
-const APPROACHING_LIMIT_OFFSET = 5;
-const APPROACHING_LIMIT_WARNING =
-  "You are approaching the tool-use turn limit. You have {remaining} turns remaining. Wrap up your current task — summarize progress and present results to the user. If you cannot finish, explain what remains and ask the user how to proceed.";
-
 export interface ResolvedSystemPrompt {
   systemPrompt: string;
   maxTokens?: number;
@@ -301,12 +301,25 @@ export class AgentLoop {
                 }
               } else if (event.type === "thinking_delta") {
                 onEvent({ type: "thinking_delta", thinking: event.thinking });
+              } else if (event.type === "tool_use_preview_start") {
+                onEvent({
+                  type: "tool_use_preview_start",
+                  toolUseId: event.toolUseId,
+                  toolName: event.toolName,
+                });
               } else if (event.type === "input_json_delta") {
                 onEvent({
                   type: "input_json_delta",
                   toolName: event.toolName,
+                  toolUseId: event.toolUseId,
                   accumulatedJson: event.accumulatedJson,
                 });
+              } else if (event.type === "server_tool_start") {
+                onEvent({
+                  type: "server_tool_start",
+                  name: event.name,
+                  toolUseId: event.toolUseId,
+                });
               }
             },
             signal,
@@ -627,32 +640,7 @@ export class AgentLoop {
 
         // Track tool-use turns and inject progress reminder every N turns
         toolUseTurns++;
-        if (
-          this.config.maxToolUseTurns &&
-          this.config.maxToolUseTurns > 0 &&
-          toolUseTurns >= this.config.maxToolUseTurns
-        ) {
-          const limitMessage = `Tool-use turn limit reached (${this.config.maxToolUseTurns}). Stopping to prevent runaway loops; ask the user for guidance.`;
-          onEvent({ type: "error", error: new Error(limitMessage) });
-          resultBlocks.push({
-            type: "text",
-            text: `<system_notice>${limitMessage}</system_notice>`,
-          });
-          history.push({ role: "user", content: resultBlocks });
-          break;
-        }
-        // Soft warning a few turns before the hard limit
-        const softLimit =
-          (this.config.maxToolUseTurns ?? 0) - APPROACHING_LIMIT_OFFSET;
-        if (softLimit > 0 && toolUseTurns === softLimit) {
-          resultBlocks.push({
-            type: "text",
-            text: `<system_notice>${APPROACHING_LIMIT_WARNING.replace(
-              "{remaining}",
-              String(APPROACHING_LIMIT_OFFSET),
-            )}</system_notice>`,
-          });
-        } else if (toolUseTurns % PROGRESS_CHECK_INTERVAL === 0) {
+        if (toolUseTurns % PROGRESS_CHECK_INTERVAL === 0) {
           resultBlocks.push({
             type: "text",
             text: `<system_notice>${PROGRESS_CHECK_REMINDER}</system_notice>`,

package/src/bundler/bundle-signer.ts

@@ -2,7 +2,7 @@
  * Bundle signing for .vellum archives.
  *
  * Computes content hashes, constructs a canonical signing payload,
- * and requests an Ed25519 signature from the Swift client via IPC.
+ * and requests an Ed25519 signature from the Swift client.
  */
 
 import { createHash } from "node:crypto";
@@ -22,8 +22,8 @@ export interface SignatureJson {
 }
 
 /**
- * Callback type for requesting a signature from the Swift client via IPC.
- * The caller provides this so the signer doesn't need to know about IPC details.
+ * Callback type for requesting a signature from the Swift client.
+ * The caller provides this so the signer doesn't need to know about transport details.
  */
 export type SigningCallback = (payload: string) => Promise<{
   signature: string; // base64-encoded
@@ -109,7 +109,7 @@ export async function signBundle(
   });
   const canonicalPayload = JSON.stringify(signingPayload);
 
-  // 4. Request signature from Swift via IPC
+  // 4. Request signature from Swift client
   const { signature, keyId } = await requestSignature(canonicalPayload);
 
   // 5. Build SignatureJson

package/src/calls/call-controller.ts

@@ -108,7 +108,7 @@ export class CallController {
   private awaitingOpeningAck = false;
   /** Monotonic run id used to suppress stale turn side effects after interruption. */
   private llmRunVersion = 0;
-  /** Optional broadcast function for emitting IPC events to connected clients. */
+  /** Optional broadcast function for emitting events to connected clients. */
   private broadcast?: (msg: ServerMessage) => void;
   /** Assistant identity for scoping guardian bindings. */
   private assistantId: string;

package/src/calls/relay-server.ts

@@ -155,7 +155,7 @@ let globalBroadcast:
   | ((msg: import("../daemon/message-protocol.js").ServerMessage) => void)
   | undefined;
 
-/** Register a broadcast function so RelayConnection can forward IPC events. */
+/** Register a broadcast function so RelayConnection can forward events to connected clients. */
 export function setRelayBroadcast(
   fn: (msg: import("../daemon/message-protocol.js").ServerMessage) => void,
 ): void {

package/src/calls/twilio-rest.ts

@@ -2,7 +2,7 @@
  * Reusable Twilio REST API helpers.
  *
  * Provides low-level building blocks (auth header, base URL, credential
- * resolution) shared across the voice provider and IPC
+ * resolution) shared across the voice provider and the
  * config handler. Uses fetch() directly — no twilio npm package.
  */
 

package/src/calls/voice-session-bridge.ts

@@ -471,7 +471,7 @@ export async function startVoiceTurn(
 
   // Fire-and-forget the agent loop
   const cleanup = () => {
-    // Reset channel capabilities so a subsequent IPC/desktop session on the
+    // Reset channel capabilities so a subsequent desktop session on the
     // same conversation is not incorrectly treated as a voice client.
     session.setChannelCapabilities(null);
     session.setTrustContext(null);
@@ -513,6 +513,8 @@ export async function startVoiceTurn(
           } else if (msg.type === "tool_use_start") {
             eventSink.onToolUse(msg.toolName, msg.input);
           }
+          // Note: tool_use_preview_start is intentionally not handled here.
+          // Voice only reacts to the definitive tool_use_start event.
         },
       );
       if (lastError) {

package/src/cli/__tests__/notifications.test.ts

@@ -23,7 +23,6 @@ mock.module("../../util/platform.js", () => ({
   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"),
@@ -190,7 +189,7 @@ describe("notifications send", () => {
       "--source-channel",
       "scheduler",
       "--source-event-name",
-      "reminder.fired",
+      "schedule.notify",
       "--message",
       "Test",
       "--urgency",
@@ -269,7 +268,7 @@ describe("notifications send", () => {
     expect(parsed.error).toContain("bogus.event");
     // Should list valid event names from the registry
     expect(parsed.error).toContain("user.send_notification");
-    expect(parsed.error).toContain("reminder.fired");
+    expect(parsed.error).toContain("schedule.notify");
   });
 
   test("send rejects invalid urgency", async () => {
@@ -399,7 +398,7 @@ describe("notifications list", () => {
 
     createEvent({
       id: `evt-filter-reminder-${Date.now()}`,
-      sourceEventName: "reminder.fired",
+      sourceEventName: "schedule.notify",
       sourceChannel: "scheduler",
       sourceSessionId: "session-filter-2",
       attentionHints: {

package/src/cli/commands/map.ts

@@ -224,9 +224,7 @@ async function startLearnSession(
 
         if (status.bootstrapFailureReason) {
           reject(
-            new Error(
-              `Learn session failed: ${status.bootstrapFailureReason}`,
-            ),
+            new Error(`Learn session failed: ${status.bootstrapFailureReason}`),
           );
           return;
         }
@@ -239,9 +237,7 @@ async function startLearnSession(
             });
           } else {
             reject(
-              new Error(
-                "Learn session completed but no recording was saved.",
-              ),
+              new Error("Learn session completed but no recording was saved."),
             );
           }
           return;