fix: strip all ANSI escape sequences in terminal command output - #838
fix: strip all ANSI escape sequences in terminal command output#838OmarAlaaeldein wants to merge 1 commit into
Conversation
|
Thanks for digging into the TUI corruption issue - the root cause analysis (control/cursor codes bleeding through A few concerns before this could be ported as-is:
Splitting this into the ANSI-stripping fix (clearly justified) and separately proposing the |
|
Marking this stale - there has been no activity here for 50 days. It will close in 7 days unless someone comments. This is backlog upkeep, not a verdict on the pull request. A single comment keeps it open, and anything closed this way can be reopened. |
e54d409 to
00ee8c9
Compare
|
Rebased onto current main and narrowed this PR to the ANSI-stripping fix only. It now uses the existing stripAnsi utility in the bounded output buffer, removes the TERM=dumb and SUDO_PROMPT changes, and does not strip bare carriage returns. Added a regression test for cursor-control sequences. Verified with: bun test sdk/src/tests/run-terminal-command.test.ts (15 passing). |
00ee8c9 to
683e277
Compare
This PR fixes TUI display and scroll corruption caused by VT/ANSI control sequences in captured terminal-command output.