Skip to content

Show diff load progress and stop delta on leave - #10

Merged
skunkworker merged 1 commit into
masterfrom
jb/2026_10_11_diff_progress
Oct 11, 2026
Merged

skunkworker merged 1 commit into
masterfrom
jb/2026_10_11_diff_progress

Conversation

@skunkworker

Copy link
Copy Markdown
Owner

Summary

  • v on a large diff showed a still "loading diff…" line for about 2s while delta ran. It looked frozen.
  • The read and the layout are now two steps. The loading line shows a spinner, the step, the line count, and the time: ⠹ delta is styling 121,800 lines… 1.4s.
  • Leaving the diff, or starting another layout, now kills the running delta. Before, it ran to the end on one CPU core.

Test plan

  • New tests: TestLeavingDiffStopsDelta (fails without the fix), TestDiffSpinnerTicksWhileLoading, TestGroupDigits.
  • go test -race passes.
  • Checked in the real app on a 121,800-line diff: the spinner turns, and no delta process is left after q.

🤖 Generated with Claude Code

A large diff spent about 2s in delta behind a still "loading diff…"
line, which looked like a frozen screen. The read and the layout are
now two steps, so the loading line shows a spinner, the step, the
line count, and the time so far.

Leaving the diff, or starting another layout, now kills the running
delta. Before, its answer was dropped but it ran to the end on one
CPU core.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@skunkworker
skunkworker merged commit 3832584 into master Oct 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant