Skip to content

fix(AC-332): fall back to ANTHROPIC_API_KEY in build_client_from_settings#448

Merged
jayscambler merged 1 commit intomainfrom
fix/ac-332-api-key-fallback
Mar 18, 2026
Merged

fix(AC-332): fall back to ANTHROPIC_API_KEY in build_client_from_settings#448
jayscambler merged 1 commit intomainfrom
fix/ac-332-api-key-fallback

Conversation

@jayscambler
Copy link
Contributor

Summary

build_client_from_settings() didn't fall back to ANTHROPIC_API_KEY env var, unlike the provider registry and judge provider which both do.

Fix

# Before:
if not settings.anthropic_api_key:
    raise ValueError("AUTOCONTEXT_ANTHROPIC_API_KEY is required...")

# After:
api_key = settings.anthropic_api_key or os.getenv("ANTHROPIC_API_KEY", "")
if not api_key:
    raise ValueError("AUTOCONTEXT_ANTHROPIC_API_KEY or ANTHROPIC_API_KEY is required...")

Test plan

  • 4 tests: AUTOCONTEXT key, ANTHROPIC_API_KEY fallback, no key raises, deterministic OK
  • ruff clean, full suite 4393 passed

…ings

build_client_from_settings() only checked settings.anthropic_api_key
(AUTOCONTEXT_ANTHROPIC_API_KEY) but didn't fall back to ANTHROPIC_API_KEY
like the provider registry and judge provider do. This caused L11/L12
escalation scripts to fail when only ANTHROPIC_API_KEY was set.

Fix: api_key = settings.anthropic_api_key or os.getenv("ANTHROPIC_API_KEY")

4 tests: AUTOCONTEXT key used when set, falls back to ANTHROPIC_API_KEY,
raises when neither set, deterministic doesn't need key.
@linear
Copy link

linear bot commented Mar 18, 2026

@jayscambler jayscambler merged commit d68446b into main Mar 18, 2026
4 checks passed
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