fix: add ensure_ascii=False to json.dumps for correct Unicode output

KatHaruto · KatHaruto · commit 3b295e47d7f2 · 2025-05-02T17:27:46.000+09:00
diff --git a/src/agents/extensions/models/litellm_model.py b/src/agents/extensions/models/litellm_model.py
@@ -96,7 +96,11 @@ async def get_response(
                 logger.debug("Received model response")
             else:
                 logger.debug(
-                    f"LLM resp:\n{json.dumps(response.choices[0].message.model_dump(), indent=2)}\n"
+                    f"LLM resp:\n{json.dumps(
+                        response.choices[0].message.model_dump(),
+                        indent=2,
+                        ensure_ascii=False
+                    )}\n"
                 )
 
             if hasattr(response, "usage"):
@@ -251,8 +255,8 @@ async def _fetch_response(
         else:
             logger.debug(
                 f"Calling Litellm model: {self.model}\n"
-                f"{json.dumps(converted_messages, indent=2)}\n"
-                f"Tools:\n{json.dumps(converted_tools, indent=2)}\n"
+                f"{json.dumps(converted_messages, indent=2, ensure_ascii=False)}\n"
+                f"Tools:\n{json.dumps(converted_tools, indent=2, ensure_ascii=False)}\n"
                 f"Stream: {stream}\n"
                 f"Tool choice: {tool_choice}\n"
                 f"Response format: {response_format}\n"
diff --git a/src/agents/mcp/util.py b/src/agents/mcp/util.py
@@ -116,7 +116,10 @@ async def invoke_mcp_tool(
         if len(result.content) == 1:
             tool_output = result.content[0].model_dump_json()
         elif len(result.content) > 1:
-            tool_output = json.dumps([item.model_dump() for item in result.content])
+            tool_output = json.dumps(
+                [item.model_dump() for item in result.content],
+                ensure_ascii=False
+            )
         else:
             logger.error(f"Errored MCP tool result: {result}")
             tool_output = "Error running tool."
diff --git a/src/agents/models/chatcmpl_converter.py b/src/agents/models/chatcmpl_converter.py
@@ -393,7 +393,8 @@ def ensure_assistant_message() -> ChatCompletionAssistantMessageParam:
                             {
                                 "queries": file_search.get("queries", []),
                                 "status": file_search.get("status"),
-                            }
+                            },
+                            ensure_ascii=False
                         ),
                     },
                 )
diff --git a/src/agents/models/openai_chatcompletions.py b/src/agents/models/openai_chatcompletions.py
@@ -74,7 +74,12 @@ async def get_response(
                 logger.debug("Received model response")
             else:
                 logger.debug(
-                    f"LLM resp:\n{json.dumps(response.choices[0].message.model_dump(), indent=2)}\n"
+                    "LLM resp:\n",
+                    f"{json.dumps(
+                        response.choices[0].message.model_dump(),
+                        indent=2,
+                        ensure_ascii=False
+                    )}\n"
                 )
 
             usage = (
@@ -222,8 +227,8 @@ async def _fetch_response(
             logger.debug("Calling LLM")
         else:
             logger.debug(
-                f"{json.dumps(converted_messages, indent=2)}\n"
-                f"Tools:\n{json.dumps(converted_tools, indent=2)}\n"
+                f"{json.dumps(converted_messages, indent=2, ensure_ascii=False)}\n"
+                f"Tools:\n{json.dumps(converted_tools, indent=2, ensure_ascii=False)}\n"
                 f"Stream: {stream}\n"
                 f"Tool choice: {tool_choice}\n"
                 f"Response format: {response_format}\n"
diff --git a/src/agents/models/openai_responses.py b/src/agents/models/openai_responses.py
@@ -89,7 +89,11 @@ async def get_response(
                 else:
                     logger.debug(
                         "LLM resp:\n"
-                        f"{json.dumps([x.model_dump() for x in response.output], indent=2)}\n"
+                        f"{json.dumps(
+                            [x.model_dump() for x in response.output],
+                            indent=2,
+                            ensure_ascii=False
+                        )}\n"
                     )
 
                 usage = (
@@ -231,8 +235,8 @@ async def _fetch_response(
         else:
             logger.debug(
                 f"Calling LLM {self.model} with input:\n"
-                f"{json.dumps(list_input, indent=2)}\n"
-                f"Tools:\n{json.dumps(converted_tools.tools, indent=2)}\n"
+                f"{json.dumps(list_input, indent=2, ensure_ascii=False)}\n"
+                f"Tools:\n{json.dumps(converted_tools.tools, indent=2, ensure_ascii=False)}\n"
                 f"Stream: {stream}\n"
                 f"Tool choice: {tool_choice}\n"
                 f"Response format: {response_format}\n"

Original file line number	Diff line number	Diff line change
`@@ -393,7 +393,8 @@ def ensure_assistant_message() -> ChatCompletionAssistantMessageParam:`
`393`	`393`	`{`
`394`	`394`	`"queries": file_search.get("queries", []),`
`395`	`395`	`"status": file_search.get("status"),`
`396`		`- }`
	`396`	`+ },`
	`397`	`+ ensure_ascii=False`
`397`	`398`	`),`
`398`	`399`	`},`
`399`	`400`	`)`