feat. Unset pydantic_data_converter if OpenAI Agents plugin being used

Author: danielmillerp

src/agentex/lib/core/clients/temporal/utils.py

@@ -74,20 +74,24 @@ async def get_temporal_client(temporal_address: str, metrics_url: str | None = N
     if plugins:
         validate_client_plugins(plugins)
 
+    # Check if OpenAI plugin is present - it needs to configure its own data converter
+    has_openai_plugin = any(
+        type(p).__name__ == 'OpenAIAgentsPlugin' for p in (plugins or [])
+    )
+
+    # Only set data_converter if OpenAI plugin is not present
+    connect_kwargs = {
+        "target_host": temporal_address,
+        "plugins": plugins,
+    }
+
+    if not has_openai_plugin:
+        connect_kwargs["data_converter"] = pydantic_data_converter
+
     if not metrics_url:
-        client = await Client.connect(
-            target_host=temporal_address,
-            # data_converter=custom_data_converter,
-            data_converter=pydantic_data_converter,
-            plugins=plugins,
-        )
+        client = await Client.connect(**connect_kwargs)
     else:
         runtime = Runtime(telemetry=TelemetryConfig(metrics=OpenTelemetryConfig(url=metrics_url)))
-        client = await Client.connect(
-            target_host=temporal_address,
-            # data_converter=custom_data_converter,
-            data_converter=pydantic_data_converter,
-            runtime=runtime,
-            plugins=plugins,
-        )
+        connect_kwargs["runtime"] = runtime
+        client = await Client.connect(**connect_kwargs)
     return client