@runnerpro/backend 1.33.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/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/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.33.0",
+  "version": "1.33.1",
   "description": "A collection of common backend functions",
   "exports": {
     ".": "./lib/cjs/index.js"