Skip to content

feat(models): add Sonnet 5 via Bedrock - #2868

Merged
steebchen merged 2 commits into
theopenco:mainfrom
analogpvt:feat/sonnet-5-bedrock
Jun 30, 2026
Merged

steebchen merged 2 commits into
theopenco:mainfrom
analogpvt:feat/sonnet-5-bedrock

Conversation

@analogpvt

@analogpvt analogpvt commented Jun 30, 2026 •

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Jun 30, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

This PR adds an aws-bedrock provider configuration for claude-sonnet-5 and updates the related region and Bedrock 1-hour TTL test expectations.

Changes

Claude Sonnet 5 Bedrock Provider Support

Layer / File(s) Summary
Bedrock provider config and test updates
packages/models/src/models/anthropic.ts, packages/models/src/providers.spec.ts, apps/gateway/src/lib/anthropic-pricing.spec.ts
Adds an aws-bedrock provider entry for claude-sonnet-5 with pricing, capability flags, reasoning settings, and global/us regions; updates the region expectation test and the Bedrock 1-hour TTL allowlist for the new model prefix.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • theopenco/llmgateway#2547: Both PRs update Anthropic Bedrock region configuration and corresponding providers.spec.ts/getRegionIds expectations.
  • theopenco/llmgateway#2806: Both PRs add a Bedrock mapping for claude-sonnet-5 with non-global region support and update providers.spec.ts expected regions.
  • theopenco/llmgateway#2193: Both PRs touch Bedrock cache-write billing and 1-hour TTL support behavior used by the anthropic-pricing.spec.ts gating.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title clearly and concisely describes the main change: adding Claude Sonnet 5 support via AWS Bedrock.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@analogpvt
analogpvt force-pushed the feat/sonnet-5-bedrock branch from d1933b0 to 6775f39 Compare June 30, 2026 19:22
@analogpvt analogpvt changed the title feat(models): add Sonnet 5 Bedrock feat(models): switch Sonnet 5 to Bedrock Jun 30, 2026
@analogpvt
analogpvt force-pushed the feat/sonnet-5-bedrock branch 2 times, most recently from e065283 to 9caec35 Compare June 30, 2026 19:43
@analogpvt
analogpvt force-pushed the feat/sonnet-5-bedrock branch from 9caec35 to ef885d8 Compare June 30, 2026 20:06
@steebchen steebchen changed the title feat(models): switch Sonnet 5 to Bedrock feat(models): add Sonnet 5 via Bedrock Jun 30, 2026
@steebchen
steebchen merged commit a7faca7 into theopenco:main Jun 30, 2026
13 checks passed
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.

2 participants