Si falla la lectura del colectivo la pantalla de edicion se vuelve un borrado masivo silencioso

Hallazgo de audit-mba-l-cc el 2026-09-07 23:03 leyendo `origin/dev`; verificado por coder02-mba-l-cc contra el archivo antes de abrir el item. **NO se disparo esta noche** - la medicion de db02 sobre AA033TK dio la ficha intacta y `changes` con UNA sola clave. Es un camino LATENTE, y predatea todo lo shipeado hoy. EL MECANISMO, en dos mitades que solas no hacen daño y juntas si: MITAD 1 - `src/app/vehiculos/[dominio]/editar/colectivo/page.tsx:284-291`: ``` let colectivoData: Record<string, unknown> = {}; try { const result = await obtenerColectivo(Number(vehiculoId)); colectivoData = (result.success && result.colectivo ? result.colectivo : {}) as ...; } catch { // No colectivo data found, using defaults } ``` Si la action TIRA, o si devuelve `success:false`, `colectivoData` queda `{}` **en silencio**. Sin mensaje, sin fila de log, sin diferencia visible. Despues los campos se pueblan con `String(undefined || "")` = `""` y el formulario renderiza en blanco, con aspecto perfectamente normal. MITAD 2 - el formulario postea `formData` ENTERO en cada guardado. Lo dice el comentario del propio archivo en `:582-590`. Solo `entidadId` y `lineaId` se omiten cuando no cambiaron (`:604-605`); **todas las demas columnas del colectivo viajan siempre**, y en `repositories/vehiculos.ts:955-966` cada `""` se convierte en null explicito via `X ? parseInt(X) : null`. JUNTAS: una lectura fallida deja el formulario vacio, el usuario guarda cualquier cosa -o nada- y se escriben NULLs sobre todas las columnas del colectivo. **El usuario no ve nada raro en ningun momento**, ni al cargar ni al guardar. ASIMETRIA QUE LO HACE PEOR DE DIAGNOSTICAR: `estadoId` esta protegido POR ACCIDENTE. `:964` es el unico que saltea `''`; sus hermanos de `:955-963` y `:965` no. O sea que un vehiculo puede quedar con TODAS las columnas en null y el `estado_id` intacto - y como la ficha muestra el estado, **la pantalla sigue pareciendo sana**. Una ficha que muestra "Alta" NO es evidencia de que la fila este entera. POR QUE NO SE VIO ESTA NOCHE: la lectura anduvo, el formulario cargo con los valores reales, y entonces el post trajo los valores CORRECTOS de vuelta. Se comparo contra `beforeColectivo`, no difirieron, y `changes` quedo en `["observaciones"]`. El camino no se ejercito. ARREGLO, y hay que decidir la forma antes de escribir: 1. Que el `catch` NO se trague el error: si la lectura del colectivo falla, la pantalla no debe ofrecer un formulario editable. Mostrar el error y no dejar guardar es lo minimo. 2. Distinguir "no hay datos de colectivo" de "no se pudieron leer los datos de colectivo". Son dos estados distintos y hoy producen el mismo formulario en blanco. 3. Que el guardado mande solo lo que cambio, o que el repositorio distinga ausente de vacio para TODAS las columnas y no solo para estadoId, entidadId y lineaId. Ojo: la regla de omitir tiene su propia limitacion documentada en `:592-597` y no se toca sin medir desde el llamador real. CONTROL POSITIVO PARA CUANDO SE ARREGLE: forzar el fallo de `obtenerColectivo` y comprobar que la pantalla NO ofrece guardar. No alcanza con verificar que un guardado normal siga andando - eso es el camino que ya funciona. RELACIONADO: MOVILBA-336 (guardar sin cambios no dice nada) es el mismo silencio en el otro extremo del mismo formulario.