Skip to content

fix(script): read the DOM instead of serializing and reparsing it - #405

Merged
vishnuv688 merged 3 commits into
mainfrom
fix/mocha-example-leak-check
Oct 1, 2026
Merged

vishnuv688 merged 3 commits into
mainfrom
fix/mocha-example-leak-check

Conversation

@vishnuv688

@vishnuv688 vishnuv688 commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

What & why

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Polish (an improvement to an existing feature)
  • Breaking change (existing behavior changes for users)
  • Documentation
  • Internal (build, CI, dependencies, tooling)

Packages touched

  • shared (types and contracts)
  • core (framework-agnostic capture/reporting)
  • elements (published element/snapshot API — @wdio/elements)
  • service (WebdriverIO adapter)
  • nightwatch-devtools (Nightwatch adapter)
  • selenium-devtools (Selenium adapter)
  • selenium-devtools-py (Selenium Python adapter)
  • backend (server)
  • app (UI)
  • script (page-injected runtime)
  • trace (Trace mode)

Notes for reviewers

Screenshots / recordings

@vishnuv688 vishnuv688 linked an issue Sep 30, 2026 that may be closed by this pull request
@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Rewrites DOM serialization to read live instead of parsing HTML.

The PR appears safe to merge; no actionable new failure was established.

Summary

The PR replaces HTML serialization and reparsing with direct live-DOM traversal, reducing the injected collector bundle. It also removes pinned development Chrome drivers from the Nightwatch and Selenium adapters and adds serializer and bundle-size tests.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  DOM[Live page DOM] --> Collector[Collector traversal]
  Collector --> Wire[Simplified vnode tree]
  Wire --> Replay[Replay]
Loading

Reviews (2) · Last reviewed commit: "fix(examples): stop pinning a browser dr..."

@vishnuv688
vishnuv688 merged commit 269a436 into main Oct 1, 2026
9 checks passed
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.

Shrink the page collector bundle: parse5 and preact are 92% of it

1 participant