allow content to have empty text, which will not serialize it

dongri · Jul 17, 2024 · 55b4b78 · 55b4b78
1 parent 9b8aecb
commit 55b4b78
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/v1/chat_completion.rs b/src/v1/chat_completion.rs
@@ -115,7 +115,13 @@ impl serde::Serialize for Content {
         S: serde::Serializer,
     {
         match *self {
-            Content::Text(ref text) => serializer.serialize_str(text),
+            Content::Text(ref text) => {
+                if text.is_empty() {
+                    serializer.serialize_none()
+                } else {
+                    serializer.serialize_str(text)
+                }
+            }
             Content::ImageUrl(ref image_url) => image_url.serialize(serializer),
         }
     }