Skip to content

fix(chat): render builtin command output safely - #8

Merged
devswha merged 1 commit into
mainfrom
fix/builtin-output-boundary
Aug 16, 2026
Merged

fix(chat): render builtin command output safely#8
devswha merged 1 commit into
mainfrom
fix/builtin-output-boundary

Conversation

@devswha

@devswha devswha commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • normalize SDK builtin stdout with Node’s maintained VT-control stripper
  • emit builtin output as tagged completed plain-text messages so realtime state preserves provenance
  • render tagged output as escaped preformatted text instead of Markdown/KaTeX
  • preserve trusted requested export paths when upstream diagnostics contain corrupted replacement characters

Verification

  • focused Bun SDK contract: 1 passed
  • MessageComponent regressions: 3 passed
  • ESLint on all changed files
  • full client/server typecheck

Closes #5

@devswha
devswha force-pushed the fix/builtin-output-boundary branch from 38addd4 to 86eaff4 Compare August 16, 2026 19:08
@devswha
devswha force-pushed the fix/builtin-output-boundary branch from 86eaff4 to 4df3bbd Compare August 16, 2026 19:11
@devswha
devswha merged commit e2f7967 into main Aug 16, 2026
2 checks passed
@devswha
devswha deleted the fix/builtin-output-boundary branch August 16, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Builtin command output leaks ANSI escapes and corrupts structured text in chat

1 participant