@runnerpro/backend 1.21.14 → 1.21.16

package/lib/cjs/sendMail/index.js

@@ -31,7 +31,7 @@ const sendMail = ({ subject, title, body, to, link, attachments, bcc }) => __awa
         return;
     const bodyHTML = getBodyHTML(title, body, link);
     const sendPromise = transporter.sendMail({
-        from: process.env.GMAIL_EMAIL_USER,
+        from: `"RunnerPro" <${process.env.GMAIL_EMAIL_USER}>`,
         to: toMapped,
         bcc: process.env.NODE_ENV === 'PROD' && bcc ? bcc.map((correo) => correo).join(',') : '',
         subject,

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":"AAwRA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAsEvB,CAAC;AAuEF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutAplication.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutAplication.ts"],"names":[],"mappings":"AA4SA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAsEvB,CAAC;AAuEF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -149,19 +149,25 @@ const similar = (a, b, tol) => {
     return Math.abs(na - nb) / Math.max(na, nb) <= tol;
 };
 // Ventana de solape de hora de inicio para el dedup cross-proveedor. Las
-// grabaciones de la MISMA actividad (típicamente un mismo reloj que sincroniza a
-// Strava/Apple/Garmin) empiezan en el mismo instante; 5 min absorben cualquier
-// desfase de redondeo/zona horaria sin abrir falsos positivos (no se inician dos
-// actividades del mismo grupo de tipo en 5 min).
-const VENTANA_INICIO_MS = 5 * 60 * 1000;
-const inicioSolapa = (a, b) => {
+// grabaciones de la MISMA actividad (un mismo reloj que sincroniza a
+// Strava/Apple/Garmin) empiezan casi en el mismo instante; 30 min absorben el
+// desfase de relojes que arrancan la grabación tarde sin abrir falsos positivos
+// (no se inician dos actividades del mismo grupo de tipo en 30 min).
+const VENTANA_INICIO_MS = 30 * 60 * 1000;
+// Ventana ampliada cuando una de las dos caras es un completado MANUAL: su
+// "START AT" es una ESTIMACIÓN (hora de marcado − duración) que puede desviarse
+// bastante de la hora real del reloj (el cliente marca hecho un rato después de
+// terminar, o redondea la duración). 2h la absorben; si aun así no solapan, son
+// dos entrenos distintos y se respetan como dobles.
+const VENTANA_INICIO_MANUAL_MS = 2 * 60 * 60 * 1000;
+const inicioSolapa = (a, b, ventanaMs = VENTANA_INICIO_MS) => {
     if (!a || !b)
         return false;
     const ta = new Date(a).getTime();
     const tb = new Date(b).getTime();
     if (Number.isNaN(ta) || Number.isNaN(tb))
         return false;
-    return Math.abs(ta - tb) <= VENTANA_INICIO_MS;
+    return Math.abs(ta - tb) <= ventanaMs;
 };
 const buscarWorkoutSimilar = (q, opts, dateStr, pt) => __awaiter(void 0, void 0, void 0, function* () {
     // Pre-filtro por ventana de día ±1: el match real es por hora de inicio (no por
@@ -170,6 +176,10 @@ const buscarWorkoutSimilar = (q, opts, dateStr, pt) => __awaiter(void 0, void 0,
     // día cubre ese borde sin ensanchar el rango de más.
     const dayPrev = (0, moment_1.default)(dateStr, 'YYYY-MM-DD').subtract(1, 'day').format('YYYY-MM-DD');
     const dayNext = (0, moment_1.default)(dateStr, 'YYYY-MM-DD').add(1, 'day').format('YYYY-MM-DD');
+    // Candidatos: entrenos hechos ese día (±1) del mismo grupo de tipo con OTRA fuente
+    // — INCLUYE los MANUAL, para que un entreno sincronizado que llega pueda reclamar
+    // el que el cliente ya marcó a mano y fusionarse con él (los datos del reloj mandan,
+    // el título/estructura del plan se conservan).
     const rows = yield q(`SELECT "ID", "DURATION", "DISTANCE", "TIPO APLICATION", "START AT"
        FROM "WORKOUT"
       WHERE "ID CLIENTE" = ? AND "DATE" BETWEEN ? AND ? AND ${pt.sql}
@@ -177,12 +187,14 @@ const buscarWorkoutSimilar = (q, opts, dateStr, pt) => __awaiter(void 0, void 0,
     const cand = rows.filter((r) => {
         // Señal robusta: solape de hora de inicio. Inmune a las unidades de distancia
         // (Apple en millas) y a elapsed-vs-moving en la duración (webhook Strava).
+        // Ventana ampliada a 2h si el candidato es MANUAL (su "START AT" es estimado).
         // Sólo aplica si AMBAS filas tienen "START AT".
+        const ventana = String(r.tipoAplication) === 'MANUAL' ? VENTANA_INICIO_MANUAL_MS : VENTANA_INICIO_MS;
         if (opts.startAt && r.startAt)
-            return inicioSolapa(r.startAt, opts.startAt);
-        // Fallback legacy: filas sin "START AT" (anteriores a esta feature, sin
-        // backfill) → se mantiene el heurístico distancia+duración para no regresar.
-        return similar(r.duration, opts.duration, 0.1) && similar(r.distance, opts.distance, 0.1);
+            return inicioSolapa(r.startAt, opts.startAt, ventana);
+        // Fallback legacy: filas sin "START AT" (manuales antiguos o pre-feature) →
+        // heurístico distancia+duración para no regresar.
+        return similar(r.duration, opts.duration, 0.25) && similar(r.distance, opts.distance, 0.25);
     });
     cand.sort((a, b) => prioridadFuente(b.tipoAplication) - prioridadFuente(a.tipoAplication));
     return cand[0] || null;
@@ -224,7 +236,7 @@ const registrarFuente = (q, idWorkout, tipo, idApp) => __awaiter(void 0, void 0,
      DO UPDATE SET "ID APLICATION" = EXCLUDED."ID APLICATION", "INGESTED AT" = NOW()`, [idWorkout, String(tipo), idApp !== null && idApp !== undefined ? String(idApp) : null]);
 });
 const aplicarMerge = (q, idWorkout, opts, esFilaNueva) => __awaiter(void 0, void 0, void 0, function* () {
-    var _a, _b, _c;
+    var _a, _b, _c, _d;
     const [row] = yield q(`SELECT "DISTANCE", "DURATION", "POTENCIA", "DESNIVEL", "CALORIES", "FC MEDIA",
             "ZONE", "POLYLINE", "TIPO APLICATION", "ID APLICATION", "START AT"
        FROM "WORKOUT" WHERE "ID" = ? FOR UPDATE`, [idWorkout]);
@@ -241,14 +253,19 @@ const aplicarMerge = (q, idWorkout, opts, esFilaNueva) => __awaiter(void 0, void
     const zone = (0, paceZone_1.getZone)({ duration, distance, zones: cli });
     // POLYLINE: la primera fuente que la puso manda; nunca se pisa.
     const polyline = row && row.polyline ? row.polyline : getPolyline(opts.polyline);
-    // START AT: la primera fuente que lo puso manda; nunca se pisa (igual que
-    // POLYLINE). Es la clave del dedup cross-proveedor (solape de hora de inicio).
-    const startAt = row && row.startAt ? row.startAt : (_c = opts.startAt) !== null && _c !== void 0 ? _c : null;
-    // TIPO/ID APLICATION: sólo se setean en INSERT o si la fila no tenía fuente;
-    // nunca se reemplazan (la primera fuente se queda).
+    // ¿La fila existente es un completado MANUAL? Su "START AT" es una estimación y su
+    // fuente es la de menor prioridad, así que una fuente REAL entrante debe pisarla.
+    const existingIsManual = String(row === null || row === void 0 ? void 0 : row.tipoAplication) === 'MANUAL';
+    // START AT: la primera fuente que lo puso manda y nunca se pisa (igual que
+    // POLYLINE) — EXCEPTO si la existente es la estimación de un MANUAL y llega una
+    // fuente real: entonces la hora real del reloj pisa la estimada.
+    const startAt = row && row.startAt && !existingIsManual ? row.startAt : (_d = (_c = opts.startAt) !== null && _c !== void 0 ? _c : row === null || row === void 0 ? void 0 : row.startAt) !== null && _d !== void 0 ? _d : null;
+    // TIPO/ID APLICATION: se setean en INSERT, si la fila no tenía fuente, o si la fila
+    // era MANUAL y llega una fuente de más prioridad (reloj) que la reemplaza como
+    // principal. Entre dos fuentes reales, la primera se queda (no se reemplaza).
     let tipoFinal;
     let idFinal;
-    if (esFilaNueva || !(row === null || row === void 0 ? void 0 : row.tipoAplication)) {
+    if (esFilaNueva || !(row === null || row === void 0 ? void 0 : row.tipoAplication) || (existingIsManual && prioNueva > prioActual)) {
         tipoFinal = opts.aplicationType;
         idFinal = opts.aplicationId;
     }
@@ -294,7 +311,7 @@ _data) => __awaiter(void 0, void 0, void 0, function* () {
     // extra para clientes no-ES con doble proveedor, a cambio de no bloquear el lock.
     const translated = yield (0, titleDescriptionTranslated_1.getTitleDescriptionTranslated)(opts.idCliente, opts.title, opts.description);
     const idWorkout = yield withTx((q) => __awaiter(void 0, void 0, void 0, function* () {
-        var _d;
+        var _e;
         // Lock anti-carrera: serializa ingestas del mismo cliente/día/bucket-tipo
         // (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)}`)]);
@@ -327,7 +344,7 @@ _data) => __awaiter(void 0, void 0, void 0, function* () {
         const { title: titleNoTranslate, description: descriptionNoTranslate, titlePreferredLanguage, descriptionPreferredLanguage } = translated;
         const inserted = yield q(`INSERT INTO "WORKOUT"
          ("ID CLIENTE", "DATE", "START AT", "TITLE", "DESCRIPTION", "TITLE PREFERRED LANGUAGE", "DESCRIPTION PREFERRED LANGUAGE", "TYPE", "DONE", "SHOW CLIENT")
-       VALUES (?, ?, ?, ?, ?, ?, ?, ?, TRUE, TRUE) RETURNING "ID"`, [opts.idCliente, dateStr, (_d = opts.startAt) !== null && _d !== void 0 ? _d : null, titleNoTranslate, descriptionNoTranslate, titlePreferredLanguage, descriptionPreferredLanguage, opts.type]);
+       VALUES (?, ?, ?, ?, ?, ?, ?, ?, TRUE, TRUE) RETURNING "ID"`, [opts.idCliente, dateStr, (_e = opts.startAt) !== null && _e !== void 0 ? _e : null, titleNoTranslate, descriptionNoTranslate, titlePreferredLanguage, descriptionPreferredLanguage, opts.type]);
         if (!inserted || !inserted[0])
             return null;
         const id = inserted[0].id;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@runnerpro/backend",
-  "version": "1.21.14",
+  "version": "1.21.16",
   "description": "A collection of common backend functions",
   "exports": {
     ".": "./lib/cjs/index.js"
@@ -53,7 +53,7 @@
   },
   "peerDependencies": {
     "@napi-rs/canvas": "^0.1.53",
-    "@runnerpro/common": "^1.7.0",
+    "@runnerpro/common": "^1.7.3",
     "axios": "^1.6.7",
     "image-size": "^1.0.2",
     "jimp": "^0.22.10",
@@ -81,4 +81,4 @@
     "oauth-signature": "1.5.0",
     "socket.io": "^4.7.2"
   }
-}
+}