umputun / revdiff / 24370599519

90%

push

github

Add `--include` prefix filter flag (#103)

* feat: add IncludeFilter with shared prefix helpers

Extract matchesPrefix() and normalizePrefixes() from ExcludeFilter into
shared prefix.go helpers. Create IncludeFilter that keeps only files
matching given prefixes, mirroring ExcludeFilter's decorator pattern.

* feat: add --include CLI flag with validation and tests

* feat: wire IncludeFilter into renderer factories

Add include []string parameter to makeGitRenderer, makeHgRenderer, and
makeNoVCSRenderer. IncludeFilter wraps before ExcludeFilter so include
narrows first, then exclude refines. Update setupVCSRenderer to pass
opts.Include. Add tests for include wrapping, all-files+include, and
functional include+exclude composition.

* feat: add --include flag documentation across all doc targets

* fix: address code review findings

- Add diff.IncludeFilter to Renderer implementors list in CLAUDE.md
- Update gotcha bullet with --include filtering and --stdin mutual exclusivity
- Add --include example to Pi plugin SKILL.md

* chore: remove implementation plan file

* refactor: move shared prefix helpers from prefix.go into diff.go

Move matchesPrefix and normalizePrefixes to the bottom of diff.go and
their tests to diff_test.go. Drop duplicate tests from include_test.go
and exclude_test.go. Delete prefix.go.

* refactor: remove unreachable include/exclude from makeNoVCSRenderer

No-VCS mode requires --only, which is mutually exclusive with --include.
--exclude on FileReader (which only returns --only files) is a no-op.
Remove both filter params and simplify to direct FileReader return.

* fix: handle empty normalized include prefixes as no-op

When all --include prefixes normalize to empty (e.g. REVDIFF_INCLUDE=,),
treat as no-op instead of silently dropping all files. Also remove
unreachable include/exclude params from makeNoVCSRenderer since no-VCS
mode requires --only which is mutually exclusive with --include.

56 of 59 new or added lines in 4 files covered. (94.92%)

6580 of 7412 relevant lines covered (88.77%)

349.18 hits per line