umputun / revdiff

93%

master: 89%

LAST BUILD ON BRANCH status-bar-line-number

branch: status-bar-line-number

CHANGE BRANCH

x

Reset

Sync Branches

• status-bar-line-number
• add-collapsed-flag
• all-files-mode
• annotation-list-popup
• ci-shellcheck-yaml
• code-smells-cleanup
• codex-plugin
• collapsed-diff-mode
• commit-info-popup
• community-themes
• custom-keybindings
• daulet/new_file
• daulet/word_highlight
• dependabot/go_modules/github.com/charmbracelet/x/ansi-0.11.7
• dependabot/go_modules/github.com/mattn/go-runewidth-0.0.23
• diff-search
• discard-quit
• docs/pi-package-review-fixes
• external-editor-annotations
• feat/cmux-support
• feat/history-skill-fallback
• feat/jj-support
• feat/launch-script-improvements
• feat/one-light-chroma-style
• feat/opencode-config-docs
• feat/reviewed-files-and-status-indicators
• feat/scroll-indicators
• feat/stdin-scratch-buffer
• feat/toggle-tree-pane
• feat/word-diff-highlighting
• feat/word-diff-opt-in
• feat/zed-revdiff-task
• feature/binary-file-detection
• feature/layout-agnostic-keys
• feature/layout-agnostic-keys-v2
• feature/new-file-and-untracked-support
• feature/range-hunk-annotations
• fix-ctrl-du-half-page
• fix-file-annotation-bugs
• fix-initial-loading-flash
• fix-tmux-title-compat
• fix-wrap-right-padding
• fix/center-hunk-in-viewport
• fix/cmux-send-stdout-leak
• fix/diff-viewport-visual-height-scroll
• fix/ghostty-improvements
• fix/launcher-argument-quoting
• fix/overlay-path-inheritance
• fix/staged-only-empty-diff
• fix/subdirectory-support
• global-hunk-navigation
• hunk-keyword-expansion
• include-filter
• intraline-word-diff
• iterm2-osascript
• line-numbers
• markdown-toc-navigation
• master
• mercurial-support
• move-code-to-app
• only-filter
• overlay-extraction
• refs/tags/v0.1.0
• refs/tags/v0.1.1
• refs/tags/v0.10.0
• refs/tags/v0.11.0
• refs/tags/v0.12.0
• refs/tags/v0.13.0
• refs/tags/v0.14.0
• refs/tags/v0.14.1
• refs/tags/v0.15.0
• refs/tags/v0.15.1
• refs/tags/v0.15.2
• refs/tags/v0.15.3
• refs/tags/v0.16.0
• refs/tags/v0.16.1
• refs/tags/v0.17.0
• refs/tags/v0.18.0
• refs/tags/v0.18.1
• refs/tags/v0.19.0
• refs/tags/v0.19.1
• refs/tags/v0.19.2
• refs/tags/v0.2.0
• refs/tags/v0.2.1
• refs/tags/v0.2.2
• refs/tags/v0.2.3
• refs/tags/v0.2.4
• refs/tags/v0.3.0
• refs/tags/v0.4.0
• refs/tags/v0.4.1
• refs/tags/v0.4.2
• refs/tags/v0.5.0
• refs/tags/v0.6.0
• refs/tags/v0.7.0
• refs/tags/v0.7.1
• refs/tags/v0.7.2
• refs/tags/v0.8.0
• refs/tags/v0.9.0
• review-findings
• review-history
• sidepane-extraction
• single-column-line-numbers-full-context
• single-file-mode
• skill-opencode-timeout-fix
• skill-scripts-hg-jj
• skill-use-claude-skill-dir
• status-line-help-overlay
• structural-refactor
• style-extraction
• theme-support
• two-ref-diff
• ui-package-split
• word-wrap
• worddiff-extraction
• worktree-bugfix
• worktree-no-git-file-review

Build # 24008374759

Build Type

Pull #23

github

Committed by shamanod

Commit Message

fix: use dynamic denominator in line number segment based on change type

The denominator in L:42/380 was using max(OldNum, NewNum) across all
diff lines, which could show the old file length when lines were removed
from the end. Now uses maxOld on removed lines and maxNew on
context/added lines, so the denominator always matches the file side
the cursor is on.

Note: revdiff uses -U1000000 (full file context), so the denominator
always covers the entire file — but the old/new distinction still
matters when lines are added or removed.

Also fixes 3x testifylint: assert.Empty instead of assert.Equal(t, "").

Pull Request Pull Request #23: feat(ui): show current line number in status bar

Coverage Stats

34 of 38 new or added lines in 1 file covered. (89.47%)

3 existing lines in 2 files now uncovered.

3216 of 3465 relevant lines covered (92.81%)

479.03 hits per line