umputun / revdiff / 24050513781

90%

push

github

Add git blame gutter toggle (B key) (#38)

* Add git blame gutter toggle (B key)

Show author name and relative commit age per line in a gutter column,
toggled with the B key. Blame data loads asynchronously via git blame
and is keyed by new-side line numbers (blank for removed lines/dividers).

- Add diff/blame.go with FileBlame parser and RelativeAge formatter
- Add Blamer interface in ui/, wired via ModelConfig from cmd/main.go
- Add blameGutter rendering in diffview.go alongside lineNumGutter
- Extract shared gutter helpers (lineGutters, gutterExtra, gutterBlanks,
  applyHorizontalScroll) to reduce duplication across render paths
- Add ActionToggleBlame keymap action bound to B
- Update status bar mode icons to show @ when blame is active
- Update README, docs, site, CLAUDE.md, and plugin references

* Fix all golangci-lint warnings

- Extract writeStagedBlameFile() to reduce nestif complexity in blame.go
- Combine param types in FileBlame signature (gocritic)
- Handle errcheck/gosec on temp file cleanup paths
- Extract deletePlaceholderVisualHeight() to reduce nestif in annotate.go
- Remove stale //nolint:gosec directives from diff.go and diff_test.go
- Pre-allocate slices in directory.go, fallback.go, theme.go, filetree.go,
  mdtoc.go, and model_test.go; preserve nil-vs-empty semantics in fallback.go

* Fix blame review follow-ups and persist view modes

* Fix blameTargetRef to handle triple-dot ref syntax

strings.Split("A...B", "..") produced ".B" which caused git blame
to fail or blame the wrong revision. Use strings.Cut and check for
"..." before ".." to correctly extract the target ref.

* Restore //nolint:gosec for internal exec.Command calls

CI golangci-lint flags G204 on exec.Command with variable args.
These are safe — git args are constructed internally, not from user input.

233 of 304 new or added lines in 13 files covered. (76.64%)

4080 of 4475 relevant lines covered (91.17%)

469.19 hits per line