Auditoría de separación lógica del cliente

Alcance y referencia

Revisión de modules/core/src/web, modules/identity/src/web, modules/assets/src/web, modules/resources/src/web y modules/companies/src/web, usando AGENTS.md, la documentación normativa y el código de customer-invoices.zip como referencia de separación de responsabilidades. Se excluyen los cambios en apps/web/src, vendor, backend, schemas HTTP y persistencia.

La referencia aporta controllers, adapters y acceso a API por operaciones. No se incorporan sus dependencias, factories, tipos genéricos, memoización o servicios ajenos al proyecto.

Áreas revisadas

Core: health y home; la consulta ya tiene API y claves separadas. No se añade un controller que únicamente envuelva esa consulta.
Identity: sesión, autenticación, logout, contraseñas, gestión de usuarios y workspace.
Assets: inventario, detalle, características, relaciones, puestos, marcas, catálogos y soporte.
Resources: catálogos, plantillas, editor, valores iniciales e integraciones de recursos.
Companies: compañías, logos, personas, departamentos, sedes, asignaciones, accesos, dashboard y workspace; correo, usos de buzón, perfiles, carpetas, aplicaciones, certificados e impresoras; tickets, intervenciones, plantillas, procesos, teletrabajo y remotos.

La inspección combina inventario y análisis sintáctico del código, revisión de dependencias, lectura de las responsabilidades y diffs afectados y comprobaciones por módulos.

Incumplimientos y cambios

Formularios y páginas coordinaban consultas, mutaciones, invalidación, defaults y grandes bloques visuales en el mismo componente. Se extraen 78 controllers con responsabilidades concretas, conservando RHF y TanStack Query como fuentes de verdad.
Cálculos reutilizados se exportaban desde .tsx, incluso para usarlos desde coordinación. Se separan filtros, integración, destino remoto, navegación, fechas y progreso de pasos.
Schemas locales y contratos de formularios se mezclaban con la presentación. Se colocan en schemas y types, sin duplicar DTOs HTTP. Los adapters separan conversiones reales de personas, sedes e instrucciones de procesos.
Clientes compartidos agrupaban varias entidades. Se trasladan 31 operaciones de identidad, inventario, soporte, compañías y procesos a sus ámbitos. Accesos y buzones distinguen consulta y guardado. Los clientes pequeños que ya tienen un ámbito cohesivo se conservan.
Los callbacks de altas anidadas exponían refs de montaje a UI. Su coordinación y protección ante desmontaje quedan privadas de los controllers.
Los índices se ajustan a sus consumidores y a la responsabilidad que exportan, evitando compatibilidad artificial desde UI y ciclos por composición de API.

Los componentes simples, helpers visuales privados y hooks existentes que ya resuelven su responsabilidad permanecen próximos a su consumidor. No se fragmenta JSX trivial, se duplica estado remoto ni se añaden dependencias o nuevas capas de negocio.

Se preservan el comportamiento y las composiciones públicas requeridas por la aplicación y los demás módulos, incluidas identidad, inventario y catálogos. No cambian endpoints, schemas, datos persistidos, migraciones ni textos de interfaz.

Comprobaciones

Typechecks y pruebas dirigidas de los módulos y de la aplicación, sin modificar esta última.
Doce pruebas nuevas incluidas en los scripts de paquete: sesión y autenticación, rutas, métodos HTTP, CSRF, errores seguros, respuestas inválidas, accesos, usos de buzón, adapters, integración y límites de logos.
Comparación sintáctica de los cuerpos y parámetros de las 31 operaciones trasladadas, normalizando únicamente el nombre del transporte compartido.
Análisis de dependencias relativas de los cinco módulos, sin ciclos detectados.
Comparación de elementos JSX y atributos estáticos de los 252 componentes inventariados frente a Git, sin cambios accidentales en estructura, clases o atributos estáticos.

Resultados globales del cierre de esta revisión:

Comando	Resultado
`pnpm typecheck`	Correcto
`pnpm lint`	Correcto, sin avisos
`pnpm test`	Correcto: 236 pruebas, ningún fallo
`pnpm check`	Correcto; se corrigió el orden de imports detectado en core
`pnpm build`	Correcto, con aviso de bundle web superior a 500 kB

Las pruebas de UI existentes comprueban renderizado, estados, traducciones y contratos observables. No sustituyen una prueba manual en navegador de foco, teclado y apertura/cierre de todas las combinaciones de modales; esta revisión no incorpora un runner de navegador.

Pendientes y trabajo ajeno

No se ha necesitado una decisión arquitectónica para esta separación. Los cambios previos del usuario en TODO.md y el ZIP de referencia se conservan. Las nuevas funcionalidades anotadas allí, como búsqueda de logos SVG y routing, no forman parte de esta auditoría. El aviso de tamaño del bundle no impide el build. Su posible división requiere un trabajo específico de carga de pantallas y no se introduce como efecto secundario de esta separación.

5.1 KiB Raw Blame History