Skip to content

fix(cli): align terminal output by display width - #140

Merged
hughgrigg merged 1 commit into
mainfrom
fix/terminal-display-width
Sep 1, 2026
Merged

fix(cli): align terminal output by display width#140
hughgrigg merged 1 commit into
mainfrom
fix/terminal-display-width

Conversation

@hughgrigg

Copy link
Copy Markdown
Contributor

Terminal tables and help text now measure terminal display width. CJK text, combining marks, and joined emoji align with ASCII columns. The build bundles string-width into the display-width module and keeps packed CLI help self-contained. Tests cover the affected text shapes. pnpm check passes through the coverage step, where 81 Yulin-backed tests fail locally because their simulated S3 buckets and Athena workgroup are unavailable.

  • Conventional commit message, used as the title

  • Conventional branch name, like feat/concise-description

  • Full check with pnpm run check passed

  • Rebased off latest main

  • User-facing behaviour is documented in docs/

See https://www.conventionalcommits.org/en/v1.0.0/

@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 48 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 90 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 9730df9c-9a11-473e-af65-a032bf58bd2e

📥 Commits

Reviewing files that changed from the base of the PR and between dc52255 and 247e25d.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (6)
  • package.json
  • src/cli/display-width.ts
  • src/cli/output/table.test.ts
  • src/cli/output/table.ts
  • src/cli/text-layout.test.ts
  • src/cli/text-layout.ts

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

@hughgrigg
hughgrigg merged commit d3ef295 into main Sep 1, 2026
8 checks passed
@hughgrigg
hughgrigg deleted the fix/terminal-display-width branch September 1, 2026 11:48
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