@vellumai/assistant 0.4.42 → 0.4.44

package/src/daemon/session-notifiers.ts

@@ -18,8 +18,10 @@ import {
   unregisterCallTranscriptNotifier,
 } from "../calls/call-state.js";
 import { getCallSession } from "../calls/call-store.js";
-import * as conversationStore from "../memory/conversation-store.js";
-import { provenanceFromTrustContext } from "../memory/conversation-store.js";
+import {
+  addMessage,
+  provenanceFromTrustContext,
+} from "../memory/conversation-crud.js";
 import type { Message } from "../providers/types.js";
 import type { WatchSession } from "../tools/watch/watch-state.js";
 import {
@@ -31,7 +33,7 @@ import {
   unregisterWatchCompletionNotifier,
   unregisterWatchStartNotifier,
 } from "../tools/watch/watch-state.js";
-import type { ServerMessage } from "./ipc-protocol.js";
+import type { ServerMessage } from "./message-protocol.js";
 import type { TrustContext } from "./session-runtime-assembly.js";
 import {
   lastCommentaryBySession,
@@ -106,16 +108,16 @@ export function registerSessionNotifiers(
       const callee = callSession?.toNumber ?? "the caller";
       const questionText = `**Live call question** (to ${callee}):\n\n${question}\n\n_Use the call answer API to respond._`;
 
-      await conversationStore.addMessage(
+      await addMessage(
         conversationId,
         "assistant",
         JSON.stringify([{ type: "text", text: questionText }]),
         {
           ...provenanceFromTrustContext(ctx.trustContext),
-          userMessageChannel: "voice",
-          assistantMessageChannel: "voice",
-          userMessageInterface: "voice",
-          assistantMessageInterface: "voice",
+          userMessageChannel: "phone",
+          assistantMessageChannel: "phone",
+          userMessageInterface: "phone",
+          assistantMessageInterface: "phone",
         },
       );
 

package/src/daemon/session-process.ts

@@ -17,21 +17,28 @@ import type {
 import { parseChannelId, parseInterfaceId } from "../channels/types.js";
 import { getConfig } from "../config/loader.js";
 import { listPendingRequestsByConversationScope } from "../memory/canonical-guardian-store.js";
-import * as conversationStore from "../memory/conversation-store.js";
-import { provenanceFromTrustContext } from "../memory/conversation-store.js";
+import {
+  addMessage,
+  provenanceFromTrustContext,
+  setConversationOriginChannelIfUnset,
+  setConversationOriginInterfaceIfUnset,
+} from "../memory/conversation-crud.js";
 import { extractPreferences } from "../notifications/preference-extractor.js";
 import { createPreference } from "../notifications/preferences-store.js";
 import type { Message } from "../providers/types.js";
 import { routeGuardianReply } from "../runtime/guardian-reply-router.js";
 import { getLogger } from "../util/logger.js";
-import { resolveGuardianVerificationIntent } from "./guardian-verification-intent.js";
-import type { UsageStats } from "./ipc-contract.js";
-import type { ServerMessage, UserMessageAttachment } from "./ipc-protocol.js";
+import type {
+  ServerMessage,
+  UsageStats,
+  UserMessageAttachment,
+} from "./message-protocol.js";
 import type { MessageQueue } from "./session-queue-manager.js";
 import type { QueueDrainReason } from "./session-queue-manager.js";
 import type { TrustContext } from "./session-runtime-assembly.js";
 import { resolveSlash, type SlashContext } from "./session-slash.js";
 import type { TraceEmitter } from "./trace-emitter.js";
+import { resolveVerificationSessionIntent } from "./verification-session-intent.js";
 
 const log = getLogger("session-process");
 
@@ -287,7 +294,7 @@ export async function drainQueue(
             createUserMessage(next.displayContent, next.attachments).content,
           )
         : JSON.stringify(userMsg.content);
-      await conversationStore.addMessage(
+      await addMessage(
         session.conversationId,
         "user",
         contentToPersist,
@@ -296,7 +303,7 @@ export async function drainQueue(
       session.messages.push(userMsg);
 
       const assistantMsg = createAssistantMessage(slashResult.message);
-      await conversationStore.addMessage(
+      await addMessage(
         session.conversationId,
         "assistant",
         JSON.stringify(assistantMsg.content),
@@ -305,13 +312,13 @@ export async function drainQueue(
       session.messages.push(assistantMsg);
 
       if (queuedTurnCtx) {
-        conversationStore.setConversationOriginChannelIfUnset(
+        setConversationOriginChannelIfUnset(
           session.conversationId,
           queuedTurnCtx.userMessageChannel,
         );
       }
       if (queuedInterfaceCtx) {
-        conversationStore.setConversationOriginInterfaceIfUnset(
+        setConversationOriginInterfaceIfUnset(
           session.conversationId,
           queuedInterfaceCtx.userMessageInterface,
         );
@@ -373,16 +380,17 @@ export async function drainQueue(
   // loop receives the rewritten instruction.
   let agentLoopContent = resolvedContent;
   if (slashResult.kind === "passthrough") {
-    const guardianIntent = resolveGuardianVerificationIntent(resolvedContent);
-    if (guardianIntent.kind === "direct_setup") {
+    const verificationIntent =
+      resolveVerificationSessionIntent(resolvedContent);
+    if (verificationIntent.kind === "direct_setup") {
       log.info(
         {
           conversationId: session.conversationId,
-          channelHint: guardianIntent.channelHint,
+          channelHint: verificationIntent.channelHint,
         },
-        "Guardian verification intent intercepted in queue — forcing skill flow",
+        "Verification session intent intercepted in queue — forcing skill flow",
       );
-      agentLoopContent = guardianIntent.rewrittenContent;
+      agentLoopContent = verificationIntent.rewrittenContent;
       session.preactivatedSkillIds = ["guardian-verify-setup"];
     }
   }
@@ -566,7 +574,7 @@ export async function processMessage(
       };
 
       const userMsg = createUserMessage(content, attachments);
-      const persisted = await conversationStore.addMessage(
+      const persisted = await addMessage(
         session.conversationId,
         "user",
         JSON.stringify(userMsg.content),
@@ -580,7 +588,7 @@ export async function processMessage(
           ? "Decision applied."
           : "Request already resolved.");
       const assistantMsg = createAssistantMessage(replyText);
-      await conversationStore.addMessage(
+      await addMessage(
         session.conversationId,
         "assistant",
         JSON.stringify(assistantMsg.content),
@@ -636,7 +644,7 @@ export async function processMessage(
     const contentToPersist = displayContent
       ? JSON.stringify(createUserMessage(displayContent, attachments).content)
       : JSON.stringify(userMsg.content);
-    const persisted = await conversationStore.addMessage(
+    const persisted = await addMessage(
       session.conversationId,
       "user",
       contentToPersist,
@@ -645,7 +653,7 @@ export async function processMessage(
     session.messages.push(userMsg);
 
     const assistantMsg = createAssistantMessage(slashResult.message);
-    await conversationStore.addMessage(
+    await addMessage(
       session.conversationId,
       "assistant",
       JSON.stringify(assistantMsg.content),
@@ -654,13 +662,13 @@ export async function processMessage(
     session.messages.push(assistantMsg);
 
     if (pmTurnCtx) {
-      conversationStore.setConversationOriginChannelIfUnset(
+      setConversationOriginChannelIfUnset(
         session.conversationId,
         pmTurnCtx.userMessageChannel,
       );
     }
     if (pmInterfaceCtx) {
-      conversationStore.setConversationOriginInterfaceIfUnset(
+      setConversationOriginInterfaceIfUnset(
         session.conversationId,
         pmInterfaceCtx.userMessageInterface,
       );
@@ -698,16 +706,17 @@ export async function processMessage(
   // rewritten content only for the agent loop instruction.
   let agentLoopContent = resolvedContent;
   if (slashResult.kind === "passthrough") {
-    const guardianIntent = resolveGuardianVerificationIntent(resolvedContent);
-    if (guardianIntent.kind === "direct_setup") {
+    const verificationIntent =
+      resolveVerificationSessionIntent(resolvedContent);
+    if (verificationIntent.kind === "direct_setup") {
       log.info(
         {
           conversationId: session.conversationId,
-          channelHint: guardianIntent.channelHint,
+          channelHint: verificationIntent.channelHint,
         },
-        "Guardian verification intent intercepted — forcing skill flow",
+        "Verification session intent intercepted — forcing skill flow",
       );
-      agentLoopContent = guardianIntent.rewrittenContent;
+      agentLoopContent = verificationIntent.rewrittenContent;
       session.preactivatedSkillIds = ["guardian-verify-setup"];
     }
   }

package/src/daemon/session-queue-manager.ts

@@ -10,7 +10,7 @@ import type {
   TurnInterfaceContext,
 } from "../channels/types.js";
 import { getLogger } from "../util/logger.js";
-import type { ServerMessage, UserMessageAttachment } from "./ipc-protocol.js";
+import type { ServerMessage, UserMessageAttachment } from "./message-protocol.js";
 
 const log = getLogger("session-queue");
 

package/src/daemon/session-runtime-assembly.ts

@@ -25,7 +25,7 @@ import { channelStatusToMemberStatus } from "../runtime/routes/inbound-stages/ac
  * interacting.  Used to gate UI-specific references and permission asks.
  */
 export interface ChannelCapabilities {
-  /** The raw channel identifier (e.g. "vellum", "telegram", "sms"). */
+  /** The raw channel identifier (e.g. "vellum", "telegram"). */
   channel: string;
   /** Whether this channel can render the dashboard UI (apps, dynamic pages). */
   dashboardCapable: boolean;
@@ -81,8 +81,6 @@ export interface TrustContext {
   requesterExternalUserId?: string;
   /** Chat/conversation ID the requester is interacting through. */
   requesterChatId?: string;
-  /** Access denial reason, if applicable. See {@link DenialReason}. */
-  denialReason?: "no_binding" | "no_identity";
 }
 
 /**
@@ -113,8 +111,6 @@ export interface InboundActorContext {
   memberStatus?: string;
   /** Member policy when the actor has a contact record. */
   memberPolicy?: string;
-  /** Denial reason when access is blocked. */
-  denialReason?: string;
   /** Free-text notes about this contact. */
   contactNotes?: string;
   /** Number of prior interactions with this contact. */
@@ -138,7 +134,6 @@ export function inboundActorContextFromTrustContext(
     actorMemberDisplayName: ctx.requesterMemberDisplayName,
     trustClass: ctx.trustClass,
     guardianIdentity: ctx.guardianExternalUserId,
-    denialReason: ctx.denialReason,
   };
 }
 
@@ -162,28 +157,22 @@ export function inboundActorContextFromTrust(
       ? channelStatusToMemberStatus(ctx.memberRecord.channel.status)
       : undefined,
     memberPolicy: ctx.memberRecord?.channel.policy ?? undefined,
-    denialReason: ctx.denialReason,
     contactNotes: ctx.memberRecord?.contact.notes ?? undefined,
     contactInteractionCount:
       ctx.memberRecord?.contact.interactionCount ?? undefined,
   };
 }
 
-/** Legacy push-to-talk activation key values (pre-custom-key support). */
-const PTT_KEY_LEGACY = new Set(["fn", "ctrl", "fn_shift", "none"]);
-
 /**
- * Validate a PTT activation key string. Accepts both legacy string values
- * (fn, ctrl, fn_shift, none) and JSON PTTActivator payloads from the
- * custom key feature. Returns the key as-is if valid, undefined otherwise.
+ * Validate a PTT activation key string. Accepts JSON PTTActivator payloads
+ * from the custom key feature. Returns the key as-is if valid, undefined otherwise.
  */
 export function sanitizePttActivationKey(
   key: string | undefined | null,
 ): string | undefined {
   if (key == null) return undefined;
-  if (PTT_KEY_LEGACY.has(key)) return key;
 
-  // Try parsing as a JSON PTTActivator payload
+  // Parse as a JSON PTTActivator payload
   if (key.startsWith("{")) {
     try {
       const parsed = JSON.parse(key) as { kind?: string };
@@ -247,14 +236,8 @@ const KEY_CODE_NAMES: Record<number, string> = {
   57: "Caps Lock",
 };
 
-/** Derive a human-readable label from a PTT activation key value (legacy string or JSON). */
+/** Derive a human-readable label from a PTT activation key JSON value. */
 function pttKeyLabel(raw: string): string {
-  // Legacy string values
-  if (raw === "fn") return "Fn (Globe)";
-  if (raw === "ctrl") return "Ctrl";
-  if (raw === "fn_shift") return "Fn+Shift";
-  if (raw === "none") return "none";
-
   // JSON PTTActivator payload
   if (raw.startsWith("{")) {
     try {
@@ -362,8 +345,7 @@ export function resolveChannelCapabilities(
       };
     }
     case "telegram":
-    case "sms":
-    case "voice":
+    case "phone":
     case "whatsapp":
     case "slack":
     case "email":
@@ -439,13 +421,6 @@ export function injectActiveSurfaceContext(
       "8. Keep your text response to 1 brief sentence confirming what you changed.",
     );
 
-    if (ctx.html.includes('data-vellum-home-base="v1"')) {
-      lines.push(
-        "9. This is the prebuilt Home Base scaffold. Preserve layout anchors:",
-        "   `home-base-root`, `home-base-onboarding-lane`, and `home-base-starter-lane`.",
-      );
-    }
-
     // File tree with sizes (capped at 50 files to bound prompt size)
     const files = ctx.appFiles ?? listAppFiles(ctx.appId);
     const MAX_FILE_TREE_ENTRIES = 50;
@@ -779,7 +754,6 @@ export function buildInboundActorContextBlock(
   if (ctx.memberPolicy) {
     lines.push(`member_policy: ${ctx.memberPolicy}`);
   }
-  lines.push(`denial_reason: ${ctx.denialReason ?? "none"}`);
   // Contact metadata — only included when the sender has a contact record
   // with non-default values.
   if (ctx.contactNotes) {

package/src/daemon/session-surfaces.ts

@@ -1,9 +1,5 @@
 import { v4 as uuid } from "uuid";
 
-import {
-  findSeededHomeBaseApp,
-  getPrebuiltHomeBasePreview,
-} from "../home-base/prebuilt/seed.js";
 import { getApp, getAppPreview, updateApp } from "../memory/app-store.js";
 import type { ToolExecutionResult } from "../tools/types.js";
 import { getLogger } from "../util/logger.js";
@@ -20,8 +16,8 @@ import type {
   TableRow,
   TableSurfaceData,
   UiSurfaceShow,
-} from "./ipc-protocol.js";
-import { INTERACTIVE_SURFACE_TYPES } from "./ipc-protocol.js";
+} from "./message-protocol.js";
+import { INTERACTIVE_SURFACE_TYPES } from "./message-protocol.js";
 import { buildSessionErrorMessage } from "./session-error.js";
 
 const log = getLogger("session-surfaces");
@@ -78,14 +74,27 @@ function normalizeCardShowData(
     normalized.templateData = input.templateData;
   }
 
-  // task_progress cards need a title for Swift parsing; fall back when missing.
+  // The LLM sometimes sends `title` or `body` at the top-level tool input
+  // instead of nesting them inside `data`. The Swift client requires `title`
+  // inside the card data dict — without it `parseCardData` returns nil and
+  // the surface is silently dropped. Copy them from input when missing.
+  if (
+    typeof normalized.title !== "string" &&
+    typeof input.title === "string" &&
+    input.title.trim().length > 0
+  ) {
+    normalized.title = input.title;
+  }
+  if (typeof normalized.body !== "string" && typeof input.body === "string") {
+    normalized.body = input.body;
+  }
+
+  // task_progress cards: additional fallbacks for title from templateData.
   if (
     normalized.template === "task_progress" &&
     typeof normalized.title !== "string"
   ) {
-    if (typeof input.title === "string" && input.title.trim().length > 0) {
-      normalized.title = input.title;
-    } else if (
+    if (
       isPlainObject(normalized.templateData) &&
       typeof normalized.templateData.title === "string"
     ) {
@@ -166,7 +175,17 @@ export interface SurfaceSessionContext {
   >;
   surfaceState: Map<
     string,
-    { surfaceType: SurfaceType; data: SurfaceData; title?: string }
+    {
+      surfaceType: SurfaceType;
+      data: SurfaceData;
+      title?: string;
+      actions?: Array<{
+        id: string;
+        label: string;
+        style?: string;
+        data?: Record<string, unknown>;
+      }>;
+    }
   >;
   surfaceUndoStacks: Map<string, string[]>;
   /** Request IDs that originated from surface action button clicks (not regular user messages). */
@@ -176,7 +195,12 @@ export interface SurfaceSessionContext {
     surfaceType: SurfaceType;
     title?: string;
     data: SurfaceData;
-    actions?: Array<{ id: string; label: string; style?: string }>;
+    actions?: Array<{
+      id: string;
+      label: string;
+      style?: string;
+      data?: Record<string, unknown>;
+    }>;
     display?: string;
   }>;
   onEscalateToComputerUse?: (task: string, sourceSessionId: string) => boolean;
@@ -517,8 +541,91 @@ export function handleSurfaceAction(
   data?: Record<string, unknown>,
 ): void {
   const pending = ctx.pendingSurfaceActions.get(surfaceId);
+
+  // When surfaces are restored from history (e.g. onboarding cards), there is
+  // no in-memory pendingSurfaceActions entry.  For relay_prompt / agent_prompt
+  // actions the client already sends the full payload (including { prompt }),
+  // so we can handle them without stored state.
   if (!pending) {
-    log.warn({ surfaceId, actionId }, "No pending surface action found");
+    const isRelay = actionId === "relay_prompt" || actionId === "agent_prompt";
+    const prompt =
+      isRelay && typeof data?.prompt === "string" ? data.prompt.trim() : "";
+
+    if (!prompt) {
+      log.warn({ surfaceId, actionId }, "No pending surface action found");
+      return;
+    }
+
+    const requestId = uuid();
+    ctx.surfaceActionRequestIds.add(requestId);
+    const onEvent = (msg: ServerMessage) => ctx.sendToClient(msg);
+
+    ctx.traceEmitter.emit("request_received", "Surface action received", {
+      requestId,
+      status: "info",
+      attributes: { source: "surface_action", surfaceId, actionId },
+    });
+
+    const result = ctx.enqueueMessage(
+      prompt,
+      [],
+      onEvent,
+      requestId,
+      surfaceId,
+    );
+    if (result.rejected) {
+      log.error({ surfaceId, actionId }, "Relay prompt rejected — queue full");
+      onEvent(
+        buildSessionErrorMessage(ctx.conversationId, {
+          code: "QUEUE_FULL",
+          userMessage:
+            "Message queue is full (max depth: 10). Please wait for current messages to be processed.",
+          retryable: true,
+          debugDetails: "Relay prompt rejected — session queue is full",
+        }),
+      );
+      return;
+    }
+
+    // Echo the prompt to the client so it appears in the chat UI.
+    // Sent after enqueue succeeds so the user doesn't see a prompt that
+    // won't be processed.
+    ctx.sendToClient({
+      type: "user_message_echo",
+      text: prompt,
+      sessionId: ctx.conversationId,
+    });
+
+    if (result.queued) {
+      log.info(
+        { surfaceId, actionId, requestId },
+        "Relay prompt queued (session busy, history-restored)",
+      );
+      return;
+    }
+
+    // Session is idle — process the message immediately.
+    log.info(
+      { surfaceId, actionId, requestId },
+      "Processing relay prompt immediately (history-restored)",
+    );
+    ctx
+      .processMessage(prompt, [], onEvent, requestId, surfaceId)
+      .catch((err) => {
+        const message = err instanceof Error ? err.message : String(err);
+        log.error(
+          { err, surfaceId, actionId },
+          "Failed to process history-restored relay prompt",
+        );
+        onEvent(
+          buildSessionErrorMessage(ctx.conversationId, {
+            code: "SESSION_PROCESSING_FAILED",
+            userMessage: `Something went wrong: ${message}`,
+            retryable: false,
+            debugDetails: `History-restored relay prompt processing failed: ${message}`,
+          }),
+        );
+      });
     return;
   }
   const retainPending = pending.surfaceType === "dynamic_page";
@@ -538,32 +645,39 @@ export function handleSurfaceAction(
     handleDocumentContentChanged(ctx, surfaceId, data);
     return;
   }
-  ctx.lastSurfaceAction.set(surfaceId, { actionId, data });
+  // Merge stored action-level data (from ui_show definition) with client-sent
+  // data. This is critical for relay_prompt buttons: the client only sends the
+  // actionId, but the prompt payload lives in the action definition's data.
+  const stored = ctx.surfaceState.get(surfaceId);
+  const actionDef = stored?.actions?.find((a) => a.id === actionId);
+  const mergedData: Record<string, unknown> | undefined =
+    actionDef?.data || data ? { ...actionDef?.data, ...data } : undefined;
+
+  ctx.lastSurfaceAction.set(surfaceId, { actionId, data: mergedData });
   const shouldRelayPrompt =
     actionId === "relay_prompt" || actionId === "agent_prompt";
   const prompt =
-    shouldRelayPrompt && typeof data?.prompt === "string"
-      ? data.prompt.trim()
+    shouldRelayPrompt && typeof mergedData?.prompt === "string"
+      ? mergedData.prompt.trim()
       : "";
 
   // Build a human-readable summary so the LLM clearly understands the
   // user's decision instead of parsing raw JSON.
-  const stored = ctx.surfaceState.get(surfaceId);
   const surfaceData = stored?.data as Record<string, unknown> | undefined;
   const summary = buildCompletionSummary(
     pending.surfaceType,
     actionId,
-    data,
+    mergedData,
     surfaceData,
   );
   let fallbackContent = `[User action on ${pending.surfaceType} surface: ${summary}]`;
   // Append structured data so the LLM has access to IDs/values it needs
   // to act on (e.g. selectedIds for archiving).
-  if (data && Object.keys(data).length > 0) {
-    fallbackContent += `\n\nAction data: ${JSON.stringify(data)}`;
+  if (mergedData && Object.keys(mergedData).length > 0) {
+    fallbackContent += `\n\nAction data: ${JSON.stringify(mergedData)}`;
   }
   // Append deselection context for table/list surfaces so the LLM knows what the user chose to keep.
-  const selectedIds = data?.selectedIds as string[] | undefined;
+  const selectedIds = mergedData?.selectedIds as string[] | undefined;
   if (
     selectedIds &&
     (pending.surfaceType === "table" || pending.surfaceType === "list")
@@ -574,11 +688,28 @@ export function handleSurfaceAction(
       selectedIds,
     );
   }
-  const content = prompt || fallbackContent;
+  // When a relay_prompt button also carries selection data (e.g. list/table
+  // surface with a canned prompt + user-selected rows), append the selection
+  // context so the LLM sees both the prompt and the user's selections.
+  let content = prompt || fallbackContent;
+  if (prompt && selectedIds && mergedData) {
+    if (pending.surfaceType === "table" || pending.surfaceType === "list") {
+      content += buildDeselectionDescription(
+        pending.surfaceType,
+        stored,
+        selectedIds,
+      );
+    }
+  }
   // Show the user plain-text instead of raw JSON action data.
   const displayContent = prompt
     ? undefined
-    : buildUserFacingLabel(pending.surfaceType, actionId, data, surfaceData);
+    : buildUserFacingLabel(
+        pending.surfaceType,
+        actionId,
+        mergedData,
+        surfaceData,
+      );
 
   const requestId = uuid();
   ctx.surfaceActionRequestIds.add(requestId);
@@ -912,7 +1043,12 @@ export async function surfaceProxyResolver(
           : rawData
     ) as SurfaceData;
     const actions = input.actions as
-      | Array<{ id: string; label: string; style?: string }>
+      | Array<{
+          id: string;
+          label: string;
+          style?: string;
+          data?: Record<string, unknown>;
+        }>
       | undefined;
     // Interactive surfaces default to awaiting user action.
     const hasActions = Array.isArray(actions) && actions.length > 0;
@@ -942,9 +1078,6 @@ export async function surfaceProxyResolver(
       }
     }
 
-    // Track surface state for ui_update merging
-    ctx.surfaceState.set(surfaceId, { surfaceType, data, title });
-
     const display = (input.display as string) === "panel" ? "panel" : "inline";
 
     const mappedActions = actions?.map((a) => ({
@@ -954,8 +1087,31 @@ export async function surfaceProxyResolver(
         | "primary"
         | "secondary"
         | "destructive",
+      ...(a.data ? { data: a.data } : {}),
     }));
 
+    // Track surface state for ui_update merging (includes actions so we can
+    // look up per-action data payloads when the client sends an action back).
+    ctx.surfaceState.set(surfaceId, {
+      surfaceType,
+      data,
+      title,
+      actions: mappedActions,
+    });
+
+    log.info(
+      {
+        surfaceId,
+        surfaceType,
+        title,
+        dataKeys: Object.keys(data),
+        actionCount: mappedActions?.length ?? 0,
+        display,
+        conversationId: ctx.conversationId,
+      },
+      "Sending ui_surface_show to client",
+    );
+
     ctx.sendToClient({
       type: "ui_surface_show",
       sessionId: ctx.conversationId,
@@ -1106,14 +1262,10 @@ export async function surfaceProxyResolver(
     const openMode = input.open_mode as string | undefined;
     const app = getApp(appId);
     if (!app) return { content: `App not found: ${appId}`, isError: true };
-    const seededHomeBase = findSeededHomeBaseApp();
-    const defaultPreview =
-      seededHomeBase && seededHomeBase.id === app.id
-        ? getPrebuiltHomeBasePreview()
-        : // Generate a minimal fallback preview from app metadata so that the
-          // surface is always rendered as a clickable preview card (not an
-          // un-clickable fallback chip) after session restart.
-          { title: app.name, subtitle: app.description };
+    // Generate a minimal fallback preview from app metadata so that the
+    // surface is always rendered as a clickable preview card (not an
+    // un-clickable fallback chip) after session restart.
+    const defaultPreview = { title: app.name, subtitle: app.description };
 
     const storedPreview = getAppPreview(app.id);
     const surfaceData: DynamicPageSurfaceData = {

package/src/daemon/session-tool-setup.ts

@@ -40,7 +40,7 @@ import {
   isDoordashCommand,
   markDoordashStepInProgress,
 } from "./doordash-steps.js";
-import type { ServerMessage, UiSurfaceShow } from "./ipc-protocol.js";
+import type { ServerMessage, UiSurfaceShow } from "./message-protocol.js";
 import type { TrustContext } from "./session-runtime-assembly.js";
 import {
   projectSkillTools,