fix(cli): honor saved coordinator host

[adity982]

Summary

Organization and project commands now use the coordinator URL saved by arcade login when no connection flags are provided, matching the behavior of the rest of the CLI. Explicit host, port, and TLS flags continue to take precedence.

Resolves: #896

Design decisions

The fallback is centralized in resolve_coordinator_url so both command groups share the same precedence rules. The visible --host default remains the production coordinator for help output, while None internally distinguishes an omitted flag from an explicit override.

Test plan

• uv run pytest libs/tests/cli/test_org_project.py (3 passed)
• uv run ruff check for the changed Python files
• uv run ruff format --check for the changed Python files

Author checklist

Before moving this PR from Draft to Ready for Review:

• Linked to a GitHub issue (above)
• I understand every change in the diff
• Runs locally, exercised through the end-user path (not just unit tests)
• make check and make test are green locally; CI is expected to pass
• I've reviewed my own diff top-to-bottom
• I'd merge it myself if a teammate said LGTM right now

---

Note

Low Risk
CLI-only coordinator URL resolution for org/project commands; explicit flags and production fallback unchanged in spirit, with new unit test coverage.

Overview
arcade org and arcade project no longer always target the production coordinator when connection flags are omitted. They now resolve the coordinator through new resolve_coordinator_url, which uses the URL saved at arcade login when --host is not passed, matching the rest of the CLI.

--host defaults to None internally (help still shows production via show_default), so an omitted flag can fall back to config. Explicit --host, --port, and --tls / --no-tls still win; partial overrides reuse the saved host/scheme where appropriate. Missing or invalid config falls back to https://{PROD_COORDINATOR_HOST}.

Adds libs/tests/cli/test_org_project.py for these precedence rules and bumps arcade-mcp to 1.15.3.

^{Reviewed by Cursor Bugbot for commit b584205. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 5daa4a2. Configure here.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines	Coverage Δ
libs/arcade-cli/arcade_cli/org.py	34.84% <100.00%> (+13.95%)	⬆️
libs/arcade-cli/arcade_cli/project.py	38.46% <100.00%> (+17.24%)	⬆️
libs/arcade-cli/arcade_cli/utils.py	57.98% <100.00%> (+0.93%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

This pull request has been automatically marked as stale because it has had no activity for 14 days. It will be closed in 14 days if no further activity occurs. If this is still relevant, please leave a comment or remove the stale label.

[adity982]

Addressed the missing-config review finding in commit 3aab68d and merged current upstream main.
esolve_coordinator_url now falls back to the production coordinator when no saved config exists, with a CLI regression covering the no-config path. Post-merge validation: 4 focused tests passed; Ruff lint/format and lockfile checks passed. The review thread is now automatically resolved/outdated on the new head.

[EricGustin]

@greptileai

[greptile-apps]

Greptile Summary

Organization and project commands now resolve their coordinator URL from saved login configuration when no connection flags are supplied, while retaining explicit-option and production fallbacks.

• Adds a shared coordinator URL resolver.
• Updates both command callbacks to distinguish omitted hosts from explicit overrides.
• Adds tests for saved URLs, host overrides, and missing configuration.

Confidence Score: 4/5

The PR appears safe to merge after addressing the non-blocking configuration-error handling and required package version update.

The intended saved-host precedence is implemented consistently, but invalid existing configuration can now escape from callbacks as a traceback, and the repository-required library version increment is absent.

Files Needing Attention: libs/arcade-cli/arcade_cli/utils.py and pyproject.toml

Important Files Changed

Filename	Overview
libs/arcade-cli/arcade_cli/utils.py	Adds centralized saved-URL resolution, but non-missing configuration errors can escape from command callbacks.
libs/arcade-cli/arcade_cli/org.py	Changes the organization callback to distinguish an omitted host and delegate coordinator resolution.
libs/arcade-cli/arcade_cli/project.py	Applies the same coordinator-resolution behavior to project commands.
libs/tests/cli/test_org_project.py	Covers saved coordinator URLs, explicit host precedence, and production fallback, but not invalid configuration files.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Organization or project callback] --> B{Any connection flag supplied?}
    B -- Yes --> C[Build URL from explicit options and production host fallback]
    B -- No --> D[Load saved configuration]
    D --> E{Saved coordinator URL exists?}
    E -- Yes --> F[Use saved coordinator URL]
    E -- No or file missing --> G[Use production coordinator URL]

Loading

_{Reviews (1): Last reviewed commit: "Merge branch 'main' into agent/use-saved..." | Re-trigger Greptile}

[github-actions]

This pull request has been automatically marked as stale because it has had no activity for 14 days. It will be closed in 14 days if no further activity occurs. If this is still relevant, please leave a comment or remove the stale label.

[greptile-apps]

Your trial has ended. Reactivate Greptile to resume code reviews.