Yoast / wordpress-seo / 424209e48bf6f764975ff41e9027e948a174febb

54%

trunk: 53%

Build Type

Pull #23344

github

Committed by vraja-pro

Commit Message

refactor(elementor-v4): reduce complexity in content-walker and add position map

- Extract toPlainNodes() shared helper — removes duplicated Backbone
  collection handling from both walk functions.
- Extract shouldSkipNode() predicate — collapses the two per-node guard
  conditions into one call, reducing cyclomatic complexity of both
  walkAtomicTree and walkAtomicTreeWithMap.
- Simplify child-offset relocation in walkAtomicTreeWithMap from a for
  loop to a map + spread.
- Add getWidgetMap() in marks.js and expose it on window.yoastElementorV4
  so premium's mark applicator can resolve widget positions without
  reimplementing the tree walk.
- Add walk-atomic-tree-with-map.test.js with full coverage of positions,
  nested container offset adjustment, excluded types, Backbone collections,
  empty/no-id nodes, and whitespace normalisation.
- Fix extractors.test.js: remove e-tab (now excluded), add e-youtube.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #23344: Add Elementor V4 atomic editor content extraction

Coverage Stats

10124 of 18207 branches covered (55.6%)

Branch coverage included in aggregate %.

161 of 221 new or added lines in 9 files covered. (72.85%)

156 existing lines in 2 files now uncovered.

37773 of 70095 relevant lines covered (53.89%)

42932.98 hits per line