@runnerpro/backend 1.21.0 → 1.21.2

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

@@ -1 +1 @@
-{"version":3,"file":"estructuraWorkout.d.ts","sourceRoot":"","sources":["../../../../src/workout/estructuraWorkout.ts"],"names":[],"mappings":"AAuCA,QAAA,MAAM,mBAAmB,YAAa,GAAG,cAAc,GAAG,QAiCzD,CAAC;AAEF;;;;;;;;;;;;;;;;GAgBG;AACH,QAAA,MAAM,wBAAwB,cAAqB,GAAG,kBAkBrD,CAAC;AAuOF,OAAO,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,CAAC"}
+{"version":3,"file":"estructuraWorkout.d.ts","sourceRoot":"","sources":["../../../../src/workout/estructuraWorkout.ts"],"names":[],"mappings":"AAuCA,QAAA,MAAM,mBAAmB,YAAa,GAAG,cAAc,GAAG,QAiCzD,CAAC;AAEF;;;;;;;;;;;;;;;;GAgBG;AACH,QAAA,MAAM,wBAAwB,cAAqB,GAAG,kBAkBrD,CAAC;AAuPF,OAAO,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,CAAC"}

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":"AAsPA;;;;;;;;;GASG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAmDvB,CAAC;AA+DF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutAplication.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutAplication.ts"],"names":[],"mappings":"AAwPA;;;;;;;;;GASG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CA0DvB,CAAC;AA+DF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}

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

@@ -1 +1 @@
-{"version":3,"file":"saveWorkoutLaps.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutLaps.ts"],"names":[],"mappings":"AAGA;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AACH,QAAA,MAAM,eAAe,cAAqB,GAAG,QAAQ,GAAG,EAAE,mBAAmB,GAAG,KAAG,QAAQ,IAAI,CAuB9F,CAAC;AAEF,OAAO,EAAE,eAAe,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutLaps.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutLaps.ts"],"names":[],"mappings":"AAGA;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AAEH,QAAA,MAAM,eAAe,cAAqB,GAAG,QAAQ,GAAG,EAAE,mBAAmB,GAAG,KAAG,QAAQ,IAAI,CAuB9F,CAAC;AAEF,OAAO,EAAE,eAAe,EAAE,CAAC"}

package/lib/cjs/workout/estructuraWorkout.js

@@ -116,7 +116,17 @@ const saveDoneStructuraWorkoutWatch = (idWorkout, streamData, laps, workout, est
     });
     const numPasos = estructura.length;
     const numLaps = laps.length;
-    if (numPasos !== numLaps && numPasos !== numLaps + 1) {
+    // Casos aceptados:
+    //  - numLaps === numPasos: cada paso tiene su lap (1:1).
+    //  - numLaps === numPasos - 1: el cooldown no se marcó como lap aparte
+    //    (típico cuando el plan acaba en el último intervalo de trabajo y el
+    //    runner enfría sin pulsar lap).
+    //  - numLaps === numPasos + 1: el runner siguió moviéndose tras la vuelta
+    //    a la calma planeada antes de parar el reloj → el lap "extra" final
+    //    se absorbe en el último paso del plan (ver bucle abajo).
+    // Cualquier otra desviación es anómala (faltan laps en medio, doble extra,
+    // etc.) y se reporta para auditoría manual.
+    if (numPasos !== numLaps && numPasos !== numLaps + 1 && numLaps !== numPasos + 1) {
         return (0, sendMail_1.sendMail)({
             title: 'Estructura workout: laps no coinciden con pasos',
             subject: 'Dedup cross-proveedor - Error saving steps structure workout',
@@ -155,7 +165,13 @@ const saveDoneStructuraWorkoutWatch = (idWorkout, streamData, laps, workout, est
         }
         if (lapIndex !== null && lapIndex >= 0 && lapIndex < laps.length) {
             const currentLap = laps[lapIndex];
-            const nextLap = laps[lapIndex + 1];
+            // Si hay un lap más que pasos y estamos en el último paso (cooldown),
+            // absorbemos el lap extra (post-cooldown): ignoramos laps[lapIndex+1]
+            // como límite y consideramos toda la actividad hasta workout.duration
+            // como parte de este paso. La duración resultante = cooldown planeado
+            // + lo que el runner siguió moviéndose después.
+            const isLastStepAbsorbingExtraLap = numLaps === numPasos + 1 && i === numPasos - 1;
+            const nextLap = isLastStepAbsorbingExtraLap ? null : laps[lapIndex + 1];
             if (nextLap) {
                 duration = nextLap.durationStartLap - currentLap.durationStartLap;
             }

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -95,6 +95,7 @@ function camelizeRow(row) {
     });
     return aux;
 }
+// eslint-disable-next-line no-unused-vars, @typescript-eslint/no-unused-vars
 const withTx = (fn) => __awaiter(void 0, void 0, void 0, function* () {
     const client = yield db_1.pool.connect();
     try {
@@ -252,9 +253,16 @@ _data) => __awaiter(void 0, void 0, void 0, function* () {
         // (Strava-webhook y Garmin-push pueden entrar casi a la vez).
         yield q('SELECT pg_advisory_xact_lock(?)', [hashInt(`${opts.idCliente}|${dateStr}|${bucketTipo(opts.type)}`)]);
         // 1) dedup intra-proveedor / completar planificado (comportamiento actual común).
+        //    Ambas ramas filtran por "ID CLIENTE": el activityId de un proveedor es
+        //    global pero pertenece a un único usuario en su plataforma; si por
+        //    cualquier motivo (test, re-conexión OAuth con cuenta distinta, fallo
+        //    de resolución upstream) llegara con un idCliente distinto al que ya
+        //    tenía esa fila, NO debemos hacer merge cross-cliente.
         const [exist] = yield q(`SELECT "ID", "TYPE" FROM "WORKOUT"
-        WHERE ("ID CLIENTE" = ? AND "DATE" = ? AND ${pt.sql} AND "DONE" = FALSE AND "SHOW CLIENT" = TRUE)
-           OR ("ID APLICATION" = ? AND "TIPO APLICATION" = ?)
+        WHERE "ID CLIENTE" = ? AND (
+              ("DATE" = ? AND ${pt.sql} AND "DONE" = FALSE AND "SHOW CLIENT" = TRUE)
+              OR ("ID APLICATION" = ? AND "TIPO APLICATION" = ?)
+        )
         LIMIT 1`, [opts.idCliente, dateStr, ...pt.params, opts.aplicationId, opts.aplicationType]);
         if (exist) {
             yield aplicarMerge(q, exist.id, opts, false);

package/lib/cjs/workout/saveWorkoutLaps.js

@@ -37,6 +37,7 @@ const estructuraWorkout_1 = require("./estructuraWorkout");
  * @param {{index:number,durationStartLap:number}[]} laps
  * @param {number} aplicationType  TIPO_APLICACION.* (reservado para trazabilidad)
  */
+// eslint-disable-next-line no-unused-vars, @typescript-eslint/no-unused-vars
 const saveWorkoutLaps = (idWorkout, laps, aplicationType) => __awaiter(void 0, void 0, void 0, function* () {
     if (!idWorkout)
         return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@runnerpro/backend",
-  "version": "1.21.0",
+  "version": "1.21.2",
   "description": "A collection of common backend functions",
   "exports": {
     ".": "./lib/cjs/index.js"