umputun / revdiff / 24578021559 / 1
90%
master: 90%

DEFAULT BRANCH: master
Ran
Files 57
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 88.362% (+0.02%) from 88.345%
24578021559.1

push

github

web-flow 
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.

7031 of 7957 relevant lines covered (88.36%)

330.68 hits per line

Source Files on job 24578021559.1