Repository navigation
fix(gemini): switch thinking default logic - #50841
Conversation
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
|
This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window. Feel free to open a new pull request that follows our guidelines. |
Skip the default thinkingLevel / OpenRouter reasoning effort when a model's variants are token-budget shaped, so a budget variant can never be merged with an effort. Also limit Gemma 4 variants to the minimal/high levels the Gemini API accepts.
Unlisted Gemini models are assumed to use efforts; only explicitly listed older models take the token path.
|
This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window. Feel free to open a new pull request that follows our guidelines. |
Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Type of change
What does this PR do?
Switch logic for Gemini thinking such that Gemini 3 is the default, and older models are exceptional.
How did you verify your code works?
Updated & existing unit tests all pass.
Checklist