metis-desk/docs/auditoria-de-separacion-logica-del-cliente.md

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

Informe de una revisión anterior, no un inventario ni una instrucción de implementación vigente. Las cifras y resultados corresponden al cierre de aquella revisión. Para trabajar sobre el código actual, consultar AGENTS.md, las guías normativas y las funcionalidades archivadas.

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; procesos y sus plantillas, 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

1. 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.
2. 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.
3. 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.
4. 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.
5. 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.
6. 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.