Fix #152 empty arguments parsing error in ai-sdk extension

Author: seratch

.changeset/common-carrots-wait.md

@@ -0,0 +1,5 @@
+---
+'@openai/agents-extensions': patch
+---
+
+Fix #152 empty arguments parsing error in ai-sdk extension

packages/agents-extensions/src/aiSdk.ts

@@ -168,7 +168,7 @@ export function itemsToLanguageV1Messages(
           type: 'tool-call',
           toolCallId: item.callId,
           toolName: item.name,
-          args: JSON.parse(item.arguments),
+          args: parseArguments(item.arguments),
           providerMetadata: {
             ...(item.providerData ?? {}),
           },
@@ -775,3 +775,15 @@ export class AiSdkModel implements Model {
 export function aisdk(model: LanguageModelV1) {
   return new AiSdkModel(model);
 }
+
+export function parseArguments(args: string | undefined | null): any {
+  if (!args) {
+    return {};
+  }
+
+  try {
+    return JSON.parse(args);
+  } catch (_) {
+    return {};
+  }
+}

packages/agents-extensions/test/aiSdk.test.ts

@@ -3,6 +3,7 @@ import {
   AiSdkModel,
   getResponseFormat,
   itemsToLanguageV1Messages,
+  parseArguments,
   toolToLanguageV1Tool,
 } from '../src/aiSdk';
 import { protocol, withTrace, UserError } from '@openai/agents';
@@ -802,4 +803,23 @@ describe('AiSdkModel', () => {
       },
     ]);
   });
+
+  describe('parseArguments', () => {
+    test('should parse valid JSON', () => {
+      expect(parseArguments(undefined)).toEqual({});
+      expect(parseArguments(null)).toEqual({});
+      expect(parseArguments('')).toEqual({});
+      expect(parseArguments(' ')).toEqual({});
+      expect(parseArguments('{ ')).toEqual({});
+      expect(parseArguments('foo')).toEqual({});
+      expect(parseArguments('{}')).toEqual({});
+      expect(parseArguments('{ }')).toEqual({});
+
+      expect(parseArguments('"foo"')).toEqual('foo');
+      expect(parseArguments('[]')).toEqual([]);
+      expect(parseArguments('[1,2,3]')).toEqual([1, 2, 3]);
+      expect(parseArguments('{"a":1}')).toEqual({ a: 1 });
+      expect(parseArguments('{"a":1,"b":"c"}')).toEqual({ a: 1, b: 'c' });
+    });
+  });
 });