umputun / ralphex / 27316234290

84%

master: 84%

Build Type

Pull #383

github

Committed by vlondon

Commit Message

fix: emit task_end boundaries on progress-file parse paths so plan panel marks finished tasks done

the web dashboard plan panel never marked previous tasks as completed
while a run progressed - only the active task was highlighted, and
earlier tasks reverted to pending until a manual page refresh re-read
the plan file with checked boxes.

root cause: only BroadcastLogger (in-process runs) emitted task_end
events. the progress-file parse paths - loadProgressFileIntoSession
(replay on page load) and the Tailer (watch-mode live tail) - emitted
task_start without ever ending the previous task, so the frontend's
clearActiveTasksExcept demoted finished tasks back to pending. the
non-deferred Tailer path (parseLine) emitted no task boundaries at all,
so the active highlight never advanced live.

changes:
- buildPendingSectionEvents tracks the current task and emits task_end
  before the next task_start or on a phase transition away from the
  task phase, mirroring BroadcastLogger semantics
- Tailer.parseLine routes section lines through the same builder so the
  live tail path gets task boundaries too
- the active task number is carried across tailer restarts (Tailer
  CurrentTask / TailerConfig.InitialTask / Session.lastTask), matching
  the existing lastPhase/pendingSection carryover
- frontend: clearActiveTasksExcept marks a lower-numbered previously
  active task as done instead of demoting it to pending
- frontend: the plan panel scrolls the newly active task into view

fixes #380

Pull Request Pull Request #383: fix: plan panel marks finished tasks done and scrolls to the active task

Coverage Stats

76 of 79 new or added lines in 3 files covered. (96.2%)

2 existing lines in 2 files now uncovered.

7721 of 9240 relevant lines covered (83.56%)

227.28 hits per line