dcdpr / jp / 26624962241

66%

Build Type

push

github

Committed by web-flow

Commit Message

fix(cli): Fix indentation and trailing blank in fenced code inside lists (#681)

Two bugs affected the rendering of fenced code blocks that appear inside
list items.

First, `indent_lines` was treating ANSI reset sequences (`\x1b[0m`)
emitted by syntect *after* a trailing `\n` as the start of a new visible
line. This caused the indent prefix to be applied twice, pushing all
syntax-highlighted code content `indent` columns too far to the right.

Second, the `FencedCodeEnd` handler was unconditionally appending a
blank-line separator after every closing fence. At top level this is
correct behaviour — it keeps the conventional gap between a fence and
the next block — but inside a list item it injected a spurious blank
line that broke the visual flow of the surrounding list.

The fix updates `indent_lines` to skip over CSI/SGR escape sequences
when deciding where to emit the prefix: the prefix is now inserted only
before the first *visible* character on a new line. The `FencedCodeEnd`
handler is updated to call `render_code_fence` (replacing the removed
`render_closing_fence`) and to append the separator only when `indent ==
0`.

---------

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

Coverage Stats

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

9 existing lines in 1 file now uncovered.

31794 of 48120 relevant lines covered (66.07%)

226.43 hits per line