Repository navigation
Add explicit reasoning_effort on account OpenAI, Claude and TrueFoundry Managed models - #5646
Merged
Merged
Conversation
Contributor
SDK live feature checksResult: failed (11/107 scenarios) Informational — does not block merge.
Failures
|
bhaveshpatel640
marked this pull request as ready for review
October 21, 2026 14:30
thesujai
approved these changes
Oct 21, 2026
thesujai
enabled auto-merge (squash)
October 22, 2026 04:43
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.
Add explicit reasoning_effort on account OpenAI, Claude and TrueFoundry Managed models
none.none,low,medium,high,xhighnonemedium.none,low,medium,high,xhighmediummedium.none,low,medium,high,xhigh,maxmedium.none,low,medium,high,xhigh,maxmedium.none,low,medium,high,xhigh,maxmedium.low,medium,high,xhigh,maxnonereturns HTTP 400. The page does not name a default. The reasoning guide statesmediumfor the other GPT-6 modelsmedium.none,low,medium,high,xhigh,maxmediummedium.low,medium,high,xhigh,maxnoneandminimalare rejectedhigh.low,medium,high,xhigh,maxhighhigh.low,medium,high,maxxhighis not available on Opus 4.6high.low,medium,high,xhigh,maxhighhigh.low,medium,high,xhigh,maxhighmedium.low,medium,high,xhigh,maxmediumhigh.low,medium,high,maxxhighis not available on Sonnet 4.6high.low,medium,high,xhigh,maxhighhigh.low,medium,high,xhigh,maxhighmedium.low,medium,high,xhigh,maxhigh.low,medium,high,xhigh,maxhigh.none,low,high,maxhigh.minimalmaps tolow.mediumandxhighmap tohighhigh.none,low,high,maxhigh.none,low,high,maxhigh,low,high,maxfor this snapshotmax.low,high,maxmax.low,high,maxmedium.low,medium,high,xhigh,maxmedium.none,low,medium,high,xhigh,maxmedium.low,medium,high,xhigh,maxmedium.low,medium,highhigh.low,medium,high,xhighreasoning_effortis not supported. Thinking is always onmax.low,high,maxmedium.none,minimal,low,medium,high,xhigh,maxultra, which the catalog enum does not allow. Onlynoneturns thinking off. Every other value enables thinking at the same depth. Xiaomi does not namemediumas the defaultultra. Default is oursreasoning_effort. Depth tuning is only on M3.1-Flash (low,medium,high,xhigh,max, defaultmax)high.none,medium,highxhigh.low,medium,xhighnoneand mapshighandmaxtoxhighNote
Low Risk
Catalog-only YAML updates to inference parameters; no application runtime, auth, or billing logic changes.
Overview
This PR aligns model catalog YAML with provider-specific
reasoning_effortbehavior so callers get explicit defaults, allowed enums, and models that ignore the knob no longer expose it.Anthropic (
claude-fable-5,claude-opus-4-8,claude-opus-5,claude-sonnet-5): addsreasoning_effortdefaulthighwithlow/medium/high/xhigh/max.TrueFoundry Managed: adds or tightens
reasoning_efforton DeepSeek v4 variants (none,low,high,max, defaulthigh), Kimi K3 (low,high,max, defaultmax), MiMo v2.6 flash/pro (full ladder includingnone/minimal, defaultmedium), Nemotron 3 Ultra, and Qwen3.8-Max;supportedValuesongpt-oss-120b(low,medium,high).removeParams: reasoning_efforton Kimi K2.7 Code and MiniMax M3 where the upstream API does not honor it.These definitions feed catalogue-driven validation and SDK live-test reasoning scenarios.
Reviewed by Cursor Bugbot for commit 19fefd4. Bugbot is set up for automated code reviews on this repo. Configure here.