Skip to content

feat(git): migrate diff pager to delta and fix worktree-aware branch cleanup#5

Merged
iagodahlem merged 1 commit into
masterfrom
iago/git-delta-migration
Jun 19, 2026
Merged

feat(git): migrate diff pager to delta and fix worktree-aware branch cleanup#5
iagodahlem merged 1 commit into
masterfrom
iago/git-delta-migration

Conversation

@iagodahlem

Copy link
Copy Markdown
Owner

What

Fixes three issues reported on Hades (TASK-040):

  1. git df / git br failingdiff-so-fancy was set as core.pager, so it also filtered git branch output, which it can't parse. Migrated the pager to git-delta (passes non-diff through cleanly), and added interactive.diffFilter + a [delta] block (navigate, line-numbers).

  2. git-clean / bam trying to delete branch '+' — git marks worktree-checked-out branches with a leading + (like * for current). Both helpers parsed git branch output without skipping that marker, so + reached git branch -D. Both now exclude ^[+*].

  3. Manifests + docs — swapped diff-so-fancygit-delta in packages/Brewfile, packages/pacman.txt, and TOOLS.md.

Notes

  • git-delta is in Arch extra (pacman -S git-delta) and Homebrew (brew install git-delta) — no AUR needed.
  • After merge: git -C ~/.dotfiles pull makes it live (symlinks already point at the repo).
  • Verified: gitconfig parses, zsh -n passes on .functions, awk fix tested (skips +/* markers).

…ee-aware

- Replace diff-so-fancy with git-delta as core pager; delta passes non-diff
  output through, so git branch no longer breaks under the pager
- Add delta config (navigate, line-numbers) and interactive diffFilter
- Fix git-clean: skip worktree (+) and current (*) markers when parsing
  git branch -vv, so it no longer runs git branch -D +
- Fix bam alias the same way (exclude ^[+*])
- Swap diff-so-fancy -> git-delta in Brewfile and pacman.txt; update TOOLS.md
@iagodahlem
iagodahlem merged commit 5fb7fa7 into master Jun 19, 2026
6 checks passed
@iagodahlem
iagodahlem deleted the iago/git-delta-migration branch June 19, 2026 17:19
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