Skip to content

chore: move cheerio to devDependencies - #5768

Open
DavertMik wants to merge 3 commits into
4.xfrom
chore/cheerio-dev-dependency
Open

DavertMik wants to merge 3 commits into
4.xfrom
chore/cheerio-dev-dependency

Conversation

@DavertMik

Copy link
Copy Markdown
Contributor

cheerio is listed as a runtime dependency but nothing in lib/ or bin/ imports it. The only consumer is test/unit/html_test.js; runtime HTML handling uses parse5 and @xmldom/xmldom.

Moving it to devDependencies drops cheerio and its transitive tree (htmlparser2, domhandler, css-select, undici, …) from user installs.

  • test/unit/html_test.js passes locally (20 passing, 1 pending)

🤖 Generated with Claude Code

DavertMik and others added 3 commits October 25, 2026 23:34
…p timing, role waits

Found while running the testomat.io e2e suite on Obscura.

CDPBrowser (Obscura, Lightpanda, Chrome over CDP):
- visibility: an inline or display:contents element with no box of its
  own (e.g. <a> wrapping a block card) is visible when a child is.
  Obscura reports no client rects for such inline elements.
- synthetic click fires pointerdown/mousedown/focus/pointerup/mouseup
  before click, so widgets that react to mousedown/mouseup (ember
  power-select, Radix) work.
- synthetic click on a non-interactive wrapper lands on the control at
  its center (elementFromPoint), like a coordinate click would.
- traffic is recorded once the response finished loading (matching
  Playwright's requestfinished), and redirect hops are recorded.

Playwright: waitFor* with role locators used [role="..."] css, ignoring
native roles and the accessible name; build an internal:role selector.

Core: HelperStep end time is taken when the async helper call settles,
so step durations are no longer ~0ms.

aiTrace: when a Before hook fails, only the running test writes the
trace, instead of every test in the suite overwriting it.

globalTimeout: ignore hook.finished for a suite hook that never fired
hook.started (TypeError on __prevSuiteTimeout.slice).

Co-Authored-By: Claude Opus 5.5 <[email protected]>
cheerio is only used by test/unit/html_test.js; lib/ parses HTML with parse5.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

This branch has not been deployed

No deployments
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