Skip to content

Commit 01cf831

Browse files
revert(acp): remove claude-login from auth method map
The previous fix added claude-login to _AUTH_METHOD_ENV_MAP, but Claude Code ACP doesn't support this authentication flow via conn.authenticate(). Instead, Claude Code uses bypassPermissions mode which works without explicit authentication. Reverting to the previous behavior where claude-login triggers a warning but falls back to bypassPermissions mode. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent cab3618 commit 01cf831

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

openhands-sdk/openhands/sdk/agent/acp_agent.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,9 @@ def _make_dummy_llm() -> LLM:
118118
# ACP auth method ID → environment variable that supplies the credential.
119119
# When the server reports auth_methods, we pick the first method whose
120120
# required env var is set.
121+
# Note: claude-login is intentionally NOT included because Claude Code ACP
122+
# uses bypassPermissions mode instead of API key authentication.
121123
_AUTH_METHOD_ENV_MAP: dict[str, str] = {
122-
"claude-login": "ANTHROPIC_API_KEY",
123124
"codex-api-key": "CODEX_API_KEY",
124125
"openai-api-key": "OPENAI_API_KEY",
125126
}

0 commit comments

Comments
 (0)