fix(web): migrar PuntoDeVentaForm a sintaxis Zod v4 (closes #21) #23

dmolinari · 2026-04-18T11:47:35Z

dmolinari commented

2026-04-18 11:47:35 +00:00

Descripción

Migración de PuntoDeVentaForm.tsx de sintaxis Zod v3 a v4, siguiendo el mismo pattern aplicado en los 4 componentes fiscales de ADM-009 (PR #22).

Problema

La sintaxis z.coerce.number({ invalid_type_error: '...' }) es inválida en Zod v4 — el objeto de opciones ya no acepta invalid_type_error. Esto causaba 5 errores en cascada de TypeScript en @hookform/resolvers: los tipos medioId y numeroAFIP inferían unknown en vez de number, rompiendo Resolver, Control (3 instancias) y SubmitHandler.

Fix aplicado

// Antes (v3 — inválido en v4):
medioId: z.coerce.number().refine(...)
numeroAFIP: z.coerce.number({ invalid_type_error: 'El número AFIP debe ser un número' })

// Después (v4 — genérico explícito para que hookform resuelva bien):
medioId: z.coerce.number<number>().refine(...)
numeroAFIP: z.coerce.number<number>('El número AFIP debe ser un número')

Verificación

tsc -b: 0 errores en PuntoDeVentaForm.tsx post-fix
Tests: 275/275 pasando sin regresiones

Closes #21

## Descripción Migración de `PuntoDeVentaForm.tsx` de sintaxis Zod v3 a v4, siguiendo el mismo pattern aplicado en los 4 componentes fiscales de ADM-009 (PR #22). ## Problema La sintaxis `z.coerce.number({ invalid_type_error: '...' })` es inválida en Zod v4 — el objeto de opciones ya no acepta `invalid_type_error`. Esto causaba **5 errores en cascada** de TypeScript en `@hookform/resolvers`: los tipos `medioId` y `numeroAFIP` inferían `unknown` en vez de `number`, rompiendo `Resolver`, `Control` (3 instancias) y `SubmitHandler`. ## Fix aplicado ```ts // Antes (v3 — inválido en v4): medioId: z.coerce.number().refine(...) numeroAFIP: z.coerce.number({ invalid_type_error: 'El número AFIP debe ser un número' }) // Después (v4 — genérico explícito para que hookform resuelva bien): medioId: z.coerce.number<number>().refine(...) numeroAFIP: z.coerce.number<number>('El número AFIP debe ser un número') ``` ## Verificación - `tsc -b`: 0 errores en `PuntoDeVentaForm.tsx` post-fix - Tests: 275/275 pasando sin regresiones Closes #21

dmolinari added 1 commit 2026-04-18 11:47:35 +00:00

fix(web/adm-008): migrar PuntoDeVentaForm a sintaxis Zod v4 (closes #21 ) 0863ed8682

dmolinari merged commit d4b2183628 into main

2026-04-18 11:47:39 +00:00

dmolinari deleted branch fix/ADM-008-zod-v4

2026-04-18 11:47:39 +00:00

dmolinari referenced this issue from a commit

2026-04-18 11:47:41 +00:00

Merge pull request 'fix(web): migrar PuntoDeVentaForm a sintaxis Zod v4 (closes #21)' (#23) from fix/ADM-008-zod-v4 into main

dmolinari referenced this pull request

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

[WEB] Fixear 25 errores TS preexistentes en main (MedioForm + SeccionForm + pagination + tests) #26