@runnerpro/backend 1.31.0 → 1.31.2

package/lib/cjs/chat/api/conversation.js

@@ -476,47 +476,61 @@ const sendFile = (req, res, { sendNotification, firebaseMessaging, isClient, buc
     let thumbnail;
     const { userid } = req.session;
     const filePath = path_1.default.join('./uploads', req.file.filename);
-    if (req.file.mimetype.includes('video')) {
-        duration = yield getVideoDuration(filePath);
-        thumbnail = yield getThumbnailFromVideo(filePath, duration);
-    }
-    const [{ id: idFile }] = yield (0, index_1.query)('INSERT INTO [CHAT MESSAGE] ([ID CLIENTE], [ID SENDER], [TEXT], [MIMETYPE], [DURATION], [TYPE]) VALUES (?, ?, ?, ?, ?, ?) RETURNING [ID]', [isClient ? userid : idCliente, userid, req.file.originalname, req.file.mimetype, duration || null, type || 2]);
-    // Logros: adjunto enviado por el cliente también cuenta como mensaje. Await
-    // acotado para devolver los recién desbloqueados al FE en la respuesta (push).
+    // El adjunto de multer vive en ./uploads, que en Cloud Run es tmpfs: disco que es RAM.
+    // El unlink va en un finally porque antes solo se alcanzaba si TODO lo anterior salía
+    // bien: una subida a Storage o una query que fallara dejaban la foto, el audio o el
+    // vídeo —decenas de MB— ocupando memoria hasta el reinicio (la fuga de T-211).
+    let idFile;
     let logrosDesbloqueados = [];
-    if (isClient) {
-        logrosDesbloqueados = yield (0, index_1.evaluateAchievementsBounded)(userid, 'chat', {}, 1500);
+    try {
+        if (req.file.mimetype.includes('video')) {
+            duration = yield getVideoDuration(filePath);
+            thumbnail = yield getThumbnailFromVideo(filePath, duration);
+        }
+        [{ id: idFile }] = yield (0, index_1.query)('INSERT INTO [CHAT MESSAGE] ([ID CLIENTE], [ID SENDER], [TEXT], [MIMETYPE], [DURATION], [TYPE]) VALUES (?, ?, ?, ?, ?, ?) RETURNING [ID]', [isClient ? userid : idCliente, userid, req.file.originalname, req.file.mimetype, duration || null, type || 2]);
+        // Logros: adjunto enviado por el cliente también cuenta como mensaje. Await
+        // acotado para devolver los recién desbloqueados al FE en la respuesta (push).
+        if (isClient) {
+            logrosDesbloqueados = yield (0, index_1.evaluateAchievementsBounded)(userid, 'chat', {}, 1500);
+        }
+        const fileData = fs_1.default.readFileSync(filePath);
+        const files = [];
+        if (req.file.mimetype.includes('image')) {
+            try {
+                const fileDataResize = yield resizeImage(fileData, filePath, req.file.mimetype);
+                const fileDataOriented = yield rotateOrientationImage(fileData, filePath, null, req.file.mimetype);
+                files.push({ data: fileDataResize, id: idFile });
+                files.push({ data: fileDataOriented, id: `${idFile}-original` });
+            }
+            catch (error) {
+                (0, index_1.err)(null, null, error, null);
+            }
+        }
+        else if (req.file.mimetype.includes('video')) {
+            files.push({ data: fileData, id: idFile });
+            files.push({ data: thumbnail, id: `${idFile}-thumbnail` });
+        }
+        else {
+            files.push({ data: fileData, id: idFile });
+        }
+        res.send({ idFile, logrosDesbloqueados });
+        // Procesar el adjunto en background, sea del tipo que sea: transcripción, descripción, resumen
+        // o ficha del archivo. El router de mediaProcessing decide qué toca. Va ANTES de subir a Storage
+        // a propósito: si la subida falla, el mensaje se quedaría sin FILE TEXT de forma permanente y
+        // eso bloquea al entrenador, que no distingue "sin texto" de "aún sin procesar".
+        (0, mediaProcessing_1.processMediaFile)(idFile, fileData, req.file.mimetype, req.file.originalname);
+        for (const file of files) {
+            yield bucket.file(`Chat/${file.id}`).save(file.data);
+        }
     }
-    const fileData = fs_1.default.readFileSync(filePath);
-    const files = [];
-    if (req.file.mimetype.includes('image')) {
+    finally {
         try {
-            const fileDataResize = yield resizeImage(fileData, filePath, req.file.mimetype);
-            const fileDataOriented = yield rotateOrientationImage(fileData, filePath, null, req.file.mimetype);
-            files.push({ data: fileDataResize, id: idFile });
-            files.push({ data: fileDataOriented, id: `${idFile}-original` });
+            fs_1.default.unlinkSync(filePath);
         }
-        catch (error) {
-            (0, index_1.err)(null, null, error, null);
+        catch (cleanupError) {
+            // Ya no está (o nunca llegó a escribirse): no hay nada que limpiar.
         }
     }
-    else if (req.file.mimetype.includes('video')) {
-        files.push({ data: fileData, id: idFile });
-        files.push({ data: thumbnail, id: `${idFile}-thumbnail` });
-    }
-    else {
-        files.push({ data: fileData, id: idFile });
-    }
-    res.send({ idFile, logrosDesbloqueados });
-    // Procesar el adjunto en background, sea del tipo que sea: transcripción, descripción, resumen
-    // o ficha del archivo. El router de mediaProcessing decide qué toca. Va ANTES de subir a Storage
-    // a propósito: si la subida falla, el mensaje se quedaría sin FILE TEXT de forma permanente y
-    // eso bloquea al entrenador, que no distingue "sin texto" de "aún sin procesar".
-    (0, mediaProcessing_1.processMediaFile)(idFile, fileData, req.file.mimetype, req.file.originalname);
-    for (const file of files) {
-        yield bucket.file(`Chat/${file.id}`).save(file.data);
-    }
-    fs_1.default.unlinkSync(filePath);
     if (!isClient) {
         let textFile = 'Archivo adjunto';
         if (Number(type) === 4)

package/lib/cjs/image/generateShareMap.js

@@ -20,17 +20,43 @@ const common_1 = require("@runnerpro/common");
 const translation_1 = require("../translation");
 const path_1 = __importDefault(require("path"));
 const jimp_1 = __importDefault(require("jimp"));
-const uuidv4_1 = require("uuidv4");
-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);
@@ -131,13 +157,13 @@ const addShadowShareWorkoutmap = (context, useDefaultPhoto) => __awaiter(void 0,
     context.fillRect(0, 1080 - shadowHeight, 1080, shadowHeight);
 });
 const addImageShareWorkoutmap = (context, data) => __awaiter(void 0, void 0, void 0, function* () {
+    // Sin fichero temporal: antes el JPEG intermedio pasaba por ./uploads y el unlink
+    // solo llegaba si el render entero salía bien — un fallo por medio lo dejaba
+    // huérfano, y en Cloud Run ./uploads es tmpfs, o sea RAM que no se recupera hasta
+    // el reinicio (la fuga de T-211). loadImage acepta el Buffer directamente.
     const image = yield jimp_1.default.read(data);
-    const filename2 = `share-${(0, uuidv4_1.uuid)()}.jpeg`;
-    const uploadsDir = path_1.default.resolve(process.cwd(), 'uploads');
-    yield image.cover(1080, 1080).writeAsync(path_1.default.join(uploadsDir, filename2));
-    const imagePath2 = path_1.default.join(uploadsDir, filename2);
-    const image2 = yield (0, canvas_1.loadImage)(imagePath2);
+    const buffer = yield image.cover(1080, 1080).getBufferAsync(jimp_1.default.MIME_JPEG);
+    const image2 = yield (0, canvas_1.loadImage)(buffer);
     context.drawImage(image2, 0, 0, 1080, 1080);
     context.fillStyle = 'rgba(0, 0, 0, 0.5)';
-    yield fs_1.default.promises.unlink(imagePath2);
 });

package/lib/cjs/image/reduceSizeImage.js

@@ -13,40 +13,30 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.reduceSizeImage = void 0;
-const util_1 = require("util");
 const jimp_1 = __importDefault(require("jimp"));
 const fs_1 = __importDefault(require("fs"));
-const uuid_1 = require("uuid");
-const path_1 = __importDefault(require("path"));
 const exifr_1 = __importDefault(require("exifr"));
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const sizeOf = require('image-size');
 const reduceSizeImage = (file, newWidth = 100, quality = 80) => __awaiter(void 0, void 0, void 0, function* () {
     try {
-        let filePath;
-        if (typeof file === 'object') {
-            // Write this image to a file in uploads
-            const uid = (0, uuid_1.v4)();
-            filePath = path_1.default.join(process.cwd(), 'uploads', `${uid}.webp`);
-            fs_1.default.writeFileSync(filePath, file);
-        }
-        else {
-            // If it's not an object, it's already a file path
-            filePath = file;
-        }
-        // eslint-disable-next-line @typescript-eslint/no-var-requires
-        const sizeOf = (0, util_1.promisify)(require('image-size'));
-        const dimensions = yield sizeOf(filePath);
+        // Todo en memoria, sin fichero temporal. Antes esto escribía el buffer en ./uploads
+        // para medirlo y leerlo desde disco, y solo lo borraba si TODO salía bien: con un
+        // formato que Jimp no lee (el HEIC de los iPhone, sin ir más lejos) se saltaba al
+        // catch y el fichero quedaba huérfano. En Cloud Run ./uploads es tmpfs —disco que
+        // es RAM— así que cada foto fallida eran MiB de memoria perdidos hasta el reinicio
+        // (la fuga de T-211). Jimp e image-size aceptan el Buffer directamente.
+        const source = typeof file === 'object' ? file : fs_1.default.readFileSync(file);
+        const dimensions = sizeOf(source);
         const imageWidth = dimensions.width;
         const imageHeight = dimensions.height;
         const newHeight = Math.round((imageHeight * newWidth) / imageWidth);
-        let image = yield jimp_1.default.read(filePath);
+        let image = yield jimp_1.default.read(source);
         image = yield rotateOrientationImage(image, file);
         image = yield image.resize(newWidth, newHeight);
         image = yield image.quality(quality);
         // @ts-ignore
-        image = yield image.getBufferAsync(jimp_1.default.MIME_JPEG);
-        if (typeof file === 'object')
-            fs_1.default.unlinkSync(filePath);
-        return image;
+        return yield image.getBufferAsync(jimp_1.default.MIME_JPEG);
     }
     catch (error) {
         if (typeof file === 'object')

package/lib/cjs/types/chat/api/conversation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conversation.d.ts","sourceRoot":"","sources":["../../../../../src/chat/api/conversation.ts"],"names":[],"mappings":"AAmBA,QAAA,MAAM,iBAAiB,0BAA2B,GAAG,SA0BpD,CAAC;AAoGF;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,QAAA,MAAM,uBAAuB,qCAAkC,GAAG,iBAwCjE,CAAC;AAmGF;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,QAAA,MAAM,WAAW,0EAAuE,GAAG,kBA0E1F,CAAC;AAEF,QAAA,MAAM,gBAAgB;;;;mBAqBrB,CAAC;AA2TF,OAAO,EAAE,iBAAiB,EAAE,gBAAgB,EAAE,WAAW,EAAE,uBAAuB,EAAE,CAAC"}
+{"version":3,"file":"conversation.d.ts","sourceRoot":"","sources":["../../../../../src/chat/api/conversation.ts"],"names":[],"mappings":"AAmBA,QAAA,MAAM,iBAAiB,0BAA2B,GAAG,SA0BpD,CAAC;AAoGF;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,QAAA,MAAM,uBAAuB,qCAAkC,GAAG,iBAwCjE,CAAC;AAmGF;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,QAAA,MAAM,WAAW,0EAAuE,GAAG,kBA0E1F,CAAC;AAEF,QAAA,MAAM,gBAAgB;;;;mBAqBrB,CAAC;AAuUF,OAAO,EAAE,iBAAiB,EAAE,gBAAgB,EAAE,WAAW,EAAE,uBAAuB,EAAE,CAAC"}

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":";AAYA;;;;;;;;;;;;;;;GAeG;AACH,QAAA,MAAM,gBAAgB,+DA+BrB,CAAC;AA0HF,OAAO,EAAE,gBAAgB,EAAE,CAAC"}

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

@@ -1 +1 @@
-{"version":3,"file":"reduceSizeImage.d.ts","sourceRoot":"","sources":["../../../../src/image/reduceSizeImage.ts"],"names":[],"mappings":"AAOA,QAAA,MAAM,eAAe,kEAqCpB,CAAC;AAaF,OAAO,EAAE,eAAe,EAAE,CAAC"}
+{"version":3,"file":"reduceSizeImage.d.ts","sourceRoot":"","sources":["../../../../src/image/reduceSizeImage.ts"],"names":[],"mappings":"AAOA,QAAA,MAAM,eAAe,kEA4BpB,CAAC;AAaF,OAAO,EAAE,eAAe,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.2",
   "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"
   },