Skip to content

Conversation

@Fl0p
Copy link

@Fl0p Fl0p commented Jan 17, 2026

  • Detect CLAUDE_CODE_USE_BEDROCK env var for Bedrock auth
  • Hide OAuth login UI for Bedrock users (auth via AWS credentials)
  • Pass isBedrock flag through auth status to frontend

Summary by CodeRabbit

  • New Features
    • Added support for AWS Bedrock as a Claude authentication option
    • Simplified authentication experience for Bedrock users
    • Authentication status now indicates Bedrock connection method

✏️ Tip: You can customize this high-level summary in your review settings.

- Detect CLAUDE_CODE_USE_BEDROCK env var for Bedrock auth
- Hide OAuth login UI for Bedrock users (auth via AWS credentials)
- Pass isBedrock flag through auth status to frontend

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 17, 2026

Walkthrough

This PR adds AWS Bedrock authentication support to Claude credential checking. A new isBedrock flag is introduced in the authentication status payload, propagated from server to client, and used to conditionally hide login UI for Bedrock-authenticated users. The server now checks an environment variable to enable Bedrock mode, bypassing local credential file reading.

Changes

Cohort / File(s) Summary
Bedrock Authentication Support
server/routes/cli-auth.js
Added Bedrock path detection via CLAUDE_CODE_USE_BEDROCK env variable; /claude/status endpoint now includes isBedrock flag and derives auth method as 'bedrock' or 'credentials_file' based on this flag; early return for Bedrock auth with email set to 'AWS Bedrock'
Bedrock Authentication Support
src/components/Settings.jsx
Integrated isBedrock flag from auth status payload into Claude auth state object; defaults to false when absent
Bedrock Authentication Support
src/components/settings/AccountContent.jsx
Added conditional guard to hide login/re-authentication UI when authStatus.isBedrock is true; login button and related auth UI now only render for non-Bedrock users

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A Bedrock path now lights the way,
Where AWS credentials dance and play,
The login button hops aside so clean,
For Bedrock users, a serene scene,
Authentication flags hop through the stack! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: add AWS Bedrock authentication support' accurately describes the main objective of the pull request, which adds support for AWS Bedrock as an authentication method.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant