fix(mothership): execute display-named frames by their execName

Companion to mothership b4910ed8. The worker's CLI frames carry cli_*
display identities; execution now dispatches on the frame's execName
(ToolCallState.execName) while rendering and persistence keep the display
name. Regenerated stream contract (execName + ui.simExecutable); the
generator's formatter step also un-broken (biome refuses stdin paths its
config excludes, so generated output formats under a neutral path).

Claude-Session: https://claude.ai/code/session_01CgaxNAaeD3taGdghbXn17w

Author: Sg312

apps/sim/lib/mothership/generated/mothership-stream-v1-schema.ts

@@ -1179,6 +1179,9 @@ export const MOTHERSHIP_STREAM_V1_SCHEMA: JsonSchema = {
         arguments: {
           $ref: '#/$defs/MothershipStreamV1AdditionalPropertiesMap',
         },
+        execName: {
+          type: 'string',
+        },
         executor: {
           $ref: '#/$defs/MothershipStreamV1ToolExecutor',
         },
@@ -1351,6 +1354,9 @@ export const MOTHERSHIP_STREAM_V1_SCHEMA: JsonSchema = {
         internal: {
           type: 'boolean',
         },
+        simExecutable: {
+          type: 'boolean',
+        },
       },
       type: 'object',
     },

apps/sim/lib/mothership/generated/mothership-stream-v1.ts

@@ -145,6 +145,7 @@ export interface MothershipStreamV1ToolCallEventEnvelope {
 }
 export interface MothershipStreamV1ToolCallDescriptor {
   arguments?: MothershipStreamV1AdditionalPropertiesMap
+  execName?: string
   executor: MothershipStreamV1ToolExecutor
   mode: MothershipStreamV1ToolMode
   partial?: boolean
@@ -162,6 +163,7 @@ export interface MothershipStreamV1ToolUI {
   hidden?: boolean
   inbandOwned?: boolean
   internal?: boolean
+  simExecutable?: boolean
 }
 export interface MothershipStreamV1ToolArgsDeltaEventEnvelope {
   payload: MothershipStreamV1ToolArgsDeltaPayload

apps/sim/lib/mothership/request/handlers/handlers.test.ts

@@ -444,6 +444,36 @@ describe('sse-handlers tool lifecycle', () => {
     expect(context.finalAssistantContent).toBe('Final answer only.')
   })
 
+  it('executes a display-named frame by its execName (the worker CLI class)', async () => {
+    // The worker wires sim_cli frames with toolName cli_* (display identity) and
+    // execName sim_cli. Dispatching by the display name sent every CLI call into
+    // the app registry as "Tool not found" — caught by the bench suite.
+    executeTool.mockResolvedValueOnce({ success: true, output: { exitCode: 0, stdout: '[]' } })
+    await sseHandlers.tool(
+      {
+        type: MothershipStreamV1EventType.tool,
+        payload: {
+          toolCallId: 'cli-1',
+          toolName: 'cli_workflows_list',
+          execName: 'sim_cli',
+          arguments: { args: ['workflows', 'list'] },
+          executor: MothershipStreamV1ToolExecutor.sim,
+          mode: MothershipStreamV1ToolMode.async,
+          phase: MothershipStreamV1ToolPhase.call,
+          ui: { simExecutable: true },
+        },
+      } satisfies StreamEvent,
+      context,
+      execContext,
+      { interactive: false, timeout: 1000 }
+    )
+    await sleep(0)
+    expect(executeTool).toHaveBeenCalledTimes(1)
+    expect(executeTool.mock.calls[0][0]).toBe('sim_cli')
+    // Rendering and persistence keep the display identity.
+    expect(context.toolCalls.get('cli-1')?.name).toBe('cli_workflows_list')
+  })
+
   it('executes tool_call and emits tool_result', async () => {
     executeTool.mockResolvedValueOnce({ success: true, output: { ok: true } })
     const onEvent = vi.fn()

apps/sim/lib/mothership/request/handlers/tool.ts

@@ -509,6 +509,7 @@ async function handleCallPhase(
   // into the server tool context — this is what scopes the prepare_file_edit ->
   // apply_file_edit intent handoff to one file subagent under concurrency.
   if (parentToolCallId) toolCall.parentToolCallId = parentToolCallId
+  if (data.execName) toolCall.execName = data.execName
 
   const readPath = typeof args?.path === 'string' ? args.path : undefined
   if (toolName === 'read' && readPath?.startsWith('internal/')) return

apps/sim/lib/mothership/request/tools/executor.ts

@@ -298,8 +298,11 @@ export function buildToolExecutionContext(
  * eventual settlement is ignored.
  */
 async function executeToolWithWatchdog(toolCall: ToolCallState, toolContext: ExecutionContext) {
-  const timeoutMs = toolWatchdogTimeoutMs(toolCall.name)
-  const execution = executeTool(toolCall.name, toolCall.params || {}, toolContext)
+  // The frame's wire name can be a display identity (the worker's cli_* names);
+  // execution always dispatches on the model's real tool name.
+  const executableName = toolCall.execName ?? toolCall.name
+  const timeoutMs = toolWatchdogTimeoutMs(executableName)
+  const execution = executeTool(executableName, toolCall.params || {}, toolContext)
   let timer: ReturnType<typeof setTimeout> | undefined
   try {
     return await Promise.race([

apps/sim/lib/mothership/request/types.ts

@@ -27,6 +27,9 @@ export function isTerminalToolCallStatus(status?: string): boolean {
 export interface ToolCallState {
   id: string
   name: string
+  /** The model's tool name when `name` is a display identity (the worker's cli_*
+   * names). Execution dispatches on this; rendering and persistence keep `name`. */
+  execName?: string
   status: ToolCallStatus
   /** Bounded registry ID of the agent that invoked this tool. */
   agentId?: string

scripts/format-generated-source.ts

@@ -1,16 +1,22 @@
 import { spawnSync } from 'node:child_process'
 import { localBin } from './local-bin'
+import { join } from 'node:path'
 
 export function formatGeneratedSource(source: string, stdinFilePath: string, cwd: string): string {
-  const result = spawnSync(localBin('biome'), ['format', '--stdin-file-path', stdinFilePath], {
+  // biome.json excludes the generated output dirs, and biome refuses to format a
+  // stdin whose declared path is excluded — so declare a neutral path instead;
+  // formatting rules do not vary by location, only ignores do.
+  void stdinFilePath
+  const neutralPath = join(cwd, 'scripts', '.generated-format-buffer.ts')
+  const result = spawnSync(localBin('biome'), ['format', '--stdin-file-path', neutralPath], {
     cwd,
     encoding: 'utf8',
     input: source,
   })
 
   if (result.status !== 0) {
     throw new Error(
-      `Failed to format generated source for ${stdinFilePath}:\n${
+      `Failed to format generated source for ${neutralPath}:\n${
         result.stderr || result.stdout || 'unknown error'
       }`
     )