fix: persist container_id in codeInterpreterTool history (fixes #253)

Author: vrtnis

.changeset/thin-kids-wink.md

@@ -0,0 +1,5 @@
+---
+'@openai/agents-openai': patch
+---
+
+Persist container_id in codeInterpreterTool history (fixes #253)

packages/agents-openai/src/openaiResponsesModel.ts

@@ -33,7 +33,10 @@ import {
   ImageGenerationStatus,
   WebSearchStatus,
 } from './tools';
-import { camelOrSnakeToSnakeCase } from './utils/providerData';
+import {
+  camelOrSnakeToSnakeCase,
+  snakeToCamelCase,
+} from './utils/providerData';
 import { ProviderData } from '@openai/agents-core/types';
 
 type ToolChoice = ToolChoiceOptions | ToolChoiceTypes | ToolChoiceFunction;
@@ -710,7 +713,7 @@ function convertToOutputItem(
         name: item.type,
         status,
         output: outputData,
-        providerData: remainingItem,
+        providerData: snakeToCamelCase(remainingItem),
       };
       return output;
     } else if (item.type === 'function_call') {
@@ -722,7 +725,7 @@ function convertToOutputItem(
         name,
         status,
         arguments: args,
-        providerData,
+        providerData: snakeToCamelCase(providerData),
       };
       return output;
     } else if (item.type === 'computer_call') {
@@ -733,7 +736,7 @@ function convertToOutputItem(
         callId: call_id,
         status,
         action,
-        providerData,
+        providerData: snakeToCamelCase(providerData),
       };
       return output;
     } else if (item.type === 'mcp_list_tools') {
@@ -744,7 +747,7 @@ function convertToOutputItem(
         name: item.type,
         status: 'completed',
         output: undefined,
-        providerData,
+        providerData: snakeToCamelCase(providerData),
       };
       return output;
     } else if (item.type === 'mcp_approval_request') {
@@ -755,7 +758,7 @@ function convertToOutputItem(
         name: 'mcp_approval_request',
         status: 'completed',
         output: undefined,
-        providerData,
+        providerData: snakeToCamelCase(providerData),
       };
       return output;
     } else if (item.type === 'mcp_call') {
@@ -767,7 +770,7 @@ function convertToOutputItem(
         name: item.type,
         status: 'completed',
         output: outputData || undefined,
-        providerData,
+        providerData: snakeToCamelCase(providerData),
       };
       return output;
     } else if (item.type === 'reasoning') {
@@ -782,10 +785,10 @@ function convertToOutputItem(
           return {
             type: 'input_text',
             text,
-            providerData: remainingContent,
+            providerData: snakeToCamelCase(remainingContent),
           };
         }),
-        providerData,
+        providerData: snakeToCamelCase(providerData),
       };
       return output;
     }

packages/agents-openai/src/utils/providerData.ts

@@ -19,3 +19,25 @@ export function camelOrSnakeToSnakeCase<
   }
   return result;
 }
+
+/**
+ * Converts snake_case keys of an object to camelCase recursively.
+ */
+export function snakeToCamelCase<T extends Record<string, any> | undefined>(
+  providerData: T | undefined,
+): Record<string, any> | undefined {
+  if (
+    !providerData ||
+    typeof providerData !== 'object' ||
+    Array.isArray(providerData)
+  ) {
+    return providerData;
+  }
+
+  const result: Record<string, any> = {};
+  for (const [key, value] of Object.entries(providerData)) {
+    const camelKey = key.replace(/_([a-z])/g, (_, c) => c.toUpperCase());
+    result[camelKey] = snakeToCamelCase(value);
+  }
+  return result;
+}

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

@@ -328,4 +328,21 @@ describe('convertToOutputItem', () => {
       ] as any),
     ).toThrow();
   });
+
+  it('converts snake_case provider data to camelCase', () => {
+    const out = convertToOutputItem([
+      {
+        type: 'code_interpreter_call',
+        id: 'ci',
+        status: 'completed',
+        container_id: 'xyz',
+        code: 'print()',
+        outputs: [],
+      },
+    ] as any);
+    expect(out[0]).toMatchObject({
+      type: 'hosted_tool_call',
+      providerData: { containerId: 'xyz' },
+    });
+  });
 });

packages/agents-openai/test/utils/providerData.test.ts

@@ -1,5 +1,8 @@
 import { describe, it, expect } from 'vitest';
-import { camelOrSnakeToSnakeCase } from '../../src/utils/providerData';
+import {
+  camelOrSnakeToSnakeCase,
+  snakeToCamelCase,
+} from '../../src/utils/providerData';
 
 describe('camelToSnakeCase', () => {
   it('converts flat camelCase keys to snake_case', () => {
@@ -68,3 +71,25 @@ describe('camelToSnakeCase', () => {
     });
   });
 });
+
+describe('snakeToCamelCase', () => {
+  it('converts snake_case keys to camelCase', () => {
+    expect(snakeToCamelCase({ foo_bar: 1, baz_qux: 2 })).toEqual({
+      fooBar: 1,
+      bazQux: 2,
+    });
+  });
+
+  it('handles nested objects', () => {
+    expect(
+      snakeToCamelCase({
+        outer_key: { inner_key: 42, another: { deep_key: 'x' } },
+      }),
+    ).toEqual({ outerKey: { innerKey: 42, another: { deepKey: 'x' } } });
+  });
+
+  it('leaves arrays and primitives unchanged', () => {
+    expect(snakeToCamelCase([1, 2, 3])).toEqual([1, 2, 3]);
+    expect(snakeToCamelCase(undefined)).toBe(undefined);
+  });
+});