@runnerpro/backend 1.30.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/prompt/ai.js

@@ -214,6 +214,38 @@ function withLaxSchemaDefaults(options) {
     const po = options.providerOptions;
     return Object.assign(Object.assign({}, options), { providerOptions: Object.assign(Object.assign({}, po), { openai: Object.assign({ strictJsonSchema: false }, po === null || po === void 0 ? void 0 : po.openai), azure: Object.assign({ strictJsonSchema: false }, po === null || po === void 0 ? void 0 : po.azure) }) });
 }
+/**
+ * Quita `temperature` cuando el modelo que va a atender ESTA llamada la descarta.
+ *
+ * Los GPT-5.x de Azure son "reasoning models": el SDK borra `temperature` y emite un warning
+ * ("temperature is not supported for reasoning models") en CADA llamada. No rompe nada —la
+ * llamada sigue y no salta al fallback—, pero ensucia los logs del pipeline (~80 call sites
+ * pasan temperature) y hace creer que el valor se está aplicando cuando no es así.
+ *
+ * ⚠️ Se aplica POR INTENTO, no una vez al entrar: la cola de la cascada es Gemini, y ahí
+ * `temperature` SÍ se respeta. Los modelos de Google/Bedrock no llevan la etiqueta
+ * `_ignoresTemperature`, así que conservan el valor que pase el caller.
+ *
+ * @param options - Opciones tal y como se van a mandar al SDK en este intento
+ * @returns Las mismas opciones, sin `temperature` si el modelo la fuese a descartar
+ */
+function withoutUnsupportedTemperature(options) {
+    var _a, _b, _c;
+    const model = options.model;
+    if (!(model === null || model === void 0 ? void 0 : model._ignoresTemperature))
+        return options;
+    if (typeof options.temperature !== 'number')
+        return options;
+    // Única vía de escape del SDK: reasoningEffort 'none' en las familias que aún admiten
+    // parámetros de muestreo (5.1–5.5). Si el caller la pide, respetamos su temperature.
+    const po = options.providerOptions;
+    const reasoningEffort = (_b = (_a = po === null || po === void 0 ? void 0 : po.openai) === null || _a === void 0 ? void 0 : _a.reasoningEffort) !== null && _b !== void 0 ? _b : (_c = po === null || po === void 0 ? void 0 : po.azure) === null || _c === void 0 ? void 0 : _c.reasoningEffort;
+    if (reasoningEffort === 'none' && model._supportsNonReasoningParameters)
+        return options;
+    const rest = Object.assign({}, options);
+    delete rest.temperature;
+    return rest;
+}
 function generateObject(options) {
     var _a;
     return __awaiter(this, void 0, void 0, function* () {
@@ -236,7 +268,7 @@ function generateObject(options) {
         let fallbackIdx = 0;
         for (let attempt = 0; attempt < maxAttempts; attempt++) {
             try {
-                const { object, usage } = yield (0, ai_1.generateObject)(currentOptions);
+                const { object, usage } = yield (0, ai_1.generateObject)(withoutUnsupportedTemperature(currentOptions));
                 const modelName = extractModelName(currentOptions.model);
                 const cost = (0, modelPricing_1.calculateCost)(modelName, usage === null || usage === void 0 ? void 0 : usage.inputTokens, usage === null || usage === void 0 ? void 0 : usage.outputTokens);
                 const tracker = costTrackingStorage.getStore();
@@ -397,7 +429,7 @@ function generateText(options) {
         let fallbackIdx = 0;
         for (let attempt = 0; attempt < maxAttempts; attempt++) {
             try {
-                const { text, usage } = yield (0, ai_1.generateText)(currentOptions);
+                const { text, usage } = yield (0, ai_1.generateText)(withoutUnsupportedTemperature(currentOptions));
                 const modelName = extractModelName(currentOptions.model);
                 const cost = (0, modelPricing_1.calculateCost)(modelName, usage === null || usage === void 0 ? void 0 : usage.inputTokens, usage === null || usage === void 0 ? void 0 : usage.outputTokens);
                 const cleanResult = yield ensureNoChainOfThought({ rawText: text, baseCost: cost, originalOptions: options, modelLabel: modelName });
@@ -459,9 +491,11 @@ function ensureNoChainOfThought(params) {
         const accumulatedCost = Object.assign({}, baseCost);
         const baseTemperature = typeof originalOptions.temperature === 'number' ? originalOptions.temperature : 0.7;
         const retryTemperature = Math.min(1, baseTemperature + 0.1);
-        // Paso 2: retry con system anti-CoT y temperatura ligeramente superior
+        // Paso 2: retry con system anti-CoT y temperatura ligeramente superior.
+        // En los reasoning models de Azure el empujón de temperatura no llega a aplicarse (el SDK la
+        // descarta), así que allí lo que desatasca el retry es únicamente el `system` reforzado.
         try {
-            const { text: retryText, usage: retryUsage } = yield (0, ai_1.generateText)(Object.assign(Object.assign({}, originalOptions), { system: buildAntiCotSystem(originalOptions.system), temperature: retryTemperature }));
+            const { text: retryText, usage: retryUsage } = yield (0, ai_1.generateText)(withoutUnsupportedTemperature(Object.assign(Object.assign({}, originalOptions), { system: buildAntiCotSystem(originalOptions.system), temperature: retryTemperature })));
             const retryModelLabel = extractModelName(originalOptions.model);
             const retryCost = (0, modelPricing_1.calculateCost)(retryModelLabel, retryUsage === null || retryUsage === void 0 ? void 0 : retryUsage.inputTokens, retryUsage === null || retryUsage === void 0 ? void 0 : retryUsage.outputTokens);
             accumulatedCost.inputTokens = (accumulatedCost.inputTokens || 0) + (retryCost.inputTokens || 0);

package/lib/cjs/prompt/azureModel.js

@@ -20,10 +20,34 @@ function getAzureProvider() {
     const baseURL = process.env.AZURE_BASE_URL;
     return (0, azure_1.createAzure)(Object.assign(Object.assign({ apiKey: process.env.AZURE_API_KEY }, (baseURL ? { baseURL } : { resourceName: process.env.AZURE_RESOURCE_NAME })), (apiVersion ? { apiVersion } : {})));
 }
+// Prefijos de razonamiento que NO son de la familia gpt-5 (esa se trata aparte por la
+// excepción de gpt-5-chat, que es el único gpt-5* conversacional).
+const OPENAI_REASONING_PREFIXES = ['o1', 'o3', 'o4-mini'];
+// Familias que recuperan los parámetros de muestreo si se pide `reasoningEffort: 'none'`.
+// ⚠️ La 5.6 NO está en la lista: en terra/luna `temperature` se descarta siempre, pase lo
+// que pase. Réplica de `supportsNonReasoningParameters` en @ai-sdk/openai.
+const NON_REASONING_PARAM_PREFIXES = ['gpt-5.1', 'gpt-5.2', 'gpt-5.3', 'gpt-5.4', 'gpt-5.5'];
+/**
+ * Indica si el SDK va a tratar el deployment como "reasoning model" y, por tanto, va a
+ * DESCARTAR `temperature` y `topP` antes de llamar (avisando con un warning por llamada).
+ *
+ * Réplica de `getOpenAILanguageModelCapabilities` de @ai-sdk/openai: los modelos `.responses()`
+ * de @ai-sdk/azure comparten ese parser, así que la regla aplica igual a los deployments de Azure.
+ * Los DeepSeek quedan fuera a propósito (no empiezan por gpt-5/o*), que es justo lo que hace el SDK.
+ *
+ * @param deploymentName - Nombre del deployment (ej: 'gpt-5.6-terra')
+ * @returns true si el SDK descartará los parámetros de muestreo
+ */
+function isReasoningDeployment(deploymentName) {
+    if (deploymentName.startsWith('gpt-5'))
+        return !deploymentName.startsWith('gpt-5-chat');
+    return OPENAI_REASONING_PREFIXES.some((prefix) => deploymentName.startsWith(prefix));
+}
 /**
  * Crea una instancia del modelo de Azure OpenAI a partir del nombre del deployment.
  * Etiqueta el objeto con _productionModelName (nombre limpio, sin el prefijo "azure:")
- * para el cálculo de costes.
+ * para el cálculo de costes, y con _ignoresTemperature / _supportsNonReasoningParameters
+ * para que `generateObject`/`generateText` no manden `temperature` a un modelo que la descarta.
  *
  * @param deploymentName - Nombre del deployment en Azure (ej: 'gpt-5')
  * @returns Instancia del modelo de Azure compatible con AI SDK
@@ -42,6 +66,8 @@ function createAzureModelFromString(deploymentName) {
     const model = azure.responses(deploymentName);
     model._productionModelName = deploymentName;
     model._fallbackModelNames = [];
+    model._ignoresTemperature = isReasoningDeployment(deploymentName);
+    model._supportsNonReasoningParameters = NON_REASONING_PARAM_PREFIXES.some((prefix) => deploymentName.startsWith(prefix));
     return model;
 }
 exports.createAzureModelFromString = createAzureModelFromString;

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/prompt/ai.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ai.d.ts","sourceRoot":"","sources":["../../../../src/prompt/ai.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,IAAI,sBAAsB,EAAE,YAAY,IAAI,oBAAoB,EAAE,MAAM,IAAI,CAAC;AAEpG,OAAO,EAAiB,KAAK,UAAU,EAAE,MAAM,gBAAgB,CAAC;AAoNhE,iBAAe,cAAc,CAAC,OAAO,EAAE,UAAU,CAAC,OAAO,sBAAsB,CAAC,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,GAAG,CAAC;IAAC,IAAI,EAAE,UAAU,CAAA;CAAE,CAAC,CAoD/H;AAkFD;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,iBAAe,YAAY,CAAC,OAAO,EAAE,UAAU,CAAC,OAAO,oBAAoB,CAAC,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,UAAU,CAAA;CAAE,CAAC,CA4C5H;AA+ED;;;;;;;;;;;;;;;;;GAiBG;AACH,iBAAe,mBAAmB,CAAC,CAAC,EAAE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,CAAC,CAAC;IAAC,SAAS,EAAE,UAAU,CAAC;IAAC,SAAS,EAAE,MAAM,CAAA;CAAE,CAAC,CAgB5H;AAED,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,mBAAmB,EAAE,CAAC"}
+{"version":3,"file":"ai.d.ts","sourceRoot":"","sources":["../../../../src/prompt/ai.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,IAAI,sBAAsB,EAAE,YAAY,IAAI,oBAAoB,EAAE,MAAM,IAAI,CAAC;AAEpG,OAAO,EAAiB,KAAK,UAAU,EAAE,MAAM,gBAAgB,CAAC;AAmPhE,iBAAe,cAAc,CAAC,OAAO,EAAE,UAAU,CAAC,OAAO,sBAAsB,CAAC,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,GAAG,CAAC;IAAC,IAAI,EAAE,UAAU,CAAA;CAAE,CAAC,CAoD/H;AAkFD;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,iBAAe,YAAY,CAAC,OAAO,EAAE,UAAU,CAAC,OAAO,oBAAoB,CAAC,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,UAAU,CAAA;CAAE,CAAC,CA4C5H;AAmFD;;;;;;;;;;;;;;;;;GAiBG;AACH,iBAAe,mBAAmB,CAAC,CAAC,EAAE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,CAAC,CAAC;IAAC,SAAS,EAAE,UAAU,CAAC;IAAC,SAAS,EAAE,MAAM,CAAA;CAAE,CAAC,CAgB5H;AAED,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,mBAAmB,EAAE,CAAC"}

package/lib/cjs/types/prompt/azureModel.d.ts

@@ -1,7 +1,8 @@
 /**
  * Crea una instancia del modelo de Azure OpenAI a partir del nombre del deployment.
  * Etiqueta el objeto con _productionModelName (nombre limpio, sin el prefijo "azure:")
- * para el cálculo de costes.
+ * para el cálculo de costes, y con _ignoresTemperature / _supportsNonReasoningParameters
+ * para que `generateObject`/`generateText` no manden `temperature` a un modelo que la descarta.
  *
  * @param deploymentName - Nombre del deployment en Azure (ej: 'gpt-5')
  * @returns Instancia del modelo de Azure compatible con AI SDK

package/lib/cjs/types/prompt/azureModel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"azureModel.d.ts","sourceRoot":"","sources":["../../../../src/prompt/azureModel.ts"],"names":[],"mappings":"AAyBA;;;;;;;;;;;;;GAaG;AACH,iBAAS,0BAA0B,CAAC,cAAc,EAAE,MAAM,GAAG,GAAG,CAS/D;AAED,OAAO,EAAE,0BAA0B,EAAE,CAAC"}
+{"version":3,"file":"azureModel.d.ts","sourceRoot":"","sources":["../../../../src/prompt/azureModel.ts"],"names":[],"mappings":"AAkDA;;;;;;;;;;;;;;GAcG;AACH,iBAAS,0BAA0B,CAAC,cAAc,EAAE,MAAM,GAAG,GAAG,CAW/D;AAED,OAAO,EAAE,0BAA0B,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.30.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"
   },