8000 Add Claude Sonnet 4 and Claude Opus 4 models with thinking variants by shariqriazz · Pull Request #3844 · RooCodeInc/Roo-Code · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add Claude Sonnet 4 and Claude Opus 4 models with thinking variants #3844

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 22, 2025

Conversation

shariqriazz
Copy link
@shariqriazz shariqriazz commented May 22, 2025

adds claude 4 sonnet / opus with thinking variant to anthropic, vertex, bedrock


Important

Add Claude Sonnet 4 and Claude Opus 4 models with thinking variants to the system, updating anthropic.ts and api.ts.

  • Models Added:
    • Added claude-sonnet-4-20250514 and claude-opus-4-20250514 models with thinking variants to anthropicModels, bedrockModels, and vertexModels in api.ts.
  • Behavior:
    • Updated AnthropicHandler in anthropic.ts to handle new models and their thinking variants.
    • Added support for thinking-128k variants in createMessage() and getModel() methods.
  • Misc:
    • Updated PROMPT_CACHING_MODELS and COMPUTER_USE_MODELS sets in api.ts to include new models.

This description was created by Ellipsis for 83beb49. You can customize this summary. It will automatically update as commits are pushed.

@@ -201,8 +201,8 @@ export const parseOpenRouterModel = ({
// and we want to use 128k.
if (id.startsWith("anthropic/claude-3.7-sonnet")) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this isn't quite right

Copy link
Author

Choose a reason for hiding this comment

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

sorry i am too sleep deprived idk how i managed to even close this lol thanks @cte

@cte cte reopened this May 22, 2025
@github-project-automation github-project-automation bot moved this from Done to New in Roo Code Roadmap May 22, 2025
@github-project-automation github-project-automation bot moved this from Done to New in Roo Code Roadmap May 22, 2025
@cte
Copy link
Collaborator
cte commented May 22, 2025

@shariqriazz - Mind if we use this a the official Claude 4 PR?

@shariqriazz
Copy link
Author

@shariqriazz - Mind if we use this a the official Claude 4 PR?

not at all we good :)

@cte cte merged commit 8a8fadd into RooCodeInc:main May 22, 2025
12 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 22, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 22, 2025
@lupuletic
Copy link
lupuletic commented May 22, 2025

Hey! I think the vertex models are missing the @

I just tested it and it doesn't work - @mrubens @shariqriazz
image
image

Update: tried locally with @ and it works now with:

  • claude-sonnet-4@20250514:thinking
  • claude-opus-4@20250514:thinking
image

@mrubens
Copy link
Collaborator
mrubens commented May 22, 2025

Thank you! #3849

886B

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants
0