fix: use max_completion_tokens for reasoning models (o1/o3/o4/gpt-5) #544

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

di-sukharev merged 4 commits into di-sukharev:master from majiayu000:fix/issue-529-max-completion-tokens

Mar 31, 2026

+39 −5

Merged

fix: use max_completion_tokens for reasoning models (o1/o3/o4/gpt-5) #544

fix: resolve CI failures — revert gemini test mock path and fix prett…

GitHub Advanced Security / CodeQL succeeded Mar 29, 2026 in 2s

No new alerts in code changed by this pull request

View all branch alerts.