From d16da502f41b1746d2bca5726a48fc3c4462dc5c Mon Sep 17 00:00:00 2001
From: dmolinari <dmolinari@eldia.com>
Date: Sat, 18 Apr 2026 10:54:19 -0300
Subject: [PATCH 1/5] fix(web): corregir import type-only de ButtonProps en
 pagination.tsx (TS1484)

---
 src/web/src/components/ui/pagination.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/web/src/components/ui/pagination.tsx b/src/web/src/components/ui/pagination.tsx
index ea40d19..ca20226 100644
--- a/src/web/src/components/ui/pagination.tsx
+++ b/src/web/src/components/ui/pagination.tsx
@@ -2,7 +2,8 @@ import * as React from "react"
 import { ChevronLeft, ChevronRight, MoreHorizontal } from "lucide-react"
 
 import { cn } from "@/lib/utils"
-import { ButtonProps, buttonVariants } from "@/components/ui/button"
+import type { ButtonProps } from "@/components/ui/button"
+import { buttonVariants } from "@/components/ui/button"
 
 const Pagination = ({ className, ...props }: React.ComponentProps<"nav">) => (
   <nav
-- 
2.49.1


From 8b369b69ee06640d6cbd6ecf5bafe529893b3d71 Mon Sep 17 00:00:00 2001
From: dmolinari <dmolinari@eldia.com>
Date: Sat, 18 Apr 2026 10:54:23 -0300
Subject: [PATCH 2/5] =?UTF-8?q?fix(web):=20migrar=20MedioForm=20a=20sintax?=
 =?UTF-8?q?is=20Zod=20v4=20(TS2322=20=E2=80=94=20coerce.number<number>())?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/web/src/features/medios/components/MedioForm.tsx | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/web/src/features/medios/components/MedioForm.tsx b/src/web/src/features/medios/components/MedioForm.tsx
index 1c87169..c53ca75 100644
--- a/src/web/src/features/medios/components/MedioForm.tsx
+++ b/src/web/src/features/medios/components/MedioForm.tsx
@@ -34,11 +34,10 @@ const medioFormSchema = z.object({
     .string()
     .min(1, 'El nombre es requerido')
     .max(100, 'Máximo 100 caracteres'),
-  tipo: z.coerce.number().refine((v) => v >= 1, 'Seleccioná un tipo válido'),
+  tipo: z.coerce.number<number>().refine((v) => v >= 1, 'Seleccioná un tipo válido'),
   plataformaEmpresaId: z
-    .union([z.coerce.number().int().positive('Debe ser un número positivo'), z.literal('')])
-    .optional()
-    .transform((v) => (v === '' || v === undefined ? null : Number(v))),
+    .union([z.coerce.number<number>().int().positive('Debe ser un número positivo'), z.literal('')])
+    .optional(),
 })
 
 export type MedioFormValues = z.infer<typeof medioFormSchema>
-- 
2.49.1


From 202d267e16c267488498b7b96af27f5f492f5ef2 Mon Sep 17 00:00:00 2001
From: dmolinari <dmolinari@eldia.com>
Date: Sat, 18 Apr 2026 10:54:28 -0300
Subject: [PATCH 3/5] =?UTF-8?q?fix(web):=20migrar=20SeccionForm=20a=20sint?=
 =?UTF-8?q?axis=20Zod=20v4=20(errorMap=20=E2=86=92=20error,=20coerce.numbe?=
 =?UTF-8?q?r<number>())?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/web/src/features/secciones/components/SeccionForm.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/web/src/features/secciones/components/SeccionForm.tsx b/src/web/src/features/secciones/components/SeccionForm.tsx
index 1d188fb..4b9cb34 100644
--- a/src/web/src/features/secciones/components/SeccionForm.tsx
+++ b/src/web/src/features/secciones/components/SeccionForm.tsx
@@ -29,7 +29,7 @@ import type { SeccionDetail, TipoSeccion } from '../types'
 const TIPO_SECCION_VALUES = ['clasificados', 'notables', 'suplementos'] as const
 
 const seccionFormSchema = z.object({
-  medioId: z.coerce.number().refine((v) => v >= 1, 'Seleccioná un medio'),
+  medioId: z.coerce.number<number>().refine((v) => v >= 1, 'Seleccioná un medio'),
   codigo: z
     .string()
     .min(1, 'El código es requerido')
@@ -38,7 +38,7 @@ const seccionFormSchema = z.object({
     .string()
     .min(1, 'El nombre es requerido')
     .max(100, 'Máximo 100 caracteres'),
-  tipo: z.enum(TIPO_SECCION_VALUES, { errorMap: () => ({ message: 'Seleccioná un tipo válido' }) }),
+  tipo: z.enum(TIPO_SECCION_VALUES, { error: 'Seleccioná un tipo válido' }),
 })
 
 export type SeccionFormValues = z.infer<typeof seccionFormSchema>
-- 
2.49.1


From a39427865f0b818044cb326f24c5cc2ee0a65610 Mon Sep 17 00:00:00 2001
From: dmolinari <dmolinari@eldia.com>
Date: Sat, 18 Apr 2026 10:54:40 -0300
Subject: [PATCH 4/5] fix(web/tests): eliminar imports no usados en tests
 (TS6133)

---
 .../features/auth/ProtectedRoute.test.tsx     | 32 -------------------
 .../fiscal/iva/TipoDeIvaFormModal.test.tsx    |  2 +-
 .../tests/features/medios/MedioForm.test.tsx  |  2 +-
 .../secciones/SeccionesListPage.test.tsx      |  1 -
 .../features/users/UsersListPage.test.tsx     |  2 +-
 5 files changed, 3 insertions(+), 36 deletions(-)

diff --git a/src/web/src/tests/features/auth/ProtectedRoute.test.tsx b/src/web/src/tests/features/auth/ProtectedRoute.test.tsx
index 59aeb99..37bfc69 100644
--- a/src/web/src/tests/features/auth/ProtectedRoute.test.tsx
+++ b/src/web/src/tests/features/auth/ProtectedRoute.test.tsx
@@ -5,42 +5,10 @@ import { useAuthStore } from '../../../stores/authStore'
 import { ProtectedRoute } from '../../../components/routing/ProtectedRoute'
 
 // Helper components for testing
-function HomePage() {
-  return <div>Home Page</div>
-}
-
 function SecurePage() {
   return <div>Secure Page</div>
 }
 
-// Renders ProtectedRoute at a given path with optional navigation target capture
-function renderProtected(
-  props: {
-    requiredRoles?: string[]
-    requiredPermissions?: string[]
-    children?: React.ReactNode
-  },
-  { initialPath = '/' }: { initialPath?: string } = {},
-) {
-  const { children, ...routeProps } = props
-  return render(
-    <MemoryRouter initialEntries={[initialPath]}>
-      <Routes>
-        <Route path="/login" element={<div>Login Page</div>} />
-        <Route path="/" element={<div>Root</div>} />
-        <Route
-          path="/secure"
-          element={
-            <ProtectedRoute {...routeProps}>
-              {children ?? <SecurePage />}
-            </ProtectedRoute>
-          }
-        />
-      </Routes>
-    </MemoryRouter>,
-  )
-}
-
 beforeEach(() => {
   useAuthStore.setState({ user: null, accessToken: null, refreshToken: null, expiresAt: null })
 })
diff --git a/src/web/src/tests/features/fiscal/iva/TipoDeIvaFormModal.test.tsx b/src/web/src/tests/features/fiscal/iva/TipoDeIvaFormModal.test.tsx
index 53edf89..c634114 100644
--- a/src/web/src/tests/features/fiscal/iva/TipoDeIvaFormModal.test.tsx
+++ b/src/web/src/tests/features/fiscal/iva/TipoDeIvaFormModal.test.tsx
@@ -1,6 +1,6 @@
 // T600.5 — TDD: TipoDeIvaFormModal
 // CRÍTICO: verifica que el modal de Editar NO tiene campo Porcentaje [REQ-UI-003]
-import { describe, it, expect, beforeAll, afterAll, afterEach, vi } from 'vitest'
+import { describe, it, expect, vi } from 'vitest'
 import { render, screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
diff --git a/src/web/src/tests/features/medios/MedioForm.test.tsx b/src/web/src/tests/features/medios/MedioForm.test.tsx
index 021ee0d..be03489 100644
--- a/src/web/src/tests/features/medios/MedioForm.test.tsx
+++ b/src/web/src/tests/features/medios/MedioForm.test.tsx
@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeAll, afterAll, afterEach, vi } from 'vitest'
+import { describe, it, expect, vi } from 'vitest'
 import { render, screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
diff --git a/src/web/src/tests/features/secciones/SeccionesListPage.test.tsx b/src/web/src/tests/features/secciones/SeccionesListPage.test.tsx
index 7cc82d9..6071319 100644
--- a/src/web/src/tests/features/secciones/SeccionesListPage.test.tsx
+++ b/src/web/src/tests/features/secciones/SeccionesListPage.test.tsx
@@ -1,6 +1,5 @@
 import { describe, it, expect, beforeAll, afterAll, afterEach, vi } from 'vitest'
 import { render, screen, waitFor } from '@testing-library/react'
-import userEvent from '@testing-library/user-event'
 import { http, HttpResponse } from 'msw'
 import { setupServer } from 'msw/node'
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
diff --git a/src/web/src/tests/features/users/UsersListPage.test.tsx b/src/web/src/tests/features/users/UsersListPage.test.tsx
index 7c14d5b..8ad6730 100644
--- a/src/web/src/tests/features/users/UsersListPage.test.tsx
+++ b/src/web/src/tests/features/users/UsersListPage.test.tsx
@@ -1,5 +1,5 @@
 import { describe, it, expect, beforeAll, afterAll, afterEach, vi } from 'vitest'
-import { render, screen, waitFor, within } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { http, HttpResponse } from 'msw'
 import { setupServer } from 'msw/node'
-- 
2.49.1


From b5ec0c25a9923e4335fa5ecee96b3a5ebb204196 Mon Sep 17 00:00:00 2001
From: dmolinari <dmolinari@eldia.com>
Date: Sat, 18 Apr 2026 10:54:44 -0300
Subject: [PATCH 5/5] fix(web/tests): alinear updateUserPermisosOverrides mock
 con UsuarioPermisos shape (TS2339)

---
 .../src/tests/features/secciones/SeccionForm.test.tsx  |  2 +-
 .../features/users/updateUserPermisosOverrides.test.ts | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/web/src/tests/features/secciones/SeccionForm.test.tsx b/src/web/src/tests/features/secciones/SeccionForm.test.tsx
index 741f4bd..78ae621 100644
--- a/src/web/src/tests/features/secciones/SeccionForm.test.tsx
+++ b/src/web/src/tests/features/secciones/SeccionForm.test.tsx
@@ -80,7 +80,7 @@ describe('SeccionForm — create mode', () => {
     await userEvent.type(screen.getByLabelText(/nombre/i), 'Mi Sección')
     await userEvent.click(screen.getByRole('button', { name: /crear sección/i }))
     await waitFor(() =>
-      expect(screen.getByText(/seleccioná un tipo/i)).toBeInTheDocument(),
+      expect(screen.getByText(/seleccioná un tipo válido/i)).toBeInTheDocument(),
     )
   })
 
diff --git a/src/web/src/tests/features/users/updateUserPermisosOverrides.test.ts b/src/web/src/tests/features/users/updateUserPermisosOverrides.test.ts
index b46707f..ba9d0fd 100644
--- a/src/web/src/tests/features/users/updateUserPermisosOverrides.test.ts
+++ b/src/web/src/tests/features/users/updateUserPermisosOverrides.test.ts
@@ -6,11 +6,11 @@ import { updateUserPermisosOverrides } from '../../../features/users/api/updateU
 const API_URL = 'http://localhost:5000'
 
 const mockResponse = {
-  usuarioId: 42,
-  rol: 'cajero',
   rolPermisos: ['ventas:contado:crear'],
-  grant: ['textos:editar'],
-  deny: [],
+  overrides: {
+    grant: ['textos:editar'],
+    deny: [],
+  },
   effective: ['ventas:contado:crear', 'textos:editar'],
 }
 
@@ -35,7 +35,7 @@ describe('updateUserPermisosOverrides api client', () => {
       deny: [],
     })
 
-    expect(result.grant).toEqual(['textos:editar'])
+    expect(result.overrides.grant).toEqual(['textos:editar'])
     expect(result.effective).toContain('textos:editar')
     expect(capturedBody).toMatchObject({ grant: ['textos:editar'], deny: [] })
   })
-- 
2.49.1