terminal-pilot: screenshot serialization retains inverse after SGR 27

[kamilio]

Summary

terminal-pilot's screenshot serialization retains inverse styling after the PTY
explicitly disables it with SGR 27. Reproduced September 5, 2026 with the installed
terminal-pilot 0.0.1 package, Node 22.22.0, Linux. This reproduces without any
application UI library or framework.

Reproduction

Save this as /tmp/terminal-pilot-ansi.mjs:

process.stdout.write("\x1b[31mRED\x1b[0m  \x1b[36mCYAN\x1b[0m  plain\r\n\x1b[7;37m INVERSE \x1b[27;36m NORMAL CYAN \x1b[0m DEFAULT \r\n");
setInterval(() => {}, 1000);

terminal-pilot create-session --session ansi-reset --cwd /tmp --cols 64 --rows 12 node /tmp/terminal-pilot-ansi.mjs
terminal-pilot screenshot --session ansi-reset -o /tmp/ansi-reset.png
terminal-pilot close-session --session ansi-reset

Expected: only INVERSE has an inverted background. NORMAL CYAN should be
cyan text on the default dark background, since SGR 27 disabled inverse before
the text was written. DEFAULT should have default styling.

Actual: NORMAL CYAN has a cyan filled background with dark text. The preceding
inverse attribute survives until the later SGR 0. The top-row RED/CYAN controls
render correctly, so this is not a general color-support or font issue.

Likely boundary and impact

The installed terminal buffer's full-style serialization omits disabled flags
when switching cells. Emitting the next cell's enabled style without resetting
the prior style loses inverse-off (and may lose other attribute-off transitions).
Please verify TerminalBuffer.renderLine() and serializeStyleState() in the
source version corresponding to the distributed package.

This makes screenshot-based TUI QA show false selection/highlight remnants. It
should not be addressed by changing application output that already uses valid
SGR sequences.

Suggested coverage

• Round-trip cells with inverse-on → inverse-off plus a foreground change.
• Cover bold/dim/underline disable transitions and explicit default colors.
• Assert serialized raw lines reconstruct the same styles as emulator cells.
• Screenshot the minimal fixture and verify the normal segment's background.

The reproduction PNG was captured and visually inspected; no private repository
content is needed to reproduce this defect.

[kamilio]

Validated on current main 848ca91: the real PTY/public SDK reproduces the cyan filled background after SGR 27, and 13 initial in-memory serialization regressions fail before the fix. I am handling this issue. The change resets the previous full style before emitting a different complete cell style; application output and SGR parsing are unchanged. The terminal-pilot package cohort now passes 253 tests. Completing full workspace gates and after-fix visual verification before pushing; I will close this issue immediately after verified remote-main delivery.

[kamilio]

Delivered to origin/main at ac6c3a0; remote SHA verified before this immediate post-push closure. The actual PTY/public SDK reproduced the reported cyan-background leakage. renderLine now resets a previous nonempty style before emitting a different complete cell style, preventing disabled attributes from leaking. Application output and SGR parsing are unchanged. Added explicit cell-style and round-trip/raw-line controls; all 253 terminal-pilot tests pass. Normal full build/npm test/lint/typechecks passed on 520bb4e and merged 96bf3fd. Concurrent upstream SafeJS commits were preserved, not overwritten; the final integration passed a fresh normal build, all 10,505 SafeJS/terminal-pilot cohort tests, root/package/workflow lint and full maintained typechecking. Full final-tree CI is monitored with release publication. Before/after screenshots from the identical real-PTY fixture confirm NORMAL CYAN now has cyan text on the default background, with INVERSE and top-row controls unchanged; both sessions were closed. A separate pre-existing standalone zero-red truecolor parser edge is documented, not claimed fixed. The seven-day kamilio-only monitor remains active.

[kamilio]

Release verification complete: terminal-pilot and terminal-pilot-mcp 0.0.56 published from ac6c3a0 (run 33941297695), and both registry gitHead values match. A fresh integrity-checked terminal-pilot tarball passes explicit style round-tripping and a real-PTY/public-SDK screenshot after building its required node-pty native dependency; the published image was visually inspected. The original root run was cancelled by a concurrent main push, not a failed test. Successor cad3dee is a verified descendant with no terminal-pilot changes; its complete CI passed and actually published poe-code 14.0.43 (run 33941378321). Registry gitHead matches that successor. Issue closure occurred immediately after verified delivery, before either release. Seven-day author-filtered monitoring continues.