Skip to content

Comments

fix: correct Vertex AI claude-sonnet-4-6 model ID#11626

Merged
hannesrudolph merged 1 commit intomainfrom
fix/vertex-sonnet-4-6-model-id
Feb 20, 2026
Merged

fix: correct Vertex AI claude-sonnet-4-6 model ID#11626
hannesrudolph merged 1 commit intomainfrom
fix/vertex-sonnet-4-6-model-id

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Feb 20, 2026

Related GitHub Issue

Closes: #11625

Description

This PR attempts to address Issue #11625. The Vertex AI model ID claude-sonnet-4-6@20260114 includes a date version suffix (@20260114) that causes a 404 error on Google Vertex AI. The correct model ID should be claude-sonnet-4-6 (without the date suffix), consistent with how claude-opus-4-6 is already defined in the same file.

Changes:

  • packages/types/src/providers/vertex.ts: Renamed model key from claude-sonnet-4-6@20260114 to claude-sonnet-4-6 in the vertexModels object, and updated the reference in the VERTEX_1M_CONTEXT_MODEL_IDS array.
  • src/api/providers/__tests__/anthropic-vertex.spec.ts: Updated the test referencing the old model ID.

Feedback and guidance are welcome.

Test Procedure

  • Ran npx vitest run api/providers/__tests__/anthropic-vertex.spec.ts from the src/ directory: all 31 tests pass.
  • All lint checks pass via turbo lint.
  • All type checks pass via turbo check-types.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Documentation Impact: No documentation updates are required.
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Documentation Updates

  • No documentation updates are required.

Additional Notes

The claude-opus-4-6 entry in the same file already follows the correct pattern (no date suffix), so this change aligns claude-sonnet-4-6 with that convention.

Start a new Roo Code Cloud session on this branch

@yuvarajl
Copy link

@hannesrudolph approval ?

@hannesrudolph hannesrudolph marked this pull request as ready for review February 20, 2026 15:47
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Feb 20, 2026
Copy link
Collaborator

@hannesrudolph hannesrudolph left a comment

Choose a reason for hiding this comment

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

LGTM. Merging per Hannes' request.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 20, 2026
@hannesrudolph hannesrudolph merged commit 93415c7 into main Feb 20, 2026
23 checks passed
@hannesrudolph hannesrudolph deleted the fix/vertex-sonnet-4-6-model-id branch February 20, 2026 17:47
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Feb 20, 2026
@roomote roomote bot mentioned this pull request Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Vertex Sonnet 4.6 Model id is not right

3 participants