From a1c8bdc9ce0aed489d5d452168ef7c5d9431fadb Mon Sep 17 00:00:00 2001 From: Marcin Tojek Date: Tue, 22 Jul 2025 13:26:47 +0200 Subject: [PATCH] fix: do not warn on Claude API key if Bedrock in use --- cli/exp_mcp.go | 24 +++++++++++++++++------- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/cli/exp_mcp.go b/cli/exp_mcp.go index 5cfd9025134fd..d5ea26739085b 100644 --- a/cli/exp_mcp.go +++ b/cli/exp_mcp.go @@ -127,6 +127,7 @@ func (r *RootCmd) mcpConfigureClaudeCode() *serpent.Command { appStatusSlug string testBinaryName string aiAgentAPIURL url.URL + claudeUseBedrock string deprecatedCoderMCPClaudeAPIKey string ) @@ -154,14 +155,15 @@ func (r *RootCmd) mcpConfigureClaudeCode() *serpent.Command { configureClaudeEnv[envAgentURL] = agentClient.SDK.URL.String() configureClaudeEnv[envAgentToken] = agentClient.SDK.SessionToken() } - if claudeAPIKey == "" { - if deprecatedCoderMCPClaudeAPIKey == "" { - cliui.Warnf(inv.Stderr, "CLAUDE_API_KEY is not set.") - } else { - cliui.Warnf(inv.Stderr, "CODER_MCP_CLAUDE_API_KEY is deprecated, use CLAUDE_API_KEY instead") - claudeAPIKey = deprecatedCoderMCPClaudeAPIKey - } + + if deprecatedCoderMCPClaudeAPIKey != "" { + cliui.Warnf(inv.Stderr, "CODER_MCP_CLAUDE_API_KEY is deprecated, use CLAUDE_API_KEY instead") + claudeAPIKey = deprecatedCoderMCPClaudeAPIKey + } + if claudeAPIKey == "" && claudeUseBedrock != "1" { + cliui.Warnf(inv.Stderr, "CLAUDE_API_KEY is not set.") } + if appStatusSlug != "" { configureClaudeEnv[envAppStatusSlug] = appStatusSlug } @@ -280,6 +282,14 @@ func (r *RootCmd) mcpConfigureClaudeCode() *serpent.Command { Value: serpent.StringOf(&testBinaryName), Hidden: true, }, + { + Name: "claude-code-use-bedrock", + Description: "Use Amazon Bedrock.", + Env: "CLAUDE_CODE_USE_BEDROCK", + Flag: "claude-code-use-bedrock", + Value: serpent.StringOf(&claudeUseBedrock), + Hidden: true, + }, }, } return cmd