@vellumai/assistant 0.8.5 → 0.8.6

package/src/subagent/types.ts

@@ -69,6 +69,12 @@ export interface SubagentConfig {
    * profile, every spawned subagent inherits it automatically.
    */
   overrideProfile?: string;
+  /**
+   * Tool-use id of the `skill_execute` call that spawned this subagent.
+   * Forwarded into the `subagent_spawned` event so the client can anchor the
+   * inline subagent card to the exact spawn tool call.
+   */
+  parentToolUseId?: string;
 }
 
 // ── State (runtime) ─────────────────────────────────────────────────────

package/src/tasks/tool-sanitizer.ts

@@ -1,4 +1,4 @@
-import { getAllTools, getTool } from "../tools/registry.js";
+import { getAllTools, getTool, getToolOwner } from "../tools/registry.js";
 
 /**
  * Deduplicate and sort a list of tool names, validating against the live
@@ -24,7 +24,7 @@ export function sanitizeToolList(tools: string[]): string[] {
  */
 export function getRegisteredToolNames(): string[] {
   return getAllTools()
-    .filter((t) => t.executionMode !== "proxy" && t.origin !== "skill")
+    .filter((t) => getToolOwner(t.name)?.kind !== "skill")
     .map((t) => t.name)
     .sort();
 }

package/src/tools/apps/definitions.ts

@@ -9,16 +9,31 @@
  */
 
 import { RiskLevel } from "../../permissions/types.js";
-import type { Tool, ToolExecutionResult } from "../types.js";
+import type { Tool, ToolContext, ToolExecutionResult } from "../types.js";
 
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------
 
-function proxyExecute(): Promise<ToolExecutionResult> {
-  throw new Error(
-    "Proxy tool: execution must be forwarded to the connected client",
-  );
+/**
+ * Forward execution to the connected macOS client via the request-bound
+ * `proxyToolResolver`. Returns a structured error when no resolver is
+ * configured (e.g. no client connected) so callers see a normal tool
+ * failure rather than an unhandled throw.
+ */
+function proxyExecute(toolName: string) {
+  return async (
+    input: Record<string, unknown>,
+    context: ToolContext,
+  ): Promise<ToolExecutionResult> => {
+    if (!context.proxyToolResolver) {
+      return {
+        content: `No proxy resolver configured for proxy tool "${toolName}". This tool requires an external resolver (e.g. a connected macOS client).`,
+        isError: true,
+      };
+    }
+    return context.proxyToolResolver(toolName, input);
+  };
 }
 
 // ---------------------------------------------------------------------------
@@ -31,27 +46,26 @@ const appOpenTool: Tool = {
     "Open a persistent app in a dynamic_page surface on the connected client.",
   category: "apps",
   defaultRiskLevel: RiskLevel.Low,
-  executionMode: "proxy",
   executionTarget: "host",
 
   input_schema: {
-        type: "object",
-        properties: {
-          app_id: {
-            type: "string",
-            description: "The ID of the app to open",
-          },
-          open_mode: {
-            type: "string",
-            enum: ["preview", "workspace"],
-            description:
-              "Display mode. 'preview' shows an inline preview card in chat. 'workspace' opens the full app in a workspace panel. Defaults to 'workspace'.",
-          },
-        },
-        required: ["app_id"],
+    type: "object",
+    properties: {
+      app_id: {
+        type: "string",
+        description: "The ID of the app to open",
+      },
+      open_mode: {
+        type: "string",
+        enum: ["preview", "workspace"],
+        description:
+          "Display mode. 'preview' shows an inline preview card in chat. 'workspace' opens the full app in a workspace panel. Defaults to 'workspace'.",
       },
+    },
+    required: ["app_id"],
+  },
 
-  execute: proxyExecute,
+  execute: proxyExecute("app_open"),
 };
 
 // ---------------------------------------------------------------------------

package/src/tools/browser/__tests__/browser-execution-acquire.test.ts

@@ -9,6 +9,7 @@
  */
 import { beforeEach, describe, expect, mock, test } from "bun:test";
 
+import { createMockLoggerModule } from "../../../__tests__/helpers/mock-logger.js";
 import type { ToolContext } from "../../types.js";
 import type { BrowserMode, CdpClientKind } from "../cdp-client/types.js";
 
@@ -111,14 +112,7 @@ mock.module("../runtime-check.js", () => ({
   }),
 }));
 
-mock.module("../../../util/logger.js", () => ({
-  getLogger: () => ({
-    debug: () => {},
-    warn: () => {},
-    info: () => {},
-    error: () => {},
-  }),
-}));
+mock.module("../../../util/logger.js", () => createMockLoggerModule());
 
 // Import under test after all mock.module calls.
 const { executeBrowserAttach } = await import("../browser-execution.js");