feat(providers): add AWS Bedrock provider by loafoe · Pull Request #1903 · sipeed/picoclaw

loafoe · 2026-03-22T21:12:20Z

📝 Description

Add support for AWS Bedrock as an LLM provider using the Converse API. The provider is behind a build tag to keep binary size small.

🗣️ Type of Change

🐞 Bug fix (non-breaking change which fixes an issue)
✨ New feature (non-breaking change which adds functionality)
📖 Documentation update
⚡ Code refactoring (no functional changes, no api changes)

🤖 AI Code Generation

🤖 Fully AI-generated (100% AI, 0% Human)
🛠️ Mostly AI-generated (AI draft, Human verified/modified)
👨‍💻 Mostly Human-written (Human lead, AI assisted or none)

🔗 Related Issue

Refs #283
Implements #252

📚 Technical Context (Skip for Docs)

Example config:

  {"model": "bedrock/us.anthropic.claude-opus-4-5-20251101-v1:0", "api_base": "us-east-1"}

🧪 Test Environment

Hardware: Macbook Pro M3, Raspberry Pi 5
OS: Ubuntu 26.04, MacOS Tahoe
Model/Provider: Bedrock / Claude Opus 4.5

☑️ Checklist

My code/docs follow the style of this project.
I have performed a self-review of my own changes.
I have updated the documentation accordingly.

CLAassistant · 2026-03-22T21:12:30Z

All committers have signed the CLA.

Copilot

Pull request overview

Adds AWS Bedrock as a new LLM provider (Converse API) behind a bedrock build tag, wiring it into the provider factory and documenting configuration.

Changes:

Add bedrock protocol handling in CreateProviderFromConfig (region via api_base, AWS SDK credentials chain).
Implement Bedrock provider (Converse API) plus a non-tag stub provider that errors with build-tag guidance.
Update docs and Go module dependencies for AWS SDK v2 Bedrock Runtime.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
pkg/providers/factory_provider.go	Adds `bedrock` protocol branch to provider factory (region/options + provider initialization).
pkg/providers/bedrock/provider_stub.go	Stub implementation for non-`bedrock` builds (returns build-tag error).
pkg/providers/bedrock/provider_bedrock.go	Real Bedrock provider implementation using AWS SDK v2 Converse API.
go.mod	Adds AWS SDK v2 dependencies needed for Bedrock Runtime client.
go.sum	Adds checksums for newly introduced AWS SDK modules.
README.md	Documents Bedrock provider, build tag requirement, and region configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

pkg/providers/factory_provider.go

pkg/providers/bedrock/provider_bedrock.go

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support - Image handling with proper base64 decoding - Build tag gating (-tags bedrock) - Comprehensive test coverage - Timeout protection for AWS credential resolution Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

pkg/providers/factory_provider.go

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support - Image handling with proper base64 decoding - Build tag gating (-tags bedrock) - Comprehensive test coverage - Timeout protection for AWS credential resolution - Uses shared common.AsInt/AsFloat helpers Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/factory_provider.go

pkg/providers/bedrock/provider_bedrock.go

pkg/providers/bedrock/provider_bedrock_test.go

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support - Image handling with proper base64 decoding and size limits - Build tag gating (-tags bedrock) - Comprehensive test coverage - Timeout protection for AWS credential resolution - Uses shared common.AsInt/AsFloat helpers - Region validation (rejects URLs in api_base) Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support with Function fallback - Image handling with proper base64 decoding and size limits - Build tag gating (-tags bedrock) - Comprehensive test coverage - Timeout protection for AWS credential resolution - Uses shared common.AsInt/AsFloat helpers - Region validation (rejects URLs in api_base) Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/factory_provider.go

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support with Function fallback - Image handling with proper base64 decoding and size limits - Build tag gating (-tags bedrock) - Comprehensive test coverage including factory tests - Timeout protection for AWS credential resolution - Uses shared common.AsInt/AsFloat helpers - Region validation (rejects URLs in api_base) Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support with Function fallback - Image handling with proper base64 decoding and size limits - Build tag gating (-tags bedrock) - Comprehensive test coverage including factory tests - Timeout protection for AWS credential resolution - Uses shared common.AsInt/AsFloat helpers - Proper endpoint configuration via api_base (URL or region name) - Only sets ToolConfig when valid tools are present - Consistent build-tag error messages in stub - Region validation on provider creation - Deterministic factory tests with dummy AWS env vars api_base supports both: - Region name: us-east-1 (sets both region and endpoint) - Full endpoint URL (requires AWS_REGION separately) Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/factory_provider.go

Add AWS Bedrock provider using the Converse API with: - Full tool/function calling support with Function fallback - Image handling with proper base64 decoding and size limits - Build tag gating (-tags bedrock) - Comprehensive test coverage including factory tests - Timeout protection for AWS credential resolution - Uses shared common.AsInt/AsFloat helpers - Proper endpoint configuration via api_base (URL or region name) - Only sets ToolConfig when valid tools are present - Consistent build-tag error messages in stub - Region validation on provider creation - Deterministic factory tests with dummy AWS env vars - AWS SDK endpoint resolution for all partitions (aws, aws-cn, aws-us-gov) api_base supports both: - Region name: us-east-1 (AWS SDK resolves endpoint for all partitions) - Full endpoint URL (for custom endpoints or testing) Addresses PR sipeed#1903 Copilot review feedback.

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

pkg/providers/factory_provider.go

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

pkg/providers/factory_provider_test.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/factory_provider.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

go.mod

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

pkg/providers/factory_provider_test.go

loafoe · 2026-03-23T13:44:05Z

@jmahotiedu @Leeaandrob @shresthapranesh with #213 landed added AWS Bedrock support. Tested with different models in different AWS regions, also went through several Copilot review iterations to align with existing code. Please consider reviewing / merging, thanks for this awesome project!

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pkg/providers/bedrock/provider_bedrock.go

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Add support for AWS Bedrock as an LLM provider using the Converse API. The implementation is behind a build tag (-tags bedrock) to keep the default binary size small. Features: - AWS SDK v2 with automatic credential chain (env vars, profiles, IAM roles) - Converse API for unified access to Claude, Llama, Mistral models - Tool/function calling support with proper document handling - Image support with base64 decoding and size limits - Request timeout configuration - Region validation and endpoint resolution for all AWS partitions Usage: go build -tags bedrock model: bedrock/us.anthropic.claude-sonnet-4-20250514-v1:0 api_base: us-east-1 (or full endpoint URL)

yinwm

LGTM!

Review Summary:

Clean implementation with proper build tag separation ( / )
AWS SDK v2 integration with automatic credential chain
Excellent test coverage (~95%) including edge cases
Correct handling of Bedrock-specific requirements (tool results merging)
All CI checks passed

Minor notes (non-blocking):

Binary size increases ~10MB when built with (expected for AWS SDK)
Consider returning error instead of silent skip for oversized images (future improvement)

Thanks for the contribution!

yinwm

LGTM!

Review Summary:

Clean implementation with proper build tag separation
AWS SDK v2 integration with automatic credential chain
Excellent test coverage (~95%) including edge cases
Correct handling of Bedrock-specific requirements (tool results merging)
All CI checks passed

Minor notes (non-blocking):

Binary size increases ~10MB when built with bedrock tag (expected for AWS SDK)
Consider returning error instead of silent skip for oversized images (future improvement)

Thanks for the contribution!

Orgmar · 2026-03-24T03:05:41Z

@loafoe Awesome work bringing AWS Bedrock support to PicoClaw! Using the Converse API and keeping it behind a build tag to avoid bloating the binary is a thoughtful design choice. Testing on both Mac and Raspberry Pi is a nice touch too. This has been requested since #283, so a lot of users will appreciate it.

We have a PicoClaw Dev Group on Discord where contributors connect and collaborate. If you'd like to join, send an email to support@sipeed.com with subject [Join PicoClaw Dev Group] loafoe and we'll send you the invite link!

Copilot AI review requested due to automatic review settings March 22, 2026 21:12

Copilot started reviewing on behalf of loafoe March 22, 2026 21:12 View session

Copilot AI reviewed Mar 22, 2026

View reviewed changes

sipeed-bot bot added type: enhancement New feature or request domain: provider go Pull requests that update go code labels Mar 22, 2026

loafoe force-pushed the main branch from 865c293 to 41a2989 Compare March 22, 2026 21:53

loafoe requested a review from Copilot March 22, 2026 22:06

Copilot started reviewing on behalf of loafoe March 22, 2026 22:07 View session

Copilot AI reviewed Mar 22, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Outdated Show resolved Hide resolved

pkg/providers/bedrock/provider_bedrock.go Outdated Show resolved Hide resolved

pkg/providers/factory_provider.go Show resolved Hide resolved

loafoe force-pushed the main branch from 41a2989 to f6fd37c Compare March 22, 2026 22:12

loafoe requested a review from Copilot March 22, 2026 22:15

Copilot started reviewing on behalf of loafoe March 22, 2026 22:15 View session

Copilot AI reviewed Mar 22, 2026

View reviewed changes

pkg/providers/factory_provider.go Outdated Show resolved Hide resolved

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

pkg/providers/bedrock/provider_bedrock_test.go Outdated Show resolved Hide resolved

loafoe force-pushed the main branch from f6fd37c to 759f77f Compare March 22, 2026 22:19

loafoe requested a review from Copilot March 22, 2026 22:20

Copilot started reviewing on behalf of loafoe March 22, 2026 22:20 View session

Copilot AI reviewed Mar 22, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

loafoe force-pushed the main branch from 759f77f to 45e49ef Compare March 22, 2026 22:25

loafoe requested a review from Copilot March 22, 2026 22:30

Copilot started reviewing on behalf of loafoe March 22, 2026 22:30 View session

Copilot AI reviewed Mar 22, 2026

View reviewed changes

pkg/providers/factory_provider.go Show resolved Hide resolved

github-actions bot mentioned this pull request Mar 23, 2026

🦞 OpenClaw 生态日报 2026-03-23 gsscsd/big_model_radar#81

Open

loafoe force-pushed the main branch from 45e49ef to 201a0c9 Compare March 23, 2026 05:32

Copilot AI reviewed Mar 23, 2026

View reviewed changes

loafoe force-pushed the main branch from 3af3b15 to c2be469 Compare March 23, 2026 10:38

loafoe requested a review from Copilot March 23, 2026 10:42

Copilot started reviewing on behalf of loafoe March 23, 2026 10:43 View session

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/factory_provider.go Outdated Show resolved Hide resolved

loafoe force-pushed the main branch from c2be469 to 5489dff Compare March 23, 2026 11:02

loafoe requested a review from Copilot March 23, 2026 11:05

Copilot started reviewing on behalf of loafoe March 23, 2026 11:06 View session

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Outdated Show resolved Hide resolved

pkg/providers/factory_provider.go Outdated Show resolved Hide resolved

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

loafoe requested a review from Copilot March 23, 2026 11:37

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

pkg/providers/factory_provider_test.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Outdated Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/factory_provider.go Outdated Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Outdated Show resolved Hide resolved

go.mod Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Outdated Show resolved Hide resolved

pkg/providers/factory_provider_test.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

pkg/providers/bedrock/provider_bedrock.go Show resolved Hide resolved

Copilot AI reviewed Mar 23, 2026

View reviewed changes

yinwm approved these changes Mar 23, 2026

View reviewed changes

github-actions bot mentioned this pull request Mar 24, 2026

🦞 OpenClaw 生态日报 2026-03-24 gsscsd/big_model_radar#87

Open

loafoe mentioned this pull request Mar 25, 2026

feat(bedrock): detect SSO token expiration and provide actionable error #1988

Open

10 tasks

Conversation

loafoe commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 Description

🗣️ Type of Change

🤖 AI Code Generation

🔗 Related Issue

📚 Technical Context (Skip for Docs)

🧪 Test Environment

☑️ Checklist

Uh oh!

CLAassistant commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

loafoe commented Mar 22, 2026 •

edited

Loading

CLAassistant commented Mar 22, 2026 •

edited

Loading