[regnull]

Summary

Follow-up cleanup to the terminal client (#314), folding in two carried-forward nits:

• connect now actually probes the server. It builds an OpenBotClient from the saved settings and calls the existing ping() against the public /api/v1/health route, so a bad URL or key surfaces immediately. A failed probe is a warning (not fatal) — the config is still saved so the user can retry. (ping() already existed and was tested; it simply wasn't wired to any command.)
• Dropped the inline __import__("pathlib") in cli.py in favour of a top-level from pathlib import Path import.

What changed

• backend/openbot_cli/cli.py
  • connect: saves config, then builds a client and pings; prints Connection OK (/api/v1/health) on success or a Warning: on failure (config still saved).
  • top-level from pathlib import Path; import OpenBotClientError.
• backend/tests/test_openbot_cli.py
  • test_cli_connect_saves_and_pings: config saved, client built with correct base_url/api_key, ping() called, output reports OK.
  • test_cli_connect_warns_when_server_unreachable: warning path + config still persisted on a failed probe.

Verification

• uv run ruff check . — only pre-existing errors remain, all in out-of-scope files I did not touch: the pre-existing untracked WIP backend/openbot/cli_client.py (someone else's) and scripts/test_makefile.py on main. The openbot_cli package is ruff-clean.
• uv run pytest -q tests/test_openbot_cli.py tests/test_config.py → 50 passed. The two new connect tests pass.
• No changes to web-UI code; the shared-workspace untracked WIP (cli_client.py, delivery.py, pyproject.toml, test_actor_system.py) was left exactly as found.

Note for reviewers

PR #314 is already merged, so this is a small follow-up branch off current main. Draft until CI is green.

[regnull]

PASS — Ready to merge (posted as COMMENTED: self-approval is blocked on the shared fleet account, not an approval).

Verified against the actual diff + production code:

• Scope: exactly 2 files (backend/openbot_cli/cli.py, backend/tests/test_openbot_cli.py). No web-UI files touched.
• Nit #1 (README/connect /health claim) RESOLVED: connect now probes the server via probe.ping() -> GET /api/v1/health (client.py:71-73, documented on the public unauthenticated router). On failure it warns and still saves config — matches the stated design. The earlier "claims a /health probe it does not perform" nit is closed.
• Nit #2 (inline __import__("pathlib")) RESOLVED: now a top-level from pathlib import Path.
• OpenBotClientError is a real exported class (client.py:30) and is imported correctly.
• Tests: two new tests assert the discriminating behavior (config saved + pinged=True; warning on OpenBotClientError with config still saved) — not vacuous.
• CI green on exact head per Chief of Staff (combined review+QA gate, re-check skipped).

Both carried-forward nits from #314 are now closed. No blockers.

[regnull]

PASS / LGTM (diff-only re-check). Risk 2. CI is green (backend and frontend) and the merge state is CLEAN at 63c82ac.

Musts: none. Shoulds: none.

Verified

• git diff b0d8a223 63c82acd is empty: the trees are identical. 63c82ac is a re-commit of b0d8a22 on the same parent (0bda7e3, #314), and only the commit message changed (backticks).
• main is still 0bda7e3, so main brought in nothing new.
• uv run pytest -q tests/test_openbot_cli.py passed 17/17, and ruff check is clean on openbot_cli and the test file.