dcdpr / jp / 28665285449

69%

Build Type

push

github

Committed by web-flow

Commit Message

fix(cli): Keep waiting indicator alive through keep-alives (#836)

The waiting indicator no longer disappears the moment any provider event
arrives. Previously a single SSE keep-alive ping tore down the
indicator, leaving the terminal blank until the first real token
arrived; a user watching the indicator vanish and then nothing happen
could reasonably conclude JP had crashed. The indicator now survives
non-rendering events (keep-alives, history patches, part-less flushes)
and only finishes on an event that is about to write to the terminal.

The indicator also gains a status detail shown alongside the elapsed
time, e.g. `⏱ Waiting… 9.2s (receiving response data)`, updated as the
turn loop moves through sending the request, waiting for the first
tokens, and receiving response data. `LineTimer` replaces the raw
`(CancellationToken, JoinHandle)` pair used by the lock-wait, drain, and
reasoning timers, exposing `set_status` and an async `finish` that
awaits the line-clear before persistent output is rendered.

Adds RFD 091, proposing a printer-owned status line that consolidates
this and six other bespoke timer/temp-line mechanisms in `jp_cli` behind
one primitive in `jp_printer`, removing the per-site clear-before-write
discipline this fix still relies on. RFD 048 is updated to list RFD 091
as an extension.

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Coverage Stats

53 of 66 new or added lines in 5 files covered. (80.3%)

1 existing line in 1 file now uncovered.

38181 of 56499 relevant lines covered (67.58%)

38675.04 hits per line