Skip to content

Comments

chore: bump bedrock models to sonnet-4.5#3859

Open
joanagmaia wants to merge 2 commits intomainfrom
chore/bump-bedrock-models
Open

chore: bump bedrock models to sonnet-4.5#3859
joanagmaia wants to merge 2 commits intomainfrom
chore/bump-bedrock-models

Conversation

@joanagmaia
Copy link
Contributor

@joanagmaia joanagmaia commented Feb 18, 2026

Note

Medium Risk
Changes the underlying LLM model and Bedrock region for multiple production workflows, which can alter output quality/behavior and has potential availability/latency implications, but is configuration-scoped with minimal logic changes.

Overview
Upgrades Amazon Bedrock model usage to Claude Sonnet 4.5 (us.anthropic.claude-sonnet-4-5-20250929-v1:0) across the codebase, including crowdgit’s invoke_bedrock and the Temporal merge-suggestion workflows.

Introduces LlmModelType.CLAUDE_SONNET_4_5 and wires it into shared LLM_SETTINGS, plus adds region/pricing mappings; merge workflows also switch from hardcoded model IDs to the enum and move execution to us-west-2.

Written by Cursor Bugbot for commit 15af6b2. This will update automatically on new commits. Configure here.

@joanagmaia joanagmaia requested a review from skwowet February 18, 2026 17:22
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

2 similar comments
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copy link
Member

@skwowet skwowet left a comment

Choose a reason for hiding this comment

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

Just left a comment about the region prefix, but LGTM overall.


try:
modelId = "us.anthropic.claude-sonnet-4-20250514-v1:0"
modelId = "anthropic.claude-sonnet-4-5-20250929-v1:0"
Copy link
Member

Choose a reason for hiding this comment

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

Don’t we need the us region prefix here?

Maybe we can test it with curl to confirm.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@skwowet updated. Also ran the curl and returned with a valid response

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

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