Skip to content

test(plugin): add alignment regression coverage for Tiny Actor Grid (#1273)#1282

Merged
JeremyDev87 merged 1 commit into
masterfrom
taskmaestro/1775268085/pane-2
Apr 4, 2026
Merged

test(plugin): add alignment regression coverage for Tiny Actor Grid (#1273)#1282
JeremyDev87 merged 1 commit into
masterfrom
taskmaestro/1775268085/pane-2

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

Summary

  • Add 23 regression tests for Tiny Actor Grid display-width alignment
  • Cover Unicode faces, ASCII fallback, CJK labels, emoji, ANSI color codes, and narrow terminals
  • Reuse display_width, pad_to_display_width, render_face_banner, render_box_line from buddy_renderer.py
  • Mock card data defined inline — no external fixtures

Test plan

  • python3 -m pytest tests/test_tiny_actor_alignment.py -v — 23 passed
  • yarn workspace codingbuddy test — 5887 passed
  • Lint/type-check: no new issues

Closes #1273

…1273)

Add 23 regression tests verifying display-width consistency across:
- Unicode faces (◕‿◕, ⬡‿⬡, ⊙‿⊙)
- ASCII fallback mode (o_o, ^_^)
- CJK labels (Korean 보안/성능, Chinese 安全/性能)
- Emoji faces (🤖, 👻, 🔥)
- ANSI color codes (zero-width verification)
- Narrow terminal widths (40/80 columns)
- Mixed content grid rows

Closes #1273
@JeremyDev87 JeremyDev87 added test plugin packages/claude-code-plugin wow-experience Plugin Wow Experience Design labels Apr 4, 2026
@vercel

vercel Bot commented Apr 4, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
codingbuddy-landing Ready Ready Preview, Comment Apr 4, 2026 2:10am

@JeremyDev87 JeremyDev87 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: APPROVE ✅

CI Status: ALL PASS (25/25)

Code Quality

  • 348-line comprehensive alignment regression suite
  • Leverages existing buddy_renderer helpers (display_width, pad_to_display_width, render_box_line, strip_ansi, truncate_to_display_width)
  • 7 test sections: Unicode, ASCII, CJK, emoji, ANSI, narrow terminal, mixed grid
  • Width-consistency focused — verifies uniform display width, not visual appearance
  • Mock data inline — no external fixtures needed

Key Coverage

  • CJK double-width: Korean (보안, 성능, 접근성) + Chinese (安全, 性能)
  • ANSI zero-width: strip + width calculation verified
  • Narrow terminals: 40-col and 80-col scenarios
  • Mixed grid row: Unicode + CJK + emoji + ANSI in same row

No Issues Found

Recommendation: APPROVE

@JeremyDev87 JeremyDev87 merged commit f8dc64d into master Apr 4, 2026
25 checks passed
@JeremyDev87 JeremyDev87 deleted the taskmaestro/1775268085/pane-2 branch April 4, 2026 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plugin packages/claude-code-plugin test wow-experience Plugin Wow Experience Design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(plugin): add alignment regression coverage for Tiny Actor Grid across CJK, emoji, ANSI, and asciiMode

1 participant