Skip to content

fix(cli): report a snippet the browser does not support as not supported - #437

Open
nucliweb wants to merge 1 commit into
fix/extension-stored-settingfrom
fix/cli-unsupported-output
Open

nucliweb wants to merge 1 commit into
fix/extension-stored-settingfrom
fix/cli-unsupported-output

Conversation

@nucliweb

Copy link
Copy Markdown
Owner

Summary

Stacked on #436.

A result with the unsupported status fell through to the metric row of the human and markdown reports, which printed undefined as its value and, in the markdown table, · undefined as its status.

  • Both reports say not supported and the reason the snippet gives, or that the browser lacks the API when it gives none.
  • The JSON output keeps the result as it is.

Test plan

  • Failing tests first: · LCP undefined in the human report and | LCP | undefined | · undefined | in the markdown one.
  • npm run test:unit --prefix cli locally: 607 of 607.
  • npm run lint locally.
  • The real CLI against a local page that removes largest-contentful-paint from PerformanceObserver.supportedEntryTypes: the human report prints · LCP not supported: largest-contentful-paint entries not supported in this browser, the markdown one | LCP | · not supported | largest-contentful-paint entries not supported in this browser |, and --json keeps "status":"unsupported". Through a pipe with NO_COLOR, the line is the same without color codes.
  • The e2e tests of the CLI were not run locally: the change only touches the reporters.

A result with the unsupported status fell through to the metric row of the human and markdown reports, which printed "undefined" as its value and, in the markdown table, "· undefined" as its status.

Both reports say "not supported" and the reason the snippet gives, or that the browser lacks the API when it gives none.
@vercel

vercel Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
webperf-snippets Ready Ready Preview Nov 14, 2026 8:57pm UTC

This branch was successfully deployed

1 active deployment
Preview — 091f7c4f Deployed Nov 14, 2026 by vercel[bot]
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