@runnerpro/backend 1.31.2 → 1.32.0

package/README.md

@@ -16,6 +16,19 @@ Connection with the PostgresSQL database
 - <b>Param</b>: array of values
 - <b>Return</b>: promise of array of values (each item is a row of the table)
 
+> ⚠️ **T-219 — un fallo de CONEXIÓN puede rechazar, y se enciende por servicio.** `query()` no
+> lanzaba nunca: tanto «la BD no respondió» como «no hay filas» devolvían `[]`. Desde la **1.32.0**,
+> con `RP_DB_RECHAZA_FALLO_CONEXION=1` en el entorno, un fallo de conexión —agotado el reintento (1,
+> a los 500 ms)— **rechaza la promesa**. Un error de SQL (sintaxis, constraint, columna inexistente)
+> se sigue logueando y devolviendo `[]` en los dos modos, igual que siempre.
+>
+> **Sin esa variable no cambia nada**, y por eso puede salir en una menor: a un `^1.x` le entra en
+> cuanto alguien refresque su lock (`npm install`/`npm update`, sin tocar el rango) y ahí se comporta
+> como la 1.31.x. Con `npm ci` el lock manda, así que hasta ese refresco no llega. **Antes de encenderla en un
+> servicio, lee [docs/T-219-auditoria-consumidores.md](./docs/T-219-auditoria-consumidores.md)**: ahí
+> está, repo por repo y con fichero y línea, dónde aterriza ese rechazo hoy y qué hay que cerrar
+> antes. Encenderla sin cerrar esos puntos deja peticiones sin respuesta.
+
 ### batchQuery
 
 Connection with PostgreSQL database in batches. Use for faster query execution when we need to execute many queries.

package/lib/cjs/achievements/index.js

@@ -27,6 +27,29 @@ for (const meters of Object.keys(catalog_1.PR_DISTANCE_BADGE)) {
     BADGE_PR_DISTANCE[catalog_1.PR_DISTANCE_BADGE[Number(meters)]] = Number(meters);
 }
 const typesList = (types) => types.map((t) => `'${t}'`).join(',');
+/**
+ * Traza el fallo de un grupo de logros sin tumbar el resto.
+ *
+ * Los grupos van aislados a propósito (un fallo no debe impedir evaluar los demás), pero hasta
+ * T-219 lo hacían con un `catch (_) {}` mudo: cuando el pool no conectaba, los logros de ese
+ * cliente simplemente no se evaluaban y no quedaba ni rastro. Ahora sigue sin tumbar el resto,
+ * pero se sabe a quién le pasó y en qué grupo.
+ */
+const avisoLogros = (etiqueta, userId, error) => {
+    // eslint-disable-next-line no-console
+    console.error(`achievements: fallo evaluando ${etiqueta} de idCliente=${userId} (${(error === null || error === void 0 ? void 0 : error.message) || error}). Ese grupo se queda sin evaluar.`);
+};
+/** Igual que `avisoLogros` pero para lo que NO va aislado: deja la traza y vuelve a lanzar. */
+const conTraza = (etiqueta, userId, fn) => __awaiter(void 0, void 0, void 0, function* () {
+    try {
+        return yield fn();
+    }
+    catch (error) {
+        // eslint-disable-next-line no-console
+        console.error(`achievements: fallo leyendo/escribiendo ${etiqueta} de idCliente=${userId} (${(error === null || error === void 0 ? void 0 : error.message) || error}).`);
+        throw error;
+    }
+});
 const countScalar = (sql, params) => __awaiter(void 0, void 0, void 0, function* () {
     const [row] = yield (0, db_1.query)(sql, params);
     return Number(row === null || row === void 0 ? void 0 : row.c) || 0;
@@ -49,7 +72,9 @@ const getAllAggregates = (userId, sinceDate) => __awaiter(void 0, void 0, void 0
         params.push(sinceDate);
     }
     const runList = typesList(catalog_1.RUNNING_TYPES);
-    const [row] = yield (0, db_1.query)(`SELECT
+    // ⚠️ T-219: 9 de los 176 client_login_timeout de la semana medida caen aquí. No va aislado a
+    // propósito: sin agregados no hay nada que evaluar, así que se traza y se propaga.
+    const [row] = yield conTraza('los agregados de workouts', userId, () => (0, db_1.query)(`SELECT
        COALESCE(MAX("DISTANCE") FILTER (WHERE "TYPE" IN (${runList})), 0) AS "RUN MAX",
        COALESCE(SUM("DISTANCE") FILTER (WHERE "TYPE" IN (${runList})), 0) AS "RUN TOTAL",
        COUNT(*) FILTER (WHERE "TYPE" = 'FUERZA') AS "STR COUNT",
@@ -62,7 +87,7 @@ const getAllAggregates = (userId, sinceDate) => __awaiter(void 0, void 0, void 0
        COUNT(*) FILTER (WHERE "TYPE" = 'COMPETICION') AS "RACE COUNT",
        COUNT(*) FILTER (WHERE "FC MEDIA" IS NOT NULL) AS "HR COUNT"
      FROM "WORKOUT"
-     WHERE "ID CLIENTE" = ? AND "DONE" = true${sinceFilter}`, params);
+     WHERE "ID CLIENTE" = ? AND "DONE" = true${sinceFilter}`, params));
     return {
         runMax: Number(row === null || row === void 0 ? void 0 : row.runMax) || 0,
         runTotal: Number(row === null || row === void 0 ? void 0 : row.runTotal) || 0,
@@ -311,10 +336,12 @@ const persistLogros = (userId, ids, markSeen = false) => __awaiter(void 0, void
     const params = [];
     for (const id of uniq)
         params.push(userId, id);
-    const rows = yield (0, db_1.query)(`INSERT INTO "CLIENTE LOGRO" ("ID CLIENTE", "LOGRO ID", "FECHA VISTO")
+    // ⚠️ T-219: es el alta de los logros. Si esto se pierde en silencio, el cliente no desbloquea
+    // nada y no queda rastro; se traza con el idCliente y se propaga.
+    const rows = yield conTraza(`el alta de ${uniq.length} logro(s)`, userId, () => (0, db_1.query)(`INSERT INTO "CLIENTE LOGRO" ("ID CLIENTE", "LOGRO ID", "FECHA VISTO")
      VALUES ${valuesSql}
      ON CONFLICT ("ID CLIENTE", "LOGRO ID") DO NOTHING
-     RETURNING "LOGRO ID" AS "ID", "FECHA DESBLOQUEO"`, params);
+     RETURNING "LOGRO ID" AS "ID", "FECHA DESBLOQUEO"`, params));
     return rows.map((r) => ({ id: r.id, fechaDesbloqueo: r.fechaDesbloqueo }));
 });
 // ── Reglas por categoría ────────────────────────────────────────────────────
@@ -366,7 +393,9 @@ const evalCount = (rules, count) => rules.filter((r) => count >= r.count).map((r
  * los 4 bloques (base/desarrollo/pico/taper). Devuelve null si no hay plan.
  */
 const getPlanContext = (userId) => __awaiter(void 0, void 0, void 0, function* () {
-    const [row] = yield (0, db_1.query)(`SELECT
+    // ⚠️ T-219: va dentro de un grupo aislado (`evaluatePlan`), así que el error lo recoge
+    // `avisoLogros` arriba; aquí se deja la traza con el idCliente para saber a quién le pasó.
+    const [row] = yield conTraza('el contexto del plan', userId, () => (0, db_1.query)(`SELECT
        "ID PLAN CARRERA" AS "PLAN ID",
        "START DATE PLAN"::date AS "PLAN START",
        "GOAL DATE"::date AS "PLAN GOAL",
@@ -380,7 +409,7 @@ const getPlanContext = (userId) => __awaiter(void 0, void 0, void 0, function* (
        "ALGORITMO BLOQUE 3 FIN"::date AS "PICO FIN",
        "ALGORITMO BLOQUE 4 INICIO"::date AS "TAPER INI",
        "ALGORITMO BLOQUE 4 FIN"::date AS "TAPER FIN"
-     FROM "CLIENTE" WHERE "ID" = ?`, [userId]);
+     FROM "CLIENTE" WHERE "ID" = ?`, [userId]));
     return row || null;
 });
 /**
@@ -682,8 +711,8 @@ const evaluateAchievements = (userId, event = 'all', opts = {}) => __awaiter(voi
             else
                 nuevos.push(...(yield recomputeAllPRs(userId, anchor, markSeen)));
         }
-        catch (_) {
-            // ignore
+        catch (error) {
+            avisoLogros('los récords personales (PR)', userId, error);
         }
         const [streak, agg] = yield Promise.all([
             (0, streak_1.computeStreakStats)(userId, anchor),
@@ -697,8 +726,8 @@ const evaluateAchievements = (userId, event = 'all', opts = {}) => __awaiter(voi
         try {
             yield (0, db_1.query)('UPDATE "CLIENTE" SET "RACHA MAXIMA" = GREATEST(COALESCE("RACHA MAXIMA", 0), ?) WHERE "ID" = ?', [streak.longest, userId]);
         }
-        catch (_) {
-            // ignore
+        catch (error) {
+            avisoLogros('la racha máxima', userId, error);
         }
         eligible.push(...evaluateConsistency(streak.longest), ...evaluateDistance(agg.runMax), ...evaluateVolume(agg.runTotal), ...evaluateStrength(agg.strCount), ...evaluateCycling(agg), ...evaluateSwim(agg), ...evaluateTriSwim(agg), ...evaluateTrail(agg));
         const raceDates = agg.raceCount >= 3 ? yield getRaceDates(userId, anchor) : [];
@@ -711,31 +740,41 @@ const evaluateAchievements = (userId, event = 'all', opts = {}) => __awaiter(voi
         try {
             eligible.push(...(yield evaluatePlan(userId, anchor)));
         }
-        catch (_) { /* ignore */ }
+        catch (error) {
+            avisoLogros('el plan y sus hitos', userId, error);
+        }
         try {
             const strengthDates = yield getStrengthDates(userId, anchor);
             eligible.push(...evaluateStrengthFrequency(strengthDates));
         }
-        catch (_) { /* ignore */ }
+        catch (error) {
+            avisoLogros('la frecuencia de fuerza', userId, error);
+        }
         try {
             eligible.push(...(yield evaluateBrick(userId, anchor)));
         }
-        catch (_) { /* ignore */ }
+        catch (error) {
+            avisoLogros('los brick', userId, error);
+        }
         try {
             eligible.push(...(yield evaluateRecovery(userId, anchor)));
         }
-        catch (_) { /* ignore */ }
+        catch (error) {
+            avisoLogros('la recuperación', userId, error);
+        }
         try {
             eligible.push(...(yield evaluateCoachCall(userId)));
         }
-        catch (_) { /* ignore */ }
+        catch (error) {
+            avisoLogros('la llamada con el entrenador', userId, error);
+        }
     }
     if (want('chat') || want('survey') || want('availability') || want('nutrition') || want('food_photo') || want('wearable')) {
         try {
             eligible.push(...(yield evaluateNonWorkout(userId, event)));
         }
-        catch (_) {
-            // ignore
+        catch (error) {
+            avisoLogros(`los logros de '${event}'`, userId, error);
         }
     }
     // 'goal' (añadir objetivo) re-evalúa los logros de Plan (incl. pc_nuevo_capitulo).
@@ -744,7 +783,9 @@ const evaluateAchievements = (userId, event = 'all', opts = {}) => __awaiter(voi
         try {
             eligible.push(...(yield evaluatePlan(userId, anchor)));
         }
-        catch (_) { /* ignore */ }
+        catch (error) {
+            avisoLogros('el plan y sus hitos', userId, error);
+        }
     }
     nuevos.push(...(yield persistLogros(userId, eligible, markSeen)));
     // pace_*/pr_* se derivan del mapa de PRs ya persistido en los PARAMS de dist_*.
@@ -753,8 +794,8 @@ const evaluateAchievements = (userId, event = 'all', opts = {}) => __awaiter(voi
             const prMap = yield getPrMap(userId);
             nuevos.push(...(yield persistLogros(userId, [...evaluatePace(prMap), ...evaluateTimePR(prMap)])));
         }
-        catch (_) {
-            // ignore
+        catch (error) {
+            avisoLogros('los logros de ritmo y marca', userId, error);
         }
     }
     // Dedup por id, preservando el PRIMERO de cada id. applyPRs se acumula antes que

package/lib/cjs/db/index.js

@@ -34,31 +34,167 @@ pool.on('error', (err) => {
     console.error('PG pool error', err);
     // El pool se encarga automáticamente del manejo
 });
-// ✅ FUNCIÓN PRINCIPAL - queryDirect (recomendada para 90% de casos)
+// Reintento de conexión: mismos números que `queryWithClient` (1 reintento, 500 ms).
+const REINTENTOS_CONEXION = 1;
+const ESPERA_REINTENTO_MS = 500;
+// Códigos y mensajes que NO son un error de SQL sino del pool o del socket. La clase 08 de
+// PostgreSQL es "Connection Exception"; 57P01 es "admin_shutdown"; el resto son errno de Node.
+const CODIGOS_CONEXION = [
+    'EPIPE',
+    'ECONNRESET',
+    'ECONNREFUSED',
+    'ETIMEDOUT',
+    'ENOTFOUND',
+    'EHOSTUNREACH',
+    '57P01',
+    '08000',
+    '08001',
+    '08003',
+    '08004',
+    '08006',
+    '08P01',
+];
+// pg y pg-pool no ponen `code` en sus propios timeouts, así que hay que mirar el mensaje.
+// Los cuatro primeros son, literalmente, los que salen en prod-third-party-backend (T-219).
+const MENSAJES_CONEXION = [
+    'client_login_timeout',
+    'timeout exceeded when trying to connect',
+    'connection terminated due to connection timeout',
+    'query read timeout',
+    'connection terminated unexpectedly',
+    'connection ended unexpectedly',
+    'client has encountered a connection error',
+];
+/**
+ * ¿Este error es "no he podido hablar con la BD" y no "tu SQL está mal"?
+ *
+ * Sirve para decidir dos cosas: si merece la pena reintentar y si hay que devolver la conexión
+ * al pool como rota. Un error de sintaxis o una constraint violada NO entran aquí: reintentarlos
+ * es tirar el tiempo y el resultado sería el mismo.
+ */
+const esErrorDeConexion = (error) => {
+    if (!error)
+        return false;
+    if (error.code && CODIGOS_CONEXION.includes(String(error.code)))
+        return true;
+    const mensaje = String(error.message || '').toLowerCase();
+    return MENSAJES_CONEXION.some((m) => mensaje.includes(m));
+};
+/**
+ * Interruptor del contrato de T-219, APAGADO por defecto y a propósito.
+ *
+ * El rechazo salió primero como 2.0.0 justamente para que `^1.x` no se lo llevara solo. Sale ahora
+ * en una menor —que cualquier `^1.x` se lleva en cuanto refresque su lock, y esos repos suben
+ * dependencias a menudo— así que la barrera tiene que seguir en pie de otra forma: apagado, la
+ * librería se comporta exactamente como la 1.31.x.
+ *
+ * Cada servicio lo enciende con `RP_DB_RECHAZA_FALLO_CONEXION=1` CUANDO haya cerrado sus puntos
+ * de `docs/T-219-auditoria-consumidores.md` (hoy: Cliente-backend 10, Dashboard-backend 6,
+ * Cron-backend 3, IA-backend 0, back-office sin auditar).
+ *
+ * Se lee en cada llamada, no al cargar el módulo: así se puede encender por entorno sin depender
+ * del orden de los imports, y las pruebas pueden ir y volver.
+ */
+const rechazaFalloDeConexion = () => process.env.RP_DB_RECHAZA_FALLO_CONEXION === '1';
+const mapearFilas = (result) => ((result === null || result === void 0 ? void 0 : result.rows) || []).map((row) => {
+    const json = Object.keys(row);
+    const aux = {};
+    json.forEach((key) => {
+        aux[camelize(key)] = row[key];
+    });
+    return aux;
+});
+/**
+ * Coge una conexión del pool reintentando si el fallo es de conexión/login.
+ *
+ * Se separa del `client.query` a propósito (T-219): aquí todavía no se ha mandado nada a
+ * PostgreSQL, así que reintentar es seguro incluso con un INSERT o un UPDATE. Reintentar la
+ * query ya enviada no lo sería —podría duplicar una escritura—, por eso `query()` no lo hace.
+ */
+const conectarConReintento = () => __awaiter(void 0, void 0, void 0, function* () {
+    for (let intento = 0;; intento++) {
+        try {
+            return yield pool.connect();
+        }
+        catch (error) {
+            if (intento >= REINTENTOS_CONEXION || !esErrorDeConexion(error))
+                throw error;
+            // eslint-disable-next-line no-console
+            console.error(`PG connect error (intento ${intento + 1}/${REINTENTOS_CONEXION + 1}): ${error === null || error === void 0 ? void 0 : error.message}. Reintentando en ${ESPERA_REINTENTO_MS} ms.`);
+            yield sleep(ESPERA_REINTENTO_MS);
+        }
+    }
+});
+/**
+ * ✅ FUNCIÓN PRINCIPAL - queryDirect (recomendada para 90% de casos)
+ *
+ * ⚠️ CONTRATO (T-219): con `RP_DB_RECHAZA_FALLO_CONEXION=1`, un fallo de CONEXIÓN, agotados los
+ * reintentos, **rechaza**. Sin esa variable —el caso por defecto— devuelve `[]` como toda la 1.x.
+ * Un error de SQL (sintaxis, constraint, columna que no existe) se sigue logueando y
+ * devolviendo `[]` en los dos modos, como siempre.
+ *
+ * Antes todo devolvía `[]`, así que "la BD no ha conectado" y "no hay filas" eran la misma
+ * cosa para quien llamaba. En 7 días eso escondió 176 `client_login_timeout` en
+ * prod-third-party-backend: unos pocos reventaban con un TypeError y la mayoría se saltaban
+ * trabajo en silencio (entrenos que se quedaron sin estructura, notificaciones que no
+ * salieron). Ver docs de T-219.
+ *
+ * Para el que llama: si la fila puede no existir de verdad, sigue comprobándolo. Lo que ya no
+ * hace falta es adivinar si el `[]` era un fallo de BD.
+ *
+ * El cambio sale publicado en **1.32.0** con el interruptor apagado: la versión puede entrar en
+ * cualquier servicio sin avisar y aun así no cambia nada hasta que ese servicio lo enciende. Antes de
+ * encenderlo, `docs/T-219-auditoria-consumidores.md` dice, con fichero y línea, dónde cae ese
+ * rechazo en cada repo y qué queda por cerrar.
+ */
 const query = (queryText, values = []) => __awaiter(void 0, void 0, void 0, function* () {
+    let client;
+    try {
+        client = yield conectarConReintento();
+    }
+    catch (error) {
+        // Agotados los reintentos: se propaga para que el llamante decida, en vez de fingir "0 filas".
+        const enhancedError = enhancePostgreSQLError(error, queryText, values);
+        // eslint-disable-next-line no-console
+        console.error('PG query error', enhancedError);
+        if (rechazaFalloDeConexion())
+            throw error;
+        // Interruptor apagado = contrato de la 1.x: se devuelve []. La traza de arriba es lo único
+        // que queda del fallo, y es justo lo que T-219 vino a arreglar: enciéndelo cuando puedas.
+        return [];
+    }
+    let clienteRoto = false;
     try {
         const text = getParseQuery(queryText, values);
-        const result = (yield pool.query({
+        const result = (yield client.query({
             text,
-            // @ts-ignore
             values: values.flat(),
         }));
-        return (result.rows || []).map((row) => {
-            const json = Object.keys(row);
-            const aux = {};
-            json.forEach((key) => {
-                aux[camelize(key)] = row[key];
-            });
-            return aux;
-        });
+        return mapearFilas(result);
     }
     catch (error) {
         // Logging mejorado con información detallada del error
         const enhancedError = enhancePostgreSQLError(error, queryText, values);
         // eslint-disable-next-line no-console
         console.error('PG query error', enhancedError);
+        // Se cayó el socket a mitad de la query: la conexión ya no vale y el llamante tiene que
+        // enterarse. Un error de SQL, en cambio, mantiene el contrato de siempre y devuelve [].
+        // Marcar el cliente como roto NO depende del interruptor: devolver al pool una conexión
+        // muerta es un fallo por sí solo, y arreglarlo no cambia lo que ve quien llama.
+        clienteRoto = esErrorDeConexion(error);
+        if (clienteRoto && rechazaFalloDeConexion())
+            throw error;
         return [];
     }
+    finally {
+        try {
+            client.release(clienteRoto);
+        }
+        catch (releaseError) {
+            // eslint-disable-next-line no-console
+            console.error('Error releasing client:', releaseError);
+        }
+    }
 });
 exports.query = query;
 // ✅ Para casos especiales que requieren control manual de conexiones
@@ -72,14 +208,7 @@ const queryWithClient = (queryText, values = [], _retry = false) => __awaiter(vo
             text,
             values: values.flat(),
         }));
-        return (result.rows || []).map((row) => {
-            const json = Object.keys(row);
-            const aux = {};
-            json.forEach((key) => {
-                aux[camelize(key)] = row[key];
-            });
-            return aux;
-        });
+        return mapearFilas(result);
     }
     catch (error) {
         // Logging mejorado con información detallada del error
@@ -93,7 +222,7 @@ const queryWithClient = (queryText, values = [], _retry = false) => __awaiter(vo
                 released = true;
             }
             if (!_retry) {
-                yield sleep(500);
+                yield sleep(ESPERA_REINTENTO_MS);
                 return queryWithClient(queryText, values, true);
             }
         }
@@ -144,14 +273,7 @@ const longRunningQuery = (queryText, values = [], timeoutMs = 300000 // 5 minuto
             // @ts-ignore
             values: values.flat(),
         }));
-        return (result.rows || []).map((row) => {
-            const json = Object.keys(row);
-            const aux = {};
-            json.forEach((key) => {
-                aux[camelize(key)] = row[key];
-            });
-            return aux;
-        });
+        return mapearFilas(result);
     }
     finally {
         client.release();

package/lib/cjs/image/generateShareMap.js

@@ -41,20 +41,36 @@ canvas_1.GlobalFonts.registerFromPath(path_1.default.join(__dirname, '../../..',
  */
 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.
+    // ⚠️ El guard va ANTES de usar `workout`: `query` devuelve [] cuando el entreno no existe.
+    // Leer `workout.idCliente` sin comprobarlo rompía el webhook con TypeError (T-066).
+    // Desde T-219 un fallo de CONEXIÓN ya no llega como [] sino como excepción, así que se
+    // captura aquí para mantener el contrato de la función (devolver `null`, no lanzar) y se
+    // distingue en el log de "el entreno no existe".
+    let workout, cliente;
+    try {
+        [workout] = yield (0, index_1.query)('SELECT [ID], [ID CLIENTE], [TYPE], [DISTANCE], [DURATION], [DESNIVEL] FROM [WORKOUT] WHERE [ID] = ?', [idWorkout]);
+    }
+    catch (error) {
+        // eslint-disable-next-line no-console
+        console.error(`generateShareMap: la BD no respondió al leer el idWorkout=${idWorkout} (${error === null || error === void 0 ? void 0 : error.message}). No se genera la imagen de compartir.`);
+        return null;
+    }
     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.`);
+        console.error(`generateShareMap: sin fila de WORKOUT para idWorkout=${idWorkout}; el entreno no existe. No se genera la imagen de compartir.`);
+        return null;
+    }
+    try {
+        [cliente] = yield (0, index_1.query)('SELECT [PREFERRED LANGUAGE] FROM [CLIENTE] WHERE [ID] = ?', [workout.idCliente]);
+    }
+    catch (error) {
+        // eslint-disable-next-line no-console
+        console.error(`generateShareMap: la BD no respondió al leer el idCliente=${workout.idCliente} del idWorkout=${idWorkout} (${error === null || error === void 0 ? void 0 : error.message}). 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 (!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.`);
+        console.error(`generateShareMap: sin fila de CLIENTE idCliente=${workout.idCliente} del idWorkout=${idWorkout}; el cliente no existe. No se genera la imagen de compartir.`);
         return null;
     }
     const width = 1080;

package/lib/cjs/sendNotification/index.js

@@ -13,16 +13,30 @@ exports.sendNotification = void 0;
 const index_1 = require("../db/index");
 const common_1 = require("@runnerpro/common");
 const sendNotification = ({ firebaseMessaging, idCliente, title, body, screen = common_1.NOTIFICATION_SCREEN_TYPES.HOME, screenParams = {}, }) => __awaiter(void 0, void 0, void 0, function* () {
-    const devices = yield (0, index_1.query)('SELECT [SUBSCRIPTION], [TYPE] FROM [PUSH MANAGER] WHERE [ID CLIENTE] = ?', [idCliente]);
-    const [{ id: idNotification }] = yield (0, index_1.query)('INSERT INTO "CLIENTE NOTIFICACION" ("ID CLIENTE", "TIMESTAMP", "TITLE", "BODY", "PARAMS") VALUES (?, NOW(), ?, ?, ?)  RETURNING "ID"', [
-        idCliente,
-        title !== null && title !== void 0 ? title : '',
-        body,
-        JSON.stringify({
-            screen,
-            screenParams,
-        }),
-    ]);
+    // ⚠️ T-219: el INSERT lleva RETURNING y se destructura directo, así que un [] revienta con
+    // "TypeError: Cannot read properties of undefined (reading 'id')" —2 veces en la semana
+    // medida—. Se traza con el idCliente y se propaga: sin fila no hay notificación que mandar.
+    let devices, idNotification;
+    try {
+        devices = yield (0, index_1.query)('SELECT [SUBSCRIPTION], [TYPE] FROM [PUSH MANAGER] WHERE [ID CLIENTE] = ?', [idCliente]);
+        const [fila] = yield (0, index_1.query)('INSERT INTO "CLIENTE NOTIFICACION" ("ID CLIENTE", "TIMESTAMP", "TITLE", "BODY", "PARAMS") VALUES (?, NOW(), ?, ?, ?)  RETURNING "ID"', [
+            idCliente,
+            title !== null && title !== void 0 ? title : '',
+            body,
+            JSON.stringify({
+                screen,
+                screenParams,
+            }),
+        ]);
+        if (!fila)
+            throw new Error('el INSERT de CLIENTE NOTIFICACION no ha devuelto fila');
+        idNotification = fila.id;
+    }
+    catch (error) {
+        // eslint-disable-next-line no-console
+        console.error(`sendNotification: no se ha podido registrar la notificación de idCliente=${idCliente} (${error === null || error === void 0 ? void 0 : error.message}). No se envía.`);
+        throw error;
+    }
     const screenParamsString = JSON.stringify(screenParams);
     for (const device of devices) {
         yield notificationWEB(firebaseMessaging, {

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/achievements/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,kBAAkB,EAAE,MAAM,UAAU,CAAC;AAiK9C,qGAAqG;AAErG,eAAO,MAAM,qBAAqB,YAAa,GAAG,UAAU,GAAG,KAAG,GAuBjE,CAAC;AAkjBF;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,oBAAoB,WACvB,GAAG,UACJ,MAAM,SACP,GAAG,KACR,QAAQ,MAAM;IAAE,EAAE,EAAE,MAAM,CAAC;IAAC,eAAe,EAAE,GAAG,CAAC;IAAC,MAAM,CAAC,EAAE,GAAG,CAAA;CAAE,CAAC,CAyGnE,CAAC;AAEF;;;;;;;;;GASG;AACH,eAAO,MAAM,2BAA2B,WAC9B,GAAG,UACJ,MAAM,SACP,GAAG,yBAER,QAAQ,MAAM;IAAE,EAAE,EAAE,MAAM,CAAC;IAAC,eAAe,EAAE,GAAG,CAAC;IAAC,MAAM,CAAC,EAAE,GAAG,CAAA;CAAE,CAAC,CAWnE,CAAC;AAEF,uFAAuF;AACvF,eAAO,MAAM,uBAAuB,WAAkB,GAAG,KAAG,QAAQ,MAAM,EAAE,CAO3E,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/achievements/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,kBAAkB,EAAE,MAAM,UAAU,CAAC;AA6L9C,qGAAqG;AAErG,eAAO,MAAM,qBAAqB,YAAa,GAAG,UAAU,GAAG,KAAG,GAuBjE,CAAC;AA0jBF;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,oBAAoB,WACvB,GAAG,UACJ,MAAM,SACP,GAAG,KACR,QAAQ,MAAM;IAAE,EAAE,EAAE,MAAM,CAAC;IAAC,eAAe,EAAE,GAAG,CAAC;IAAC,MAAM,CAAC,EAAE,GAAG,CAAA;CAAE,CAAC,CAyGnE,CAAC;AAEF;;;;;;;;;GASG;AACH,eAAO,MAAM,2BAA2B,WAC9B,GAAG,UACJ,MAAM,SACP,GAAG,yBAER,QAAQ,MAAM;IAAE,EAAE,EAAE,MAAM,CAAC;IAAC,eAAe,EAAE,GAAG,CAAC;IAAC,MAAM,CAAC,EAAE,GAAG,CAAA;CAAE,CAAC,CAWnE,CAAC;AAEF,uFAAuF;AACvF,eAAO,MAAM,uBAAuB,WAAkB,GAAG,KAAG,QAAQ,MAAM,EAAE,CAO3E,CAAC"}

package/lib/cjs/types/db/index.d.ts

@@ -1,5 +1,27 @@
 import postgresql from 'pg';
 declare const pool: postgresql.Pool;
+/**
+ * ✅ FUNCIÓN PRINCIPAL - queryDirect (recomendada para 90% de casos)
+ *
+ * ⚠️ CONTRATO (T-219): con `RP_DB_RECHAZA_FALLO_CONEXION=1`, un fallo de CONEXIÓN, agotados los
+ * reintentos, **rechaza**. Sin esa variable —el caso por defecto— devuelve `[]` como toda la 1.x.
+ * Un error de SQL (sintaxis, constraint, columna que no existe) se sigue logueando y
+ * devolviendo `[]` en los dos modos, como siempre.
+ *
+ * Antes todo devolvía `[]`, así que "la BD no ha conectado" y "no hay filas" eran la misma
+ * cosa para quien llamaba. En 7 días eso escondió 176 `client_login_timeout` en
+ * prod-third-party-backend: unos pocos reventaban con un TypeError y la mayoría se saltaban
+ * trabajo en silencio (entrenos que se quedaron sin estructura, notificaciones que no
+ * salieron). Ver docs de T-219.
+ *
+ * Para el que llama: si la fila puede no existir de verdad, sigue comprobándolo. Lo que ya no
+ * hace falta es adivinar si el `[]` era un fallo de BD.
+ *
+ * El cambio sale publicado en **1.32.0** con el interruptor apagado: la versión puede entrar en
+ * cualquier servicio sin avisar y aun así no cambia nada hasta que ese servicio lo enciende. Antes de
+ * encenderlo, `docs/T-219-auditoria-consumidores.md` dice, con fichero y línea, dónde cae ese
+ * rechazo en cada repo y qué queda por cerrar.
+ */
 declare const query: (queryText: string, values?: (string | number | boolean)[]) => Promise<any>;
 declare const queryWithClient: (queryText: string, values?: (string | number | boolean)[], _retry?: boolean) => any;
 declare const batchQuery: (queries: any[], batchSize?: number) => Promise<any[]>;

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/db/index.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,IAAI,CAAC;AAI5B,QAAA,MAAM,IAAI,iBAYR,CAAC;AASH,QAAA,MAAM,KAAK,cAAqB,MAAM,WAAU,CAAC,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,EAAE,iBAyB5E,CAAC;AAGF,QAAA,MAAM,eAAe,cAAqB,MAAM,WAAU,CAAC,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,EAAE,0BAmDtF,CAAC;AAGF,QAAA,MAAM,UAAU,YAAmB,GAAG,EAAE,uCAmBvC,CAAC;AAGF,QAAA,MAAM,gBAAgB,cACT,MAAM,WACT,CAAC,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,EAAE,qCAyBtC,CAAC;AAGF,QAAA,MAAM,SAAS,QAAS,MAAM,EAAE,WAM/B,CAAC;AAoJF,OAAO,EACL,KAAK,EAAE,oCAAoC;AAC3C,eAAe,EAAE,wBAAwB;AACzC,gBAAgB,EAAE,uBAAuB;AACzC,UAAU,EAAE,aAAa;AACzB,SAAS,EAAE,WAAW;AACtB,IAAI,GACL,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/db/index.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,IAAI,CAAC;AAI5B,QAAA,MAAM,IAAI,iBAYR,CAAC;AAyGH;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,QAAA,MAAM,KAAK,cAAqB,MAAM,WAAU,CAAC,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,EAAE,iBA+C5E,CAAC;AAGF,QAAA,MAAM,eAAe,cAAqB,MAAM,WAAU,CAAC,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,EAAE,0BA4CtF,CAAC;AAGF,QAAA,MAAM,UAAU,YAAmB,GAAG,EAAE,uCAmBvC,CAAC;AAGF,QAAA,MAAM,gBAAgB,cACT,MAAM,WACT,CAAC,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,EAAE,qCAkBtC,CAAC;AAGF,QAAA,MAAM,SAAS,QAAS,MAAM,EAAE,WAM/B,CAAC;AAoJF,OAAO,EACL,KAAK,EAAE,oCAAoC;AAC3C,eAAe,EAAE,wBAAwB;AACzC,gBAAgB,EAAE,uBAAuB;AACzC,UAAU,EAAE,aAAa;AACzB,SAAS,EAAE,WAAW;AACtB,IAAI,GACL,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":";AAYA;;;;;;;;;;;;;;;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,+DA6CrB,CAAC;AA0HF,OAAO,EAAE,gBAAgB,EAAE,CAAC"}

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/sendNotification/index.ts"],"names":[],"mappings":"AAEA,UAAU,YAAY;IACpB,iBAAiB,EAAE,GAAG,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,GAAG,CAAC;CACpB;AAED,QAAA,MAAM,gBAAgB,yEAOnB,YAAY,kBA8Bd,CAAC;AAuBF,OAAO,EAAE,gBAAgB,EAAE,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/sendNotification/index.ts"],"names":[],"mappings":"AAEA,UAAU,YAAY;IACpB,iBAAiB,EAAE,GAAG,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,GAAG,CAAC;CACpB;AAED,QAAA,MAAM,gBAAgB,yEAOnB,YAAY,kBA0Cd,CAAC;AAuBF,OAAO,EAAE,gBAAgB,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":"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"}
+{"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,kBAgCrD,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":"AAuTA;;;;;;;;;;GAUG;AACH,QAAA,MAAM,qBAAqB,SACnB,GAAG,UAED,GAAG,KACV,QAAQ,MAAM,GAAG,IAAI,CA6EvB,CAAC;AA+EF,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;AA0GF,OAAO,EAAE,qBAAqB,EAAE,CAAC"}

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

@@ -1 +1 @@
-{"version":3,"file":"saveWorkoutLaps.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutLaps.ts"],"names":[],"mappings":"AAGA;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AAEH,QAAA,MAAM,eAAe,cAAqB,GAAG,QAAQ,GAAG,EAAE,mBAAmB,GAAG,KAAG,QAAQ,IAAI,CAuB9F,CAAC;AAEF,OAAO,EAAE,eAAe,EAAE,CAAC"}
+{"version":3,"file":"saveWorkoutLaps.d.ts","sourceRoot":"","sources":["../../../../src/workout/saveWorkoutLaps.ts"],"names":[],"mappings":"AAGA;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AAEH,QAAA,MAAM,eAAe,cAAqB,GAAG,QAAQ,GAAG,EAAE,mBAAmB,GAAG,KAAG,QAAQ,IAAI,CAyC9F,CAAC;AAEF,OAAO,EAAE,eAAe,EAAE,CAAC"}

package/lib/cjs/workout/estructuraWorkout.js

@@ -85,12 +85,25 @@ exports.getStructuraWorkout = getStructuraWorkout;
  * @param {number|string} idWorkout
  */
 const saveDoneStructuraWorkout = (idWorkout) => __awaiter(void 0, void 0, void 0, function* () {
-    const [laps, streamData, [workout], estructura] = yield Promise.all([
-        (0, db_1.query)('SELECT * FROM "WORKOUT LAP" WHERE "ID WORKOUT" = ? ORDER BY "INDEX"', [idWorkout]),
-        (0, db_1.query)('SELECT * FROM "WORKOUT STREAM" WHERE "ID WORKOUT" = ?', [idWorkout]),
-        (0, db_1.query)('SELECT "DURATION", "ID CLIENTE", "TYPE" FROM "WORKOUT" WHERE "ID" = ?', [idWorkout]),
-        (0, db_1.query)('SELECT * FROM "WORKOUT STRUCTURE" WHERE "ID WORKOUT" = ? ORDER BY "INDEX"', [idWorkout]),
-    ]);
+    // ⚠️ T-219: estas 4 lecturas son el punto donde más veces se cae el pool (135 de los 176
+    // client_login_timeout de una semana en prod-third-party-backend). Antes el fallo llegaba
+    // como `[]` y el `return` de abajo lo confundía con "este entreno no es por series": el
+    // entreno se quedaba sin estructura real y nadie se enteraba. Ahora se traza con el
+    // idWorkout y se propaga, para que el webhook que lo llamó lo registre.
+    let laps, streamData, workout, estructura;
+    try {
+        [laps, streamData, [workout], estructura] = yield Promise.all([
+            (0, db_1.query)('SELECT * FROM "WORKOUT LAP" WHERE "ID WORKOUT" = ? ORDER BY "INDEX"', [idWorkout]),
+            (0, db_1.query)('SELECT * FROM "WORKOUT STREAM" WHERE "ID WORKOUT" = ?', [idWorkout]),
+            (0, db_1.query)('SELECT "DURATION", "ID CLIENTE", "TYPE" FROM "WORKOUT" WHERE "ID" = ?', [idWorkout]),
+            (0, db_1.query)('SELECT * FROM "WORKOUT STRUCTURE" WHERE "ID WORKOUT" = ? ORDER BY "INDEX"', [idWorkout]),
+        ]);
+    }
+    catch (error) {
+        // eslint-disable-next-line no-console
+        console.error(`saveDoneStructuraWorkout: no se han podido leer los datos del idWorkout=${idWorkout} (${error === null || error === void 0 ? void 0 : error.message}). El entreno se queda SIN estructura real.`);
+        throw error;
+    }
     // Workout sin estructura (no es entrenamiento por series) → nada que hacer.
     if (!estructura || estructura.length === 0 || !workout)
         return;

package/lib/cjs/workout/saveWorkoutAplication.js

@@ -391,7 +391,21 @@ const postHooks = (idWorkout, opts) => __awaiter(void 0, void 0, void 0, functio
         // 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);
+    // ⚠️ T-219: aislado como el resto de hooks, y por un motivo concreto. El WORKOUT ya está
+    // commiteado y la estructura real es un DERIVADO idempotente que se vuelve a calcular solo
+    // (saveWorkoutLaps la re-dispara al llegar los laps, y cualquier merge posterior también).
+    // Si el rechazo escapara de aquí subiría por saveWorkoutAplication hasta el llamante y
+    // abortaría lo que este hace DESPUÉS —los laps de Strava y la notificación de sentimientos—,
+    // que sí son irrecuperables: esos webhooks ya han respondido 200 y nadie los reintenta.
+    // Como aquí es donde más se cae el pool (135 de los 176 client_login_timeout), dejarlo
+    // propagar convertía el fallo más frecuente en la pérdida más cara. saveDoneStructuraWorkout
+    // ya traza el idWorkout antes de lanzar, así que el fallo no se pierde de vista.
+    try {
+        yield (0, estructuraWorkout_1.saveDoneStructuraWorkout)(idWorkout);
+    }
+    catch (_) {
+        // ignore — trazado dentro; la estructura se recalcula en cuanto lleguen laps o un merge.
+    }
     // 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
     // (incremental). Aislado: un fallo de logros NUNCA debe romper la ingesta.
@@ -429,7 +443,15 @@ const saveMap = (idWorkout, polyline) => __awaiter(void 0, void 0, void 0, funct
     // @ts-ignore
     yield storage.bucket(process.env.CLOUD_STORAGE_BUCKET_PUBLIC).file(`Workout/${idWorkout}`).save(imageBuffer);
     const urlMap = `https://storage.googleapis.com/${process.env.CLOUD_STORAGE_BUCKET_PUBLIC}/Workout/${idWorkout}`;
-    yield (0, db_1.query)('UPDATE "WORKOUT" SET "HAVE MAP IMAGE" = TRUE, "PHOTO URL" = ? WHERE "ID" = ?', [urlMap, idWorkout]);
+    try {
+        yield (0, db_1.query)('UPDATE "WORKOUT" SET "HAVE MAP IMAGE" = TRUE, "PHOTO URL" = ? WHERE "ID" = ?', [urlMap, idWorkout]);
+    }
+    catch (error) {
+        // T-219: la imagen ya está en Storage; si esto falla, el entreno se queda sin su mapa en BD.
+        // eslint-disable-next-line no-console
+        console.error(`saveMap: mapa subido pero NO guardado en BD para idWorkout=${idWorkout} (${error === null || error === void 0 ? void 0 : error.message}).`);
+        throw error;
+    }
 });
 const saveShareWorkoutImage = (id, type) => __awaiter(void 0, void 0, void 0, function* () {
     // @ts-ignore
@@ -451,5 +473,13 @@ const saveShareWorkoutImage = (id, type) => __awaiter(void 0, void 0, void 0, fu
     // @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`;
-    yield (0, db_1.query)('UPDATE "WORKOUT" SET "PHOTO URL SHARE" = ? WHERE "ID" = ?', [urlShare, id]);
+    try {
+        yield (0, db_1.query)('UPDATE "WORKOUT" SET "PHOTO URL SHARE" = ? WHERE "ID" = ?', [urlShare, id]);
+    }
+    catch (error) {
+        // T-219: la imagen de compartir ya está subida; sin esta URL el usuario no la ve en la app.
+        // eslint-disable-next-line no-console
+        console.error(`saveShareWorkoutImage: imagen subida pero NO guardada en BD para idWorkout=${id} (${error === null || error === void 0 ? void 0 : error.message}).`);
+        throw error;
+    }
 });

package/lib/cjs/workout/saveWorkoutLaps.js

@@ -44,16 +44,34 @@ const saveWorkoutLaps = (idWorkout, laps, aplicationType) => __awaiter(void 0, v
     const nuevos = Array.isArray(laps)
         ? laps.filter((l) => l && l.index !== null && l.index !== undefined && l.durationStartLap !== null && l.durationStartLap !== undefined)
         : [];
-    const [countRow] = yield (0, db_1.query)('SELECT COUNT(*)::int AS "N" FROM "WORKOUT LAP" WHERE "ID WORKOUT" = ?', [idWorkout]);
-    const existentes = Number((countRow === null || countRow === void 0 ? void 0 : countRow.n) || 0);
-    if (nuevos.length > existentes) {
-        yield (0, db_1.query)('DELETE FROM "WORKOUT LAP" WHERE "ID WORKOUT" = ?', [idWorkout]);
-        for (const lap of nuevos) {
-            yield (0, db_1.query)('INSERT INTO "WORKOUT LAP" ("ID WORKOUT", "INDEX", "DURATION START LAP") VALUES (?, ?, ?)', [idWorkout, lap.index, lap.durationStartLap]);
+    // ⚠️ T-219: si el pool se cae en mitad del DELETE + INSERT, los laps quedan a medias. Se traza
+    // con el idWorkout y cuántos laps había que escribir, y se propaga para que quede registrado.
+    try {
+        const [countRow] = yield (0, db_1.query)('SELECT COUNT(*)::int AS "N" FROM "WORKOUT LAP" WHERE "ID WORKOUT" = ?', [idWorkout]);
+        const existentes = Number((countRow === null || countRow === void 0 ? void 0 : countRow.n) || 0);
+        if (nuevos.length > existentes) {
+            yield (0, db_1.query)('DELETE FROM "WORKOUT LAP" WHERE "ID WORKOUT" = ?', [idWorkout]);
+            for (const lap of nuevos) {
+                yield (0, db_1.query)('INSERT INTO "WORKOUT LAP" ("ID WORKOUT", "INDEX", "DURATION START LAP") VALUES (?, ?, ?)', [idWorkout, lap.index, lap.durationStartLap]);
+            }
         }
     }
+    catch (error) {
+        // eslint-disable-next-line no-console
+        console.error(`saveWorkoutLaps: fallo guardando los ${nuevos.length} laps del idWorkout=${idWorkout} (${error === null || error === void 0 ? void 0 : error.message}). Pueden haber quedado a medias.`);
+        throw error;
+    }
     // si nuevos.length <= existentes: se conserva el conjunto existente (más detalle gana)
     // Re-dispara el recálculo de la estructura real ahora que hay laps en BD.
-    yield (0, estructuraWorkout_1.saveDoneStructuraWorkout)(idWorkout);
+    // ⚠️ T-219: misma política que en los postHooks de saveWorkoutAplication — la ESCRITURA de
+    // laps de arriba propaga (es dato que se pierde), pero este recálculo es un derivado de lo
+    // que ya está en BD: si se deja propagar tumba lo que el llamante haga después (en Strava,
+    // la notificación de sentimientos) por un fallo que se puede rehacer. Se traza dentro.
+    try {
+        yield (0, estructuraWorkout_1.saveDoneStructuraWorkout)(idWorkout);
+    }
+    catch (_) {
+        // ignore — trazado dentro; los laps ya están persistidos y el recálculo es rehacible.
+    }
 });
 exports.saveWorkoutLaps = saveWorkoutLaps;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@runnerpro/backend",
-  "version": "1.31.2",
+  "version": "1.32.0",
   "description": "A collection of common backend functions",
   "exports": {
     ".": "./lib/cjs/index.js"
@@ -18,6 +18,7 @@
     "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",
+    "test:pg-query-conexion": "ts-node --transpile-only -P configs/tsconfig.cjs.json scripts/testPgQueryConexion.ts",
     "format": "prettier --write \"src/**/*.{js,jsx,ts,tsx,json,css,scss,md}\"",
     "prepare": "husky"
   },
@@ -31,7 +32,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://gitlab.com/runner-pro/runnerpro-backend.git"
+    "url": "https://github.com/david-jimenez-lamas/runnerpro-backend.git"
   },
   "author": "Runner Pro",
   "license": "MIT",