fix: add Bedrock models publishers/anthropic/models/claude-opus-4-8 +3 more#676
fix: add Bedrock models publishers/anthropic/models/claude-opus-4-8 +3 more#676github-actions[bot] wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2aa1c8a2f0
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "Codex (@codex) review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "Codex (@codex) address that feedback".
| "locations": [ | ||
| "global" | ||
| ], |
There was a problem hiding this comment.
Respect configured Vertex locations
For Vertex Anthropic requests, resolveVertexLocation gives modelSpec.locations precedence over the secret metadata location, so adding locations: ["global"] here forces every Claude Opus 4.8 Vertex call to /locations/global/. In environments configured with a regional or multi-region Vertex location such as us-central1, us, or eu for residency/availability, this new model silently ignores that configuration, unlike the existing Anthropic Vertex models and the existing test coverage that expects metadata locations to be honored when model locations are absent.
Useful? React with 👍 / 👎.
fix: add Bedrock models publishers/anthropic/models/claude-opus-4-8 +3 more
Closes #671
Source issue: #671
Summary
publishers/anthropic/models/claude-opus-4-8anthropic.claude-opus-4-8global.anthropic.claude-opus-4-8us.anthropic.claude-opus-4-8publishers/anthropic/models/claude-opus-4-8anthropic.claude-opus-4-8global.anthropic.claude-opus-4-8us.anthropic.claude-opus-4-82
3
Verified metadata
Verification notes
Verification
Official source verification
anthropic.claude-opus-4-8for Bedrock,claude-opus-4-8for Vertex AI), context window (1M tokens), max output (128k tokens), multimodal (vision) support, adaptive thinking support, pricing ($5 input / $25 output per MTok).claude-opus-4-8, global/multi-region/regional endpoint types. Regional and multi-region carry 10% premium over global. Docs state "model availability varies by region" but do not list specific supported regions for Claude models.sync_models cross-check
sync_models reference:
https://raw.githubusercontent.com/BerriAI/litellm/refs/heads/main/litellm/model_prices_and_context_window_backup.jsonanthropic.claude-opus-4-8(sync_models provider: bedrock_converse):global.anthropic.claude-opus-4-8(sync_models provider: bedrock_converse):us.anthropic.claude-opus-4-8(sync_models provider: bedrock_converse):publishers/anthropic/models/claude-opus-4-8(Vertex):publishers/anthropic/models/claude-opus-4-7catalog entry.Deviations from sync_models
No deviations. All proposed numeric values match sync_models exactly for the three Bedrock variants. The Vertex variant has no sync_models entry to compare against.
Fields not published or not applicable
supported_regionsfor the Vertex model: Omitted. The Anthropic Vertex AI documentation states "model availability varies by region" but does not publish a specific region list for Claude models. The Google locations page referenced in the instructions covers Google-owned models (Gemini), not partner models (Anthropic). No official region list found; field omitted per instructions.deprecation/deprecation_date: Not applicable; Claude Opus 4.8 is the current latest Opus model with no announced deprecation.input_cache_write_5m_cost_per_mil_tokensandinput_cache_write_1h_cost_per_mil_tokensfor Bedrock variants: Omitted for Bedrock entries, matching the existingclaude-opus-4-7Bedrock pattern. Only the Vertex entry includes tiered cache pricing, matchingpublishers/anthropic/models/claude-opus-4-7.Resolver failure root causes addressed
[BOT ISSUE] Bedrock/Vertex: add missing claude-opus-4-8 provider variantsdoes not match any of the resolver's three title regex patterns (single-model, multi-model parenthesized, multi-model colon). The metadata block must supplyproviderandmodelssince the title cannot be parsed."provider": "bedrock,vertex"is rejected bynormalizeProvider()which expects a single provider string. Fixed to"bedrock"(majority provider; 3 of 4 models are Bedrock).model_specsormodel_spec, so the resolver could not determine format, pricing, or token limits. Added completemodel_specsmap.requiresExplicitLocations()returns true for models starting withpublishers/. Added"locations": ["global"]based on Anthropic Vertex documentation which shows global as the recommended and default endpoint.sync_models vs proposed update
sync_models cross-check found differences. Official provider verification was used for the applied values, and sync_models discrepancies are listed below for review.