Skip to content

fix: propagate double-tracked inspection errors - #190

Merged
junior-ricon merged 1 commit into
junior/fail-closed-shell-producersfrom
johnson/fix-double-tracked-inspection
Jul 28, 2026
Merged

fix: propagate double-tracked inspection errors#190
junior-ricon merged 1 commit into
junior/fail-closed-shell-producersfrom
johnson/fix-double-tracked-inspection

Conversation

@johnson-ricon

@johnson-ricon johnson-ricon commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make notes stage fail when double-tracked path inspection fails
  • make notes status propagate the same safety-check failure instead of reporting zero conflicts
  • add CLI-level regressions proving stage leaves the index untouched and status exits non-zero

Context

Focused fix-it for #189, based directly on reviewed head 173a2ee7fe17ad090ed9f83a963d19c7c7dea82b.

Validation

  • mise run test changes (20 passed)
  • mise run test (438 BATS passed, 9 Python passed; 6 existing skips)
  • codebase lint "$PWD" (8 rules passed)
  • mise run doctor (passed; existing optional local-hook warning only)
  • readme build --check
  • bash -n .mise/tasks/stage .mise/tasks/status test/changes.bats
  • git diff --check

Local git pre-commit and git pre-push commands are unavailable in this checkout, so direct repository checks were used.

@junior-ricon
junior-ricon merged commit 5f01423 into junior/fail-closed-shell-producers Jul 28, 2026
2 checks passed
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.

2 participants