1
90%
master: 90%

Ran 23 Apr 2026 07:10PM UTC

Files 60

Run time 3s

Badge

Embed ▾

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

Job # 24853784489.1

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

8347 of 9310 relevant lines covered (89.66%)

558.28 hits per line

umputun / revdiff / 24853784489 / 1
90%
master: 90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24853784489.1

umputun / revdiff / 24853784489 / 1 90% master: 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24853784489.1

umputun / revdiff / 24853784489 / 1
90%
master: 90%

README BADGES
x