Skip to content

fix(terminal): align table wrapping for wide graphemes#42249

Merged
vincentkoc merged 14 commits intomainfrom
vincentkoc-code/fix-terminal-table-width
Mar 12, 2026
Merged

fix(terminal): align table wrapping for wide graphemes#42249
vincentkoc merged 14 commits intomainfrom
vincentkoc-code/fix-terminal-table-width

Conversation

@vincentkoc
Copy link
Copy Markdown
Member

@vincentkoc vincentkoc commented Mar 10, 2026

Superseded by #42254 for the follow-up Terminal.app fixes. This original draft still covers the base wide-grapheme/table-wrap fix, but the newer PR includes the width-clamp follow-up and Terminal-safe icon adjustments.

@vincentkoc vincentkoc self-assigned this Mar 10, 2026
@openclaw-barnacle openclaw-barnacle Bot added the maintainer Maintainer-authored PR label Mar 10, 2026
@vincentkoc
Copy link
Copy Markdown
Member Author

Manual Terminal.app verification update:

The earlier narrow Terminal.app capture was invalid because the window was resized after openclaw skills had already started, so the process read a smaller stdout.columns value and rendered a narrow table.

I reran the check correctly by sizing Terminal.app first and then starting node dist/entry.js skills. Result: the table renders wide and the columns remain aligned in Terminal.app, matching the expected iTerm behavior.

Verified command:

node dist/entry.js skills | tail -n 120

Observed Terminal.app output stays aligned for wide-grapheme rows such as 🎞️ video-frames, 🌤️ weather, and 𝕏 xurl.

@vincentkoc
Copy link
Copy Markdown
Member Author

Superseded by #42849 for the Terminal.app follow-up fixes.

@openclaw-barnacle openclaw-barnacle Bot added cli CLI command changes commands Command implementations size: M and removed size: S labels Mar 12, 2026
@vincentkoc vincentkoc merged commit cb7b381 into main Mar 12, 2026
7 checks passed
@vincentkoc vincentkoc deleted the vincentkoc-code/fix-terminal-table-width branch March 12, 2026 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli CLI command changes commands Command implementations maintainer Maintainer-authored PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant