fix(openai): handle pydantic BaseModel as metadata (#1449)

Author: hassiebp

langfuse/openai.py

@@ -400,7 +400,10 @@ def _get_langfuse_data_from_kwargs(resource: OpenAiDefinition, kwargs: Any) -> A
         and not isinstance(metadata, NotGiven)
         and not isinstance(metadata, dict)
     ):
-        raise TypeError("metadata must be a dictionary")
+        if isinstance(metadata, BaseModel):
+            metadata = metadata.model_dump()
+        else:
+            metadata = {}
 
     model = kwargs.get("model", None) or None
 

tests/test_openai.py

@@ -607,16 +607,6 @@ def test_fails_wrong_name(openai):
         )
 
 
-def test_fails_wrong_metadata(openai):
-    with pytest.raises(TypeError, match="metadata must be a dictionary"):
-        openai.OpenAI().completions.create(
-            metadata="metadata",
-            model="gpt-3.5-turbo-instruct",
-            prompt="1 + 1 = ",
-            temperature=0,
-        )
-
-
 def test_fails_wrong_trace_id(openai):
     with pytest.raises(TypeError, match="trace_id must be a string"):
         openai.OpenAI().completions.create(