Skip to content

fix(wizard-ci): render snapshot frames in colour via xterm.js - #2704

Open
gewenyu99 wants to merge 2 commits into
mainfrom
posthog-code/wizard-ci-snapshot-colours
Open

fix(wizard-ci): render snapshot frames in colour via xterm.js#2704
gewenyu99 wants to merge 2 commits into
mainfrom
posthog-code/wizard-ci-snapshot-colours

Conversation

@gewenyu99

@gewenyu99 gewenyu99 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Downstream of PostHog/wizard#730, which made the captured snapshot frames carry ANSI colour.

Renders each captured .ans frame in a real xterm.js terminal in the report page so Playwright screenshots the actual coloured TUI, with no bespoke ANSI-to-HTML converter.

gewenyu99 and others added 2 commits July 10, 2026 19:05
Bring back the ansi-html.ts converter (deleted in 96685c4) and render
.ans frames through it, so the report PNGs show the TUI's colours instead
of raw escape codes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…verter

Drop the bespoke ansi-html converter and render each captured .ans frame in
a real xterm.js terminal in the report page, so Playwright screenshots the
actual coloured terminal. xterm does the colour; no SGR logic of our own.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gewenyu99 gewenyu99 changed the title fix(wizard-ci): render snapshot frames in colour fix(wizard-ci): render snapshot frames in colour via xterm.js Jul 10, 2026
@gewenyu99
gewenyu99 marked this pull request as ready for review July 10, 2026 23:26
@gewenyu99
gewenyu99 requested a review from a team as a code owner July 10, 2026 23:26
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