fix(web): cleanup 25 TS errors preexistentes en main (closes #26) #27

dmolinari · 2026-04-18T13:55:28Z

dmolinari commented

2026-04-18 13:55:28 +00:00

Resumen

Fixea los 25 errores de TypeScript preexistentes en main que hacían fallar npm run build. Descubiertos durante el smoke E2E de PR #25 (UDT-011). Cierra #26.

Archivos modificados

Archivo	Fix aplicado
`src/web/src/components/ui/pagination.tsx`	`import type { ButtonProps }` (TS1484)
`src/web/src/features/medios/components/MedioForm.tsx`	Zod v4: `z.coerce.number<number>()`, eliminado `.transform()` con mismatch de tipos
`src/web/src/features/secciones/components/SeccionForm.tsx`	Zod v4: `z.coerce.number<number>()`, `errorMap` → `error` en `z.enum()`
`src/web/src/tests/features/auth/ProtectedRoute.test.tsx`	Eliminar `HomePage` y `renderProtected` no usados (TS6133)
`src/web/src/tests/features/fiscal/iva/TipoDeIvaFormModal.test.tsx`	Eliminar `beforeAll`, `afterAll`, `afterEach` no usados (TS6133)
`src/web/src/tests/features/medios/MedioForm.test.tsx`	Eliminar `beforeAll`, `afterAll`, `afterEach` no usados (TS6133)
`src/web/src/tests/features/secciones/SeccionesListPage.test.tsx`	Eliminar `userEvent` no usado (TS6133)
`src/web/src/tests/features/secciones/SeccionForm.test.tsx`	Selector más específico `/seleccioná un tipo válido/i`
`src/web/src/tests/features/users/updateUserPermisosOverrides.test.ts`	`mockResponse` alineado con shape real de `UsuarioPermisos` — `overrides.grant` (TS2339)
`src/web/src/tests/features/users/UsersListPage.test.tsx`	Eliminar `within` no usado (TS6133)

Contexto

Mismo patrón que PR #23 (fix Zod v4 para PuntoDeVentaForm). Preparatorio para el merge de UDT-011 (PR #25) — sin este fix el branch feature/UDT-011 hereda errores de TS de main.

Verificación

npm run build → 0 errores (build limpio)
npm run test -- --run → 275/275 passing (sin regresión)

Closes #26

## Resumen Fixea los 25 errores de TypeScript preexistentes en `main` que hacían fallar `npm run build`. Descubiertos durante el smoke E2E de PR #25 (UDT-011). Cierra #26. ## Archivos modificados | Archivo | Fix aplicado | |---------|--------------| | `src/web/src/components/ui/pagination.tsx` | `import type { ButtonProps }` (TS1484) | | `src/web/src/features/medios/components/MedioForm.tsx` | Zod v4: `z.coerce.number<number>()`, eliminado `.transform()` con mismatch de tipos | | `src/web/src/features/secciones/components/SeccionForm.tsx` | Zod v4: `z.coerce.number<number>()`, `errorMap` → `error` en `z.enum()` | | `src/web/src/tests/features/auth/ProtectedRoute.test.tsx` | Eliminar `HomePage` y `renderProtected` no usados (TS6133) | | `src/web/src/tests/features/fiscal/iva/TipoDeIvaFormModal.test.tsx` | Eliminar `beforeAll`, `afterAll`, `afterEach` no usados (TS6133) | | `src/web/src/tests/features/medios/MedioForm.test.tsx` | Eliminar `beforeAll`, `afterAll`, `afterEach` no usados (TS6133) | | `src/web/src/tests/features/secciones/SeccionesListPage.test.tsx` | Eliminar `userEvent` no usado (TS6133) | | `src/web/src/tests/features/secciones/SeccionForm.test.tsx` | Selector más específico `/seleccioná un tipo válido/i` | | `src/web/src/tests/features/users/updateUserPermisosOverrides.test.ts` | `mockResponse` alineado con shape real de `UsuarioPermisos` — `overrides.grant` (TS2339) | | `src/web/src/tests/features/users/UsersListPage.test.tsx` | Eliminar `within` no usado (TS6133) | ## Contexto Mismo patrón que PR #23 (fix Zod v4 para `PuntoDeVentaForm`). Preparatorio para el merge de UDT-011 (PR #25) — sin este fix el branch `feature/UDT-011` hereda errores de TS de main. ## Verificación - `npm run build` → **0 errores** (build limpio) - `npm run test -- --run` → **275/275 passing** (sin regresión) Closes #26

dmolinari added 5 commits 2026-04-18 13:55:28 +00:00

fix(web): corregir import type-only de ButtonProps en pagination.tsx (TS1484) d16da502f4

fix(web): migrar MedioForm a sintaxis Zod v4 (TS2322 — coerce.number<number>()) 8b369b69ee

fix(web): migrar SeccionForm a sintaxis Zod v4 (errorMap → error, coerce.number<number>()) 202d267e16

fix(web/tests): eliminar imports no usados en tests (TS6133) a39427865f

fix(web/tests): alinear updateUserPermisosOverrides mock con UsuarioPermisos shape (TS2339) b5ec0c25a9

dmolinari merged commit 50f713dc10 into main

2026-04-18 13:55:33 +00:00

dmolinari deleted branch fix/ADM-011-web-ts-cleanup

2026-04-18 13:55:33 +00:00

dmolinari referenced this issue from a commit

2026-04-18 13:55:35 +00:00