Skip to content

feat(review): add "All files" diff type#629

Merged
backnotprop merged 3 commits intomainfrom
feat/review-all
Apr 29, 2026
Merged

feat(review): add "All files" diff type#629
backnotprop merged 3 commits intomainfrom
feat/review-all

Conversation

@backnotprop
Copy link
Copy Markdown
Owner

Summary

  • Adds an "All files" diff type that diffs the empty tree against HEAD, showing every tracked file as an addition
  • Solves the case where a repo has only committed changes (no working tree diffs) and there's no way to launch a review
  • Available in the diff type dropdown, as a default preference, and works with worktrees

Test plan

  • Open a review on a repo with no uncommitted/staged/unstaged changes — select "All files" from the dropdown
  • Verify every tracked file appears as additions
  • Verify file expansion (click to expand context) works correctly
  • Verify "All files" works in a worktree
  • Verify it can be set as the default diff type via the setup dialog or config

Adds an "all" diff type that diffs the empty tree against HEAD, showing
every tracked file as an addition. Solves the case where a repo has only
committed changes and no working tree diffs — previously there was no way
to launch a review at all.

For provenance purposes, this commit was AI assisted.
The Settings panel (gear icon) had its own DEFAULT_DIFF_TYPE_OPTIONS list
that was missing the new "all" diff type, so users couldn't select or see
it as the default from the settings UI.

For provenance purposes, this commit was AI assisted.
Makes it clear this mode shows committed state at HEAD, not the working
tree. Updated tooltip to distinguish from "Committed changes" (which
diffs against a base branch).

For provenance purposes, this commit was AI assisted.
@backnotprop backnotprop merged commit a68463b into main Apr 29, 2026
8 of 9 checks passed
@backnotprop backnotprop deleted the feat/review-all branch April 29, 2026 23:43
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.

1 participant