remove top level tool calls

Author: charleslien

packages/agent-runtime/src/mcp.ts

@@ -42,7 +42,6 @@ export async function getMCPToolData(
         for (const { name, description, inputSchema } of mcpData) {
           writeTo[mcpName + '/' + name] = {
             inputJsonSchema: inputSchema,
-            endsAgentStep: true,
             description,
           }
         }

packages/agent-runtime/src/run-programmatic-step.ts

@@ -270,12 +270,6 @@ export async function runProgrammaticStep(
       const excludeToolFromMessageHistory = toolCall?.includeToolCall === false
       // Add assistant message with the tool call before executing it
       if (!excludeToolFromMessageHistory) {
-        onResponseChunk({
-          type: 'tool_call',
-          toolCallId: toolCall.toolCallId,
-          toolName: toolCall.toolName,
-          input: toolCall.input,
-        })
         state.messages.push(
           assistantMessage({
             ...toolCall,

packages/agent-runtime/src/tools/definitions/tool/end-turn.ts

@@ -14,7 +14,7 @@ Only use this tool to hand control back to the user.
 - Effect: Signals the UI to wait for the user's reply; any pending tool results will be ignored.
 
 *INCORRECT USAGE*:
-${getToolCallString('some_tool_that_produces_results', { query: 'some example search term' }, false)}
+${getToolCallString('some_tool_that_produces_results', { query: 'some example search term' })}
 
 ${getToolCallString(toolName, {})}
 

sdk/src/run.ts

@@ -223,7 +223,11 @@ export async function run({
       return
     }
     const { chunk } = action
-    addToPendingAssistantMessage: if (typeof chunk === 'string') {
+    if (typeof chunk !== 'string' && chunk.type === 'tool_call') {
+      logger?.info({ stack: new Error().stack, chunk }, `asdf onResponseChunk`)
+    }
+
+    if (typeof chunk === 'string') {
       pendingAgentResponse += chunk
     } else if (
       chunk.type === 'reasoning_delta' &&