scopeRouteVariants repeats shared-input work for every route in compose
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- php
- Área
- performance
Línea de trabajo
Start by reading CssStylesheetTransformer::scopeRouteVariants, RouteSelectorMatchIndex::inertOutside, and the cited scopeRouteUnit and markerCarriers paths; compare their work across routes during compose. Profile the 112-page import described in the issue and check output against the current byte-identical result. Done means shared-input work runs once, repeated element/arm matching is avoided, and output remains byte-identical.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
In compose, CssStylesheetTransformer::scopeRouteVariants costs 60 s of a 310 s compose on a 112-page real import (BE trunk after #3675/#3676). Profile, by child:
RouteSelectorMatchIndex::inertOutside→CssSelectorMatcher::matches: 19 s. The same selector arm is matched against each route DOM in turn, through candidate lists rebuilt per (arm, owner).scopeRouteUnit→scopeRouteArms→CssSelectorTokenizer::tokenize/pseudoElementOffset/splitSelectorList: ~8 s. Identical shared units are tokenized again for each route's probe validation.markerCarriers: 4 s.isWellFormedStylesheetover shared plus page sheets for each route: ~3 s. The shared sheets are re-checked for every route.
This work cannot move to page workers: base selection and attribute-marker reconciliation span all routes (#2089).
Expected
Inside compose, work that depends only on the shared input is done once rather than once per route: shared-sheet well-formedness, unit keys, probe results and parsed arms. inertOutside answers "does arm X match any route outside the set" without re-matching the same element/arm pairs per owner. Output stays byte-identical.
- Lenguaje dominante
- PHP
- Estrellas
- 22
- Forks
- 3
- Merge medio
- 6 h 51 min
- PR fusionados (30 d)
- 759
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Automattic/blocks-engine
-
Solved site promotion gate fails on runner PHP patch drift (expects 8.2.33, runner installs 8.2.34)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Automattic/blocks-engine#2161 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Automattic/blocks-engine#2061 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Automattic/blocks-engine#1701 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Automattic/blocks-engine#775 ·
Los mantenedores suelen responder en 1 día
-
Compute per-document site-plan facts in staged page compilationPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Automattic/blocks-engine#3699 ·
Los mantenedores suelen responder en 1 día
Todos los issues de Automattic/blocks-engine
Issues similares
-
sync-en
Dificultad 1/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bug-report bug-report-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Codeinwp/wp-menu-icons#354 ·
-
🔍 Triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
coollabsio/coolify#12373 ·
Los mantenedores suelen responder en 1 día
-
False RedundantFlag for FILTER_FLAG_GLOBAL_RANGE on PHP 8.5 (hardcoded flag value is outdated)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
vimeo/psalm#12587 · 1 comentario ·
Los mantenedores suelen responder en 1 día