Skip to content

docs(roadmap): add #332 — doctor json missing top-level status field#2841

Open
code-yeongyu wants to merge 1 commit intomainfrom
docs/roadmap-325-doctor-no-status-field
Open

docs(roadmap): add #332 — doctor json missing top-level status field#2841
code-yeongyu wants to merge 1 commit intomainfrom
docs/roadmap-325-doctor-no-status-field

Conversation

@code-yeongyu
Copy link
Copy Markdown
Collaborator

Pinpoint #325

doctor --output-format json has no status field — breaks unified health-check pattern.

Live on e7074f4:

claw doctor --output-format json | python3 -c "import sys,json; print(json.load(sys.stdin).get('status'))"
# → None

claw status --output-format json | python3 -c "import sys,json; print(json.load(sys.stdin).get('status'))"  
# → ok

Doctor uses has_failures (bool) + summary: {failures, ok, warnings}; every other JSON command uses "status": "ok"|"warn"|"error". Automation checking result.get("status") == "ok" silently misreads doctor warnings as absent status.

Fix: add "status": "ok"|"warn"|"error" top-level field to doctor JSON, derived from existing has_failures/warning-count logic.

ROADMAP.md only: 1 file changed, 2 insertions(+)

@code-yeongyu code-yeongyu force-pushed the docs/roadmap-325-doctor-no-status-field branch from 50674e7 to 6f17c79 Compare April 29, 2026 11:34
@code-yeongyu code-yeongyu changed the title docs(roadmap): add #325 — doctor json missing top-level status field docs(roadmap): add #327 — doctor json missing top-level status field Apr 29, 2026
@code-yeongyu code-yeongyu force-pushed the docs/roadmap-325-doctor-no-status-field branch from 6f17c79 to 3b5bafc Compare April 29, 2026 13:04
@code-yeongyu code-yeongyu changed the title docs(roadmap): add #327 — doctor json missing top-level status field docs(roadmap): add #332 — doctor json missing top-level status field Apr 29, 2026
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