Repository navigation
fix(models): correct Grok effort tiers - #3669
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan includes up to 4 reviews per rolling hour; 1 remains after this review. WalkthroughThe xAI model metadata documents reasoning-effort aliases for ChangesxAI reasoning effort support
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This PR updates Grok reasoning-effort options and forwards the newly supported tiers; no actionable merge-blocking risk remains after normal checks and review. 🚥 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 |
Problem
xAI accepts compatibility aliases in addition to each model's distinct reasoning controls. The Grok 4.6 catalogue exposed
minimaleven though xAI normalizes it tolow, while Grok 4.5's intentionally narrower list was undocumented.Changes
low,medium, andhighcontrolslow,medium,high, andxhighcontrolsminimal,none, andmaxare not advertisedThe existing xAI adapter already forwards
reasoning_effortverbatim. Chat and Responses normalization already preserves reasoning text, summaries, reasoning-token usage, and cost details, so no adapter changes are needed.Verification
Live xAI Chat Completions and native Responses probes verified both deployments:
minimalnormalizes tolow; xAI documentsxhighas equivalent tohigh;noneandmaxrejectminimalnormalizes tolow;low,medium,high, andxhighremain distinct;noneandmaxrejectreasoning_contentand nested reasoning-token usageCommands:
TEST_MODELS=xai/grok-4-6 FULL_MODE=true pnpm test:e2e: 105 passed, 94 skippedpnpm formatpnpm buildSummary by CodeRabbit
minimalandxhighare aliases for existing reasoning tiers where applicable.grok-4-6to includelow,medium,high, andxhigh.