Repository navigation
fix: reset stale reasoning effort, clarify docs - #3030
Conversation
Reset a stale reasoning effort selection also when the model doesn't declare its supported efforts, reusing the same fallback options the selector renders. Clarify in API descriptions and docs that tiers are never downgraded but budget-based providers translate them natively. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FPnpiisz8mM8nRGyVpWmUh
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
WalkthroughReasoning-effort documentation now describes non-downgraded provider handling. The playground centralizes fallback effort options and resets unsupported selections using catalog or model-based fallback options. ChangesReasoning effort updates
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FPnpiisz8mM8nRGyVpWmUh
Summary
Follow-up to #3028, addressing the CodeRabbit review comments that couldn't land there because the branch was locked in the merge queue.
maxon gpt-5.6-sol) survived switching to a model with no declaredreasoningEffortsin the catalog, leaving the selector with no matching item while still sending the unsupportedreasoning_effortupstream — which now fails with a provider 4xx since feat: per-mapping reasoning efforts, forward as-is #3028 removed the downgrade. Both reset effects (main chat and comparison panel) now validate against the same effective options the selector renders, via a sharedgetFallbackReasoningEffortOptionshelper that also replaces the inline fallback in the dropdown.reasoning_effortdescriptions in the chat completions schema, the Anthropic endpoint comment, and the reasoning docs page.Testing
turbo run build --filter=playground --filter=gatewaypasses.🤖 Generated with Claude Code
https://claude.ai/code/session_01FPnpiisz8mM8nRGyVpWmUh
Generated by Claude Code
Summary by CodeRabbit
New Features
Bug Fixes
Documentation