Skip to content

fix: env var override for base URL in fresh CLI contexts#51

Merged
George-iam merged 1 commit intomainfrom
fix/env-base-url-override-20260319
Mar 19, 2026
Merged

fix: env var override for base URL in fresh CLI contexts#51
George-iam merged 1 commit intomainfrom
fix/env-base-url-override-20260319

Conversation

@George-iam
Copy link
Contributor

Summary

CLI creates default context with BaseURL=http://127.0.0.1:8100 when no config exists. Env var overrides (AXME_GATEWAY_BASE_URL, AXME_BASE_URL) only applied when BaseURL was empty — so they were ignored in CI.

Now env vars override the default localhost URL:

  • AXME_PORTAL_BASE_URL, AXME_GATEWAY_BASE_URL, AXME_BASE_URL → base URL
  • AXME_GATEWAY_API_KEY, AXME_API_KEY → API key

Root cause of E2E failures: CLI connected to localhost instead of staging.

Test plan

  • All existing tests pass
  • E2E tests pass after CLI release

🤖 Generated with Claude Code

@George-iam George-iam force-pushed the fix/env-base-url-override-20260319 branch from eec50aa to 450ed72 Compare March 19, 2026 13:46
When no config file exists (e.g. CI), CLI creates a default context with
BaseURL=http://127.0.0.1:8100. Previously, env var overrides only applied
when BaseURL was empty — so AXME_GATEWAY_BASE_URL was ignored.

Now env vars override the default localhost URL:
- AXME_PORTAL_BASE_URL, AXME_GATEWAY_BASE_URL, AXME_BASE_URL → base URL
- AXME_GATEWAY_API_KEY, AXME_API_KEY → API key

This fixes E2E tests in CI where CLI was connecting to localhost
instead of the staging gateway URL.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@George-iam George-iam force-pushed the fix/env-base-url-override-20260319 branch from 450ed72 to 4256e99 Compare March 19, 2026 13:49
@George-iam George-iam merged commit d3e0b50 into main Mar 19, 2026
1 check passed
@George-iam George-iam deleted the fix/env-base-url-override-20260319 branch March 19, 2026 13:50
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