@vellumai/assistant 0.12.1-staging.1 → 0.12.1

package/src/tools/acp/spawn.test.ts

@@ -124,22 +124,11 @@ const spawnMock = mock(
     _cwd: string,
     _parentConversationId: string,
     _sendToVellum: (msg: unknown) => void,
-    _options?: { parentToolUseId?: string; model?: string },
-  ): Promise<{
-    acpSessionId: string;
-    protocolSessionId: string;
-    requestedModel?: string;
-    effectiveModel?: string;
-    modelWarning?: string;
-  }> => {
-    const requestedModel = _options?.model?.trim() || undefined;
-    return {
-      acpSessionId: "acp-session-test",
-      protocolSessionId: "proto-session-test",
-      requestedModel,
-      effectiveModel: "opus",
-    };
-  },
+    _parentToolUseId?: string,
+  ) => ({
+    acpSessionId: "acp-session-test",
+    protocolSessionId: "proto-session-test",
+  }),
 );
 
 // Spread the real module's exports so transitive importers that pull other
@@ -212,8 +201,6 @@ describe("executeAcpSpawn - happy path", () => {
     const payload = JSON.parse(result.content);
     expect(payload.acpSessionId).toBe("acp-session-test");
     expect(payload.status).toBe("running");
-    expect(payload.requestedModel).toBeNull();
-    expect(payload.effectiveModel).toBe("opus");
     // The real adapter binary is spawned (no `bun x` wrapper).
     const agentConfigArg = spawnMock.mock.calls[0][1] as { command: string };
     expect(agentConfigArg.command).toBe("claude-agent-acp");
@@ -227,10 +214,8 @@ describe("executeAcpSpawn - happy path", () => {
 
     expect(result.isError).toBe(false);
     expect(spawnMock).toHaveBeenCalledTimes(1);
-    // The options bag is the 7th positional arg to spawn().
-    expect(spawnMock.mock.calls[0][6]).toEqual({
-      parentToolUseId: "toolu_abc123",
-    });
+    // parentToolUseId is the 7th positional arg to spawn().
+    expect(spawnMock.mock.calls[0][6]).toBe("toolu_abc123");
   });
 
   test("default-profile fallback when user config is empty", async () => {
@@ -287,7 +272,7 @@ describe("executeAcpSpawn — input validation", () => {
     expect(result.isError).toBe(true);
     expect(result.content).toContain("claude-agent-acp is not on PATH");
     expect(result.content).toContain(
-      "bun add -g @agentclientprotocol/claude-agent-acp@0.75.1",
+      "bun add -g @agentclientprotocol/claude-agent-acp",
     );
     expect(execFileMock).not.toHaveBeenCalled();
     expect(spawnMock).not.toHaveBeenCalled();
@@ -324,7 +309,7 @@ describe("executeAcpSpawn: sandboxed bun auto-install on missing binary", () =>
     expect(spawnMock).toHaveBeenCalledTimes(1);
     const payload = JSON.parse(result.content);
     expect(payload.message).toContain(
-      "Installed @agentclientprotocol/claude-agent-acp@0.75.1 automatically.",
+      "Installed @agentclientprotocol/claude-agent-acp automatically.",
     );
     // The real binary was spawned with cwd = the project dir and token
     // injected (trusted-binary config, no resolution at spawn).
@@ -345,7 +330,7 @@ describe("executeAcpSpawn: sandboxed bun auto-install on missing binary", () =>
     expect(args).toEqual([
       "add",
       "--global",
-      "@agentclientprotocol/claude-agent-acp@0.75.1",
+      "@agentclientprotocol/claude-agent-acp",
     ]);
   });
 
@@ -434,7 +419,7 @@ describe("executeAcpSpawn: sandboxed bun auto-install on missing binary", () =>
     expect(result.isError).toBe(true);
     expect(result.content).toContain("claude-agent-acp is not on PATH");
     expect(result.content).toContain(
-      "bun add -g @agentclientprotocol/claude-agent-acp@0.75.1",
+      "bun add -g @agentclientprotocol/claude-agent-acp",
     );
     expect(result.content).toContain("auto-install failed");
     expect(result.content).toContain("EACCES");
@@ -473,126 +458,6 @@ describe("executeAcpSpawn — per-agent resume hint", () => {
   });
 });
 
-describe("executeAcpSpawn - model selection", () => {
-  test("threads the requested model into manager.spawn's options", async () => {
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: "opus" },
-      { ...makeContext(), toolUseId: "toolu_model" },
-    );
-
-    expect(result.isError).toBe(false);
-    expect(spawnMock.mock.calls[0][6]).toEqual({
-      parentToolUseId: "toolu_model",
-      model: "opus",
-    });
-    const payload = JSON.parse(result.content);
-    expect(payload.requestedModel).toBe("opus");
-    expect(payload.effectiveModel).toBe("opus");
-    expect(payload.message).toContain(
-      'The top-level ACP session reports "opus" as its effective model.',
-    );
-  });
-
-  test("reports the manager-normalized requested model", async () => {
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: "  opus  " },
-      makeContext(),
-    );
-
-    expect(spawnMock.mock.calls[0][6]).toEqual({
-      parentToolUseId: undefined,
-      model: "  opus  ",
-    });
-    expect(JSON.parse(result.content).requestedModel).toBe("opus");
-  });
-
-  test("a null model is treated as omitted", async () => {
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: null },
-      makeContext(),
-    );
-
-    expect(result.isError).toBe(false);
-    const options = spawnMock.mock.calls[0][6] as { model?: string };
-    expect(options.model).toBeUndefined();
-  });
-
-  test("a modelWarning from the spawn is relayed in the result message", async () => {
-    spawnMock.mockImplementationOnce(async () => ({
-      acpSessionId: "acp-session-test",
-      protocolSessionId: "proto-session-test",
-      requestedModel: "nope",
-      effectiveModel: "opus",
-      modelWarning: "Unknown model: nope",
-    }));
-
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: "nope" },
-      makeContext(),
-    );
-
-    expect(result.isError).toBe(false);
-    const payload = JSON.parse(result.content);
-    expect(payload.message).toContain(
-      "The requested model was not applied: Unknown model: nope",
-    );
-    expect(payload.requestedModel).toBe("nope");
-    expect(payload.effectiveModel).toBe("opus");
-  });
-
-  test("the note relays a no-selector warning without calling it a refusal", async () => {
-    spawnMock.mockImplementationOnce(async () => ({
-      acpSessionId: "acp-session-test",
-      protocolSessionId: "proto-session-test",
-      requestedModel: "opus",
-      modelWarning:
-        'Agent "claude" does not support model selection, so the session is running on the agent\'s own model.',
-    }));
-
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: "opus" },
-      makeContext(),
-    );
-
-    const payload = JSON.parse(result.content);
-    expect(payload.message).toContain("does not support model selection");
-    expect(payload.message).not.toContain("refused");
-  });
-
-  test("no model note when the spawn reports no warning", async () => {
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: "opus" },
-      makeContext(),
-    );
-
-    expect(result.isError).toBe(false);
-    const payload = JSON.parse(result.content);
-    expect(payload.message).not.toContain(
-      "The requested model was not applied",
-    );
-  });
-
-  test("reports when the adapter does not identify the effective model", async () => {
-    spawnMock.mockImplementationOnce(async () => ({
-      acpSessionId: "acp-session-test",
-      protocolSessionId: "proto-session-test",
-      requestedModel: "sonnet",
-    }));
-
-    const result = await executeAcpSpawn(
-      { agent: "claude", task: "do something", model: "sonnet" },
-      makeContext(),
-    );
-
-    const payload = JSON.parse(result.content);
-    expect(payload.requestedModel).toBe("sonnet");
-    expect(payload.effectiveModel).toBeNull();
-    expect(payload.message).toContain(
-      "The top-level ACP session did not report an effective model.",
-    );
-  });
-});
-
 // ---------------------------------------------------------------------------
 // executeAcpSpawn — CLAUDE_CODE_OAUTH_TOKEN env injection + preflight
 //

package/src/tools/acp/spawn.ts

@@ -37,7 +37,6 @@ export const acpSpawnInputSchema = z.looseObject({
   agent: nullAsOmitted(z.string()),
   task: nullAsOmitted(z.string()),
   cwd: nullAsOmitted(z.string()),
-  model: nullAsOmitted(z.string()),
 });
 
 /**
@@ -67,7 +66,6 @@ export async function executeAcpSpawn(
   }
   const agent = parsedInput.data.agent || "claude";
   const task = parsedInput.data.task;
-  const model = parsedInput.data.model;
 
   if (!task) {
     return { content: '"task" is required.', isError: true };
@@ -126,20 +124,14 @@ export async function executeAcpSpawn(
     // Recheck: the auto-install and the agent-env preparation above are both
     // awaits, and this is the point a long-lived subprocess starts.
     throwIfCancelled(context);
-    const {
-      acpSessionId,
-      protocolSessionId,
-      requestedModel,
-      effectiveModel,
-      modelWarning,
-    } = await manager.spawn(
+    const { acpSessionId, protocolSessionId } = await manager.spawn(
       agent,
       agentConfig,
       task,
       cwd,
       context.conversationId,
       sendToClient,
-      { parentToolUseId: context.toolUseId, model },
+      context.toolUseId,
       // The manager rechecks after its own protocol handshake and session
       // creation, so a turn stopped in that window tears the child process
       // down instead of handing it the task.
@@ -158,26 +150,16 @@ export async function executeAcpSpawn(
     const installNote = autoInstalledPackage
       ? ` Installed ${autoInstalledPackage} automatically.`
       : "";
-    // Relayed verbatim: the warning already distinguishes an adapter with no
-    // selector from one that refused the value.
-    const modelNote = modelWarning
-      ? ` The requested model was not applied: ${modelWarning}`
-      : "";
-    const effectiveModelNote = effectiveModel
-      ? ` The top-level ACP session reports "${effectiveModel}" as its effective model.`
-      : " The top-level ACP session did not report an effective model.";
     const payload = JSON.stringify({
       acpSessionId,
       protocolSessionId,
       agent,
       cwd,
-      requestedModel: requestedModel ?? null,
-      effectiveModel: effectiveModel ?? null,
       status: "running",
       message:
         `ACP agent "${agent}" spawned (session: ${protocolSessionId}). ` +
         `Results stream back via SSE. You will be notified when it completes.` +
-        `${installNote}${modelNote}${effectiveModelNote}${resumeHint}`,
+        `${installNote}${resumeHint}`,
     });
 
     return { content: payload, isError: false };

package/src/tools/acp/status.ts

@@ -1,7 +1,6 @@
 import { z } from "zod";
 
 import { getAcpSessionManager } from "../../acp/index.js";
-import type { AcpSessionState } from "../../acp/types.js";
 import {
   invalidToolInputResult,
   nullAsOmitted,
@@ -17,40 +16,6 @@ export const acpStatusInputSchema = z.looseObject({
   acp_session_id: nullAsOmitted(z.string()),
 });
 
-/**
- * The session fields worth spending LLM context on. An allowlist rather than
- * an omission: `availableModels` is a picker for the human surfaces (HTTP and
- * SSE) that says nothing the agent can act on, and a field added to
- * `AcpSessionState` for those surfaces should not reach this one by default.
- * `model` stays: which model a session is running on is answerable.
- */
-function projectSession(state: AcpSessionState) {
-  return {
-    id: state.id,
-    agentId: state.agentId,
-    acpSessionId: state.acpSessionId,
-    parentConversationId: state.parentConversationId,
-    status: state.status,
-    startedAt: state.startedAt,
-    completedAt: state.completedAt,
-    error: state.error,
-    stopReason: state.stopReason,
-    task: state.task,
-    parentToolUseId: state.parentToolUseId,
-    authErrorCode: state.authErrorCode,
-    authErrorCredential: state.authErrorCredential,
-    latestUsage: state.latestUsage,
-    model: state.model,
-  };
-}
-
-/** Projects either shape `getStatus` answers with. */
-function projectStatus(status: AcpSessionState | AcpSessionState[]): unknown {
-  return Array.isArray(status)
-    ? status.map(projectSession)
-    : projectSession(status);
-}
-
 export async function executeAcpStatus(
   input: Record<string, unknown>,
   _context: ToolContext,
@@ -64,8 +29,9 @@ export async function executeAcpStatus(
 
   try {
     if (acpSessionId) {
+      const state = manager.getStatus(acpSessionId);
       return {
-        content: JSON.stringify(projectStatus(manager.getStatus(acpSessionId))),
+        content: JSON.stringify(state),
         isError: false,
       };
     }
@@ -76,10 +42,7 @@ export async function executeAcpStatus(
       return { content: "No ACP sessions found.", isError: false };
     }
 
-    return {
-      content: JSON.stringify(projectStatus(allStates)),
-      isError: false,
-    };
+    return { content: JSON.stringify(allStates), isError: false };
   } catch (err) {
     const msg = err instanceof Error ? err.message : String(err);
     return { content: msg, isError: true };

package/src/tools/app-control/skill-proxy-bridge.ts

@@ -5,7 +5,6 @@
  * the proxy resolver, which forwards the call to the connected client.
  */
 
-import { formatDesktopAppRequired } from "../capability-offer.js";
 import { throwIfCancelled } from "../shared/abort.js";
 import type { ToolContext, ToolExecutionResult } from "../types.js";
 
@@ -33,7 +32,7 @@ export function forwardAppControlProxyTool(
   }
   if (!context.proxyToolResolver) {
     return Promise.resolve({
-      content: formatDesktopAppRequired("apps"),
+      content: `Cannot execute ${toolName}: no proxy resolver available. This tool requires a connected client.`,
       isError: true,
     });
   }

package/src/tools/capability-offer.test.ts

@@ -4,26 +4,11 @@ import {
   appendLoggedInBrowserOffer,
   CHROME_WEB_STORE_INSTALL_URL,
   DESKTOP_APP_DOWNLOAD_URL,
-  formatDesktopAppRequired,
   formatLoggedInBrowserOffer,
   isPhoneSurface,
   shouldOfferLoggedInBrowser,
 } from "./capability-offer.js";
 
-describe("formatDesktopAppRequired", () => {
-  test("offers a direct desktop-app download and retry step", () => {
-    const message = formatDesktopAppRequired("shell");
-
-    expect(message).toContain(
-      "The Vellum desktop app is needed to run commands on your computer",
-    );
-    expect(message).toContain(DESKTOP_APP_DOWNLOAD_URL);
-    expect(message).toContain("Open it, then retry.");
-    expect(message).not.toContain("client");
-    expect(message).not.toContain("daemon");
-  });
-});
-
 describe("isPhoneSurface", () => {
   test("is true for iOS transport and mobile client OS", () => {
     expect(isPhoneSurface({ transportInterface: "ios" })).toBe(true);
@@ -64,9 +49,7 @@ describe("shouldOfferLoggedInBrowser", () => {
     expect(shouldOfferLoggedInBrowser('Invalid browser_mode "bogus"')).toBe(
       false,
     );
-    expect(shouldOfferLoggedInBrowser("url must use http or https")).toBe(
-      false,
-    );
+    expect(shouldOfferLoggedInBrowser("url must use http or https")).toBe(false);
   });
 });
 
@@ -88,7 +71,6 @@ describe("formatLoggedInBrowserOffer", () => {
     });
     expect(offer).toContain("no in-app browser");
     expect(offer).toContain("Do not describe a browser panel");
-    expect(offer).toContain("Mac or Windows PC");
     expect(offer).toContain(DESKTOP_APP_DOWNLOAD_URL);
     expect(offer).toContain(CHROME_WEB_STORE_INSTALL_URL);
   });
@@ -96,9 +78,7 @@ describe("formatLoggedInBrowserOffer", () => {
 
 describe("appendLoggedInBrowserOffer", () => {
   test("appends the offer once", () => {
-    const once = appendLoggedInBrowserOffer(
-      "Error: Navigation failed: timeout",
-    );
+    const once = appendLoggedInBrowserOffer("Error: Navigation failed: timeout");
     expect(once).toContain("Error: Navigation failed: timeout");
     expect(once).toContain(DESKTOP_APP_DOWNLOAD_URL);
     const twice = appendLoggedInBrowserOffer(once);

package/src/tools/capability-offer.ts

@@ -7,22 +7,7 @@ export const DESKTOP_APP_DOWNLOAD_URL = "https://www.vellum.ai/downloads";
 export const CHROME_WEB_STORE_INSTALL_URL =
   "https://chromewebstore.google.com/detail/vellum-assistant-browser/hphbdmpffeigpcdjkckleobjmhhokpne";
 
-export const DESKTOP_APP_INSTALL_HINT = `Download the Vellum desktop app: ${DESKTOP_APP_DOWNLOAD_URL}. Open it, then retry.`;
-
-export type DesktopAppCapability = "files" | "screen" | "shell" | "apps";
-
-const DESKTOP_APP_CAPABILITY_ACTION: Record<DesktopAppCapability, string> = {
-  files: "access files on your computer",
-  screen: "view or control your screen",
-  shell: "run commands on your computer",
-  apps: "control apps on your computer",
-};
-
-export function formatDesktopAppRequired(
-  capability: DesktopAppCapability,
-): string {
-  return `Error: The Vellum desktop app is needed to ${DESKTOP_APP_CAPABILITY_ACTION[capability]}. ${DESKTOP_APP_INSTALL_HINT}`;
-}
+export const DESKTOP_APP_INSTALL_HINT = `Install the Vellum desktop app: ${DESKTOP_APP_DOWNLOAD_URL}`;
 
 /** User-facing install step for status userActions and command-failure hints. */
 export const CHROME_EXTENSION_INSTALL_HINT = `Install the Vellum Assistant Chrome extension from the Chrome Web Store: ${CHROME_WEB_STORE_INSTALL_URL}`;

package/src/tools/computer-use/definitions.ts

@@ -8,7 +8,6 @@
  */
 
 import { RiskLevel } from "../../permissions/types.js";
-import { formatDesktopAppRequired } from "../capability-offer.js";
 import type {
   ToolContext,
   ToolDefinition,
@@ -32,7 +31,7 @@ function proxyExecute(toolName: string) {
   ): Promise<ToolExecutionResult> => {
     if (!context.proxyToolResolver) {
       return {
-        content: formatDesktopAppRequired("screen"),
+        content: `No proxy resolver configured for proxy tool "${toolName}". This tool requires a connected desktop client.`,
         isError: true,
       };
     }
@@ -457,8 +456,7 @@ export const computerUseObserveTool = {
         type: "integer",
         minimum: 1,
         maximum: 4294967295,
-        description:
-          "macOS only: capture this native CGWindowID instead of the desktop, including only its accessibility tree. Obtain a current native window ID first; do not guess or use a browser tab ID. Applies to this observation only, not subsequent actions. Requires a desktop helper with window-capture support. Screenshot coordinates are window-relative; use accessibility element IDs for later actions, not desktop scaling.",
+        description: "macOS only: capture this native CGWindowID instead of the desktop, including only its accessibility tree. Obtain a current native window ID first; do not guess or use a browser tab ID. Applies to this observation only, not subsequent actions. Requires a desktop helper with window-capture support. Screenshot coordinates are window-relative; use accessibility element IDs for later actions, not desktop scaling.",
       },
       target_client_id: {
         type: "string",

package/src/tools/computer-use/skill-proxy-bridge.ts

@@ -5,7 +5,6 @@
  * the proxy resolver, which forwards the call to the connected desktop client.
  */
 
-import { formatDesktopAppRequired } from "../capability-offer.js";
 import { throwIfCancelled } from "../shared/abort.js";
 import type { ToolContext, ToolExecutionResult } from "../types.js";
 
@@ -46,7 +45,7 @@ export function forwardComputerUseProxyTool(
   }
   if (!context.proxyToolResolver) {
     return Promise.resolve({
-      content: formatDesktopAppRequired("screen"),
+      content: `Cannot execute ${toolName}: no proxy resolver available. This tool requires a connected desktop client.`,
       isError: true,
     });
   }

package/src/tools/credentials/metadata-store.ts

@@ -20,7 +20,6 @@ import {
 } from "@vellumai/credential-storage";
 import type { CredentialRecord } from "@vellumai/service-contracts/credential-rpc";
 
-import { createCesHttpRecordBackendIfConfigured } from "../../security/ces-http-record-backend.js";
 import type { CredentialRecordBackend } from "../../security/ces-rpc-record-backend.js";
 import { getLogger } from "../../util/logger.js";
 import type { CredentialInjectionTemplate } from "./policy-types.js";
@@ -56,11 +55,6 @@ const CES_MEMORY_PATH = "ces-in-memory-credential-metadata";
 let _store: StaticCredentialMetadataStore | undefined;
 let _overridePath: string | null = null;
 let _recordBackend: CredentialRecordBackend | undefined;
-/**
- * Test override for the CES HTTP listing fallback. `undefined` uses the
- * production factory. `null` disables HTTP fallback.
- */
-let _httpRecordBackendOverride: CredentialRecordBackend | null | undefined;
 let _cesRecordsLoaded = false;
 let _cesLoad: Promise<void> | undefined;
 let _cesLoadGeneration = 0;
@@ -261,93 +255,6 @@ export function listCredentialMetadata(): CredentialMetadata[] {
   return getStore().list() as CredentialMetadata[];
 }
 
-export type CredentialRecordListResult = {
-  records: CredentialMetadata[];
-  unreachable: boolean;
-};
-
-type ListedCredentialRecords = Array<{
-  account: string;
-  record: CredentialRecord;
-}>;
-
-function backendIsAvailable(backend: CredentialRecordBackend): boolean {
-  try {
-    return backend.isAvailable();
-  } catch {
-    return false;
-  }
-}
-
-async function listFromRecordBackend(
-  backend: CredentialRecordBackend,
-): Promise<ListedCredentialRecords | null> {
-  if (!backendIsAvailable(backend)) {
-    return null;
-  }
-  return backend.list();
-}
-
-function resolveHttpRecordBackend(): CredentialRecordBackend | null {
-  if (_httpRecordBackendOverride !== undefined) {
-    return _httpRecordBackendOverride;
-  }
-  return createCesHttpRecordBackendIfConfigured();
-}
-
-async function listFromHttpFallback(): Promise<ListedCredentialRecords | null> {
-  const httpBackend = resolveHttpRecordBackend();
-  if (!httpBackend) {
-    return null;
-  }
-  return listFromRecordBackend(httpBackend);
-}
-
-/**
- * Live catalog from the credential record backend. Does not read the
- * daemon in-process cache, so an outage cannot be mistaken for an empty
- * vault. `unreachable` is true when the backend is missing, down, or
- * the list RPC fails, and CES HTTP failover is also unavailable.
- *
- * When the attached backend is CES RPC (or none is attached), listing
- * fails over to CES HTTP in containerized deployments so a down RPC
- * socket does not hide a healthy vault sidecar. A successful empty
- * catalog (`[]`) does not fall through to HTTP.
- *
- * The file-backed test store is the catalog when a test override path is set.
- */
-export async function listCredentialRecordsLive(): Promise<CredentialRecordListResult> {
-  if (_overridePath) {
-    return {
-      records: getStore().list() as CredentialMetadata[],
-      unreachable: false,
-    };
-  }
-  if (_recordBackend) {
-    const remote = await listFromRecordBackend(_recordBackend);
-    if (remote !== null) {
-      return {
-        records: remote.map((entry) => entry.record as CredentialMetadata),
-        unreachable: false,
-      };
-    }
-    if (_recordBackend.name !== "ces-rpc") {
-      return { records: [], unreachable: true };
-    }
-  }
-  const httpRemote = await listFromHttpFallback();
-  if (httpRemote !== null) {
-    log.warn(
-      "CES RPC credential record list unavailable. Failing over to CES HTTP.",
-    );
-    return {
-      records: httpRemote.map((entry) => entry.record as CredentialMetadata),
-      unreachable: false,
-    };
-  }
-  return { records: [], unreachable: true };
-}
-
 /**
  * Delete metadata for a credential.
  */
@@ -380,10 +287,3 @@ export function _setMetadataPath(path: string | null): void {
     }
   }
 }
-
-/** @internal Test-only: override the CES HTTP listing fallback. */
-export function _setHttpRecordBackendForTests(
-  backend: CredentialRecordBackend | null | undefined,
-): void {
-  _httpRecordBackendOverride = backend;
-}

package/src/tools/host-filesystem/edit.test.ts

@@ -71,9 +71,8 @@ describe("host_file_edit cross-client guards", () => {
     );
     expect(result.isError).toBe(true);
     expect(result.content).toContain(
-      "The Vellum desktop app is needed to access files on your computer",
+      "no client with host_file capability is connected",
     );
-    expect(result.content).toContain("https://www.vellum.ai/downloads");
   });
 
   test("returns 'specified client disconnected' error when targetClientId set but proxy unavailable on web", async () => {

package/src/tools/host-filesystem/edit.ts

@@ -4,7 +4,7 @@ import { supportsHostProxy } from "../../channels/types.js";
 import { HostFileProxy } from "../../daemon/host-file-proxy.js";
 import { RiskLevel } from "../../permissions/types.js";
 import { assistantEventHub } from "../../runtime/assistant-event-hub.js";
-import { formatDesktopAppRequired } from "../capability-offer.js";
+import { desktopClientName } from "../client-os.js";
 import { FileSystemOps } from "../shared/filesystem/file-ops-service.js";
 import { formatEditDiff } from "../shared/filesystem/format-diff.js";
 import { hostPolicy } from "../shared/filesystem/path-policy.js";
@@ -115,7 +115,7 @@ export const hostFileEditTool = {
       !HostFileProxy.instance.isAvailable()
     ) {
       return {
-        content: formatDesktopAppRequired("files"),
+        content: `Error: no client with host_file capability is connected. Connect a ${desktopClientName(context)} client to use host_file from a non-desktop interface.`,
         isError: true,
       };
     }

package/src/tools/host-filesystem/read.test.ts

@@ -67,9 +67,8 @@ describe("host_file_read cross-client guards", () => {
     );
     expect(result.isError).toBe(true);
     expect(result.content).toContain(
-      "The Vellum desktop app is needed to access files on your computer",
+      "no client with host_file capability is connected",
     );
-    expect(result.content).toContain("https://www.vellum.ai/downloads");
   });
 
   test("returns 'specified client disconnected' error when targetClientId set but proxy unavailable on web", async () => {

package/src/tools/host-filesystem/read.ts

@@ -6,7 +6,7 @@ import { supportsHostProxy } from "../../channels/types.js";
 import { HostFileProxy } from "../../daemon/host-file-proxy.js";
 import { RiskLevel } from "../../permissions/types.js";
 import { assistantEventHub } from "../../runtime/assistant-event-hub.js";
-import { formatDesktopAppRequired } from "../capability-offer.js";
+import { desktopClientName } from "../client-os.js";
 import {
   AUDIO_EXTENSIONS,
   readAudioFile,
@@ -121,7 +121,7 @@ export const hostFileReadTool = {
       !HostFileProxy.instance.isAvailable()
     ) {
       return {
-        content: formatDesktopAppRequired("files"),
+        content: `Error: no client with host_file capability is connected. Connect a ${desktopClientName(context)} client to use host_file from a non-desktop interface.`,
         isError: true,
       };
     }