Skip to content

chore: bump bedrock models to sonnet-4.5#3859

Merged
joanagmaia merged 2 commits intomainfrom
chore/bump-bedrock-models
Feb 25, 2026
Merged

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

Conversation

@joanagmaia
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

@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.

Comment thread services/apps/git_integration/src/crowdgit/services/maintainer/bedrock.py Outdated
@github-actions
Copy link
Copy Markdown
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.

@joanagmaia joanagmaia requested a review from skwowet February 24, 2026 14:54
@joanagmaia joanagmaia merged commit 520bb4f into main Feb 25, 2026
15 checks passed
@joanagmaia joanagmaia deleted the chore/bump-bedrock-models branch February 25, 2026 11:06
joanagmaia added a commit that referenced this pull request Feb 25, 2026
@joanagmaia joanagmaia restored the chore/bump-bedrock-models branch February 25, 2026 11:26
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