umputun / revdiff / 24853784489

90%

master: 90%

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