Skip to content

test(e2e): fix pdf-server screenshot flake from random viewUUID#556

Open
ochafik wants to merge 1 commit intomainfrom
claude/fix-ci-pipeline
Open

test(e2e): fix pdf-server screenshot flake from random viewUUID#556
ochafik wants to merge 1 commit intomainfrom
claude/fix-ci-pipeline

Conversation

@ochafik
Copy link
Contributor

@ochafik ochafik commented Mar 19, 2026

Problem

The e2e job is failing on main since #506 merged.

Root cause: display_pdf now returns "PDF opened. viewUUID: <random-uuid>..." in its text content. The basic-host Tool Result panel shows the first 100 chars of the JSON-serialized result as a preview, which now includes the random UUID — so pdf-server.png can never match across runs.

Before #506 the result text was just "Displaying PDF: <url>" (the viewUUID lived only in _meta, past the 100-char preview cutoff).

Fix

Add pdf-server to HOST_MASKS to mask the collapsiblePanel elements (Tool Input/Result). Same pattern used by integration, basic-*, and system-monitor servers that have dynamic result content.

Next step

After this PR's CI runs, comment /update-snapshots to regenerate pdf-server.png — adding a mask changes the screenshot (masked regions render as pink boxes), plus #506 also changed the fullscreen button from to an SVG icon.

The display_pdf result now includes a random viewUUID in its text
content (visible in the first 100 chars of the Tool Result preview),
making the pdf-server screenshot non-deterministic. Mask the
collapsible panels as done for other servers with dynamic results.
@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 19, 2026

Open in StackBlitz

@modelcontextprotocol/ext-apps

npm i https://pkg.pr.new/@modelcontextprotocol/ext-apps@556

@modelcontextprotocol/server-basic-preact

npm i https://pkg.pr.new/@modelcontextprotocol/server-basic-preact@556

@modelcontextprotocol/server-basic-react

npm i https://pkg.pr.new/@modelcontextprotocol/server-basic-react@556

@modelcontextprotocol/server-basic-solid

npm i https://pkg.pr.new/@modelcontextprotocol/server-basic-solid@556

@modelcontextprotocol/server-basic-svelte

npm i https://pkg.pr.new/@modelcontextprotocol/server-basic-svelte@556

@modelcontextprotocol/server-basic-vanillajs

npm i https://pkg.pr.new/@modelcontextprotocol/server-basic-vanillajs@556

@modelcontextprotocol/server-basic-vue

npm i https://pkg.pr.new/@modelcontextprotocol/server-basic-vue@556

@modelcontextprotocol/server-budget-allocator

npm i https://pkg.pr.new/@modelcontextprotocol/server-budget-allocator@556

@modelcontextprotocol/server-cohort-heatmap

npm i https://pkg.pr.new/@modelcontextprotocol/server-cohort-heatmap@556

@modelcontextprotocol/server-customer-segmentation

npm i https://pkg.pr.new/@modelcontextprotocol/server-customer-segmentation@556

@modelcontextprotocol/server-debug

npm i https://pkg.pr.new/@modelcontextprotocol/server-debug@556

@modelcontextprotocol/server-map

npm i https://pkg.pr.new/@modelcontextprotocol/server-map@556

@modelcontextprotocol/server-pdf

npm i https://pkg.pr.new/@modelcontextprotocol/server-pdf@556

@modelcontextprotocol/server-scenario-modeler

npm i https://pkg.pr.new/@modelcontextprotocol/server-scenario-modeler@556

@modelcontextprotocol/server-shadertoy

npm i https://pkg.pr.new/@modelcontextprotocol/server-shadertoy@556

@modelcontextprotocol/server-sheet-music

npm i https://pkg.pr.new/@modelcontextprotocol/server-sheet-music@556

@modelcontextprotocol/server-system-monitor

npm i https://pkg.pr.new/@modelcontextprotocol/server-system-monitor@556

@modelcontextprotocol/server-threejs

npm i https://pkg.pr.new/@modelcontextprotocol/server-threejs@556

@modelcontextprotocol/server-transcript

npm i https://pkg.pr.new/@modelcontextprotocol/server-transcript@556

@modelcontextprotocol/server-video-resource

npm i https://pkg.pr.new/@modelcontextprotocol/server-video-resource@556

@modelcontextprotocol/server-wiki-explorer

npm i https://pkg.pr.new/@modelcontextprotocol/server-wiki-explorer@556

commit: f351976

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.

2 participants