@runnerpro/backend 1.31.1 → 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,8 +20,6 @@ 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');
@@ -159,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.map

@@ -1 +1 @@
-{"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"}
+{"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/package.json

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