umputun / revdiff / 24684143206

90%

push

github

Eager commit-log fetch — fixes #122 (stale commit overlay vs current diff) (#129)

* add plan: eager-commit-fetch

* feat: add loadCommits cmd and commitsLoadedMsg type

Introduce the async commit-log load primitive mirroring loadFiles:
- commitsLoadedMsg carries seq/list/err/truncated
- commitsState gains loadSeq for stale-result guard
- loadCommits() returns a tea.Cmd when applicable, nil otherwise

Task 1 of the eager-commit-fetch plan; no behavior change yet —
callers still go through the lazy ensureCommitsLoaded path.

* feat: add handleCommitsLoaded and wire into Update

Handler drops stale results via seq-guard (mirrors handleFilesLoaded) and
caches errors with loaded=true so the overlay surfaces the failure once
instead of re-triggering the fetch.

Related to #122

* feat: wire loadCommits into Init and triggerReload

Init now returns tea.Batch(loadFiles, loadCommits) so the commit log is
fetched in parallel with files on startup. triggerReload bumps
commits.loadSeq and re-fetches commits alongside files so the overlay
stays consistent with the diff after R reload.

* feat: delete ensureCommitsLoaded and simplify handleCommitInfo

remove the lazy commit-fetch path entirely now that eager parallel fetch
is wired in. handleCommitInfo now reads from the cache populated at
startup/reload and shows a transient "loading commits…" hint when the
fetch has not yet landed.

* feat: validate eager-commit-fetch against full test/lint/build gate

* docs: update CLAUDE.md and ARCHITECTURE.md for eager commit fetch

Replaced the lazy fetch description in CLAUDE.md Gotchas with the eager
parallel fetch pattern (Init/triggerReload tea.Batch, seq-guard, loading
hint). Updated the ARCHITECTURE.md overlay flow to describe loadCommits
and handleCommitsLoaded instead of the deleted ensureCommitsLoaded. Moved
the completed plan to docs/plans/completed/.

* fix: align commitsLoadedMsg field doc style with filesLoadedMsg

Move the seq-contract explanation from type go... (continued)

36 of 36 new or added lines in 2 files covered. (100.0%)

7643 of 8573 relevant lines covered (89.15%)

470.55 hits per line