Skip to content

Conversation

@chezsmithy
Copy link
Contributor

@chezsmithy chezsmithy commented Aug 28, 2025

Description

CLI: Add aws onboarding option.

AWS Bedrock onboarding is triggered when the environment variable CONTINUE_USE_BEDROCK=1 is detected.

This will assume AWS credentials are available via all the standard options once it uses the bedrock provider.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-general-review or @continue-detailed-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

image

Tests

Updated specs
Performed manual testing
Updated onboarding tests


Summary by cubic

Adds an AWS credentials path to CLI onboarding so users can start with Bedrock-backed Claude without pasting API keys. The flow can auto-create config.yaml using the standard AWS credential chain.

  • New Features
    • Added a third onboarding choice: “Use AWS credentials” (choice 3).
    • On selection, initialize using existing AWS creds and configure bedrock/anthropic.claude-3-5-sonnet-20241022-v2:0 (region us-east-1).
    • Updated onboarding spec and prompt validation to accept 1, 2, or 3 with clearer errors.

@chezsmithy chezsmithy requested a review from a team as a code owner August 28, 2025 05:07
@chezsmithy chezsmithy requested review from Patrick-Erichsen and removed request for a team August 28, 2025 05:07
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 28, 2025
@chezsmithy
Copy link
Contributor Author

@sestinj not sure if this is expected but I wanted to be able to test the CLI localy to validate changes.

@Patrick-Erichsen Patrick-Erichsen requested review from sestinj and removed request for Patrick-Erichsen August 29, 2025 22:22
@sestinj
Copy link
Contributor

sestinj commented Aug 29, 2025

@sestinj not sure if this is expected but I wanted to be able to test the CLI localy to validate changes.

Is there part of this message that got cut off, or what exactly went wrong here?

General feedback on the PR: we aim to keep the CLI very simple for users, so a third option becoming visible right off the bat is not directionally something I would merge. That said, I wouldn't be opposed to some kind of auto-magic detection if that is possible (e.g. show an option only if we see AWS credentials)

@chezsmithy
Copy link
Contributor Author

@sestinj I can most definitely make it work that way.

@sestinj
Copy link
Contributor

sestinj commented Aug 31, 2025

Awesome! I think this will be really nice. Is it possible to detect automatically whether the AWS credentials have access to Bedrock? One concern would be that some large portion of developers have AWS creds on their machine, but that many of them are not set up to work with Bedrock, so this would be a frustrating dead end

@chezsmithy
Copy link
Contributor Author

chezsmithy commented Sep 1, 2025

@sestinj three options. 1) we could have it work like Claude Code where it looks for the existence of a exported environment variable, or 2) have it look for the existence of a profile called "bedrock" to mirror the other extensions and only offer it then. The chances most developers have this profile is pretty slim. 3) we could go for a combo and use the environment variable and have it point to the AWS profile of choice. This might be the most robust option as it offers detection and customization of the profile.

@chezsmithy chezsmithy marked this pull request as draft September 3, 2025 07:26
@sestinj
Copy link
Contributor

sestinj commented Sep 9, 2025

is the environment variable claude code-specific? Biggest thing is to avoid false positives (profiles that cause the option to be shown, but that then do not have access to bedrock). Would definitely be ok to look for profiles that are called "bedrock", this seems like the safest spot to start

@chezsmithy
Copy link
Contributor Author

@sestinj Claude code is using an environment variable like this:

Configure for Bedrock with Claude Sonnet

export CLAUDE_CODE_USE_BEDROCK=1

@sestinj
Copy link
Contributor

sestinj commented Sep 10, 2025

ok! cool. then yes that very clearly avoids false positives. happy to go forward with that

@sestinj
Copy link
Contributor

sestinj commented Sep 22, 2025

@chezsmithy curious to check in on this. Pretty excited about this one. If you're not sure whether you'll be able to get around to it though might make sense to close, at least temporarily

@chezsmithy chezsmithy marked this pull request as ready for review September 25, 2025 12:19
@chezsmithy
Copy link
Contributor Author

chezsmithy commented Sep 25, 2025

@sestinj ready for review. The failing tests appear unrelated to my changes.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Sep 25, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Sep 26, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 26, 2025
@sestinj sestinj merged commit e1455c6 into continuedev:main Sep 26, 2025
60 of 62 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Sep 26, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 26, 2025
@sestinj
Copy link
Contributor

sestinj commented Sep 26, 2025

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Sep 26, 2025

🎉 This PR is included in version 1.22.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Oct 1, 2025

🎉 This PR is included in version 1.21.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants