Skip to content

feat(doctor): actionable fix hints on each check (#23) - #252

Merged
Destynova2 merged 1 commit into
mainfrom
feat/doctor-fix-hints
Apr 21, 2026
Merged

feat(doctor): actionable fix hints on each check (#23)#252
Destynova2 merged 1 commit into
mainfrom
feat/doctor-fix-hints

Conversation

@Destynova2

Copy link
Copy Markdown
Contributor

Summary

  • Every doctor check that reports a non-OK state now emits an explicit `→ run:` (or `→ install:` / `→ check:`) line pointing at the next step — no more "now what?".
  • Examples: missing config → `grob setup`; missing creds → `grob connect`; port in use → `grob stop` + `lsof`; missing env var → `export NAME=`.
  • Exit codes unchanged.

Test plan

  • `cargo clippy --lib --all-features -- -D warnings` clean
  • Manual: `grob doctor` on a fresh VM shows hints on every failure branch

Audit item #23 (routine, N=3).

🤖 Generated with Claude Code

Every doctor check that reports a warning, error, or informational
state now emits a `→ run:` (or `→ install:` / `→ check:`) line
pointing at the exact next step the user should take. Makes the
output self-service for new users and avoids the "now what?" moment.

* Missing config  → `grob setup`
* Missing provider credentials → `grob connect` / `grob setup --edit providers`
* Port in use → `grob stop` + `lsof -i :PORT` for diagnosis
* Service not running → `grob start -d`
* Missing env var → `export NAME=<key>` with the first missing name
* DLP / Security off → `grob setup --edit compliance`
* Storage error → `ls -la` of the storage path
* Podman missing → link to the install docs

No behavioral change in exit codes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Destynova2
Destynova2 enabled auto-merge (squash) April 21, 2026 21:21
@Destynova2
Destynova2 merged commit 176d080 into main Apr 21, 2026
42 checks passed
@Destynova2
Destynova2 deleted the feat/doctor-fix-hints branch April 21, 2026 21:40
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