Skip to content

refactor(cli): use adaptive table formatter broadly#1333

Open
thebiglabasky wants to merge 4 commits into
mainfrom
herve/adaptive-table-formatters-sweep
Open

refactor(cli): use adaptive table formatter broadly#1333
thebiglabasky wants to merge 4 commits into
mainfrom
herve/adaptive-table-formatters-sweep

Conversation

@thebiglabasky
Copy link
Copy Markdown
Contributor

Summary

  • Migrate remaining user-facing formatter tables to renderAdaptiveTable(...).
  • Remove formatter-local terminal width math, fixed-width calculations, and repeated per-cell truncation.
  • Update affected formatter snapshots for the natural adaptive spacing changes.

Why

PR #1326 added the shared adaptive table utility. This pass applies it broadly so table layout behavior lives in one renderer instead of each command carrying brittle terminal-width logic.

Validation

  • pnpm exec vitest run src/formatters/__tests__/render.spec.ts src/formatters/__tests__/checks.spec.ts src/formatters/__tests__/test-sessions.spec.ts src/formatters/__tests__/status-pages.spec.ts src/formatters/__tests__/account-members.spec.ts src/formatters/__tests__/account-plan.spec.ts src/formatters/__tests__/incidents.spec.ts src/formatters/__tests__/analytics.spec.ts
  • pnpm --filter checkly exec tsc -p tsconfig.json --skipLibCheck
  • pnpm exec eslint packages/cli/src/formatters/account-members.ts packages/cli/src/formatters/account-plan.ts packages/cli/src/formatters/batch-stats.ts packages/cli/src/formatters/checks.ts packages/cli/src/formatters/incidents.ts packages/cli/src/formatters/status-pages.ts packages/cli/src/formatters/test-sessions.ts
  • git diff --check

@thebiglabasky thebiglabasky marked this pull request as ready for review May 29, 2026 13:47
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