Skip to content

fix: preserve truth in doctor and lane status - #548

Merged
jeffhuber merged 1 commit into
mainfrom
codex/v09-truth-preserving-540
Sep 2, 2026
Merged

fix: preserve truth in doctor and lane status#548
jeffhuber merged 1 commit into
mainfrom
codex/v09-truth-preserving-540

Conversation

@jeffhuber

Copy link
Copy Markdown
Contributor

Summary

  • make merge-authority review hygiene warn instead of pass when workflow file presence was not verified
  • make lanes status render GitHub-unavailable PR/workflow/gate sections as unavailable, not empty
  • document the safer interpretation for first-run hygiene warnings and lane-status next actions

Verification

  • scripts/dev-python -m unittest tests.test_doctor_provider_review_hygiene tests.test_doctor_registry tests.test_lane_status
  • scripts/dev-python -m ruff check src/code_mower/doctor_checks/provider_review_hygiene.py src/code_mower/lane_status.py tests/test_doctor_provider_review_hygiene.py tests/test_lane_status.py
  • scripts/dev-python -m code_mower.cli lanes status --repo codemower-ai/code-mower --json
  • scripts/dev-python -m code_mower.cli doctor --adoption --repo codemower-ai/code-mower --json
  • scripts/dev-python scripts/privacy_scan.py
  • scripts/dev-python -m pytest

Closes #540.

@jeffhuber jeffhuber added builder:codex Code Mower generated label needs-claude-audit needs-gitar-audit Code Mower generated label labels Sep 2, 2026
@gitar-bot

gitar-bot Bot commented Sep 2, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Fixes doctor and lane status to preserve truth by warning when workflow file presence is unverified and rendering unavailable GitHub sections accurately instead of empty. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@jeffhuber

Copy link
Copy Markdown
Contributor Author

Claude audit (merge-authority lane)

Head SHA: 2205a1d03b6a868826f16f4138ad92c9de7483c2
Findings: P0=0, P1=0, P2=0, P3=1 (blocker policy: any P0/P1/P2 -> BLOCKED)

Claude Audit: PASS

Summary:

Truth-preserving fixes to doctor review_hygiene check and lane_status next_action/render_text; well tested, no P0-P2 issues found.

Findings:

  • [P3] No blocking issues -- src/code_mower/lane_status.py:328
    Finding ID: claude:13ae01a11129777a53e7
    No blocker findings; change is a correctness fix consistent with tests and docs.

@jeffhuber jeffhuber added claude-audit-done gitar-audit-done Code Mower generated label and removed needs-claude-audit needs-gitar-audit Code Mower generated label labels Sep 2, 2026
@jeffhuber
jeffhuber merged commit 5649f04 into main Sep 2, 2026
25 checks passed
@jeffhuber
jeffhuber deleted the codex/v09-truth-preserving-540 branch September 2, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builder:codex Code Mower generated label claude-audit-done gitar-audit-done Code Mower generated label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v0.9: make doctor and lanes status truth-preserving

1 participant