@runnerpro/backend 1.34.0 → 1.34.1

package/lib/cjs/types/workout/saveWorkoutAplication.d.ts

@@ -1,3 +1,24 @@
+/**
+ * Algunas ingestas entregan la distancia del RESUMEN en millas pero rotulada en metros
+ * (9.051 m reales -> se guarda 5.600, que son 5,6 millas x 1000). El gráfico y los
+ * parciales salen bien porque se calculan del stream, pero la distancia total y el ritmo
+ * medio quedan ~1,61x peores, y `getZone` clasifica mal porque sale de duración/distancia.
+ *
+ * El stream SÍ viene en metros, así que hace de árbitro: se corrige sólo si reconstruir
+ * "millas x 1000" desde el stream reproduce la distancia candidata con menos de un 1% de
+ * desvío.
+ *
+ * OJO con leer el filtro de ratio de más abajo como un segundo criterio independiente:
+ * NO lo es, es la misma cuenta despejada. Sirve para descartar barato el 99,99% de las
+ * ingestas sin dividir. El rango que decide de verdad es ratio ∈ [1,5932 – 1,6254].
+ *
+ * Ese pasillo es estrecho a propósito, y por eso la corrección deja traza: en producción
+ * hay entrenos SANOS pegados a los dos bordes —w699672 a ratio 1,5784 y w694890 a 1,6301,
+ * con el stream más largo que el resumen por pausas o deriva de GPS—, y entre 1,40 y 1,80
+ * hay una nube continua de entrenos legítimos. Si alguno acaba cayendo dentro, el rastro
+ * es lo único que permitirá distinguirlo de un fallo de la fuente.
+ */
+declare const corregirDistanciaEnMillas: (distance: any, streamFinal: number | null) => any;
 /**
  * @param opts {{
  *   date, type, title, description, distance, duration, power, desnivel, polyline,
@@ -10,5 +31,5 @@
  * @returns {Promise<number|null>} idWorkout (o null si no se pudo crear) — firma intacta
  */
 declare const saveWorkoutAplication: (opts: any, _data?: any) => Promise<number | null>;
-export { saveWorkoutAplication };
+export { saveWorkoutAplication, corregirDistanciaEnMillas };
 //# sourceMappingURL=saveWorkoutAplication.d.ts.map

package/lib/cjs/types/workout/saveWorkoutAplication.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"saveWorkoutAplication.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutAplication.ts"],"names":[],"mappings":"AAqTA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAqFvB,CAAC;AAuGF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutAplication.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutAplication.ts"],"names":[],"mappings":"AA2RA;;;;;;;;;;;;;;;;;;;GAmBG;AACH,QAAA,MAAM,yBAAyB,aAAc,GAAG,eAAe,MAAM,GAAG,IAAI,KAAG,GAe9E,CAAC;AA8FF;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAqFvB,CAAC;AAuGF,OAAO,EAAE,qBAAqB,EAAE,yBAAyB,EAAE,CAAC"}

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -12,7 +12,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.saveWorkoutAplication = void 0;
+exports.corregirDistanciaEnMillas = exports.saveWorkoutAplication = void 0;
 const db_1 = require("../db");
 const almacenamiento_1 = require("../almacenamiento");
 const axios_1 = __importDefault(require("axios"));
@@ -243,6 +243,95 @@ const registrarFuente = (q, idWorkout, tipo, idApp) => __awaiter(void 0, void 0,
      ON CONFLICT ("ID WORKOUT", "TIPO APLICATION")
      DO UPDATE SET "ID APLICATION" = EXCLUDED."ID APLICATION", "INGESTED AT" = NOW()`, [idWorkout, String(tipo), idApp !== null && idApp !== undefined ? String(idApp) : null]);
 });
+// =============================================================================
+// Guardarraíl de unidad: distancia en MILLAS rotulada como metros
+// =============================================================================
+const METROS_POR_MILLA = 1609.344;
+// Desvío máximo admitido al reconstruir la candidata desde el stream. NO se puede
+// apretar: Strava redondea la distancia del resumen a una décima de milla, y en un
+// entreno corto ese redondeo ya vale casi un 1% (0,05 mi sobre 3,9 mi = 1,3%).
+const TOLERANCIA_RECONSTRUCCION = 0.01;
+/**
+ * Último valor útil del stream de distancia, en metros. Mira primero el stream que
+ * entra con esta ingesta y, si no viene, el que ya esté guardado para el workout
+ * (p.ej. lo dejó otra fuente). Devuelve null si no hay ninguno: sin árbitro no se
+ * toca la distancia.
+ *
+ * La consulta pide SÓLO el último punto (`-> -1`) y no el array entero: un stream de
+ * distancia son miles de puntos en un campo TOAST, y esto corre dentro de la tx que
+ * sostiene el advisory lock de la ingesta. Traerlo completo para leer un número sería
+ * alargar el lock que ya es el punto donde más se cae el pool.
+ */
+const distanciaFinalDelStream = (q, idWorkout, streams) => __awaiter(void 0, void 0, void 0, function* () {
+    const valido = (v) => {
+        const n = Number(v);
+        return Number.isFinite(n) && n > 0 ? n : null;
+    };
+    const entrantes = streams === null || streams === void 0 ? void 0 : streams.distance;
+    if (Array.isArray(entrantes) && entrantes.length > 0) {
+        const ultimo = valido(entrantes[entrantes.length - 1]);
+        if (ultimo !== null)
+            return ultimo;
+    }
+    const [row] = yield q(`SELECT CASE WHEN jsonb_typeof("DATA"::jsonb) = 'array' AND jsonb_array_length("DATA"::jsonb) > 0
+                 THEN "DATA"::jsonb -> -1 ELSE NULL END AS ultimo
+       FROM "WORKOUT STREAM" WHERE "ID WORKOUT" = ? AND "METRIC TYPE" = ?`, [idWorkout, 'distance']);
+    return row ? valido(row.ultimo) : null;
+});
+/**
+ * Algunas ingestas entregan la distancia del RESUMEN en millas pero rotulada en metros
+ * (9.051 m reales -> se guarda 5.600, que son 5,6 millas x 1000). El gráfico y los
+ * parciales salen bien porque se calculan del stream, pero la distancia total y el ritmo
+ * medio quedan ~1,61x peores, y `getZone` clasifica mal porque sale de duración/distancia.
+ *
+ * El stream SÍ viene en metros, así que hace de árbitro: se corrige sólo si reconstruir
+ * "millas x 1000" desde el stream reproduce la distancia candidata con menos de un 1% de
+ * desvío.
+ *
+ * OJO con leer el filtro de ratio de más abajo como un segundo criterio independiente:
+ * NO lo es, es la misma cuenta despejada. Sirve para descartar barato el 99,99% de las
+ * ingestas sin dividir. El rango que decide de verdad es ratio ∈ [1,5932 – 1,6254].
+ *
+ * Ese pasillo es estrecho a propósito, y por eso la corrección deja traza: en producción
+ * hay entrenos SANOS pegados a los dos bordes —w699672 a ratio 1,5784 y w694890 a 1,6301,
+ * con el stream más largo que el resumen por pausas o deriva de GPS—, y entre 1,40 y 1,80
+ * hay una nube continua de entrenos legítimos. Si alguno acaba cayendo dentro, el rastro
+ * es lo único que permitirá distinguirlo de un fallo de la fuente.
+ */
+const corregirDistanciaEnMillas = (distance, streamFinal) => {
+    const d = Number(distance);
+    if (!Number.isFinite(d) || d <= 0 || streamFinal === null || !(streamFinal > 0))
+        return distance;
+    // Filtro barato: fuera de la horquilla de la milla no hay nada que mirar.
+    const ratio = streamFinal / d;
+    if (ratio < 1.58 || ratio > 1.64)
+        return distance;
+    // La fuente trató las MILLAS como si fueran kilómetros: el valor que nos llega es
+    // "millas x 1000" (5,6 mi -> 5600). Reconstruimos ese número desde el stream y exigimos
+    // que reproduzca la candidata; si no, el desajuste tiene otra causa y no se toca.
+    const millasEscaladas = (streamFinal / METROS_POR_MILLA) * 1000;
+    if (Math.abs(millasEscaladas - d) / d > TOLERANCIA_RECONSTRUCCION)
+        return distance;
+    return Math.round(streamFinal / 10) * 10;
+};
+exports.corregirDistanciaEnMillas = corregirDistanciaEnMillas;
+const resolverDistanciaConStream = (q, idWorkout, distance, streams) => __awaiter(void 0, void 0, void 0, function* () {
+    const d = Number(distance);
+    if (!Number.isFinite(d) || d <= 0)
+        return distance;
+    const streamFinal = yield distanciaFinalDelStream(q, idWorkout, streams);
+    const corregida = corregirDistanciaEnMillas(distance, streamFinal);
+    // Traza obligatoria: esto pisa un dato del cliente por heurística y en un margen fino.
+    // Sin rastro, un "mi entreno marca de más" no se podría distinguir de un fallo de la
+    // fuente, ni se sabría cuántas veces ha actuado el guardarraíl.
+    if (Number(corregida) !== d) {
+        // eslint-disable-next-line no-console
+        console.warn(`corregirDistanciaEnMillas: idWorkout=${idWorkout} DISTANCE ${d} -> ${corregida} ` +
+            `(stream=${streamFinal}, ratio=${(Number(streamFinal) / d).toFixed(4)}). ` +
+            'La fuente entregó la distancia del resumen en millas.');
+    }
+    return corregida;
+});
 const aplicarMerge = (q, idWorkout, opts, esFilaNueva) => __awaiter(void 0, void 0, void 0, function* () {
     var _a, _b, _c, _d;
     const [row] = yield q(`SELECT "DISTANCE", "DURATION", "POTENCIA", "DESNIVEL", "CALORIES", "FC MEDIA",
@@ -250,7 +339,10 @@ const aplicarMerge = (q, idWorkout, opts, esFilaNueva) => __awaiter(void 0, void
        FROM "WORKOUT" WHERE "ID" = ? FOR UPDATE`, [idWorkout]);
     const prioNueva = prioridadFuente(opts.aplicationType);
     const prioActual = esFilaNueva ? -1 : prioridadFuente(row === null || row === void 0 ? void 0 : row.tipoAplication);
-    const distance = resolverEscalar(row === null || row === void 0 ? void 0 : row.distance, opts.distance, prioActual, prioNueva);
+    const distanceMerge = resolverEscalar(row === null || row === void 0 ? void 0 : row.distance, opts.distance, prioActual, prioNueva);
+    // El stream (metros) arbitra sobre la distancia del resumen: corrige el caso en que la
+    // fuente la entrega en millas. Va ANTES de getZone, que se calcula con esta distancia.
+    const distance = yield resolverDistanciaConStream(q, idWorkout, distanceMerge, opts.streams);
     const duration = resolverEscalar(row === null || row === void 0 ? void 0 : row.duration, opts.duration, prioActual, prioNueva);
     const potencia = resolverEscalar(row === null || row === void 0 ? void 0 : row.potencia, opts.power, prioActual, prioNueva);
     const desnivel = resolverEscalar(row === null || row === void 0 ? void 0 : row.desnivel, opts.desnivel, prioActual, prioNueva);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@runnerpro/backend",
-  "version": "1.34.0",
+  "version": "1.34.1",
   "description": "A collection of common backend functions",
   "exports": {
     ".": "./lib/cjs/index.js"
@@ -20,7 +20,8 @@
     "test:generate-share-map-guard": "ts-node --transpile-only -P configs/tsconfig.cjs.json scripts/testGenerateShareMapGuard.ts",
     "test:pg-query-conexion": "ts-node --transpile-only -P configs/tsconfig.cjs.json scripts/testPgQueryConexion.ts",
     "format": "prettier --write \"src/**/*.{js,jsx,ts,tsx,json,css,scss,md}\"",
-    "prepare": "husky"
+    "prepare": "husky",
+    "test:distancia-millas": "ts-node --transpile-only -P configs/tsconfig.cjs.json scripts/testDistanciaMillas.ts"
   },
   "release": {
     "branches": [