refactor(litellm): use transform_openai_content_part directly

LiteLLM uses OpenAI's message format, so use the OpenAI-specific
transform_openai_content_part function directly for better performance.

Also update test to reflect correct behavior: data URIs without base64
encoding are still inline data and should be treated as blobs, not URIs.

Author: constantinius

sentry_sdk/integrations/litellm.py

@@ -8,7 +8,7 @@
     get_start_span_function,
     set_data_normalized,
     truncate_and_annotate_messages,
-    transform_message_content,
+    transform_openai_content_part,
 )
 from sentry_sdk.consts import SPANDATA
 from sentry_sdk.integrations import DidNotEnable, Integration
@@ -41,7 +41,7 @@ def _get_metadata_dict(kwargs: "Dict[str, Any]") -> "Dict[str, Any]":
 def _convert_message_parts(messages: "List[Dict[str, Any]]") -> "List[Dict[str, Any]]":
     """
     Convert the message parts from OpenAI format to the `gen_ai.request.messages` format
-    using the shared transform_message_content function.
+    using the OpenAI-specific transformer (LiteLLM uses OpenAI's message format).
 
     Deep copies messages to avoid mutating original kwargs.
     """
@@ -51,8 +51,17 @@ def _convert_message_parts(messages: "List[Dict[str, Any]]") -> "List[Dict[str,
     for message in messages:
         if not isinstance(message, dict):
             continue
-        if "content" in message:
-            message["content"] = transform_message_content(message["content"])
+        content = message.get("content")
+        if isinstance(content, (list, tuple)):
+            transformed = []
+            for item in content:
+                if isinstance(item, dict):
+                    result = transform_openai_content_part(item)
+                    # If transformation succeeded, use the result; otherwise keep original
+                    transformed.append(result if result is not None else item)
+                else:
+                    transformed.append(item)
+            message["content"] = transformed
     return messages
 
 

tests/integrations/litellm/test_litellm.py

@@ -939,7 +939,7 @@ def test_convert_message_parts_does_not_mutate_original():
 
 
 def test_convert_message_parts_data_url_without_base64():
-    """Data URLs without ;base64, marker should be treated as regular URIs."""
+    """Data URLs without ;base64, marker are still inline data and should be blobs."""
     messages = [
         {
             "role": "user",
@@ -952,10 +952,12 @@ def test_convert_message_parts_data_url_without_base64():
         }
     ]
     converted = _convert_message_parts(messages)
-    uri_item = converted[0]["content"][0]
-    # Should be converted to uri type, not blob (since no base64 encoding)
-    assert uri_item["type"] == "uri"
-    assert uri_item["uri"] == "data:image/png,rawdata"
+    blob_item = converted[0]["content"][0]
+    # Data URIs (with or without base64 encoding) contain inline data and should be blobs
+    assert blob_item["type"] == "blob"
+    assert blob_item["modality"] == "image"
+    assert blob_item["mime_type"] == "image/png"
+    assert blob_item["content"] == "rawdata"
 
 
 def test_convert_message_parts_image_url_none():