Files
Trevin Chow f435b994aa fix(review): commit safe fixes on a clean tree; gate the push, not the commit
Refine the interactive apply model after design review. The permanence gate is
the push, not the commit: a local commit is private and reversible, and leaving
applied fixes uncommitted on a clean tree just creates a chore plus a window
where the user's next edits entangle with them.

- Stage 5c: when the working tree was clean before the review, commit the
  applied fixes as one isolated `fix(review):` commit; on a dirty tree apply but
  leave them for the user's commit (the fixes can't be isolated from WIP).
  Never push, open PRs, or file tickets.
- Operating principle, After Review, the Applied report step, the output
  template, the skill doc, and the plan all updated to commit-on-clean /
  never-push (was "never commits or pushes").
- Contract tests updated to the new contract.

bun test: contract 29/0; full suite at the 47 pre-existing failures, zero new.
2026-06-02 19:21:50 -07:00
..