fix: use valid model ID for default LLM settings#876
Conversation
The default model was set to "sonnet" which is not a valid model manifest ID, causing 400 errors when sessions are created without an explicit model (e.g. from CI workflows via ambient-action). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughA one-line configuration change updates the default LLM model identifier in session creation from "sonnet" to "claude-sonnet-4-5". No functional logic or control flow modifications are introduced. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 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 |
🧪 CI InsightsHere's what we observed from your CI run for 376a419. 🟢 All jobs passed!But CI Insights is watching 👀 |
|
Test comment from Claude Code to verify GitHub token permissions. |
Summary
"sonnet"which isn't a valid model manifest ID"claude-sonnet-4-5"which matches the manifestambient-action)Test plan
claude-sonnet-4-5🤖 Generated with Claude Code