Send the room's reasoning setting as OpenRouter's unified parameter, with an explicit off value - #6124
Draft
jurgenwerk wants to merge 1 commit into
Draft
Send the room's reasoning setting as OpenRouter's unified parameter, with an explicit off value#6124jurgenwerk wants to merge 1 commit into
jurgenwerk wants to merge 1 commit into
Conversation
…with an explicit off value
A Kimi K2.7 Code card set to medium effort changed nothing: the bot sent
the OpenAI-style `reasoning_effort` field, which that provider does not
list among its parameters, and the model kept thinking for minutes per
turn. The unified `reasoning` parameter is the one OpenRouter translates
for every provider, so the setting now goes out as `reasoning: { effort }`.
A card with no setting serializes as null, so null was never a usable
"off": every unset model carried it. The field gains 'none' as the
explicit off switch, sent as `reasoning: { enabled: false }`, for models
whose thinking is on or off with no levels in between. Null and undefined
both leave the parameter out, as before in effect.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Contributor
Host Test Results 1 files 1 suites 2h 7m 50s ⏱️ Results for commit 9662abb. For more details on these errors, see this check. Realm Server Test Results 1 files 223 suites 1h 19m 14s ⏱️ Results for commit 9662abb. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Kimi K2.7 Code thought for ten minutes before writing anything, and setting its ModelConfiguration card to medium effort changed nothing. The room carried the setting and the bot sent it, as the OpenAI-style
reasoning_effortfield. OpenRouter listsreasoningandinclude_reasoningamong the parameters that model accepts, notreasoning_effort, so the setting never reached the provider.reasoningparameter, which OpenRouter translates per provider: an effort level where the model grades its thinking, a budget where it budgets it, on or off where that is all it has.none, labelled "Off (no thinking)", as the explicit off switch, and the empty option is relabelled "Not specified". The bot acceptsnonealongside the levels.The boxel-catalog side, the labels and the Kimi card set to
none, is a separate PR there. A one-run smoke on Kimi after both land tells whether the provider honors the switch.🤖 Generated with Claude Code