jhheider / penknife / 29282998591

76%

Build Type

push

github

Committed by web-flow

Commit Message

feat: draw a dua-style scanning indicator during the startup scan (#47)

Now that the refresh pipeline runs off the render thread, move the
*initial* scan onto it too so a large (e.g. cloud-synced) root no longer
blanks the first frame. App::new starts with an empty tree and kicks off
`Refresh::Initial`; the first frame draws immediately and the tree pane
shows a centered spinner, "Scanning <root>…", and a live file count until
the scan lands. The counter is fed by `ScanProgress` events the scanner
emits every 256 files via a new `scan_directory_with_progress`.

- `start_refresh` now takes a `Refresh` enum (Initial / Sweep / User)
  instead of three positional booleans, so each call site's intent reads
  clearly and the progress/change-gating knobs live in one place.
- The periodic sweep and hydration hold off while `scanning` so they can't
  supersede the initial scan (which would leave the indicator stuck).
- Spinner frames come from `glyphs::spinner_frame`, ASCII-safe (`|/-\`)
  under the ASCII profile, braille otherwise - one column wide either way.

Verified in a real terminal: the "Scanning shared…" indicator shows, then
the tree populates (Files (1394)) via the async scan, and q quits.


Claude-Session: https://claude.ai/code/session_01PykYTubrtTh4te2xg71JJS

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

133 of 175 new or added lines in 9 files covered. (76.0%)

5 existing lines in 3 files now uncovered.

7361 of 9661 relevant lines covered (76.19%)

27.07 hits per line