fix: remove redundant json stringify in converters

PiperOrigin-RevId: 885762877

Author: yyyu-google

Google.GenAI.E2E.Tests/Netstandard2_0Tests/packages.lock.json

@@ -64,12 +64,12 @@
       },
       "System.Text.Json": {
         "type": "Direct",
-        "requested": "[10.0.3, )",
-        "resolved": "10.0.3",
-        "contentHash": "NTUt9DL+maqbgrIYCAmeZUbX0NoXaueySyjW/bdOlFdSUDC1l51XnsbVEuj5tuad12vdq5Sviskp9uMVGgCNLw==",
+        "requested": "[10.0.4, )",
+        "resolved": "10.0.4",
+        "contentHash": "1tRPRt8D/kzjGL7em1uJ3iJlvVIC3G/sZJ+ZgSvtVYLXGGO26Clkqy2b5uts/pyR706Yw8/xA7exeI2PI50dpw==",
         "dependencies": {
-          "System.IO.Pipelines": "10.0.3",
-          "System.Text.Encodings.Web": "10.0.3"
+          "System.IO.Pipelines": "10.0.4",
+          "System.Text.Encodings.Web": "10.0.4"
         }
       },
       "TestServerSdk": {
@@ -240,8 +240,8 @@
       },
       "System.IO.Pipelines": {
         "type": "Transitive",
-        "resolved": "10.0.3",
-        "contentHash": "WMxiA2jGdHnRBmoVK55YUq5VPaxW0Sg2frPtXV+urUMvpqHIga6lleV/YuryHIuGsAKVjQAjv6PrQ6IJpoLohQ=="
+        "resolved": "10.0.4",
+        "contentHash": "V7+RO17s/tzCpgqyj6t5vb54HFCvrRaMEwTcKDwpoQK66DRROzSff6kqtzHyiWRj6hrQQUmW80NL4pFSNhYpYA=="
       },
       "System.Management": {
         "type": "Transitive",
@@ -271,8 +271,8 @@
       },
       "System.Text.Encodings.Web": {
         "type": "Transitive",
-        "resolved": "10.0.3",
-        "contentHash": "l8QNBPp92bVzl9Kw8nNtm1uYRNNhUrdulZjM4a8YK/QGNa8z9utKsC0bDoPB+Vq8LOlbD3fIyGlabtz80jT7cw=="
+        "resolved": "10.0.4",
+        "contentHash": "6g3B7jNsPRNf4luuYt1qE4R8S3JI+zMsfGWL9Idv4Mk1Z9Gh+rCagp9sG3AejPS87yBj1DjopM4i3wSz0WnEqg=="
       },
       "YamlDotNet": {
         "type": "Transitive",
@@ -283,7 +283,7 @@
         "type": "Project",
         "dependencies": {
           "Google.Apis.Auth": "[1.69.0, )",
-          "Microsoft.Extensions.AI.Abstractions": "[10.3.0, )",
+          "Microsoft.Extensions.AI.Abstractions": "[10.4.0, )",
           "MimeTypes": "[2.5.2, )"
         }
       },
@@ -300,11 +300,11 @@
       },
       "Microsoft.Extensions.AI.Abstractions": {
         "type": "CentralTransitive",
-        "requested": "[10.3.0, )",
-        "resolved": "10.3.0",
-        "contentHash": "hDjDvUERvUH3HBMs2MDusOcGJBjAHOG5pJIU2x/HZEa4e1UthNKt89cwMi3B+ogJo6skki1XFjfgGN3ksnVqvQ==",
+        "requested": "[10.4.0, )",
+        "resolved": "10.4.0",
+        "contentHash": "t3S2H4do4YeNheIfE3GEl3MnKIrnxpbLu7a88spfApYR3in9ddhIq/GMtxgMaFjn/PUMTCFv5YH7Y6Q91dsDXQ==",
         "dependencies": {
-          "System.Text.Json": "10.0.3"
+          "System.Text.Json": "10.0.4"
         }
       },
       "MimeTypes": {

Google.GenAI.Tests/CommonTest.cs

@@ -302,5 +302,81 @@ public void GetValueByPath_ArrayCollect_PreservesValuesAcrossAllElementsAndIsClo
             Assert.AreNotSame(element0DataNode, result[0]);
             Assert.AreNotSame(element1DataNode, result[1]);
         }
+
+        [TestMethod]
+        public void ParseToJsonNode_WithJsonNodeInput_PreservesValueAndIsCloned()
+        {
+            string largeBase64 = new string('E', 100_000);
+            var sourceNode = JsonValue.Create(largeBase64);
+
+            var result = Common.ParseToJsonNode(sourceNode);
+
+            Assert.IsNotNull(result);
+            Assert.AreEqual(largeBase64, result.GetValue<string>());
+            Assert.AreNotSame(sourceNode, result);
+        }
+
+        [TestMethod]
+        public void ParseToJsonNode_WithJsonObjectInput_PreservesAllPropertiesAndIsCloned()
+        {
+            string largeBase64 = new string('F', 100_000);
+            var sourceNode = new JsonObject
+            {
+                ["data"] = largeBase64,
+                ["mimeType"] = "image/png"
+            };
+            var innerDataNode = sourceNode["data"];
+
+            var result = Common.ParseToJsonNode(sourceNode) as JsonObject;
+
+            Assert.IsNotNull(result);
+            Assert.AreEqual(largeBase64, result["data"].GetValue<string>());
+            Assert.AreEqual("image/png", result["mimeType"].GetValue<string>());
+            Assert.AreNotSame(sourceNode, result);
+            // Inner nodes must also be cloned, not the same references.
+            Assert.AreNotSame(innerDataNode, result["data"]);
+        }
+
+        [TestMethod]
+        public void ParseToJsonNode_WithNullInput_ReturnsNull()
+        {
+            var result = Common.ParseToJsonNode(null);
+
+            Assert.IsNull(result);
+        }
+
+        [TestMethod]
+        public void ParseToJsonNode_WithClrObjectInput_SerializesCorrectly()
+        {
+            var clrObject = new { mimeType = "image/png", width = 3840 };
+
+            var result = Common.ParseToJsonNode(clrObject) as JsonObject;
+
+            Assert.IsNotNull(result);
+            Assert.AreEqual("image/png", result["mimeType"].GetValue<string>());
+            Assert.AreEqual(3840, result["width"].GetValue<int>());
+        }
+
+        [TestMethod]
+        public void ParseToJsonNode_WithContentClrObject_SerializesCorrectly()
+        {
+            var content = new Types.Content
+            {
+                Role = "user",
+                Parts = new List<Types.Part>
+                {
+                    new Types.Part { Text = "Why is the sky blue?" }
+                }
+            };
+
+            var result = Common.ParseToJsonNode(content) as JsonObject;
+
+            Assert.IsNotNull(result);
+            Assert.AreEqual("user", result["role"].GetValue<string>());
+            var parts = result["parts"] as JsonArray;
+            Assert.IsNotNull(parts);
+            Assert.AreEqual(1, parts.Count);
+            Assert.AreEqual("Why is the sky blue?", parts[0]["text"].GetValue<string>());
+        }
     }
 }