Skip to content

fix(tui): disable OSC 66 text-sizing on VTE-based terminals to prevent garbled output - #1098

Open
qiaozongming wants to merge 1 commit into
XiaomiMiMo:mainfrom
qiaozongming:reopen/246
Open

fix(tui): disable OSC 66 text-sizing on VTE-based terminals to prevent garbled output#1098
qiaozongming wants to merge 1 commit into
XiaomiMiMo:mainfrom
qiaozongming:reopen/246

Conversation

@qiaozongming

Copy link
Copy Markdown
Collaborator

Resubmitting community contribution; the original PR (#246) was auto-closed by a branch history rewrite.

@qiaoshi-dot

Copy link
Copy Markdown

Opened a refreshed PR against current main: #1806.

It carries the VTE / OSC 66 workaround from this PR, updates the test location to the current tree, and resolves the merge conflict that this PR now shows.

Local validation from packages/opencode:

  • bun test test/terminal.test.ts — 16 pass / 0 fail
  • bun typecheck — pass
  • bunx oxlint packages/opencode/src/cli/cmd/tui/app.tsx packages/opencode/src/cli/cmd/tui/util/terminal.ts packages/opencode/test/terminal.test.ts — 0 errors; existing warnings in app.tsx are unrelated to this change

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.

2 participants