Skip to content

fix(claude mcp): Suppress CLAUDE_API_KEY warning when cloud providers are detected#40

Closed
cdr-robot wants to merge 1 commit into
mainfrom
fix/claude-api-key-warning
Closed

fix(claude mcp): Suppress CLAUDE_API_KEY warning when cloud providers are detected#40
cdr-robot wants to merge 1 commit into
mainfrom
fix/claude-api-key-warning

Conversation

@cdr-robot
Copy link
Copy Markdown
Contributor

Summary

  • Fixed unnecessary warning when CLAUDE_API_KEY is not set but cloud providers are configured
  • Added detection for AWS Bedrock and Google Vertex AI environment variables
  • Only shows the warning when no cloud provider credentials are detected

Test plan

  1. Set AWS credentials (AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY or AWS_PROFILE)
  2. Run the command without CLAUDE_API_KEY set
  3. Verify that the warning doesn't appear

Fixes #19
Workspace available at: https://demo.coder.com/ben/issue-19

… are detected

When using AWS Bedrock or Google Vertex AI for Claude, users don't need a direct Claude API key.
This change detects the presence of cloud provider credentials and only shows the warning
when no cloud provider is configured.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(claude mcp): WARN: CLAUDE_API_KEY is not set. is set, even if is set)

1 participant