fix(ci): actualizar guard de navegacion del catalogo tras PR #32 #33

Merged
devops merged 1 commits from fix/etapa2-subfase2-ci-nav-check into main 2026-08-25 03:17:26 +00:00
Owner

El guard grep de navbar.tsx en build.yaml seguia validando la implementacion vieja (collection=Marvel, category=Mochilas, currentCollection) que PR #32 elimino por estar rota. Se actualiza para validar el comportamiento real nuevo (arbol de categorias data-driven).

El guard grep de navbar.tsx en build.yaml seguia validando la implementacion vieja (collection=Marvel, category=Mochilas, currentCollection) que PR #32 elimino por estar rota. Se actualiza para validar el comportamiento real nuevo (arbol de categorias data-driven).
devops added 1 commit 2026-08-25 02:36:47 +00:00
fix(ci): actualizar guard de navegacion del catalogo tras PR #32
Build and Push Frontend / Construir y Subir Imagen (pull_request) Skipped
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 30s
2abfec57ae
El guard de "Validar navegación real del catálogo" en build.yaml seguia
pinneado a la implementacion vieja de navbar.tsx: greppeaba literalmente
"collection=Marvel", "category=Mochilas" y "currentCollection", que eran
exactamente los valores hardcodeados que PR #32 elimino por estar rotos
(ya no correspondian a ningun dato real desde que se quito
legacyProductMeta en PR #29). El guard quedo validando el bug, no la
correccion, y bloqueo el build post-merge de PR #32.

Ahora valida el comportamiento real nuevo: que el nav arma su arbol
desde datos reales (buildCategoryTree), que linkea por nombre real de
categoria (encodeURIComponent(root.name)) y que expone el item activo
(aria-current={isActive(...)}). El check de remontaje en
app/catalog/page.tsx tambien se actualizo: la key ya no incluye
":${section}" porque ese query param nunca se usaba (dead code
eliminado en PR #32).

Co-Authored-By: Claude Sonnet 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_013dunTcReWMKhWgCPkvAVHD
devops merged commit 20bc559a6c into main 2026-08-25 03:17:26 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: devops/apps-registry#33