feat(core): Add Amazon Bedrock provider support #809
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
enhancement
priority/P1
|
@anwschne {
"authType": "bedrock",
"model": "qwen.qwen3-coder-30b-a3b-v1:0"
}This is not the same as your configuration example. |
|
Thanks @lucidfrontier45 ! I decided the refactor and remove the short names as it just adds confusion. PR updated |
Mingholy
reviewed
Oct 22, 2025
| }, | ||
| }, | ||
| }, | ||
| } as OpenAI; |
There was a problem hiding this comment.
Wow, glad we do not use any other api of the OpenAI client.
But this hints to me that maybe we should use a duck type that requires only the APIs we actually use.
Mingholy
requested changes
Oct 22, 2025
Changes applied, thanks @Mingholy ! |
anwschne
force-pushed
the
feat/bedrock-provider-support
branch
from
abb0b86 to
30fb0e9
Compare
Implements AWS Bedrock integration for Qwen3-Coder models using the Converse API. - Add BedrockOpenAICompatibleProvider with full OpenAI format conversion - Support AWS credential chain (profiles, env vars, instance roles) - Add comprehensive test coverage (72 tests) - Add docs/cli/bedrock.md documentation - Update model mappings to use actual Bedrock model IDs Fixes QwenLM#759
…k model IDs - Remove BEDROCK_MODEL_MAP and simplify getBedrockModelId() - Update default model to use full Bedrock ID: qwen.qwen3-coder-30b-a3b-v1:0 - Update documentation to list models by region (us-east-1, us-west-2) - Add 480B and 235B models available in us-west-2 - Update all tests to use full Bedrock model IDs - Add note that model shortcuts are not supported
Move all Bedrock-specific provider files into provider/bedrock subdirectory with a barrel export file for better code organization and modularity. - Create provider/bedrock directory structure - Move bedrock.ts → bedrock/provider.ts - Move bedrockTypes.ts → bedrock/types.ts - Move bedrockConverter.ts → bedrock/converter.ts - Move test files to bedrock/ subdirectory - Add barrel file (index.ts) for clean exports - Update all import paths throughout codebase - Apply Prettier formatting to documentation and tests All 2728 tests passing (2703 core + 25 VSCode) ESLint passing with 0 warnings
anwschne
force-pushed
the
feat/bedrock-provider-support
branch
from
30fb0e9 to
0fc07b9
Compare
|
Fixed conflicts from recent merges. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Implements AWS Bedrock integration for Qwen3-Coder models using the Converse API.
This PR adds native Amazon Bedrock authentication support, allowing users to leverage Qwen models through AWS infrastructure with standard AWS credential chains.
Changes
BedrockOpenAICompatibleProviderwith full OpenAI-to-Bedrock format conversionqwen.qwen3-coder-30b-a3b-v1:0(Qwen3-Coder-30B-A3B-Instruct)qwen.qwen3-32b-v1:0(Qwen3 32B dense model)docs/cli/bedrock.mdTest Plan
Configuration Example
{ "authType": "bedrock", "model": "qwen.qwen3-coder-30b-a3b-v1:0" }export AWS_PROFILE=my-profile
export AWS_REGION=us-east-1
Fixes #759