Skip to content

[codex] Add ballin doctor health check#201

Merged
JBallin merged 2 commits into
mainfrom
codex/issue-187-doctor
Jun 30, 2026
Merged

[codex] Add ballin doctor health check#201
JBallin merged 2 commits into
mainfrom
codex/issue-187-doctor

Conversation

@JBallin

@JBallin JBallin commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #187.

Adds a small ballin doctor surface backed by the shared setup readiness helper from #173 / PR #189. The command stays focused on Ballin-managed development environment health and does not expand into broader CLI routing.

What changed

  • Adds ballin doctor handling to the existing ballin command.
  • Formats readiness helper checks as concise OK, WARN, ERROR, and INFO lines.
  • Treats warning-only doctor output as exit 0, required check failures as exit 1, and invalid ballin doctor usage as exit 2.
  • Documents the new command and explicitly keeps brew doctor out of this slice.
  • Adds isolated tests with temp config, temp PATH, and command stubs.

Scope notes

Validation

  • npm test (226 passing)
  • git diff --check

@JBallin JBallin marked this pull request as ready for review June 30, 2026 03:32
@JBallin JBallin merged commit bf90240 into main Jun 30, 2026
4 checks passed
@JBallin JBallin deleted the codex/issue-187-doctor branch June 30, 2026 04:03
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.

Design ballin doctor health check

1 participant