Skip to content

fix(tool-emulation): use gpt_native dialect for glm-5.2 - #204

Merged
dwgx merged 1 commit into
dwgx:masterfrom
lovivi:fix/glm-5.2-tool-dialect
Jun 28, 2026
Merged

fix(tool-emulation): use gpt_native dialect for glm-5.2#204
dwgx merged 1 commit into
dwgx:masterfrom
lovivi:fix/glm-5.2-tool-dialect

Conversation

@lovivi

@lovivi lovivi commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

glm-5.2 ignores the glm47 XML tool-call markup and answers in prose. Switch it and its variant SKU aliases to the gpt_native function_call JSON dialect, which is emitted reliably and parsed correctly.

glm-5.2 ignores the glm47 XML tool-call markup and answers in prose.

Switch it (and its variant SKU aliases) to the gpt_native function_call

JSON dialect, which is emitted reliably and parsed correctly.

Keep an env override (WINDSURFAPI_FORCE_TOOL_DIALECT) for tests.
@dwgx
dwgx merged commit 7ee509a into dwgx:master Jun 28, 2026
dwgx added a commit that referenced this pull request Jun 28, 2026
Add regression coverage for the dialect routing merged in #204: glm-5.2
and glm-5-2-* variants resolve to gpt_native while older GLM SKUs stay on
glm47, plus the WINDSURFAPI_FORCE_TOOL_DIALECT override (including that an
unrecognized value falls through to normal routing).
dwgx added a commit that referenced this pull request Jun 29, 2026
Backfill contributors who were merged but never added to the thanks
page, with weight tiers calibrated against existing entries:

- #64  baily-zhang (S)  — cascade transport canceled-session recovery
- #163 you922 (A)       — LS crash self-heal: exponential-backoff restart
- #204 lovivi (B+)      — glm-5.2 gpt_native tool dialect fix
- #90  smeinecke (B+)   — inline proxy on add-account form
- #88  smeinecke (B+)   — ALLOW_PRIVATE_PROXY_HOSTS opt-in for local proxy
- #89  smeinecke (B)    — login-flow i18n + error-code standardization

Synced src -> docs via sync:contributors. Also gitignore .workflow-results/
(local audit artifacts, not source). dashboard tests 17/17.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants