Address comments from @RomneyDa

Author: mcowger

core/tools/parseArgs.ts

@@ -26,19 +26,16 @@ export function getStringArg(
 
   let value = args[argName];
 
-  // Handle case where JSON content was parsed into an object by the tool call parser.
+  // Handle case where JSON was parsed into an object by the tool call parser.
   // If the arguments to the tool call are valid JSON (e.g. the model attempts to create a .json file)
-  // the earlier call to JSON.parse() will have deeply parsed the returned contents.
+  // the earlier call to JSON.parse() will have deeply parsed the returned arguments.
   // If that has happened, convert back to string.
   if (typeof value === "object" && value !== null) {
-    // Special handling for contents parameter which should always be a string
-    if (argName === "contents") {
+    try {
       value = JSON.stringify(value);
       return value;
-    } else {
-      throw new Error(
-        `Argument \`${argName}\` must be a string, not an object`,
-      );
+    } catch (e) {
+      //Swallow this, because it might be fine later.
     }
   }
 

core/tools/parseArgs.vitest.ts

@@ -176,13 +176,6 @@ describe("getStringArg", () => {
     expect(result).toBe('["item1","item2",{"key":"value"}]');
   });
 
-  it("should throw error when non-contents parameter is an object", () => {
-    const args = { filename: { invalid: "object" } };
-    expect(() => getStringArg(args, "filename")).toThrowError(
-      "Argument `filename` must be a string, not an object",
-    );
-  });
-
   it("should handle contents parameter that is already a string", () => {
     const args = { contents: "already a string" };
     const result = getStringArg(args, "contents");