Skip to content

terminal: reset style before VT replay of blank cells (on 9d8d40319) - #249

Merged
lawrencecchen merged 2 commits into
mainfrom
fix-formatter-blank-cell-style-9d8d
Sep 30, 2026
Merged

lawrencecchen merged 2 commits into
mainfrom
fix-formatter-blank-cell-style-9d8d

Conversation

@lawrencecchen

@lawrencecchen lawrencecchen commented Sep 30, 2026 •

Copy link
Copy Markdown

Same two commits as #246, on 9d8d40319 (9961d09be plus its test expectation fix). #248 used 9961d09be directly, whose own Page VT preserves a fully styled blank row test is red and blocks GhosttyKit publication. e1b8bf5f4 is excluded on purpose: on current cmux main it stops cmux DEV.app from opening its socket in FuzzRegressions.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Fixes VT replay of blank cells so they no longer inherit the previous cell's background color.

  • Resets the active style before writing accumulated blank cells as spaces.
  • Adds a regression test covering Black background followed by skipped cells via CHA.

Written for commit 5597402. Summary will update on new commits.

Review in cubic

lawrencecchen and others added 2 commits September 30, 2026 07:03
A styled cell followed by skipped default cells replays the blanks with
the previous cell's background.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The formatter wrote accumulated blank cells as spaces before it switched
to the next cell's style, so the spaces took the previous cell's colors.
Claude Code's mascot sets a black background and skips three cells with
CHA, which a cmux Cloud replay painted as a black box.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@lawrencecchen
lawrencecchen merged commit 92d2854 into main Sep 30, 2026
50 checks passed
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 64fae570-39d9-40ea-8279-ed40f1e4f622

📥 Commits

Reviewing files that changed from the base of the PR and between 572de52 and 5597402.

📒 Files selected for processing (1)
  • src/terminal/formatter.zig
 ___________________
< Look ma, no bugs! >
 -------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant