24853784489

Committed 23 Apr 2026 07:09PM UTC coverage: 89.656% (+0.3%) from 89.355%

Build # 24853784489

Build Type

Pull #147

github

Committed by

umputun

Commit Message

fix: address Copilot review findings on vim-motion preset

- center/top/bottom viewport align helpers now SetContent before SetYOffset,
  matching syncViewportToCursor. viewport.SetYOffset clamps to current content
  length; rendering first avoids mis-clamping when the cursor move mutated
  render height (wrap/annotation rows).
- jumpToLineN nudges off ChangeDivider rows after clamping. With the trailing
  divider feature shipped on master, G/NG could land the cursor on a divider
  row where no action applies. Now the cursor slides back (then forward) to
  the nearest non-divider line.
- fix REVDIFF_VIM_MOTION env var name in CLAUDE.md and the completed plan
  file — the actual go-flags tag in app/config.go is REVDIFF_VIM_MOTION to
  match the revdiff env-var convention.
- repeatDiffAction precomputes hunks once and delegates to new
  moveDiffCursorDownWithHunks / moveDiffCursorUpWithHunks variants, avoiding
  the O(N × len(diff)) rescan that would hang the UI on large counts
  (e.g. 9999j on a 10000-line diff).

Addresses Copilot inline findings on #147.

Pull Request Pull Request #147: feat: vim-motion preset (--vim-motion, off by default)

Coverage Stats

177 of 188 new or added lines in 9 files covered. (94.15%)

30 existing lines in 2 files now uncovered.

8347 of 9310 relevant lines covered (89.66%)

558.28 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

91.64

/app/diff/diff.go

Build SHA Not Found

The commit SHA "4c70c002a583667b19b6fe68c7ec2dda8cd6c6be" was not found in your repository, so the file cannot be loaded. This may be because you posted from a local development environment, or your CI created an ephemeral commit.

umputun / revdiff / 24853784489

Source File Press 'n' to go to next uncovered line, 'b' for previous

Build SHA Not Found

Source File
Press 'n' to go to next uncovered line, 'b' for previous