-
Notifications
You must be signed in to change notification settings - Fork 2.6k
feat: add claude 4 models to vertex ai specs #3843
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
Conversation
src/shared/api.ts
Outdated
| thinking: true, | ||
| }, | ||
| "claude-opus-4@20250514": { | ||
| maxTokens: 32_000, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still want this many max tokens for the non-thinking version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated it here as well
src/shared/api.ts
Outdated
| thinking: true, | ||
| }, | ||
| "claude-sonnet-4@20250514": { | ||
| maxTokens: 64_000, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question here - maybe we should make this 8192 like 3.7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will do
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated it to 8192, you were right, that value is more reasonable
|
looks like a similar PR was merged, will close this |
|
Thank you for the PR! |


Related GitHub Issue
Closes: #3842
Description
Claude 4 models are released and this PR is adding them to the list of VertexAI models.
Blog Post by Anthropic: https://www.anthropic.com/news/claude-4
Test Procedure
Type of Change
srcor test files.Pre-Submission Checklist
npm run lint).console.log) has been removed.npm test).mainbranch.npm run changesetif this PR includes user-facing changes or dependency updates.Screenshots / Videos
Documentation Updates
Does this PR necessitate updates to user-facing documentation?
Additional Notes
Get in Touch
Important
Add Claude 4 models to Vertex AI specs in
src/shared/api.ts, includingclaude-sonnet-4andclaude-opus-4with detailed attributes and pricing.claude-sonnet-4@20250514:thinkingandclaude-sonnet-4@20250514tovertexModelsinsrc/shared/api.ts.claude-opus-4@20250514:thinkingandclaude-opus-4@20250514tovertexModelsinsrc/shared/api.ts.maxTokens,contextWindow,supportsImages,supportsComputerUse,supportsPromptCache.inputPrice,outputPrice,cacheWritesPrice,cacheReadsPrice.This description was created by
for b66dc19. You can customize this summary. It will automatically update as commits are pushed.