@runnerpro/backend 1.31.0 → 1.31.1

package/lib/cjs/image/generateShareMap.js

@@ -25,12 +25,40 @@ const fs_1 = __importDefault(require("fs"));
 // install fonts
 canvas_1.GlobalFonts.registerFromPath(path_1.default.join(__dirname, '../../..', 'static/fonts', 'SofiaSans-Bold.ttf'), 'SofiaSansBold');
 canvas_1.GlobalFonts.registerFromPath(path_1.default.join(__dirname, '../../..', 'static/fonts', 'SofiaSans-Regular.ttf'), 'SofiaSans');
+/**
+ * Genera la imagen cuadrada (1080x1080) para compartir un entreno: la foto/mapa
+ * de fondo con los datos del entreno (distancia, tiempo, ritmo o desnivel) superpuestos.
+ *
+ * @param image - Buffer de la imagen de fondo (mapa del entreno o foto por defecto).
+ * @param idWorkout - ID del entreno del que se leen los datos y el idioma de su cliente.
+ * @param options - `useDefaultPhoto: true` cuando el fondo es la foto genérica (invierte el degradado superior).
+ * @returns Buffer JPEG con la imagen, o `null` si no se pudieron leer el entreno o su cliente.
+ *
+ * @example
+ * ```typescript
+ * const shareMap = await generateShareMap(image, idWorkout, { useDefaultPhoto: true });
+ * if (!shareMap) return; // ⚠️ SIEMPRE comprobar el null antes de subirla
+ * await bucket.file(`Workout/${idWorkout}-share`).save(shareMap);
+ * ```
+ */
 const generateShareMap = (image, idWorkout, options = {}) => __awaiter(void 0, void 0, void 0, function* () {
     const useDefaultPhoto = (options === null || options === void 0 ? void 0 : options.useDefaultPhoto) || false;
     const [workout] = yield (0, index_1.query)('SELECT [ID], [ID CLIENTE], [TYPE], [DISTANCE], [DURATION], [DESNIVEL] FROM [WORKOUT] WHERE [ID] = ?', [idWorkout]);
+    // ⚠️ El guard va ANTES de usar `workout`: `query` devuelve [] tanto si el entreno
+    // no existe como si la consulta falló (se traga el error y loguea 'PG query error',
+    // p.ej. client_login_timeout del pool). Leer `workout.idCliente` sin comprobarlo
+    // rompía el webhook con TypeError.
+    if (!workout) {
+        // eslint-disable-next-line no-console
+        console.error(`generateShareMap: sin fila de WORKOUT para idWorkout=${idWorkout}; el entreno no existe o la consulta falló (ver 'PG query error' justo antes). No se genera la imagen de compartir.`);
+        return null;
+    }
     const [cliente] = yield (0, index_1.query)('SELECT [PREFERRED LANGUAGE] FROM [CLIENTE] WHERE [ID] = ?', [workout.idCliente]);
-    if (!workout || !cliente)
+    if (!cliente) {
+        // eslint-disable-next-line no-console
+        console.error(`generateShareMap: sin fila de CLIENTE idCliente=${workout.idCliente} del idWorkout=${idWorkout}; el cliente no existe o la consulta falló. No se genera la imagen de compartir.`);
         return null;
+    }
     const width = 1080;
     const height = 1080;
     const canvas = (0, canvas_1.createCanvas)(width, height);

package/lib/cjs/types/image/generateShareMap.d.ts

@@ -1,4 +1,20 @@
 /// <reference types="node" />
+/**
+ * Genera la imagen cuadrada (1080x1080) para compartir un entreno: la foto/mapa
+ * de fondo con los datos del entreno (distancia, tiempo, ritmo o desnivel) superpuestos.
+ *
+ * @param image - Buffer de la imagen de fondo (mapa del entreno o foto por defecto).
+ * @param idWorkout - ID del entreno del que se leen los datos y el idioma de su cliente.
+ * @param options - `useDefaultPhoto: true` cuando el fondo es la foto genérica (invierte el degradado superior).
+ * @returns Buffer JPEG con la imagen, o `null` si no se pudieron leer el entreno o su cliente.
+ *
+ * @example
+ * ```typescript
+ * const shareMap = await generateShareMap(image, idWorkout, { useDefaultPhoto: true });
+ * if (!shareMap) return; // ⚠️ SIEMPRE comprobar el null antes de subirla
+ * await bucket.file(`Workout/${idWorkout}-share`).save(shareMap);
+ * ```
+ */
 declare const generateShareMap: (image: any, idWorkout: any, options?: {}) => Promise<Buffer>;
 export { generateShareMap };
 //# sourceMappingURL=generateShareMap.d.ts.map

package/lib/cjs/types/image/generateShareMap.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"generateShareMap.d.ts","sourceRoot":"","sources":["../../../../src/image/generateShareMap.ts"],"names":[],"mappings":";AAcA,QAAA,MAAM,gBAAgB,+DAkBrB,CAAC;AA0HF,OAAO,EAAE,gBAAgB,EAAE,CAAC"}
+{"version":3,"file":"generateShareMap.d.ts","sourceRoot":"","sources":["../../../../src/image/generateShareMap.ts"],"names":[],"mappings":";AAcA;;;;;;;;;;;;;;;GAeG;AACH,QAAA,MAAM,gBAAgB,+DA+BrB,CAAC;AA0HF,OAAO,EAAE,gBAAgB,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":"AAuTA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CA6EvB,CAAC;AAuEF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}
+{"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;AA+EF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -384,7 +384,13 @@ const postHooks = (idWorkout, opts) => __awaiter(void 0, void 0, void 0, functio
     catch (_) {
         // Un fallo de Mapbox no debe romper la ingesta (estado ya persistido).
     }
-    saveShareWorkoutImage(idWorkout, opts.type);
+    try {
+        yield saveShareWorkoutImage(idWorkout, opts.type);
+    }
+    catch (_) {
+        // La imagen de compartir es accesoria: si falla (Storage, canvas, BD) no debe
+        // romper la ingesta ni escapar como unhandled rejection (así llegaba a Sentry).
+    }
     yield (0, estructuraWorkout_1.saveDoneStructuraWorkout)(idWorkout);
     // Logros: evaluar al completar el entreno (este es el embudo de los entrenos
     // sincronizados). Se pasa el idWorkout para calcular el PR solo de ESTE entreno
@@ -438,6 +444,10 @@ const saveShareWorkoutImage = (id, type) => __awaiter(void 0, void 0, void 0, fu
     else
         [image] = yield storage.bucket(process.env.CLOUD_STORAGE_BUCKET_PUBLIC).file(`Workout/${id}`).download();
     const shareMap = yield (0, generateShareMap_1.generateShareMap)(image, id);
+    // generateShareMap devuelve null si no pudo leer el entreno o su cliente (ya deja
+    // traza del motivo): sin imagen no hay nada que subir ni URL que guardar.
+    if (!shareMap)
+        return;
     // @ts-ignore
     yield storage.bucket(process.env.CLOUD_STORAGE_BUCKET_PUBLIC).file(`Workout/${id}-share`).save(shareMap);
     const urlShare = `https://storage.googleapis.com/${process.env.CLOUD_STORAGE_BUCKET_PUBLIC}/Workout/${id}-share`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@runnerpro/backend",
-  "version": "1.31.0",
+  "version": "1.31.1",
   "description": "A collection of common backend functions",
   "exports": {
     ".": "./lib/cjs/index.js"
@@ -17,6 +17,7 @@
     "semantic-release": "semantic-release",
     "lint": "eslint --ext .ts --ignore-path .gitignore .",
     "test:translate-workout-garmin": "ts-node --transpile-only scripts/translateWorkoutGarmin.ts",
+    "test:generate-share-map-guard": "ts-node --transpile-only -P configs/tsconfig.cjs.json scripts/testGenerateShareMapGuard.ts",
     "format": "prettier --write \"src/**/*.{js,jsx,ts,tsx,json,css,scss,md}\"",
     "prepare": "husky"
   },