nagayon-935 / mping / 23639916047

83%

push

github

fix(ui): close appStop before app.Stop() to eliminate screen-teardown race

The previous fix closed appStop after app.Run() returned, but the refresh
goroutine could still call app.QueueUpdateDraw during the window between
app.Stop() and app.Run() returning (while screen.Fini() runs).

In port mode this race was more likely because port status changes triggered
appendErrorLog → errorView.SetChangedFunc → app.Draw() on every interval.

Fix: use sync.Once so closeAppStop() is idempotent, call it from the 'q'
handler before app.Stop(), and keep a fallback call after app.Run() returns
for non-interactive exits.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

5 of 5 new or added lines in 1 file covered. (100.0%)

2568 of 3095 relevant lines covered (82.97%)

0.97 hits per line