Skip to content

Improve AWS credential loading and configuration handling in BedrockProvider#5627

Closed
fbalicchia wants to merge 121 commits intoblock:mainfrom
fbalicchia:addaws-profile
Closed

Improve AWS credential loading and configuration handling in BedrockProvider#5627
fbalicchia wants to merge 121 commits intoblock:mainfrom
fbalicchia:addaws-profile

Conversation

@fbalicchia
Copy link
Contributor

@fbalicchia fbalicchia commented Nov 7, 2025

This PR refactors the BedrockProvider::from_env implementation to make AWS configuration loading more robust and user-friendly, especially for environments using AWS SSO or multiple profiles.

⸻

Key Changes
• Replaced aws_config::load_from_env() with aws_config::defaults(BehaviorVersion::latest()) to support:
• AWS SSO logins (aws sso login --profile )
• Named profiles (AWS_PROFILE)
• Explicit region configuration (AWS_REGION)
• Added explicit handling for:
• AWS_PROFILE → sets the active AWS profile
• AWS_REGION → sets the AWS region dynamically

fbalicchia and others added 29 commits November 7, 2025 13:38
@fbalicchia
manage aws profile
4fa236b 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@ARYPROGRAMMER @fbalicchia
fix: corrected my name in data (block#5414)
875f962 
Signed-off-by: Arya Pratap Singh <notaryasingh@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@M-A-SAIADITHYAA @fbalicchia
feat(prompt): add audio summarizer prompt with ElevenLabs integration (…
e8276dc 
…block#5437)

Signed-off-by: saiadithyaa <saiadithyaa2306@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@the-matrixneo @fbalicchia
Goose context7 video (block#5426)
db931e1 
Signed-off-by: Dakshata Mishra <dakshatamishralakshya@gmail.com>
Signed-off-by: Dakshata <22-783@apsdhaulakuan.onmicrosoft.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@tlongwell-block @fbalicchia
Reduce LLMism comments (block#5440)
649117c 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@tlongwell-block @fbalicchia
Remove subagent instructions from system prompt (block#5441)
d2256ab 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Shahzebdevv @taniacryptid @fbalicchia
Docs quick tip slot4 (block#4948)
28af1fe 
Signed-off-by: Mohammad Shahzeb Alam <shahzebalam14feb@gmail.com>
Co-authored-by: taniandjerry <126204004+taniandjerry@users.noreply.github.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@DOsinga @fbalicchia
Agent extension api (block#5281)
33af5dd 
Co-authored-by: Douwe Osinga <douwe@squareup.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@jamadeo @fbalicchia
Configurable search paths (block#5356)
34e3a24 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Anudhyan @fbalicchia
docs: Anudhyan's PR for Hacktoberfest issue block#4722 (block#5442)
0f6aa2e 
Signed-off-by: Anudhyan <127120520+Anudhyan@users.noreply.github.com>
Signed-off-by: Anudhyan Datta <anudhyandatta2021@gmail.com>"
Signed-off-by: Anudhyan Datta <anudhyandatta2021@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Shreyanshsingh23 @fbalicchia
Blog post goose recipes (block#5438)
36b33e9 
Signed-off-by: Shreyansh Singh Gautam <shreyanshrewa@gmail.com>
Co-authored-by: taniandjerry <126204004+taniandjerry@users.noreply.github.com>

Thank you so much for contributing this Hacktoberfest, Sheryansh!

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexhancock @fbalicchia
fix(extensions/desktop): only show env vars for stdio mcps (block#5430)
518edb4 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexhancock @fbalicchia
fix: don't use user specific paths in mcp integration tests (block#5445)
b52d54e 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Lymah123 @fbalicchia
Doc (Blog): Building Effective Prompts and Workflows for Code Review …
b27f3b6 
…with goose (block#5446)

Signed-off-by: Lymah123 <fimihanodunola625@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@dianed-square @fbalicchia
docs: otlp observability (block#5436)
fc990e0 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@anthonydmays @fbalicchia
fix: shows the correct config file update path with cli configure (bl…
fcf8f70 
…ock#5195)

Signed-off-by: Anthony D. Mays <anthony@morganlatimer.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Mohammed-Thaha @fbalicchia
docs: add blog on Optimizing Goose Performance on Windows (block#5449)
79fc898 
Signed-off-by: Mohammed Thaha <mohammedthahacse@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@anthonydmays @fbalicchia
docs: clarifies config file locations and purposes (block#5210)
79f799c 
Signed-off-by: Anthony D. Mays <anthony@morganlatimer.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexhancock @fbalicchia
feat(mcp): support sampling in a scoped way (block#5367)
0e9225e 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@the-matrixneo @fbalicchia
Add CSV File Merger recipe (block#5455)
3f306d6 
Signed-off-by: Dakshata <22-783@apsdhaulakuan.onmicrosoft.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@the-matrixneo @fbalicchia
Add Goose with Neon MCP submission (block#5454)
a7d007f 
Signed-off-by: Dakshata <22-783@apsdhaulakuan.onmicrosoft.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Better-Boy @fbalicchia
technical debt tracker recipe (block#5451)
54db894 
Signed-off-by: Better-Boy <panaroma365@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Better-Boy @fbalicchia
openapi to locust load test generator recipe (block#5447)
bf0028a 
Signed-off-by: Better-Boy <panaroma365@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@dorien-koelemeijer @fbalicchia
Add OTLP logs layer (block#5386)
8cdd000 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@zanesq @fbalicchia
Don't disable extensions after they fail to activate in new chat sess…
61b88b1 
…ion (block#5464)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@zanesq @fbalicchia
rpm install in /opt/Goose to avoid conflicts with chrome-sandbox (blo…
dbf68ed 
…ck#5421)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@katzdave @fbalicchia
Stream token usage on every agent message (block#5342)
d21e070 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Better-Boy @fbalicchia
apify doc fix (block#5460)
10d7ae0 
Signed-off-by: Better-Boy <panaroma365@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Lymah123 @fbalicchia
Doc (Blog): Managing goose Configurations Across Multiple Projects (b…
96fc9fe 
…lock#5467)

Signed-off-by: Lymah123 <fimihanodunola625@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
danielcooper and others added 22 commits November 7, 2025 13:38
@danielcooper @fbalicchia
Add environment subsition for auth blocks (block#5439)
7b8ff13 
Signed-off-by: Daniel Cooper <daniel@14lines.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexeyre @fbalicchia
Add pending extension indicator to extension panel (block#5493)
5dd210b 
Signed-off-by: Alex Holder <alexeeyre@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@lifeizhou-ap @fbalicchia
fix: customised recipe to yaml string to avoid minininjia parsing err…
b1d399f 
…or (block#5494)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexhancock @ramazansancar @fbalicchia
feat(githubcopilot): add support for newer Copilot AI Models (block#5603
76fb0b3 
)

Signed-off-by: Ramazan Sancar <ramazansancar4545@gmail.com>
Co-authored-by: Ramazan Sancar <ramazansancar4545@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@DOsinga @fbalicchia
Remove jetbrains (block#5602)
ef786ad 
Co-authored-by: Douwe Osinga <douwe@squareup.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@michaelneale @fbalicchia
fix: tidy up claude cli handling (block#5594)
2c686c8 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@katzdave @fbalicchia
Manual compaction test and fix (block#5568)
16a5065 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexhancock @fbalicchia
fix: gemini flash -> pro for mcp smoke tests (block#5574)
ce7f555 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@DOsinga @fbalicchia
Avoid web double write (block#5601)
fc7bfcd 
Co-authored-by: Douwe Osinga <douwe@squareup.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@DOsinga @fbalicchia
Support platform tools through CLI (block#5570)
6a35f3a 
Co-authored-by: Douwe Osinga <douwe@squareup.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@Abhijay007 @alexhancock @fbalicchia
feat(ui): bring back quick launcher (block#5144)
c76cc3f 
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
Co-authored-by: Alex Hancock <alexhancock@block.xyz>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@DOsinga @fbalicchia
Change the other location too (block#5608)
e2e47f9 
Co-authored-by: Douwe Osinga <douwe@squareup.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@katzdave @fbalicchia
Token state not showing on load, or after message is finished. (block…
d1d5786 
…#5606)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@dianed-square @fbalicchia
docs: add access control section to Developer tutorial (block#5615)
4b98dd6 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@jamadeo @fbalicchia
Bump openapi in prepare-release (block#5611)
bdfe145 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@sheikhlimon @fbalicchia
fix: add standard context menu items to prevent empty right-click menu (
0cbb6a3 
block#5616)

Signed-off-by: sheikhlimon <sheikhlimon404@gmail.com>
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@alexhancock @fbalicchia
fix: move goosehints/AGENTS.md handling to goose, and out of develope…
3e2a146 
…r extension (block#5575)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@github-actions @fbalicchia
Release 1.13.0
693fd9e 
Auto-merged release PR after tag v1.13.0 was pushed

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@wpfleger96 @fbalicchia
Standardize CLI argument flags and update documentation (block#5516)
22af881 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@tlongwell-block @fbalicchia
Unify subrecipe and subagent execution through shared recipe pipeline (…
675c70d 
…block#5082)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@jamadeo @fbalicchia
differentiate debug/release in cache key (block#5613)
9efad15 
Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@taniacryptid @fbalicchia
docs: October 2025 Community All-Stars spotlight, Hacktoberfest editi…
307b1d7 
…on (block#5625)

Signed-off-by: fbalicchia <fbalicchia@gmail.com>
@fbalicchia fbalicchia requested a review from a team as a code owner November 7, 2025 12:38
DOsinga
DOsinga reviewed Nov 7, 2025
View reviewed changes
Copy link
Collaborator

@DOsinga DOsinga left a comment

Choose a reason for hiding this comment

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

thanks for looking into this. I think you need to pull in main or something to clean up the diff

@alexhancock
Copy link
Collaborator

alexhancock commented Nov 12, 2025

@fbalicchia Do you want to merge main or rebase and get this one ready to go in?

@fbalicchia
Copy link
Contributor Author

fbalicchia commented Nov 12, 2025

I'm sorry alexhancock i'll do soon

@fbalicchia fbalicchia closed this Nov 12, 2025
@fbalicchia
Copy link
Contributor Author

fbalicchia commented Nov 12, 2025

alexhancock Closing in favor of #5699

@fbalicchia fbalicchia deleted the addaws-profile branch November 21, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DOsinga DOsinga DOsinga left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Comments