Skip to content

refactor: remove --api-url CLI flag, read api_url from config only - #219

Merged
sachiniyer merged 1 commit into
mainfrom
siyer/remove-cli-api-url
Mar 24, 2026
Merged

refactor: remove --api-url CLI flag, read api_url from config only#219
sachiniyer merged 1 commit into
mainfrom
siyer/remove-cli-api-url

Conversation

@sachiniyer

@sachiniyersachiniyer commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

The --api-url flag and DETAIL_API_URL env var were hidden options
intended for testing. Replace them with config-file-only api_url,
reducing the CLI surface and keeping configuration in one place.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com


Open with Devin

@sachiniyerGraphite App

sachiniyer commented Mar 24, 2026

Copy link
Copy Markdown
ContributorAuthor

@sachiniyer
sachiniyerforce-pushed the siyer/remove-cli-api-url branch from 5b388fa to a408ad3CompareMarch 24, 2026 04:19
Base automatically changed from siyer/fix-config-apiurl to mainMarch 24, 2026 04:41
@sachiniyer
sachiniyer marked this pull request as ready for review March 24, 2026 04:41
@sachiniyer
sachiniyer enabled auto-merge (squash) March 24, 2026 04:41
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@devin-ai-integrationdevin-ai-integrationBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

The `--api-url` flag and `DETAIL_API_URL` env var were hidden options
intended for testing. Replace them with config-file-only `api_url`,
reducing the CLI surface and keeping configuration in one place.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sachiniyer
sachiniyerforce-pushed the siyer/remove-cli-api-url branch from a408ad3 to 94a8ae2CompareMarch 24, 2026 05:17
@sachiniyer
sachiniyer merged commit 9842dd3 into mainMar 24, 2026
13 checks passed
@sachiniyer
sachiniyer deleted the siyer/remove-cli-api-url branch March 24, 2026 05:20
Sign up for freeto 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

@sachiniyer