Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Nov 27, 2025

Important

Adds native tool calling support to various Z.AI models in zai.ts.

  • Feature Addition:
    • Adds supportsNativeTools: true to internationalZAiModels and mainlandZAiModels in zai.ts for models glm-4.5, glm-4.5-air, glm-4.5-x, glm-4.5-airx, glm-4.5-flash, glm-4.5v, glm-4.6, and glm-4-32b-0414-128k.

This description was created by Ellipsis for 0479747. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners November 27, 2025 04:30
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. Enhancement New feature or request labels Nov 27, 2025
@roomote
Copy link
Contributor

roomote bot commented Nov 27, 2025

Rooviewer Clock   See task on Roo Cloud

Review complete. No issues found - the changes correctly enable native tool calling for Z.AI models by setting the supportsNativeTools flag, which integrates with the existing OpenAI-compatible infrastructure.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Nov 27, 2025
@mrubens mrubens merged commit fb94eb3 into main Nov 27, 2025
21 checks passed
@mrubens mrubens deleted the zai_ntc branch November 27, 2025 05:08
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Nov 27, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants