[Bugfix] Fix Unicode issues in GLM-4 tool calling

[chaunceyjiang]

Purpose

Test Plan

main

[ChatCompletionMessageFunctionToolCall(id='chatcmpl-tool-a206a0a9cae7c285', function=Function(arguments='{"city": "\\u5317\\u4eac", "date": "2024-06-27"}', name='get_weather'), type='function'), ChatCompletionMessageFunctionToolCall(id='chatcmpl-tool-9b1f3dc7e2f41093', function=Function(arguments='{"city": "\\u4e0a\\u6d77", "date": "2024-06-27"}', name='get_weather'), type='function')]

this pr

[ChatCompletionMessageFunctionToolCall(id='chatcmpl-tool-a206a0a9cae7c285', function=Function(arguments='{"city": "北京", "date": "2024-06-27"}', name='get_weather'), type='function'), ChatCompletionMessageFunctionToolCall(id='chatcmpl-tool-9b1f3dc7e2f41093', function=Function(arguments='{"city": "上海", "date": "2024-06-27"}', name='get_weather'), type='function')]

Test Result

---

Essential Elements of an Effective PR Description Checklist

• The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
• The test plan, such as providing test command.
• The test results, such as pasting the results comparison before and after, or e2e results
• (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
• (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

[gemini-code-assist]

Code Review

This pull request correctly fixes a bug in the GLM-4 tool parser where Unicode characters in tool call arguments were not being handled correctly by adding ensure_ascii=False to the json.dumps call. This prevents non-ASCII characters from being escaped. I have added one comment to further improve the robustness of the JSON serialization to prevent potential silent failures when encountering non-serializable data types.