umputun / revdiff / 25236122332

Pull #163

github

fix: round-2 review findings — stale launcher safety + tighter test/prompt

- swap compare-mode arg order from (old, new) to (new, old) in both the
  bundled launcher's 2-arg branch and the planning hook so that stale
  user-layer overrides at ${CLAUDE_PLUGIN_DATA}/scripts/ — copies of
  master's 1-arg launcher made before compare mode shipped — silently
  degrade to --only review of the NEW revision instead of opening the
  OLD one the user already reviewed. revdiff itself still receives
  --compare-old=<OLD> --compare-new=<NEW>; the launcher relabels locally.
- tighten the deny-reason text so the agent is explicitly told to ignore
  any other plan-rev-*.md marker it may have seen earlier in the
  conversation, addressing intra-session chain-mixup at the prompt layer.
- replace assert.NotEmpty on opts.compareAbsOld/New with equality against
  filepath.Abs(input) so the test would actually catch a swapped-fields
  bug rather than just confirming the cache is populated.
- document the new launcher contract (new-revision first) and the
  graceful-degradation rationale in the planning plugin README.

110 of 139 new or added lines in 7 files covered. (79.14%)

9624 of 10730 relevant lines covered (89.69%)

3210.54 hits per line