Problema
El pipeline de CI (.github/workflows/ci.yml) nunca ejecuta los tests de apps/web. El job Test solo corre pnpm --filter api test; para web solo hay lint y build. El gate documentado en AGENTS.md tampoco los incluye.
Consecuencia: toda la lógica testeada de apps/web es advisory — no protege main. Tests reales que hoy NO corren en CI:
Un cambio que rompa cualquiera de estos (p. ej. un rename de ruta que el test anti-drift debería atrapar) pasa Build+Lint y mergea igualmente. Se descubrió revisando PR #344, cuyo deliverable principal (test anti-drift) queda sin efecto real por esta brecha.
Propuesta
Criterios de aceptación
- Un test de
apps/web en rojo bloquea el merge.
- El gate de
AGENTS.md documenta pnpm --filter web test.
Contexto
Detectado en la review de PR #344 (seguimiento de #342). Afecta la confianza en el "CI verde" de PRs de web-logic (#336, #338, #344).
Problema
El pipeline de CI (
.github/workflows/ci.yml) nunca ejecuta los tests deapps/web. El jobTestsolo correpnpm --filter api test; para web solo haylintybuild. El gate documentado enAGENTS.mdtampoco los incluye.Consecuencia: toda la lógica testeada de
apps/webes advisory — no protegemain. Tests reales que hoy NO corren en CI:apps/web/src/lib/app-bar-current-path-drift.test.ts(anti-drift de rutas, PR fix(web): resourceId codificado en currentPath de pre-registro + test anti-drift #344)apps/web/src/lib/backend-error-messages.test.ts(localización de errores 4xx, PR fix(web): errores de formulario accionables — fila inválida + mensajes 4xx localizados (#296) #338)apps/web/src/lib/app-bar-context.test.ts(PR fix(web): AppBar usa el pathname exacto para el next del login (#278) #336)apps/web/src/lib/supply-lines.test.tsy demássrc/**/*.test.tsde lógica pura.Un cambio que rompa cualquiera de estos (p. ej. un rename de ruta que el test anti-drift debería atrapar) pasa
Build+Linty mergea igualmente. Se descubrió revisando PR #344, cuyo deliverable principal (test anti-drift) queda sin efecto real por esta brecha.Propuesta
pnpm --filter web testenci.yml(los tests usannode --testsobresrc/**/*.test.ts, sin jsdom/RTL — son rápidos y sin infra extra).AGENTS.md(sección "The gate").Criterios de aceptación
apps/weben rojo bloquea el merge.AGENTS.mddocumentapnpm --filter web test.Contexto
Detectado en la review de PR #344 (seguimiento de #342). Afecta la confianza en el "CI verde" de PRs de web-logic (#336, #338, #344).