Skip to content

STA-281: Support Git pathspec selection in inspect - #42

Merged
github-actions[bot] merged 1 commit into
mainfrom
feature/sta-281
Sep 17, 2026
Merged

github-actions[bot] merged 1 commit into
mainfrom
feature/sta-281

Conversation

@claudecafe

Copy link
Copy Markdown
Collaborator

User benefit

diffwalk inspect can now scope working-tree and staged captures with repeatable literal --path values or Git --pathspec expressions, including quoted globs and exclusion magic.

Accepted behavior

  • --path and --pathspec are mutually exclusive; --exclude composes with either mode.
  • Positional arguments retain revision behavior, while the old -- <path>... syntax gives migration guidance.
  • Git scope selection runs before rename detection; literal --exclude runs afterward.
  • Documentation and agent skill cover quoting, Git pathspec semantics, literal special-character filenames, empty/invalid matches, and rename boundaries.

Checks

  • pnpm check: passed
  • Typecheck: passed
  • Bun tests: 415 passed, 0 failed
  • Visual tests: 23 passed, 1 intentional skip

Walkthrough: https://review.diffwalk.dev/r/wLZL0mfHw6dJYxZ48pA29w

Add repeatable `inspect --path <path>` for literal selection in place of the `-- <path>...` interface, and repeatable `--pathspec <expression>` so Git glob and exclusion magic can scope a working-tree capture. Selection reaches Git before rename detection while literal `--exclude` still drops paths afterwards, and the old separator now fails with migration guidance instead of being read as a revision or a pattern.

Co-Authored-By: ことね <kotone@claudecafe.dev>
@github-actions
github-actions Bot enabled auto-merge (rebase) September 17, 2026 01:23
@github-actions
github-actions Bot merged commit 02d5e3a into main Sep 17, 2026
2 checks passed
@claudecafe
claudecafe deleted the feature/sta-281 branch September 17, 2026 02:04
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