El comentario del exportador nombraba como contrato una coincidencia de datos: PROD lee el expediente de vdc y DEV del periodo vigente
El comentario de src/lib/exports/vehiculos.ts:110 afirmaba que expedienteAlta es el expediente del periodo vigente que entrega v_vehiculos_export. Es cierto en DEV y falso en PROD: la vista instalada en DEV toma los dos campos de expediente del periodo normalizado vigente, y la de PROD los toma de vehiculos_datos_colectivos. Medido por coder04-mba-l-ca sobre las dos bases: el export coincide con la fuente que declara cada vista (DEV 2424/2424 Alta y Baja, PROD 2413/2413 Alta y Baja), y vdc contra el periodo normalizado vigente coincide 2413/2413 en PROD pero YA DIVERGE en DEV, 10 de 2424 en Alta y 1 de 2424 en Baja. O sea que la coincidencia de PROD es una propiedad de los datos de hoy, no un contrato, y el comentario viejo nombraba como contrato algo que puede separarse sin que nada avise. Segundo defecto, encontrado por audit sobre el mismo archivo y de la misma forma: la linea 115-116 decia 'App reads the view as-is' sin calificar, mientras la misma funcion reescribe tipoVehiculo, categoriaCarroceria, marcaChasis, modeloChasis y tramite (151-164), borra comentariosImportacion fuera de la allowlist (171) y lo vuelve a agregar desde una consulta suplementaria en hosts fieles (178-182). Se acoto a lo que la nota de MOBAAPI-661 queria decir: el export no remapea los campos de motor de la vista. Queda ABIERTO y no es parte de este item: la deriva de contrato entre las dos vistas - PROD sigue leyendo vdc.expte_alta/expte_baja y DEV el periodo - que se cierra con una migracion, no con un comentario.
Activity
-
Implementacion completa en el checkout compartido, esperando veredicto de audit. Item creado tarde: el cambio se hizo antes de que existiera el ticket, que es la razon por la que coder04-mba-l-ca tuvo que preguntar a que item colgarle los eventos de audit.
-
audit-mba-l-ca | 2026-09-23 20:58:35 -03 FAIL (first review round, recorded retrospectively after the item was created): lines 110-113 correctly described the measured DEV/PROD expediente source split, but the adjacent lines 115-116 still claimed unqualified that the app read the view as-is. That was false within the same function, which remaps several non-motor fields and conditionally replaces comentariosImportacion. Required correction: scope the statement to the intended motor-field mechanism.
-
audit-mba-l-ca | 2026-09-23 20:58:35 -03 PASS (final re-review): the isolated comment hunk accurately states that DEV sources expediente_alta/expediente_baja from the current normalized period while PROD sources both from vehiculos_datos_colectivos; it explicitly treats today's 2413/2413 PROD equality as data, not contract, and asserts no convergence plan. The prior failing sentence is removed; the replacement narrowly states that the export does not remap the view's motor fields, consistent with the implementation. passes. Coverage limitation: coder04's enumerated historical/runtime claims about validator emission, host-gated comentarios_importacion, ordering, other remaps, and dated tied-row counts remain UNEXERCISED by this review.
-
coder03-mba-l-cc, 2026-09-23 20:57 ART. Shipeado a dev: v5.6.43, commit dbaefac6, un solo hunk, src/lib/exports/vehiculos.ts y nada mas - el trabajo de MOVILBA-552 que esta sucio en el arbol quedo afuera por --only. Audit dejo en este item las DOS vueltas firmadas: el audit-finding con el FAIL original sobre las lineas 115-116 sin calificar, y el verified con el PASS posterior al acotamiento a los campos de motor, declarando NO EJERCITADAS las afirmaciones historicas y de runtime. Verifique yo contra el arbol que la frase vieja no sobrevive antes de commitear. QUEDA ABIERTO en su propio item: la deriva de contrato entre las dos vistas - PROD lee expte_alta/expte_baja de vehiculos_datos_colectivos y DEV del periodo normalizado vigente - que se cierra con una migracion, no con un comentario, y que DEV ya esta separando (10 de 2424 en Alta, 1 en Baja contra vdc).
-
Lane report: measured installed v_vehiculos_export definitions in DEV and PROD for both expediente fields. DEV sources expediente_alta and expediente_baja from the current normalized period; PROD sources both from vehiculos_datos_colectivos. The export matches each installed view's declared source (DEV 2424/2424 for Alta and Baja; PROD 2413/2413 for Alta and Baja). VDC versus current normalized period agrees 2413/2413 in PROD but diverges in DEV for 10 Alta and 1 Baja, proving the PROD equality is current data rather than contract. Updated src/lib/exports/vehiculos.ts comments to state that split without promising convergence. First audit was seen FAILING because the adjacent unqualified 'App reads the view as-is' claim contradicted remaps in the same function; narrowed it to the true motor-field claim. git diff --check passed. Audit recorded both the signed audit-finding FAIL and final verified PASS directly above, with the stated coverage limitations. coder03 subsequently committed and deployed the isolated hunk as v5.6.43 / dbaefac6 and closed the item.
-
coder04-mba-l-ca | 2026-09-23 21:02 ART SIGNED LANE REPORT. Measured the installed DEV/PROD v_vehiculos_export definitions: DEV sources both expediente fields from the current normalized period; PROD sources both from vehiculos_datos_colectivos. PROD agrees 2413/2413 today while DEV already diverges by 10 Alta and 1 Baja. Corrected the comments, then narrowed the first-audit failure 'App reads the view as-is' to the true motor-field statement. git diff --check passed. This signed event supersedes the unsigned lane progress event at 2026-09-23T23:59:14.428Z.
-
audit fail: Historical first-round audit verdict from 2026-09-23: src/lib/exports/vehiculos.ts comments at lines 110-116; audit predates the bwi audit command.
payload
{"scope":"Historical first-round audit verdict from 2026-09-23: src/lib/exports/vehiculos.ts comments at lines 110-116; audit predates the bwi audit command.","outcome":"fail","evidence":"Lines 115-116 falsely said the app read the view as-is although the same function remapped non-motor fields and conditionally replaced comentariosImportacion; required narrowing to motor fields.","reportRef":"MOVILBA-564 audit-finding event 2026-09-23T23:58:37.842Z"}