umputun / revdiff / 25207079440

90%

master: 90%

Build Type

Pull #163

github

Committed by rashpile

Commit Message

feat: add --compare flag and validation to config

feat: implement CompareReader for git diff --no-index two-file diff

feat: wire --compare flag into main.go run() function

Switch renderer selection to a switch statement covering compare,
stdin, and VCS modes. CompareReader gets workDir from filepath.Dir
of the new path. commitsApplicable and compactApplicable already
handle CompareReader correctly without changes.

Also remove stale nolintlint directive in overlay/info.go.

chore: verify acceptance criteria for --compare flag

All acceptance criteria verified: diff between two files, identical-file
empty diff, compact mode with dividers, word-diff, and all mutual-exclusion
error cases. Full test suite passes (make test).

feat: use --compare in planning plugin for plan iteration diffs

After Claude revises an annotated plan, re-opening revdiff now shows
only what changed via --compare=prev:current instead of the full file.
First open uses --only as before. Pass --cleanup to discard the snapshot.

docs: add --compare flag to README, docs, and plugin usage reference

Document the two-file diff mode in all three locations: README (Options table,
Examples, new Two-File Diff section), site/docs.html (options table, examples,
new section), and the Claude plugin usage reference.

fix: address code review findings

- validateCompareFlag: reject --include and --exclude (silently ignored in compare mode)
- main.go: use filepath.Abs before filepath.Dir to avoid "project: ." in info overlay for relative paths
- compare.go: replace pre-allocated make+append slice with composite literal (removes wrong mnd comment)
- compare_test.go: use t.TempDir() paths instead of literal /tmp/ in TestCompareReader_ChangedFiles
- renderer_setup_test.go: add CompareReader case to TestCompactApplicable, --compare case to TestCommitsApplicable
- launch-plan-review.sh: guard PLAN_ABS cd against missing directory so --cleanup survives deleted plan dirs
- CLAUDE.md: document --compar... (continued)

Pull Request Pull Request #163: feat: `--compare=old:new` — two-file diff for rolling agent reviews

Coverage Stats

110 of 146 new or added lines in 7 files covered. (75.34%)

27 existing lines in 2 files now uncovered.

9556 of 10671 relevant lines covered (89.55%)

508.56 hits per line