@runnerpro/backend 1.32.0 → 1.33.1

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

@@ -1 +1 @@
-{"version":3,"file":"garmin.d.ts","sourceRoot":"","sources":["../../../../src/workout/garmin.ts"],"names":[],"mappings":"AAoBA,UAAU,iBAAiB;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,KAAK,CAAC,EAAE,iBAAiB,EAAE,CAAC;CAC7B;AAED,UAAU,aAAa;IACrB,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,eAAe,EAAE,MAAM,CAAC;IACxB,eAAe,EAAE,MAAM,CAAC;IACxB,KAAK,EAAE,iBAAiB,EAAE,CAAC;CAC5B;AA4FD,QAAA,MAAM,sBAAsB,iCAyD3B,CAAC;AA+EF,QAAA,MAAM,mBAAmB;;;EAWxB,CAAC;AAEF,QAAA,MAAM,mBAAmB,6CASxB,CAAC;AAEF,QAAA,MAAM,mBAAmB,6CAMxB,CAAC;AAEF,QAAA,MAAM,qBAAqB,6CAU1B,CAAC;AAEF,QAAA,MAAM,2BAA2B,4DAWhC,CAAC;AAIF,OAAO,EACL,sBAAsB,EACtB,mBAAmB,EACnB,mBAAmB,EACnB,mBAAmB,EACnB,qBAAqB,EACrB,2BAA2B,GAC5B,CAAC"}
+{"version":3,"file":"garmin.d.ts","sourceRoot":"","sources":["../../../../src/workout/garmin.ts"],"names":[],"mappings":"AAoBA,UAAU,iBAAiB;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,KAAK,CAAC,EAAE,iBAAiB,EAAE,CAAC;CAC7B;AAED,UAAU,aAAa;IACrB,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,eAAe,EAAE,MAAM,CAAC;IACxB,eAAe,EAAE,MAAM,CAAC;IACxB,KAAK,EAAE,iBAAiB,EAAE,CAAC;CAC5B;AAgHD,QAAA,MAAM,sBAAsB,iCA4D3B,CAAC;AAmFF,QAAA,MAAM,mBAAmB;;;EAcxB,CAAC;AAEF,QAAA,MAAM,mBAAmB,6CASxB,CAAC;AAEF,QAAA,MAAM,mBAAmB,6CAMxB,CAAC;AAEF,QAAA,MAAM,qBAAqB,6CAU1B,CAAC;AAEF,QAAA,MAAM,2BAA2B,4DAWhC,CAAC;AAIF,OAAO,EACL,sBAAsB,EACtB,mBAAmB,EACnB,mBAAmB,EACnB,mBAAmB,EACnB,qBAAqB,EACrB,2BAA2B,GAC5B,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":"AAuTA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CA6EvB,CAAC;AA0GF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutAplication.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutAplication.ts"],"names":[],"mappings":"AAsTA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAqFvB,CAAC;AA0GF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}

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

@@ -1 +1 @@
-{"version":3,"file":"sendToWatch.d.ts","sourceRoot":"","sources":["../../../../src/workout/sendToWatch.ts"],"names":[],"mappings":"AAKA,QAAA,MAAM,kBAAkB;;;;;;EAmBvB,CAAC;AAEF,OAAO,EAAE,kBAAkB,EAAE,CAAC"}
+{"version":3,"file":"sendToWatch.d.ts","sourceRoot":"","sources":["../../../../src/workout/sendToWatch.ts"],"names":[],"mappings":"AAKA,QAAA,MAAM,kBAAkB;;;;;;EAuBvB,CAAC;AAEF,OAAO,EAAE,kBAAkB,EAAE,CAAC"}

package/lib/cjs/workout/garmin.js

@@ -65,22 +65,40 @@ const garminOauthActivity = ({ url, httpMethod, userid, params = {} }) => __awai
 const secondsPerKmToMetersPerSecond = (secondsPerKm) => {
     return Math.round((1000 / secondsPerKm) * 100) / 100;
 };
+/**
+ * Devuelve la metrica planificada como numero, o null si no la hay.
+ *
+ * OJO: Postgres devuelve las columnas `numeric` como TEXTO, asi que una distancia
+ * de 0 llega como la cadena "0", que en JS es truthy. Los entrenos de bici y
+ * natacion se planifican por tiempo y guardan `DISTANCE PLANNED` = 0: comparar
+ * en crudo hacia colar entrenos de 0 metros a Garmin.
+ */
+const metricaPlanificada = (valor) => {
+    const n = Number(valor);
+    return Number.isFinite(n) && n > 0 ? n : null;
+};
 const getWorkoutGarminDuration = (workout) => {
+    const distancia = metricaPlanificada(workout.distancePlanned);
+    const duracion = metricaPlanificada(workout.durationPlanned);
     const rangePlanning = (0, common_1.getWorkoutRangePlanning)({
         workoutType: workout.type,
         distancePlanned: workout.distancePlanned,
         durationPlanned: workout.durationPlanned,
         zonePlanned: workout.zonePlanned,
     });
-    if (rangePlanning.fixedMetric === common_1.WORKOUT_FIXED_METRIC.TIME) {
+    // `fixedMetric` solo se calcula cuando hay rango de ritmo, y eso solo pasa en
+    // CORRER/COMPETICION. Para bici y natacion siempre es null, asi que sin este
+    // fallback caian SIEMPRE por distancia (y se enviaba un paso de 0 metros).
+    const prefiereTiempo = rangePlanning.fixedMetric === common_1.WORKOUT_FIXED_METRIC.TIME;
+    if (duracion && (prefiereTiempo || !distancia)) {
         return {
             type: 'TIME',
-            value: Number(workout.durationPlanned),
+            value: duracion,
         };
     }
     return {
         type: 'DISTANCE',
-        value: Number(workout.distancePlanned),
+        value: distancia,
         valueType: 'METER',
     };
 };
@@ -126,7 +144,10 @@ const translateWorkoutGarmin = (workout) => {
         });
     }
     else if (!workout.estructura) {
-        if (!workout.distancePlanned || !workout.durationPlanned)
+        // Basta con una de las dos: bici/natacion se planifican por tiempo y correr
+        // por distancia. Exigir las dos (comparando ademas en crudo) era lo que
+        // dejaba fuera a los entrenos por tiempo.
+        if (!metricaPlanificada(workout.distancePlanned) && !metricaPlanificada(workout.durationPlanned))
             return null;
         const targetType = getWorkoutGarminTargetType(workout, null);
         const targetValue = getWorkoutGarminTargetValue(workout, null);
@@ -203,7 +224,11 @@ const getWorkoutGarminTargetValue = (workout, row) => {
         // } else {
         // El resto calcular el ritmo
         // No se puede añadir potencia ya que el valor que hay que dar es una zona, no absoluto
-        value = Number(workout.distancePlanned) / Number(workout.durationPlanned);
+        const distancia = metricaPlanificada(workout.distancePlanned);
+        const duracion = metricaPlanificada(workout.durationPlanned);
+        // Sin las dos metricas no hay ritmo objetivo: el paso va sin target (que es
+        // lo que ya se hace hoy en natacion y Garmin acepta).
+        value = distancia && duracion ? distancia / duracion : null;
         paceRange = (0, common_1.getWorkoutRangePlanning)({
             workoutType: workout.type,
             distancePlanned: workout.distancePlanned,
@@ -247,6 +272,10 @@ const getWorkoutGarminTargetValue = (workout, row) => {
 };
 const createWorkoutGarmin = (userid, workout) => __awaiter(void 0, void 0, void 0, function* () {
     const workoutGarmin = translateWorkoutGarmin(workout);
+    // Sin payload no se manda nada: antes se hacia el POST con body null y Garmin
+    // respondia un error opaco.
+    if (!workoutGarmin)
+        throw new Error(`Workout ${workout.id} no es traducible a Garmin (sin estructura y sin distancia ni duracion)`);
     const data = yield garminOauthActivity({
         url: 'https://apis.garmin.com/training-api/workout',
         userid,

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -14,7 +14,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.saveWorkoutAplication = void 0;
 const db_1 = require("../db");
-const titleDescriptionTranslated_1 = require("../translation/titleDescriptionTranslated");
 const storage_1 = require("@google-cloud/storage");
 const path_1 = __importDefault(require("path"));
 const axios_1 = __importDefault(require("axios"));
@@ -316,16 +315,24 @@ _data) => __awaiter(void 0, void 0, void 0, function* () {
     }
     const dateStr = (0, moment_1.default)(opts.date).format('YYYY-MM-DD');
     const pt = predicadoTipo(opts);
-    // Traducción FUERA de la sección crítica. getTitleDescriptionTranslated hace
-    // una llamada de red a Google Cloud Translation; si se ejecutara dentro de
-    // `withTx` retendría el advisory lock durante todo el round-trip (causa de los
-    // picos de varios minutos en pg_advisory_xact_lock cuando la API va lenta).
-    // Sólo depende de idCliente/title/description —nada de la transacción— así que
-    // se calcula antes de abrir la tx. Semántica intacta: si la traducción falla,
-    // se propaga el error y el workout no se guarda, igual que antes. En las rutas
-    // de merge (1 y 2) el valor se descarta; el único sobrecoste es una traducción
-    // 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);
+    // SIN traducción en la ingesta: título y descripción se guardan tal cual llegan
+    // del proveedor (Garmin/Strava/Apple). Antes se traducían al español con Google
+    // Cloud Translation, lo que metía una llamada de red —y una dependencia de las
+    // credenciales de GCP— en el camino crítico del guardado: si Translate fallaba,
+    // el entreno NO se guardaba y el webhook devolvía 5xx. Con la migración a Azure
+    // eso se rompió (sin metadata server, las ADC resuelven a la service account de
+    // Storage, que no tiene permiso de Translate → PERMISSION_DENIED) y dos
+    // actividades envenenadas bastaron para atascar la cola de reintentos de Garmin
+    // y frenar la ingesta del resto.
+    // Consecuencia asumida: para clientes no-ES el "TITLE" queda en el idioma del
+    // dispositivo en vez de normalizado a español —lo notan entrenadores y la IA, no
+    // el cliente: la app cae a "TITLE" cuando "TITLE PREFERRED LANGUAGE" es null.
+    const translated = {
+        title: opts.title,
+        description: opts.description,
+        titlePreferredLanguage: null,
+        descriptionPreferredLanguage: null,
+    };
     const idWorkout = yield withTx((q) => __awaiter(void 0, void 0, void 0, function* () {
         var _e;
         // Lock anti-carrera: serializa ingestas del mismo cliente/día/bucket-tipo

package/lib/cjs/workout/sendToWatch.js

@@ -28,6 +28,10 @@ const sendWorkoutToWatch = (clientId, workoutId) => __awaiter(void 0, void 0, vo
     const estructura = yield (0, db_1.query)('SELECT * FROM [WORKOUT STRUCTURE] WHERE [ID WORKOUT] = ? ORDER BY [INDEX]', [workoutId]);
     if (estructura.length > 0)
         workout.estructura = estructura;
+    // Si el entreno no se puede traducir, se avisa con un 400 legible en vez de
+    // dejar que el POST a Garmin reviente con un 500 opaco.
+    if (!(0, garmin_1.translateWorkoutGarmin)(workout))
+        return { error: 'Este entrenamiento no se puede enviar al reloj' };
     // Llamar a la función que envía el workout al Garmin
     const { workoutGarminId } = yield (0, garmin_1.createWorkoutGarmin)(clientId, workout);
     yield (0, garmin_1.scheduleWorkoutGarmin)(clientId, { date: (0, moment_1.default)(workout.date).format('YYYY-MM-DD'), workoutGarminId });

package/package.json

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

package/lib/cjs/translation/titleDescriptionTranslated.js

@@ -1,38 +0,0 @@
-"use strict";
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.getTitleDescriptionTranslated = void 0;
-const db_1 = require("../db");
-const index_1 = require("./index");
-const common_1 = require("@runnerpro/common");
-const getTitleDescriptionTranslated = (idCliente, titleOriginal, descriptionOriginal) => __awaiter(void 0, void 0, void 0, function* () {
-    let title;
-    let description;
-    let titlePreferredLanguage;
-    let descriptionPreferredLanguage;
-    const [cliente] = yield (0, db_1.query)('SELECT [PREFERRED LANGUAGE] FROM [CLIENTE] WHERE [ID] = ?', [idCliente]);
-    if ((cliente === null || cliente === void 0 ? void 0 : cliente.preferredLanguage) && cliente.preferredLanguage !== common_1.LANGUAGES.ES) {
-        // @ts-ignore
-        title = titleOriginal ? yield (0, index_1.translate)(titleOriginal, { fromLanguage: cliente.preferredLanguage, toLanguage: common_1.LANGUAGES.ES }) : null;
-        description = descriptionOriginal
-            ? // @ts-ignore
-                yield (0, index_1.translate)(descriptionOriginal, { fromLanguage: cliente.preferredLanguage, toLanguage: common_1.LANGUAGES.ES })
-            : null;
-        titlePreferredLanguage = titleOriginal;
-        descriptionPreferredLanguage = descriptionOriginal;
-    }
-    else {
-        title = titleOriginal;
-        description = descriptionOriginal;
-    }
-    return { title, description, titlePreferredLanguage, descriptionPreferredLanguage };
-});
-exports.getTitleDescriptionTranslated = getTitleDescriptionTranslated;

package/lib/cjs/types/translation/titleDescriptionTranslated.d.ts

@@ -1,8 +0,0 @@
-declare const getTitleDescriptionTranslated: (idCliente: any, titleOriginal: any, descriptionOriginal: any) => Promise<{
-    title: any;
-    description: any;
-    titlePreferredLanguage: any;
-    descriptionPreferredLanguage: any;
-}>;
-export { getTitleDescriptionTranslated };
-//# sourceMappingURL=titleDescriptionTranslated.d.ts.map

package/lib/cjs/types/translation/titleDescriptionTranslated.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"titleDescriptionTranslated.d.ts","sourceRoot":"","sources":["../../../../src/translation/titleDescriptionTranslated.ts"],"names":[],"mappings":"AAIA,QAAA,MAAM,6BAA6B;;;;;EAsBlC,CAAC;AAEF,OAAO,EAAE,6BAA6B,EAAE,CAAC"}