dcdpr / jp / 27491626702

67%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(md): ANSI tokenizer, fix compound-escape wrapping (#744)

Several related cleanups and two bug fixes in `jp_md`.

**Unified ANSI tokenizer**

`ansi::segments()` replaces three independent hand-rolled escape
scanners
in `ansi.rs`, `table.rs` (both `wrap_to_visual_width` and
`hard_break_into`). Every escape-aware routine now shares a single
termination rule so they cannot drift against each other.

**`RenderOptions` struct**

`format_terminal`, `TerminalFormatter::new`, `format_table`, and the
internal `extract_table`/`render_cell_content` helpers previously
accepted six to eight individual parameters. They all now receive a
`RenderOptions<'_>` value. Cell rendering derives a child options value
(`width: 0, indent: 0`) while inheriting theme, backgrounds, and HR
style, which makes the "cells disable wrapping" invariant explicit.

**`ListState` struct**

`State::InList` carried six inline fields. They are collapsed into a
`ListState` value so the list handlers (`handle_in_list`,
`flush_list_segment`, `classify_list_line`, etc.) take one argument
instead of six. A new `render_list_segment` helper is now shared between
the streaming flush path and the new end-of-region flush path, ensuring
both agree on stripping and renumbering.

**`Fixups` replaces `FixupChain`**

`FixupChain<I>` wrapped an iterator and was awkward to compose.
`Fixups` is an iterator-agnostic set with a single `apply(event)`
method, used as `iter.filter_map(|e| fixups.apply(e))`.
`Fixups::llm_quirks()` is a convenience constructor for the standard
LLM-output fixup set.

**`HtmlTerminator` enum**

`HtmlBlockRule::terminator()` now returns a typed `HtmlTerminator`
(`Tag(&str)` or `BlankLine`) instead of a raw string sentinel. The
handler matches on the variant directly.

**Bug fix: compound escape sequences lost on wrap**

When a line wraps, `TerminalWriter` records a restore escape that
concatenates one escape per active attribute. The code passed that
compound string to `AnsiSta... (continued)

Coverage Stats

289 of 302 new or added lines in 10 files covered. (95.7%)

9 existing lines in 3 files now uncovered.

34468 of 51838 relevant lines covered (66.49%)

225.41 hits per line