@vellumai/assistant 0.4.43 → 0.4.45

package/src/daemon/handlers/config-model.ts

@@ -11,7 +11,6 @@ import type {
 import { MODEL_TO_PROVIDER } from "../session-slash.js";
 import {
   CONFIG_RELOAD_DEBOUNCE_MS,
-  defineHandlers,
   type HandlerContext,
   log,
 } from "./shared.js";
@@ -42,10 +41,13 @@ export function getModelInfo(): ModelInfo {
 
 /**
  * Minimal interface for the side-effects needed by setModel / setImageGenModel.
- * Keeps the business logic decoupled from IPC-specific HandlerContext.
+ * Keeps the business logic decoupled from transport-specific HandlerContext.
  */
 export interface ModelSetContext {
-  sessions: Map<string, { isProcessing(): boolean; dispose(): void; markStale(): void }>;
+  sessions: Map<
+    string,
+    { isProcessing(): boolean; dispose(): void; markStale(): void }
+  >;
   suppressConfigReload: boolean;
   setSuppressConfigReload(value: boolean): void;
   updateConfigFingerprint(): void;
@@ -128,10 +130,7 @@ export function setModel(modelId: string, ctx: ModelSetContext): ModelInfo {
 /**
  * Set the image generation model. Throws on failure.
  */
-export function setImageGenModel(
-  modelId: string,
-  ctx: ModelSetContext,
-): void {
+export function setImageGenModel(modelId: string, ctx: ModelSetContext): void {
   const raw = loadRawConfig();
   raw.imageGenModel = modelId;
 
@@ -156,7 +155,7 @@ export function setImageGenModel(
 }
 
 // ---------------------------------------------------------------------------
-// IPC handlers (delegate to shared logic)
+// HTTP handlers (delegate to shared logic)
 // ---------------------------------------------------------------------------
 
 export function handleModelGet(ctx: HandlerContext): void {
@@ -195,8 +194,3 @@ export function handleImageGenModelSet(
   }
 }
 
-export const modelHandlers = defineHandlers({
-  model_get: (_msg, ctx) => handleModelGet(ctx),
-  model_set: handleModelSet,
-  image_gen_model_set: handleImageGenModelSet,
-});

package/src/daemon/handlers/config-telegram.ts

@@ -22,7 +22,7 @@ import type {
   TelegramConfigRequest,
   TelegramConfigResponse,
 } from "../message-protocol.js";
-import { defineHandlers, type HandlerContext, log } from "./shared.js";
+import { type HandlerContext, log } from "./shared.js";
 
 const TELEGRAM_BOT_TOKEN_IN_URL_PATTERN =
   /\/bot\d{8,10}:[A-Za-z0-9_-]{30,120}\//g;
@@ -53,7 +53,7 @@ export function summarizeTelegramError(err: unknown): string {
   return redactTelegramBotTokens(parts.join(" "));
 }
 
-// -- Transport-agnostic result type (omits the IPC `type` discriminant) --
+// -- Transport-agnostic result type (omits the `type` discriminant) --
 
 export type TelegramConfigResult = Omit<TelegramConfigResponse, "type">;
 
@@ -359,7 +359,7 @@ export async function setupTelegram(
   };
 }
 
-// -- IPC handler (thin wrapper over extracted functions) --
+// -- Message handler (thin wrapper over extracted functions) --
 
 export async function handleTelegramConfig(
   msg: TelegramConfigRequest,
@@ -401,8 +401,4 @@ export async function handleTelegramConfig(
       error: message,
     });
   }
-}
-
-export const telegramHandlers = defineHandlers({
-  telegram_config: handleTelegramConfig,
-});
+}

package/src/daemon/handlers/config-voice.ts

@@ -1,5 +1,5 @@
 import type { VoiceConfigUpdateRequest } from "../message-types/settings.js";
-import { defineHandlers, type HandlerContext, log } from "./shared.js";
+import { type HandlerContext, log } from "./shared.js";
 
 /**
  * Send a client_settings_update message to all connected clients.
@@ -194,7 +194,7 @@ export function normalizeActivationKey(
 }
 
 /**
- * Process a voice configuration update request from a session or IPC client.
+ * Process a voice configuration update request from a session or client.
  * Validates the activation key and broadcasts the change to all connected clients.
  */
 export function handleVoiceConfigUpdate(
@@ -214,6 +214,3 @@ export function handleVoiceConfigUpdate(
   );
 }
 
-export const voiceHandlers = defineHandlers({
-  voice_config_update: handleVoiceConfigUpdate,
-});

package/src/daemon/handlers/dictation.ts

@@ -13,7 +13,7 @@ import {
   expandSnippets,
 } from "../dictation-text-processing.js";
 import type { DictationRequest } from "../message-protocol.js";
-import { defineHandlers, type HandlerContext, log } from "./shared.js";
+import { type HandlerContext, log } from "./shared.js";
 
 // Action verbs for fast heuristic fallback (used when LLM classifier is unavailable)
 const ACTION_VERBS = [
@@ -245,13 +245,7 @@ export async function handleDictationRequest(
   // Command mode: selected text present — deterministic, no classification needed
   if (msg.context.selectedText && msg.context.selectedText.trim().length > 0) {
     log.info({ mode: "command" }, "Command mode (selected text present)");
-    await handleCommandMode(
-      msg,
-      ctx,
-      profile,
-      profileMeta,
-      stylePrompt,
-    );
+    await handleCommandMode(msg, ctx, profile, profileMeta, stylePrompt);
     return;
   }
 
@@ -477,7 +471,3 @@ async function handleCommandMode(
     });
   }
 }
-
-export const dictationHandlers = defineHandlers({
-  dictation_request: handleDictationRequest,
-});

package/src/daemon/handlers/identity.ts

@@ -1,10 +1,3 @@
-import { existsSync, readFileSync, statSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-
-import { getWorkspacePromptPath, readLockfile } from "../../util/platform.js";
-import { defineHandlers, type HandlerContext, log } from "./shared.js";
-
 export interface IdentityFields {
   name: string;
   role: string;
@@ -58,101 +51,3 @@ export function parseIdentityFields(content: string): IdentityFields {
     home: fields.home ?? "",
   };
 }
-
-function handleIdentityGet(ctx: HandlerContext): void {
-  const identityPath = getWorkspacePromptPath("IDENTITY.md");
-
-  if (!existsSync(identityPath)) {
-    ctx.send({
-      type: "identity_get_response",
-      found: false,
-      name: "",
-      role: "",
-      personality: "",
-      emoji: "",
-      home: "",
-    });
-    return;
-  }
-
-  try {
-    const content = readFileSync(identityPath, "utf-8");
-    const fields = parseIdentityFields(content);
-
-    // Read version from package.json
-    let version: string | undefined;
-    try {
-      const pkgPath = join(
-        dirname(fileURLToPath(import.meta.url)),
-        "../../../package.json",
-      );
-      const pkg = JSON.parse(readFileSync(pkgPath, "utf-8"));
-      version = pkg.version;
-    } catch {
-      // ignore
-    }
-
-    // Read createdAt from IDENTITY.md file birthtime
-    let createdAt: string | undefined;
-    try {
-      const stats = statSync(identityPath);
-      createdAt = stats.birthtime.toISOString();
-    } catch {
-      // ignore
-    }
-
-    // Read lockfile for assistantId, cloud, and originSystem
-    let assistantId: string | undefined;
-    let cloud: string | undefined;
-    let originSystem: string | undefined;
-    try {
-      const lockData = readLockfile();
-      const assistants = lockData?.assistants as
-        | Array<Record<string, unknown>>
-        | undefined;
-      if (assistants && assistants.length > 0) {
-        // Use the most recently hatched assistant
-        const sorted = [...assistants].sort((a, b) => {
-          const dateA = new Date((a.hatchedAt as string) || 0).getTime();
-          const dateB = new Date((b.hatchedAt as string) || 0).getTime();
-          return dateB - dateA;
-        });
-        const latest = sorted[0];
-        assistantId = latest.assistantId as string | undefined;
-        cloud = latest.cloud as string | undefined;
-        originSystem = cloud === "local" ? "local" : cloud;
-      }
-    } catch {
-      // ignore — lockfile may not exist
-    }
-
-    ctx.send({
-      type: "identity_get_response",
-      found: true,
-      name: fields.name,
-      role: fields.role,
-      personality: fields.personality,
-      emoji: fields.emoji,
-      home: fields.home,
-      version,
-      assistantId,
-      createdAt,
-      originSystem,
-    });
-  } catch (err) {
-    log.error({ err }, "Failed to read identity");
-    ctx.send({
-      type: "identity_get_response",
-      found: false,
-      name: "",
-      role: "",
-      personality: "",
-      emoji: "",
-      home: "",
-    });
-  }
-}
-
-export const identityHandlers = defineHandlers({
-  identity_get: (_msg, ctx) => handleIdentityGet(ctx),
-});

package/src/daemon/handlers/recording.ts

@@ -9,11 +9,7 @@ import {
 } from "../../memory/attachments-store.js";
 import { addMessage } from "../../memory/conversation-crud.js";
 import type { RecordingOptions, RecordingStatus } from "../message-protocol.js";
-import {
-  defineHandlers,
-  type HandlerContext,
-  log,
-} from "./shared.js";
+import { type HandlerContext, log } from "./shared.js";
 
 // ─── Constants ───────────────────────────────────────────────────────────────
 
@@ -501,7 +497,7 @@ export async function finalizeAndPublishRecording(params: {
   }
 
   // Restrict accepted file paths to the app's recordings directory to
-  // prevent attachment of arbitrary files via crafted IPC messages.
+  // prevent attachment of arbitrary files via crafted messages.
   let resolvedPath: string;
   try {
     resolvedPath = realpathSync(filePath);
@@ -725,7 +721,7 @@ export async function finalizeAndPublishRecording(params: {
  * after recording stops, broadcasting recording lifecycle events, and
  * triggering deferred recording restarts.
  *
- * Shared by both the IPC handler and the HTTP POST route.
+ * This is the supported entry point for both HTTP routes and tests.
  */
 export async function handleRecordingStatusCore(
   msg: RecordingStatus,
@@ -1049,19 +1045,3 @@ export function __resetRecordingState(): void {
   finalizedRecordingIds.clear();
   activeRestartToken = null;
 }
-
-// ─── IPC handler wrapper ─────────────────────────────────────────────────────
-
-/** IPC-compatible wrapper: ignores the socket (unused) and delegates to core. */
-async function handleRecordingStatusIpc(
-  msg: RecordingStatus,
-  ctx: HandlerContext,
-): Promise<void> {
-  return handleRecordingStatusCore(msg, ctx);
-}
-
-// ─── Export handler group ────────────────────────────────────────────────────
-
-export const recordingHandlers = defineHandlers({
-  recording_status: handleRecordingStatusIpc,
-});

package/src/daemon/handlers/session-history.ts

@@ -27,6 +27,43 @@ import {
   renderHistoryContent,
 } from "./shared.js";
 
+/**
+ * In light mode, strip heavy payloads (e.g. full HTML) from surface data
+ * but preserve the fields the client needs to parse and render the surface.
+ */
+function lightModeSurfaceData(s: HistorySurface): Record<string, unknown> {
+  switch (s.surfaceType) {
+    case "dynamic_page":
+      return {
+        ...(s.data.preview ? { preview: s.data.preview } : {}),
+        ...(s.data.appId ? { appId: s.data.appId } : {}),
+      };
+    case "card":
+      return {
+        ...(typeof s.data.title === "string" ? { title: s.data.title } : {}),
+        ...(typeof s.data.body === "string" ? { body: s.data.body } : {}),
+        ...(typeof s.data.template === "string"
+          ? { template: s.data.template }
+          : {}),
+        ...(s.data.templateData ? { templateData: s.data.templateData } : {}),
+      };
+    case "document_preview":
+      return {
+        ...(typeof s.data.surfaceId === "string"
+          ? { surfaceId: s.data.surfaceId }
+          : {}),
+        ...(typeof s.data.title === "string" ? { title: s.data.title } : {}),
+        ...(typeof s.data.content === "string"
+          ? { content: s.data.content }
+          : {}),
+      };
+    default:
+      // For other types (list, table, form, confirmation, etc.),
+      // preserve the full data — these are generally small.
+      return s.data;
+  }
+}
+
 export function handleHistoryRequest(
   msg: HistoryRequest,
   ctx: HandlerContext,
@@ -190,7 +227,9 @@ export function handleHistoryRequest(
             })
         : m.toolCalls;
 
-    // In light mode, strip full data from surfaces (keep metadata)
+    // In light mode, strip heavy payloads but keep essential fields so
+    // the client can still parse and render surfaces (e.g. card title/body,
+    // dynamic_page preview, document_preview metadata).
     const filteredSurfaces =
       m.surfaces.length > 0
         ? includeSurfaceData
@@ -199,14 +238,7 @@ export function handleHistoryRequest(
               surfaceId: s.surfaceId,
               surfaceType: s.surfaceType,
               title: s.title,
-              data: {
-                ...(s.surfaceType === "dynamic_page"
-                  ? {
-                      ...(s.data.preview ? { preview: s.data.preview } : {}),
-                      ...(s.data.appId ? { appId: s.data.appId } : {}),
-                    }
-                  : {}),
-              } as Record<string, unknown>,
+              data: lightModeSurfaceData(s),
               ...(s.actions ? { actions: s.actions } : {}),
               ...(s.display ? { display: s.display } : {}),
             }))
@@ -370,7 +402,7 @@ export function getMessageContent(
 }
 
 // ---------------------------------------------------------------------------
-// IPC handlers (delegate to shared logic)
+// HTTP handlers (delegate to shared logic)
 // ---------------------------------------------------------------------------
 
 export function handleConversationSearch(

package/src/daemon/handlers/sessions.ts

@@ -34,6 +34,8 @@ import * as externalConversationStore from "../../memory/external-conversation-s
 import * as pendingInteractions from "../../runtime/pending-interactions.js";
 import { getSubagentManager } from "../../subagent/index.js";
 import { truncate } from "../../util/truncate.js";
+import { HostBashProxy } from "../host-bash-proxy.js";
+import { HostFileProxy } from "../host-file-proxy.js";
 import type {
   CancelRequest,
   ConfirmationResponse,
@@ -55,13 +57,6 @@ import {
   classifySessionError,
 } from "../session-error.js";
 import {
-  handleConversationSearch,
-  handleHistoryRequest,
-  handleMessageContentRequest,
-} from "./session-history.js";
-import { handleUserMessage } from "./session-user-message.js";
-import {
-  defineHandlers,
   type HandlerContext,
   log,
   pendingStandaloneSecrets,
@@ -85,7 +80,7 @@ function parseBindingSourceChannel(
   return null;
 }
 
-export function syncCanonicalStatusFromIpcConfirmationDecision(
+export function syncCanonicalStatusFromConfirmationDecision(
   requestId: string,
   decision: ConfirmationResponse["decision"],
 ): void {
@@ -101,12 +96,12 @@ export function syncCanonicalStatusFromIpcConfirmationDecision(
   } catch (err) {
     log.debug(
       { err, requestId, targetStatus },
-      "Failed to resolve canonical request from IPC confirmation response",
+      "Failed to resolve canonical request from local confirmation response",
     );
   }
 }
 
-export function makeIpcEventSender(params: {
+export function makeEventSender(params: {
   ctx: HandlerContext;
   session: Session;
   conversationId: string;
@@ -149,7 +144,7 @@ export function makeIpcEventSender(params: {
       } catch (err) {
         log.debug(
           { err, requestId: event.requestId, conversationId },
-          "Failed to create canonical request from IPC confirmation event",
+          "Failed to create canonical request from local confirmation event",
         );
       }
     } else if (event.type === "secret_request") {
@@ -158,6 +153,18 @@ export function makeIpcEventSender(params: {
         conversationId,
         kind: "secret",
       });
+    } else if (event.type === "host_bash_request") {
+      pendingInteractions.register(event.requestId, {
+        session,
+        conversationId,
+        kind: "host_bash",
+      });
+    } else if (event.type === "host_file_request") {
+      pendingInteractions.register(event.requestId, {
+        session,
+        conversationId,
+        kind: "host_file",
+      });
     }
 
     ctx.send(event);
@@ -182,10 +189,7 @@ export function handleConfirmationResponse(
         undefined,
         { source: "button" },
       );
-      syncCanonicalStatusFromIpcConfirmationDecision(
-        msg.requestId,
-        msg.decision,
-      );
+      syncCanonicalStatusFromConfirmationDecision(msg.requestId, msg.decision);
       pendingInteractions.resolve(msg.requestId);
       return;
     }
@@ -200,10 +204,7 @@ export function handleConfirmationResponse(
         msg.selectedPattern,
         msg.selectedScope,
       );
-      syncCanonicalStatusFromIpcConfirmationDecision(
-        msg.requestId,
-        msg.decision,
-      );
+      syncCanonicalStatusFromConfirmationDecision(msg.requestId, msg.decision);
       pendingInteractions.resolve(msg.requestId);
       return;
     }
@@ -332,7 +333,7 @@ export function handleSessionList(
  */
 export function clearAllSessions(ctx: HandlerContext): number {
   const cleared = ctx.clearAllSessions();
-  // Also clear DB conversations. When a new IPC connection triggers
+  // Also clear DB conversations. When a new local connection triggers
   // sendInitialSession, it auto-creates a conversation if none exist.
   // Without this DB clear, that auto-created row survives, contradicting
   // the "clear all conversations" intent.
@@ -340,9 +341,7 @@ export function clearAllSessions(ctx: HandlerContext): number {
   return cleared;
 }
 
-export function handleSessionsClear(
-  ctx: HandlerContext,
-): void {
+export function handleSessionsClear(ctx: HandlerContext): void {
   const cleared = clearAllSessions(ctx);
   ctx.send({ type: "sessions_clear_response", cleared });
 }
@@ -389,7 +388,7 @@ export async function handleSessionCreate(
     if (title === GENERATING_TITLE) {
       queueGenerateConversationTitle({
         conversationId: conversation.id,
-        context: { origin: "ipc" },
+        context: { origin: "local" },
         userMessage: msg.initialMessage,
         onTitleUpdated: (newTitle) => {
           ctx.send({
@@ -404,13 +403,12 @@ export async function handleSessionCreate(
     const requestId = uuid();
     const transportChannel =
       parseChannelId(msg.transport?.channelId) ?? "vellum";
-    const sendEvent = makeIpcEventSender({
+    const sendEvent = makeEventSender({
       ctx,
       session,
       conversationId: conversation.id,
       sourceChannel: transportChannel,
     });
-    session.updateClient(sendEvent, false);
     session.setTurnChannelContext({
       userMessageChannel: transportChannel,
       assistantMessageChannel: transportChannel,
@@ -421,6 +419,20 @@ export async function handleSessionCreate(
       userMessageInterface: transportInterface,
       assistantMessageInterface: transportInterface,
     });
+    // Only create the host bash proxy for desktop client interfaces that can
+    // execute commands on the user's machine. Set before updateClient so
+    // updateClient's call to hostBashProxy.updateSender targets the new proxy.
+    if (transportInterface === "macos" || transportInterface === "ios") {
+      const proxy = new HostBashProxy(sendEvent, (requestId) => {
+        pendingInteractions.resolve(requestId);
+      });
+      session.setHostBashProxy(proxy);
+      const fileProxy = new HostFileProxy(sendEvent, (requestId) => {
+        pendingInteractions.resolve(requestId);
+      });
+      session.setHostFileProxy(fileProxy);
+    }
+    session.updateClient(sendEvent, false);
     session
       .processMessage(msg.initialMessage, [], sendEvent, requestId)
       .catch((err) => {
@@ -520,10 +532,7 @@ export async function handleSessionSwitch(
 /**
  * Rename a session/conversation. Returns true on success, false if not found.
  */
-export function renameSession(
-  sessionId: string,
-  name: string,
-): boolean {
+export function renameSession(sessionId: string, name: string): boolean {
   const conversation = getConversation(sessionId);
   if (!conversation) {
     return false;
@@ -572,10 +581,7 @@ export function cancelGeneration(
   return true;
 }
 
-export function handleCancel(
-  msg: CancelRequest,
-  ctx: HandlerContext,
-): void {
+export function handleCancel(msg: CancelRequest, ctx: HandlerContext): void {
   const sessionId = msg.sessionId;
   if (sessionId) {
     cancelGeneration(sessionId, ctx);
@@ -598,10 +604,7 @@ export function undoLastMessage(
   return { removedCount };
 }
 
-export function handleUndo(
-  msg: UndoRequest,
-  ctx: HandlerContext,
-): void {
+export function handleUndo(msg: UndoRequest, ctx: HandlerContext): void {
   const result = undoLastMessage(msg.sessionId, ctx);
   if (!result) {
     ctx.send({ type: "error", message: "No active session" });
@@ -616,7 +619,7 @@ export function handleUndo(
 
 /**
  * Regenerate the last assistant response for a session. The caller provides
- * a `sendEvent` callback for delivering streaming events (IPC or HTTP/SSE).
+ * a `sendEvent` callback for delivering streaming events via HTTP/SSE.
  * Returns null if the session is not found. Throws on regeneration errors.
  */
 export async function regenerateResponse(
@@ -667,7 +670,7 @@ export async function handleRegenerate(
   const regenerateChannel =
     parseChannelId(session.getTurnChannelContext()?.assistantMessageChannel) ??
     "vellum";
-  const sendEvent = makeIpcEventSender({
+  const sendEvent = makeEventSender({
     ctx,
     session,
     conversationId: msg.sessionId,
@@ -717,8 +720,12 @@ export function handleUsageRequest(
 export function deleteQueuedMessage(
   sessionId: string,
   requestId: string,
-  findSession: (id: string) => { removeQueuedMessage(requestId: string): boolean } | undefined,
-): { removed: true } | { removed: false; reason: "session_not_found" | "message_not_found" } {
+  findSession: (
+    id: string,
+  ) => { removeQueuedMessage(requestId: string): boolean } | undefined,
+):
+  | { removed: true }
+  | { removed: false; reason: "session_not_found" | "message_not_found" } {
   const session = findSession(sessionId);
   if (!session) {
     log.warn(
@@ -731,25 +738,20 @@ export function deleteQueuedMessage(
   if (removed) {
     return { removed: true };
   }
-  log.warn(
-    { sessionId, requestId },
-    "Queued message not found for deletion",
-  );
+  log.warn({ sessionId, requestId }, "Queued message not found for deletion");
   return { removed: false, reason: "message_not_found" };
 }
 
 // ---------------------------------------------------------------------------
-// IPC handler (delegates to shared logic)
+// HTTP handler (delegates to shared logic)
 // ---------------------------------------------------------------------------
 
 export function handleDeleteQueuedMessage(
   msg: DeleteQueuedMessage,
   ctx: HandlerContext,
 ): void {
-  const result = deleteQueuedMessage(
-    msg.sessionId,
-    msg.requestId,
-    (id) => ctx.sessions.get(id),
+  const result = deleteQueuedMessage(msg.sessionId, msg.requestId, (id) =>
+    ctx.sessions.get(id),
   );
   if (result.removed) {
     ctx.send({
@@ -775,24 +777,3 @@ export function handleReorderThreads(
     })),
   );
 }
-
-export const sessionHandlers = defineHandlers({
-  user_message: handleUserMessage,
-  confirmation_response: handleConfirmationResponse,
-  secret_response: handleSecretResponse,
-  session_list: (msg, ctx) =>
-    handleSessionList(ctx, msg.offset ?? 0, msg.limit ?? 50),
-  session_create: handleSessionCreate,
-  sessions_clear: (_msg, ctx) => handleSessionsClear(ctx),
-  session_switch: handleSessionSwitch,
-  session_rename: handleSessionRename,
-  cancel: handleCancel,
-  delete_queued_message: handleDeleteQueuedMessage,
-  history_request: handleHistoryRequest,
-  message_content_request: handleMessageContentRequest,
-  undo: handleUndo,
-  regenerate: handleRegenerate,
-  usage_request: handleUsageRequest,
-  conversation_search: handleConversationSearch,
-  reorder_threads: handleReorderThreads,
-});