feat: introduce opus 4.7#682
Conversation
- Bump claude-agent-sdk from 0.2.59 to 0.2.116 - Forward process.env to SDK subprocess so ANTHROPIC_BASE_URL and other env vars work - Add claude-opus-4-6 as a distinct mode
|
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 (1)
📝 WalkthroughWalkthroughThis PR bumps the Claude agent SDK dependency to version 0.2.116, modifies environment and executable path handling in the SDK integration layer, adds a new Claude Opus 4.6 model option, and adjusts TypeScript module path resolution configuration from a server-local to project-root baseline. Changes
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
server/claude-sdk.js (1)
204-206: Stale comment after addingclaude-opus-4-6.The "Valid models" comment still lists only the alias set (
sonnet, opus, haiku, opusplan, sonnet[1m]) and doesn't reflect the newly addedclaude-opus-4-6option. Consider updating it (or removing the enumeration and pointing toCLAUDE_MODELS.OPTIONSas the source of truth) so it doesn't drift further.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@server/claude-sdk.js` around lines 204 - 206, Update the stale "Valid models" comment near sdkOptions.model to reflect the current source of truth instead of an out-of-date enum; either enumerate the new claude-opus-4-6 option or (preferably) replace the hard-coded list with a short comment pointing to CLAUDE_MODELS.OPTIONS / CLAUDE_MODELS.DEFAULT so future changes (e.g., additions like claude-opus-4-6) are maintained in one place; locate the comment adjacent to sdkOptions.model and CLAUDE_MODELS and modify it accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@server/claude-sdk.js`:
- Around line 204-206: Update the stale "Valid models" comment near
sdkOptions.model to reflect the current source of truth instead of an
out-of-date enum; either enumerate the new claude-opus-4-6 option or
(preferably) replace the hard-coded list with a short comment pointing to
CLAUDE_MODELS.OPTIONS / CLAUDE_MODELS.DEFAULT so future changes (e.g., additions
like claude-opus-4-6) are maintained in one place; locate the comment adjacent
to sdkOptions.model and CLAUDE_MODELS and modify it accordingly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 57dcd84a-f189-4658-9d77-f5203931fc9d
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (3)
package.jsonserver/claude-sdk.jsshared/modelConstants.js
d0f37fd to
2611a27
Compare
* feat: introduce opus 4.7 - Bump claude-agent-sdk from 0.2.59 to 0.2.116 - Forward process.env to SDK subprocess so ANTHROPIC_BASE_URL and other env vars work - Add claude-opus-4-6 as a distinct mode * feat: add "claude" as fallback in the cli path * fix: base url config (cherry picked from commit c5e55ad)
Summary by CodeRabbit
New Features
Chores