Skip to content

🎨 Align CLI reporter with BearDen#280

Draft
Robdel12 wants to merge 2 commits into
mainfrom
rd/tdd-dynamic-content-cli
Draft

🎨 Align CLI reporter with BearDen#280
Robdel12 wants to merge 2 commits into
mainfrom
rd/tdd-dynamic-content-cli

Conversation

@Robdel12
Copy link
Copy Markdown
Contributor

@Robdel12 Robdel12 commented May 25, 2026

Why

The CLI reporter branch was still carrying the old embedded Observatory-era style contract. This brings it forward onto current main and wires the reporter to the released BearDen package while keeping CLI-specific reporter components local.

What changed

  • Rebased the existing local TDD reporter branch onto latest main after the pnpm migration.
  • Added @vizzly-testing/bear-den@^0.1.1 as a real CLI dependency.
  • Imported BearDen styles/tokens before CLI reporter-specific styles.
  • Kept reporter-owned components local for now, but renamed obsolete Observatory comments so BearDen is the current mental model.
  • Added the fresh BearDen package to the pnpm minimum-release-age allowlist.

Verification

  • pnpm run lint passes with 3 existing optional-chain warnings.
  • pnpm run build passes.
  • pnpm run test:reporter passes: 35 passed, 5 skipped.

Robdel12 added 2 commits May 25, 2026 18:05
Use the released BearDen package as the reporter style/token source, keep the CLI-specific reporter components local, and remove obsolete Observatory naming from the visual layer.
@vizzly-testing
Copy link
Copy Markdown

vizzly-testing Bot commented May 25, 2026

Vizzly - Visual Test Results

CLI Reporter - 19 changes need review
Status Count
Meaningful diffs 19
Meaningful diffs needing review (19)

search-homepage · Firefox · 1920×1080 · 39.5% diff

search-homepage

settings-page · Firefox · 1920×1080 · 68.9% diff

settings-page

search-no-results · Firefox · 1920×1080 · 33.1% diff

search-no-results

dashboard-mixed-state · Firefox · 1920×1080 · 61.1% diff

dashboard-mixed-state

fullscreen-viewer · Firefox · 1920×1080 · 10.4% diff

fullscreen-viewer

bulk-accept-dialog · Firefox · 1920×1080 · 74.9% diff

bulk-accept-dialog

...and 13 more in Vizzly.

Review changes

CLI TUI - 4 changes need review
Status Count
Passed 1
Meaningful diffs 4
Auto-approved 1
Meaningful diffs needing review (4)

vizzly-doctor · 1202×578 · 77.3% diff

vizzly-doctor

vizzly-help · 1202×1430 · 3.9% diff

vizzly-help

vizzly-tdd-help · 1202×578 · 4.6% diff

vizzly-tdd-help

vizzly-run-help · 1202×722 · 3.4% diff

vizzly-run-help

Review changes


rd/tdd-dynamic-content-cli · 6cb1bbd8

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