jedib0t / go-pretty / 32122994713

100%

main: 100%

push

github

text: stop re-opening closed escape sequences on wrapped lines

When wrapping text whose escape sequence was closed mid-string (e.g. a dim
segment followed by a reset and plain text), wrapHard/wrapSoft kept the stale
open sequence in lastSeenEscSeq: EscSeqParser.ParseString returns "" once all
codes are closed, but the 'if openEscSeq != ""' guard never let that clear the
variable. Every subsequent wrapped line was then re-opened with the old
formatting, so e.g. one dim (SGR 2) segment dimmed the rest of the wrapped
block.

Track the sequence state explicitly on both sides of each word: the state
before the word applies to wraps around/inside it (a word like "Snow\x1b[0m"
that closes its own sequence must still re-open the color for its wrapped
head), and the state after the word applies to any wraps that follow it.

Added regression tests for WrapHard, WrapSoft, and WrapText covering a
dim-then-reset string; the continuation lines after the reset now come out
unstyled.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VycbcuWhTfRp43tfdQYtmR

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

5161 of 5161 relevant lines covered (100.0%)

1.21 hits per line