Skip to content

test: guard colored table alignment#52

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/missing-test-coverage-ad5e
Draft

test: guard colored table alignment#52
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/missing-test-coverage-ad5e

Conversation

@cursor

@cursor cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown

Summary

  • add deterministic coverage for ANSI-colored renderTable output
  • verify styling leaves the visible fixed-width table identical to uncolored output

Risky behavior now covered

The live CLI status/watch table colors headers, vendors, labels, and statuses after calculating column widths. Missing re-padding can silently shift later columns when cells have different widths.

Test files added/updated

  • test/format-color.test.ts

Regression value

This directly covers the colored path around PR #12's width-calculation refactor without duplicating the open large uncolored-table test. A mutation removing status-cell re-padding causes the test to fail with a visibly shifted LAST UPDATE column.

Validation

  • npm test — 36/36 tests passed
  • npm run build — passed
  • npx prettier --check test/format-color.test.ts — passed
Open in Web View Automation 

Co-authored-by: Parvez Kose <koseparvez@gmail.com>
@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
polyagent Ready Ready Preview, Comment Jul 22, 2026 10:09am

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