fix(tui_gateway): strip ANSI from slash-worker output for desktop chat
Desktop chat bubbles render plain text, but a worker-routed command that builds its own Rich Console (e.g. /journey) picks up truecolor from the gateway's inherited COLORTERM and leaks raw escapes into the bubble. Strip ANSI at the single worker-return choke point so every command renders cleanly. The TUI opens /journey as an overlay, so it never travels this path.
This commit is contained in:
parent
428b9a0c42
commit
89cf65ab63
2 changed files with 31 additions and 1 deletions
23
tests/tui_gateway/test_slash_worker_ansi.py
Normal file
23
tests/tui_gateway/test_slash_worker_ansi.py
Normal file
|
|
@ -0,0 +1,23 @@
|
|||
"""The slash worker feeds desktop chat bubbles, which render plain text — so
|
||||
any ANSI a worker-routed command emits (e.g. /journey's own Rich Console) must
|
||||
be stripped from the worker's return value."""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
|
||||
class _FakeCLI:
|
||||
console = None
|
||||
|
||||
def process_command(self, cmd: str) -> None:
|
||||
import sys
|
||||
|
||||
sys.stdout.write("\x1b[38;2;1;2;3mcolored\x1b[0m plain")
|
||||
|
||||
|
||||
def test_run_strips_ansi_from_output():
|
||||
from tui_gateway import slash_worker
|
||||
|
||||
out = slash_worker._run(_FakeCLI(), "/anything")
|
||||
|
||||
assert "\x1b[" not in out
|
||||
assert out == "colored plain"
|
||||
|
|
@ -102,7 +102,14 @@ def _run(cli: HermesCLI, command: str) -> str:
|
|||
if old is not None:
|
||||
cli_mod._cprint = old
|
||||
|
||||
return buf.getvalue().rstrip()
|
||||
# Desktop chat bubbles render plain text, not ANSI. A worker-routed command
|
||||
# that emits Rich color (e.g. /journey building its own Console, which picks
|
||||
# up truecolor from the gateway's inherited COLORTERM) would otherwise leak
|
||||
# raw escapes; strip them at the single choke point. (The TUI opens /journey
|
||||
# as an overlay, so it never travels this path.)
|
||||
from tools.ansi_strip import strip_ansi
|
||||
|
||||
return strip_ansi(buf.getvalue().rstrip())
|
||||
|
||||
|
||||
def main():
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue