umputun / revdiff / 24578021559

90%

push

github

fix(ui): suppress initial loading flash and drop stale file-list responses (#117)

* fix(ui): suppress initial loading flash and drop stale file-list responses

Bubble Tea delivered the first WindowSizeMsg before filesLoadedMsg, so between
ready=true and the async ChangedFiles returning, View() painted the populated
two-pane layout with an empty tree and "no file selected" — a 100-500 ms flash
on larger repos.

Gate View() on a new filesLoaded flag: show "loading..." while !ready, then
"loading files..." while ready && !filesLoaded. handleFilesLoaded flips the flag
first thing, including on error, so the loading screen always exits.

Also tag filesLoadedMsg with a seq captured at load time and drop mismatched
messages in handleFilesLoaded. toggleUntracked (and any future reload site) bumps
filesLoadSeq so an older in-flight ChangedFiles can't overwrite the tree after
the user toggles state.

* docs(arch): clarify seq-check behavior in file loading diagram

Address Copilot review feedback on PR #117. The previous wording said filesLoaded
flips "on every filesLoadedMsg", which is inaccurate after the seq-check was
added — stale messages are dropped and do not flip the flag. The diagram also
coupled tree.Rebuild with the flag flip, but the flag is set before the error
check while Rebuild only runs on non-error accepted messages.

13 of 14 new or added lines in 3 files covered. (92.86%)

7031 of 7957 relevant lines covered (88.36%)

330.68 hits per line