Skip to content

feat: introduce opus 4.7#682

Merged
viper151 merged 4 commits into
mainfrom
feature/upgrade-claude-agent-sdk
Apr 21, 2026
Merged

feat: introduce opus 4.7#682
viper151 merged 4 commits into
mainfrom
feature/upgrade-claude-agent-sdk

Conversation

@viper151
Copy link
Copy Markdown
Contributor

@viper151 viper151 commented Apr 21, 2026

  • 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

Summary by CodeRabbit

  • New Features

    • Added support for Claude Opus 4.6 model selection.
  • Chores

    • Updated Claude Agent SDK dependency to latest version.

- 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
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 259243a3-fc68-4387-915b-97d7a15579b7

📥 Commits

Reviewing files that changed from the base of the PR and between d0f37fd and b0d8ea4.

📒 Files selected for processing (1)
  • server/tsconfig.json

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Dependency Update
package.json
Updated @anthropic-ai/claude-agent-sdk from ^0.2.59 to ^0.2.116 to access newer SDK features.
SDK Integration
server/claude-sdk.js
Modified environment handling to always set sdkOptions.env as a shallow copy of process environment; replaced conditional CLAUDE_CLI_PATH assignment with unconditional fallback defaulting to process.env.CLAUDE_CLI_PATH || 'claude'.
Model Constants
shared/modelConstants.js
Added new model option { value: "claude-opus-4-6", label: "Opus 4.6" } to CLAUDE_MODELS.OPTIONS.
Build Configuration
server/tsconfig.json
Updated TypeScript baseUrl from . (server directory) to .. (project root) and adjusted @/\\* path alias mapping from \\* to server/\\* for consistent module resolution.

Possibly related PRs

  • PR #208: Directly modifies the same SDK integration code (server/claude-sdk.js) with environment and CLI path handling changes, plus bumps the matching dependency.
  • PR #446: Updates the same @anthropic-ai/claude-agent-sdk package dependency and touches server/claude-sdk.js for model-usage logging enhancements.

Poem

🐰 The rabbit hops with glee today,
New Opus bounds forth to play,
Paths align from root so true,
SDK climbs to version new,
Env flows smooth, executable guides the way!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title states 'introduce opus 4.7' but the actual change adds 'claude-opus-4-6', not opus 4.7. This is a mismatch between the stated title and the actual implementation. Update the title to accurately reflect the change, such as 'feat: introduce opus 4.6' or 'feat: add claude-opus-4-6 model option'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/upgrade-claude-agent-sdk

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
server/claude-sdk.js (1)

204-206: Stale comment after adding claude-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 added claude-opus-4-6 option. Consider updating it (or removing the enumeration and pointing to CLAUDE_MODELS.OPTIONS as 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 clau­de-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 clau­de-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 clau­de-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 clau­de-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

📥 Commits

Reviewing files that changed from the base of the PR and between 89b754d and be08149.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (3)
  • package.json
  • server/claude-sdk.js
  • shared/modelConstants.js

@viper151 viper151 force-pushed the feature/upgrade-claude-agent-sdk branch 2 times, most recently from d0f37fd to 2611a27 Compare April 21, 2026 16:21
@viper151 viper151 merged commit c5e55ad into main Apr 21, 2026
5 checks passed
ethanbarclay pushed a commit to ethanbarclay/conducctor that referenced this pull request Apr 25, 2026
* 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)
@viper151 viper151 deleted the feature/upgrade-claude-agent-sdk branch April 30, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant