Fix

Author: seratch

packages/agents-core/src/runState.ts

@@ -31,6 +31,7 @@ import * as protocol from './types/protocol';
 import { AgentInputItem, UnknownContext } from './types';
 import type { InputGuardrailResult, OutputGuardrailResult } from './guardrail';
 import { safeExecute } from './utils/safeExecute';
+import { HostedMCPTool } from './tool';
 
 /**
  * The schema version of the serialized run state. This is used to ensure that the serialized
@@ -152,12 +153,28 @@ const serializedProcessedResponseSchema = z.object({
       computer: z.any(),
     }),
   ),
-  mcpApprovalRequests: z.array(
-    z.object({
-      requestItem: z.any(),
-      mcpTool: z.any(),
-    }),
-  ),
+  mcpApprovalRequests: z
+    .array(
+      z.object({
+        requestItem: z.object({
+          // protocol.HostedToolCallItem
+          rawItem: z.object({
+            type: z.literal('hosted_tool_call'),
+            name: z.string(),
+            arguments: z.string().optional(),
+            status: z.string().optional(),
+            output: z.string().optional(),
+          }),
+        }),
+        // HostedMCPTool
+        mcpTool: z.object({
+          type: z.literal('hosted_tool'),
+          name: z.literal('hosted_mcp'),
+          providerData: z.record(z.string(), z.any()),
+        }),
+      }),
+    )
+    .optional(),
 });
 
 const guardrailFunctionOutputSchema = z.object({
@@ -740,15 +757,16 @@ async function deserializeProcessedResponse<TContext = UnknownContext>(
         };
       },
     ),
-    mcpApprovalRequests: serializedProcessedResponse.mcpApprovalRequests.map(
-      (approvalRequest) => ({
-        requestItem: new RunToolApprovalItem(
-          approvalRequest.requestItem.rawItem,
-          currentAgent,
-        ),
-        mcpTool: approvalRequest.mcpTool,
-      }),
-    ),
+    mcpApprovalRequests: (
+      serializedProcessedResponse.mcpApprovalRequests ?? []
+    ).map((approvalRequest) => ({
+      requestItem: new RunToolApprovalItem(
+        approvalRequest.requestItem
+          .rawItem as unknown as protocol.HostedToolCallItem,
+        currentAgent,
+      ),
+      mcpTool: approvalRequest.mcpTool as unknown as HostedMCPTool,
+    })),
   };
 
   return {
@@ -757,6 +775,7 @@ async function deserializeProcessedResponse<TContext = UnknownContext>(
       return (
         result.handoffs.length > 0 ||
         result.functions.length > 0 ||
+        result.mcpApprovalRequests.length > 0 ||
         result.computerActions.length > 0
       );
     },

packages/agents-core/test/run.test.ts

@@ -88,7 +88,7 @@ describe('Runner.run', () => {
 
       const rawItem = {
         name: 'toolZ',
-        call_id: 'c1',
+        callId: 'c1',
         type: 'function_call',
         arguments: '{}',
       } as any;
@@ -126,6 +126,7 @@ describe('Runner.run', () => {
           },
         ],
         handoffs: [],
+        mcpApprovalRequests: [],
         computerActions: [],
       } as any;
 

packages/agents-openai/test/openaiResponsesModel.helpers.test.ts

@@ -130,9 +130,9 @@ describe('converTool', () => {
       type: 'hosted_tool',
       providerData: {
         type: 'mcp',
-        server_label: 'deepwiki',
-        server_url: 'https://mcp.deepwiki.com/mcp',
-        require_approval: 'never',
+        serverLabel: 'deepwiki',
+        serverUrl: 'https://mcp.deepwiki.com/mcp',
+        requireApproval: 'never',
       },
     } as any);