@runnerpro/backend 1.21.2 → 1.21.4

package/lib/cjs/translation/index.js

@@ -18,10 +18,19 @@ const common_1 = require("@runnerpro/common");
 Object.defineProperty(exports, "LANGUAGES", { enumerable: true, get: function () { return common_1.LANGUAGES; } });
 const fs_1 = __importDefault(require("fs"));
 const index_1 = require("../db/index");
+// Cliente singleton: instanciar TranslationServiceClient en cada llamada paga el
+// coste de init (descubrimiento de credenciales/auth) por traducción. Se reutiliza
+// un único cliente por proceso; es thread-safe para llamadas concurrentes.
+let translationClientSingleton = null;
+const getTranslationClient = () => {
+    if (!translationClientSingleton)
+        translationClientSingleton = new translate_1.TranslationServiceClient();
+    return translationClientSingleton;
+};
 const translate = (text, { fromLanguage, toLanguage }) => __awaiter(void 0, void 0, void 0, function* () {
     const projectId = process.env.GCP_PROYECT_ID;
     const location = 'global';
-    const translationClient = new translate_1.TranslationServiceClient();
+    const translationClient = getTranslationClient();
     const request = {
         parent: `projects/${projectId}/locations/${location}`,
         contents: [text],

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/translation/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAI9C,KAAK,cAAc,GAAG,MAAM,OAAO,SAAS,CAAC;AAE7C,QAAA,MAAM,SAAS,SACP,MAAM;kBACkC,cAAc;gBAAc,cAAc;MACvF,QAAQ,MAAM,CAwBhB,CAAC;AAEF,QAAA,MAAM,cAAc,aAAc,MAAM,QAAQ,MAAM;cAExC,MAAM,iBAAiB,GAAG;CAkBvC,CAAC;AAEF,QAAA,MAAM,gCAAgC;;;;;;EA4BrC,CAAC;AAEF,OAAO,EAAE,SAAS,EAAE,SAAS,EAAE,cAAc,EAAE,gCAAgC,EAAE,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/translation/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAI9C,KAAK,cAAc,GAAG,MAAM,OAAO,SAAS,CAAC;AAW7C,QAAA,MAAM,SAAS,SACP,MAAM;kBACkC,cAAc;gBAAc,cAAc;MACvF,QAAQ,MAAM,CAwBhB,CAAC;AAEF,QAAA,MAAM,cAAc,aAAc,MAAM,QAAQ,MAAM;cAExC,MAAM,iBAAiB,GAAG;CAkBvC,CAAC;AAEF,QAAA,MAAM,gCAAgC;;;;;;EA4BrC,CAAC;AAEF,OAAO,EAAE,SAAS,EAAE,SAAS,EAAE,cAAc,EAAE,gCAAgC,EAAE,CAAC"}

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

@@ -1 +1 @@
-{"version":3,"file":"estructuraWorkout.d.ts","sourceRoot":"","sources":["../../../../src/workout/estructuraWorkout.ts"],"names":[],"mappings":"AAuCA,QAAA,MAAM,mBAAmB,YAAa,GAAG,cAAc,GAAG,QAiCzD,CAAC;AAEF;;;;;;;;;;;;;;;;GAgBG;AACH,QAAA,MAAM,wBAAwB,cAAqB,GAAG,kBAkBrD,CAAC;AAuPF,OAAO,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,CAAC"}
+{"version":3,"file":"estructuraWorkout.d.ts","sourceRoot":"","sources":["../../../../src/workout/estructuraWorkout.ts"],"names":[],"mappings":"AAsCA,QAAA,MAAM,mBAAmB,YAAa,GAAG,cAAc,GAAG,QAiCzD,CAAC;AAEF;;;;;;;;;;;;;;;;GAgBG;AACH,QAAA,MAAM,wBAAwB,cAAqB,GAAG,kBAkBrD,CAAC;AAkPF,OAAO,EAAE,mBAAmB,EAAE,wBAAwB,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":"AAwPA;;;;;;;;;GASG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CA0DvB,CAAC;AA+DF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutAplication.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutAplication.ts"],"names":[],"mappings":"AAwPA;;;;;;;;;GASG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CAsEvB,CAAC;AA+DF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}

package/lib/cjs/workout/estructuraWorkout.js

@@ -11,7 +11,6 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.saveDoneStructuraWorkout = exports.getStructuraWorkout = void 0;
 const db_1 = require("../db");
-const sendMail_1 = require("../sendMail");
 const common_1 = require("@runnerpro/common");
 const calculateZone = (values) => {
     const percentage = Number(values.pacePercentage);
@@ -127,12 +126,7 @@ const saveDoneStructuraWorkoutWatch = (idWorkout, streamData, laps, workout, est
     // Cualquier otra desviación es anómala (faltan laps en medio, doble extra,
     // etc.) y se reporta para auditoría manual.
     if (numPasos !== numLaps && numPasos !== numLaps + 1 && numLaps !== numPasos + 1) {
-        return (0, sendMail_1.sendMail)({
-            title: 'Estructura workout: laps no coinciden con pasos',
-            subject: 'Dedup cross-proveedor - Error saving steps structure workout',
-            body: JSON.stringify({ idWorkout, laps, estructura }),
-            to: ['david.jimenez@runnerpro.app'],
-        });
+        return;
     }
     for (let i = 0; i < estructura.length; i++) {
         const row = estructura[i];

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -248,6 +248,16 @@ const saveWorkoutAplication = (opts,
 _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);
     const idWorkout = yield withTx((q) => __awaiter(void 0, void 0, void 0, function* () {
         // Lock anti-carrera: serializa ingestas del mismo cliente/día/bucket-tipo
         // (Strava-webhook y Garmin-push pueden entrar casi a la vez).
@@ -276,8 +286,9 @@ _data) => __awaiter(void 0, void 0, void 0, function* () {
             yield registrarFuente(q, sim.id, opts.aplicationType, opts.aplicationId);
             return sim.id;
         }
-        // 3) INSERT nuevo (comportamiento actual).
-        const { title: titleNoTranslate, description: descriptionNoTranslate, titlePreferredLanguage, descriptionPreferredLanguage } = yield (0, titleDescriptionTranslated_1.getTitleDescriptionTranslated)(opts.idCliente, opts.title, opts.description);
+        // 3) INSERT nuevo (comportamiento actual). La traducción ya se calculó antes
+        //    de abrir la tx (ver `translated` arriba) para no retener el advisory lock.
+        const { title: titleNoTranslate, description: descriptionNoTranslate, titlePreferredLanguage, descriptionPreferredLanguage } = translated;
         const inserted = yield q(`INSERT INTO "WORKOUT"
          ("ID CLIENTE", "DATE", "TITLE", "DESCRIPTION", "TITLE PREFERRED LANGUAGE", "DESCRIPTION PREFERRED LANGUAGE", "TYPE", "DONE", "SHOW CLIENT")
        VALUES (?, ?, ?, ?, ?, ?, ?, TRUE, TRUE) RETURNING "ID"`, [opts.idCliente, dateStr, titleNoTranslate, descriptionNoTranslate, titlePreferredLanguage, descriptionPreferredLanguage, opts.type]);

package/package.json

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