fix(ci): actualizar guard de navegacion del catalogo tras PR #32
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
This commit is contained in:
1 parent
f1f5f3612b
commit
2abfec57ae
1 file changed
+14
-11
+14
-11
@@ -214,22 +214,25 @@ jobs:
|
||||
exit 1
|
||||
}
|
||||
|
||||
grep -q 'currentCollection' "${NAVBAR_FILE}" || {
|
||||
echo "ERROR: navbar.tsx no calcula currentCollection."
|
||||
# Desde Etapa 2 sub-fase 2 (2026-08-25) el nav es data-driven
|
||||
# contra el arbol real de categorias de Medusa, no un array
|
||||
# hardcodeado -- esta version anterior del check no detectaba
|
||||
# que "category=Mochilas"/"collection=Marvel" eran strings
|
||||
# hardcodeadas que ya no correspondian a ningun dato real (rotas
|
||||
# en silencio desde que se elimino legacyProductMeta, PR #29).
|
||||
# "currentCollection" tampoco existe mas: las colecciones ya no
|
||||
# se usan para resaltar el item activo del menu de categorias.
|
||||
grep -q 'buildCategoryTree' "${NAVBAR_FILE}" || {
|
||||
echo "ERROR: navbar.tsx ya no arma el arbol de categorias desde datos reales."
|
||||
exit 1
|
||||
}
|
||||
|
||||
grep -q 'collection=Marvel' "${NAVBAR_FILE}" || {
|
||||
echo "ERROR: falta collection=Marvel en navbar.tsx."
|
||||
grep -q 'encodeURIComponent(root.name)' "${NAVBAR_FILE}" || {
|
||||
echo "ERROR: navbar.tsx ya no linkea por el nombre real de categoria."
|
||||
exit 1
|
||||
}
|
||||
|
||||
grep -q 'category=Mochilas' "${NAVBAR_FILE}" || {
|
||||
echo "ERROR: falta category=Mochilas en navbar.tsx."
|
||||
exit 1
|
||||
}
|
||||
|
||||
grep -q 'aria-current={active ? "page" : undefined}' "${NAVBAR_FILE}" || {
|
||||
grep -q 'aria-current={isActive(' "${NAVBAR_FILE}" || {
|
||||
echo "ERROR: navbar.tsx no expone correctamente el menú activo."
|
||||
exit 1
|
||||
}
|
||||
@@ -258,7 +261,7 @@ jobs:
|
||||
|
||||
echo "Validando remontaje por parámetros: ${PAGE_FILE}"
|
||||
|
||||
grep -q 'key={`${category}:${collection}:${query}:${section}`}' "${PAGE_FILE}" || {
|
||||
grep -q 'key={`${category}:${collection}:${query}`}' "${PAGE_FILE}" || {
|
||||
echo "ERROR: app/catalog/page.tsx no remonta CatalogClient al cambiar la URL."
|
||||
exit 1
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user