Skip to content

Conversation

@ronyblum
Copy link
Contributor

@ronyblum ronyblum commented Apr 10, 2025

Context

This PR updates all references of "AWS Bedrock" to "Amazon Bedrock" throughout the codebase to align with Amazon's official product naming. The change is due to official Amazon name is Amazon Bedrock.

The changes include:

  • Updated UI labels in the settings panel
  • Modified comments and error messages in the Bedrock provider implementation
  • Updated documentation files
  • Changed localization strings in all supported languages (en, es, de, pt-BR, hi, fr, ko, it, zh-CN, zh-TW, ca, pl, tr, vi, ja)
  • Renamed AWS_BEDROCK_REGION_INFO constant to AMAZON_BEDROCK_REGION_INFO and updated all references

Screenshots

Screenshot 2025-04-10 at 12 22 15 PM

Screenshot 2025-04-10 at 12 22 27 PM

How to Test

Open the settings panel and verify that "Amazon Bedrock" appears correctly in the provider dropdown
If you have Amazon Bedrock credentials configured:
Test that the provider still works correctly
Verify error messages display "Amazon Bedrock" instead of "AWS Bedrock"
Check that the localized versions of the settings panel show the updated terminology
Run the test suite to ensure all functionality works as expected: npm run test

Breaking changes

The constant AWS_BEDROCK_REGION_INFO has been renamed to AMAZON_BEDROCK_REGION_INFO. Any external code that directly references this constant will need to be updated.
No other breaking changes are expected as this is primarily a terminology update.


Important

Renamed 'AWS Bedrock' to 'Amazon Bedrock' across the codebase, including UI, localization, and code references.

  • Terminology Update:
    • Renamed all instances of "AWS Bedrock" to "Amazon Bedrock" in UI labels, comments, and error messages across multiple files.
    • Updated localization strings in settings.json files for all supported languages (e.g., en, es, de, pt-BR, hi, fr, ko, it, zh-CN, zh-TW, ca, pl, tr, vi, ja).
  • Code Changes:
    • Renamed AWS_BEDROCK_REGION_INFO to AMAZON_BEDROCK_REGION_INFO in aws_regions.ts and updated references in bedrock.ts and validate.ts.
  • Validation:
    • Updated validateBedrockArn() in validate.ts to reflect the new naming convention.

This description was created by Ellipsis for b7c258c. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Apr 10, 2025

⚠️ No Changeset found

Latest commit: 0b7a292

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. documentation Improvements or additions to documentation labels Apr 10, 2025
Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

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

Thank you!

@mrubens mrubens merged commit a031b74 into RooCodeInc:main Apr 10, 2025
12 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Apr 10, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants