Repository navigation
External "git" is still called multiple times for the same staged file #530
Description
It seems when staging a single file, the external "git" command is called multiple times with same parameter fast in sequence:
The expectation is that the command is only called once, but not multiple times.
I'll debug this issues...
Reacted by Hannes Achleitner
Activity
SMA-HP commentedon Mar 7, 2026
I found first issue of this: #531
But there is still more... I'll check....
hannesa2 commentedon Mar 7, 2026
Great, then I wait for a new release ....
hannesa2 commentedon Mar 17, 2026
Is this now solved with https://github.com/gitx/gitx/releases/tag/1.4 ?
Publish coherent index snapshots and reset paths from parent
Coalesce stage diff rendering after index updates
test: ratchet staging index coverage
Report coherent index snapshot counts accurately
test: ratchet verified staging coverage
Document runtime logging verification policy
goneng commentedon Sep 7, 2026
Partly fixed.
The cause SMA-HP found in #531 has been fixed since 1.4, and the follow-ups
(refresh race, update-index on every FSEvents notification, coalescing overlapping refreshes)
all shipped in 1.5.5, so the process storm is gone and that release is worth testing.
Measured on 1.5.5: staging one file runs update-index once plus two index refresh rounds,
and repeating it gives identical counts, so nothing accumulates any more.
What is left is the command in the screenshot above.
diff-files -U3for the selected file still runs 3 times per stage, down from 9,because
Resources/html/views/commit/commit.jsre-asks for the diff onevery re-render of the commit view with no caching.
Selecting a file without staging costs exactly 1.
Happy to track that separately if you would rather close this one.
Fix External git diff-files being called repeatedly for the same file…
Fix External git diff-files being called repeatedly for the same file…
Fix External git diff-files being called repeatedly for the same file…
Merge pull request #599 from goneng/fix-repeated-diff-on-stage