Fix spinner flicker in embedded terminals - #42

Merged

nodeselector merged 18 commits into

mainfrom

nodeselector/fix-spinner-flicker

Jun 12, 2026

Commits on Jun 12, 2026

ui: buffer spinner + worker rows into single synchronized writes
nodeselector
committed
0aa8e41
ui: erase spinner line before redraw, hide cursor during animation
nodeselector
committed
f0d8aa8
ui: fix data race on spinner Suffix between label updates and render goroutine
nodeselector
committed
e2e9fe5
ui: erase worker rows immediately on ClearWorkerStatuses instead of lazily
nodeselector
committed
a8ddcc9
ui: static glyph-only top line, all detail in worker rows
nodeselector
committed
58daf3b
ui: align spinner with cli/cli — label left, glyph right, race-free
nodeselector
committed
1bab682
ui: static spinner label — set once in StartProgress, never changed
nodeselector
committed
5f904fe
ui: skip worker row render on spinner erase writes
nodeselector
committed
fda2ea3
ui: move worker row glyph to right, matching main spinner style
nodeselector
committed
c2182f1
address Copilot review: tighten erase detection, lock cursor escapes, drop []byte allocs
nodeselector
committed
c6a7637
support GH_ACTIONS_PIN_WORKFLOWS_DIR for non-standard workflow locations
nodeselector
committed
df0255d
remove duplicate 'All valid' message from terminal report
nodeselector
committed
21e7a05
remove duplicate SSO URL from --no-fix path
nodeselector
committed
4d00102
pin summary: handle zero-workflow case gracefully
nodeselector
committed
833e593
TermCaution: use yellow instead of red
nodeselector
committed
be3ca41
resolution record path: use TermDetail for better visibility
nodeselector
committed
58050c6
add scenario and unit test coverage for CLI UX fixes
nodeselector
committed
090b047
fix isErase detection (wrong byte lengths), clear slot 0 on empty detail
nodeselector
committed
04d160a

Navigation Menu

Fix spinner flicker in embedded terminals - #42

Fix spinner flicker in embedded terminals#42
nodeselector merged 18 commits into
mainfrom
nodeselector/fix-spinner-flicker

Commits on Jun 12, 2026

ui: buffer spinner + worker rows into single synchronized writes

ui: erase spinner line before redraw, hide cursor during animation

ui: fix data race on spinner Suffix between label updates and render goroutine

ui: erase worker rows immediately on ClearWorkerStatuses instead of lazily

ui: static glyph-only top line, all detail in worker rows

ui: align spinner with cli/cli — label left, glyph right, race-free

ui: static spinner label — set once in StartProgress, never changed

ui: skip worker row render on spinner erase writes

ui: move worker row glyph to right, matching main spinner style

address Copilot review: tighten erase detection, lock cursor escapes, drop []byte allocs

support GH_ACTIONS_PIN_WORKFLOWS_DIR for non-standard workflow locations

remove duplicate 'All valid' message from terminal report

remove duplicate SSO URL from --no-fix path

pin summary: handle zero-workflow case gracefully

TermCaution: use yellow instead of red

resolution record path: use TermDetail for better visibility

add scenario and unit test coverage for CLI UX fixes

fix isErase detection (wrong byte lengths), clear slot 0 on empty detail