dcdpr / jp / 27700064723

67%

push

github

fix(cli, md): Preserve trailing separator after terminal tight list (#756)

When a streamed message ended with a tight list and was then flushed
(e.g. because a tool call followed), the trailing blank-line separator
was swallowed. The tight-list suppression in `jp_md` is intentional for
mid-stream items — it keeps list items visually tight while more content
is still arriving — but it should not apply to the *final* block of a
content region.

The fix threads a `force_trailing_separator` flag through
`TerminalOptions` in `jp_md`. `print_block` in the chat renderer now
distinguishes between `Event::Block` (mid-stream, passes `false`) and
`Event::Flush` (terminal, passes `true`), so only the end-of-region
drain forces the separator back on. Reasoning blocks are excluded
because they already defer their separator via
`suppress_trailing_separator`.

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

7 of 7 new or added lines in 2 files covered. (100.0%)

35470 of 53279 relevant lines covered (66.57%)

384.25 hits per line