@peopl-health/nexus 5.67.0-dev.11849 → 5.67.0-dev.11889

package/lib/clinical/providers/BaseLLMProvider.js

@@ -125,13 +125,13 @@ class BaseLLMProvider {
       assistant.toolRuntimeContext.enabledSkills = enabledSkills;
     }
 
-    const resolvedSchemas = this._resolveToolSchemas({
+    const resolvedSchemas = this._scopeOutreachDecisionTool(this._resolveToolSchemas({
       assistant, toolIds: resolvedToolIds, filtered, toolDescriptions,
-    });
+    }), isOutreach);
     const { toolChoice: flooredToolChoice, toolSchemas, narrowed } = this._applyDeliveryFloor(
       toolChoice, resolvedSchemas, Boolean(assistant?.toolRuntimeContext), turnKind);
     const effectiveToolIds = toolSchemas.map(s => s.function?.name).filter(Boolean);
-    const instructionToolIds = narrowed ? effectiveToolIds : resolvedToolIds;
+    const instructionToolIds = narrowed || isOutreach ? effectiveToolIds : resolvedToolIds;
     const requestedToolChoice = prePromptResult?.toolChoice;
     const requestedToolName = requestedToolChoice?.name || requestedToolChoice?.function?.name || null;
     const supportedGenericChoices = new Set(['auto', 'none', 'required', 'any']);
@@ -234,6 +234,17 @@ class BaseLLMProvider {
     };
   }
 
+  // The decision tool belongs to the turn kind, not to a preset: without it an outreach turn cannot
+  // close and the brief is dropped, and on any other turn it would invite a decision nobody asked for.
+  _scopeOutreachDecisionTool(schemas, isOutreach) {
+    const others = schemas.filter(s => s.function?.name !== OUTREACH_DECISION_TOOL);
+    if (!isOutreach) return others;
+    const decision = schemas.find(s => s.function?.name === OUTREACH_DECISION_TOOL)
+      || getRegistrySchemas([OUTREACH_DECISION_TOOL])[0];
+    if (!decision) throw new Error('OUTREACH_DECISION_TOOL_UNAVAILABLE');
+    return [...others, decision];
+  }
+
   _resolveToolSchemas({ assistant, toolIds, filtered, toolDescriptions }) {
     const assistantSchemas = assistant?.getToolSchemas?.() || [];
     const registrySchemas = getRegistrySchemas();

package/lib/clinical/services/nutritionPlanService.js

@@ -7,6 +7,7 @@ const NUTRITION_GUIDE_PROMPT_ID = 'nutrition_plan_guide';
 const PLAN_MAX_CHARS = 1500;
 const GUIDE_MAX_CHARS = 2000;
 const ISO_DATE_RX = /^\d{4}-\d{2}-\d{2}$/;
+const GUIDE_TIMEOUT_MS = 3000;
 
 // Each line of a plan or guide is one kind of indication, so the breaks survive and only each line is flattened.
 function keepLines(text, maxChars) {
@@ -30,14 +31,34 @@ function readPlanSection(section) {
   return { text: typeof text === 'string' ? text : '', reviewBy };
 }
 
+let lastGuide = '';
+let guideRead = null;
+
+// One shared read; a failure keeps the last guide that was read successfully.
+function readGuide() {
+  if (guideRead) return guideRead;
+  guideRead = composePrompt({ promptId: NUTRITION_GUIDE_PROMPT_ID })
+    .then(({ resolvedPrompt }) => {
+      lastGuide = keepLines(resolvedPrompt || '', GUIDE_MAX_CHARS);
+      return lastGuide;
+    })
+    .catch((error) => {
+      logger.warn('[nutritionPlanService] nutrition guide unreadable; using the last one read', { error: error.message });
+      return lastGuide;
+    })
+    .finally(() => { guideRead = null; });
+  return guideRead;
+}
+
+// The guide only adds to the plan, so a slow read must not hold the patient's turn.
 async function loadNutritionGuide() {
-  try {
-    const { resolvedPrompt } = await composePrompt({ promptId: NUTRITION_GUIDE_PROMPT_ID });
-    return keepLines(resolvedPrompt || '', GUIDE_MAX_CHARS);
-  } catch (error) {
-    logger.warn('[nutritionPlanService] nutrition guide unavailable; plan sent without it', { error: error.message });
-    return '';
-  }
+  let timer;
+  const timedOut = new Promise((resolve) => { timer = setTimeout(() => resolve(null), GUIDE_TIMEOUT_MS); });
+  const guide = await Promise.race([readGuide(), timedOut]);
+  clearTimeout(timer);
+  if (guide !== null) return guide;
+  logger.warn('[nutritionPlanService] nutrition guide read timed out; using the last one read', { timeoutMs: GUIDE_TIMEOUT_MS });
+  return lastGuide;
 }
 
 async function resolveNutritionPlan(section, { now = new Date() } = {}) {
@@ -55,5 +76,7 @@ async function resolveNutritionPlan(section, { now = new Date() } = {}) {
 
 module.exports = {
   NUTRITION_GUIDE_PROMPT_ID,
+  GUIDE_TIMEOUT_MS,
   resolveNutritionPlan,
+  _test: { reset: () => { lastGuide = ''; guideRead = null; } },
 };

package/lib/eval/EvalProvider.js

@@ -75,7 +75,7 @@ class EvalProvider {
       const { messages, promptVariables, lastUserMessage, contextSource } = await this._buildContext(numero, beforeCheckpoint);
 
       const {
-        devContent, assistant, toolSchemas, resolvedPromptId, presetModelConfig, resolvedPresetId,
+        devContent, assistant, toolSchemas, resolvedPromptId, presetModelConfig, resolvedPresetId, interpolatedVariables,
       } = await this._resolvePrompt(prompt, assistantId, thread, promptVariables);
 
       const request = this._buildRequest({
@@ -83,6 +83,7 @@ class EvalProvider {
         assistantId: resolvedPromptId || assistantId,
         presetId: resolvedPresetId,
         presetModelConfig,
+        interpolatedVariables,
       });
 
       const startTime = Date.now();
@@ -133,6 +134,7 @@ class EvalProvider {
 
   async _resolvePrompt(fallbackPrompt, assistantId, thread, promptVariables) {
     let devContent;
+    let interpolatedVariables = [];
     let resolvedPromptId = assistantId;
     let presetToolIds = null;
     let presetModelConfig = null;
@@ -143,18 +145,21 @@ class EvalProvider {
     if (this.promptSource === 'airtable' && (presetId || assistantId)) {
       const {
         resolvedPrompt, promptId, presetToolIds: toolIds, modelConfig, presetId: composedPresetId,
+        interpolatedVariables: interpolated,
       } = await composePrompt({
         presetId,
         promptId: assistantId,
         variables: promptVariables,
       });
       devContent = resolvedPrompt;
+      interpolatedVariables = interpolated || [];
       resolvedPromptId = promptId || assistantId;
       presetToolIds = toolIds;
       presetModelConfig = modelConfig || null;
       resolvedPresetId = composedPresetId || resolvedPresetId;
     } else {
       devContent = fallbackPrompt;
+      interpolatedVariables = [...new Set([...(devContent || '').matchAll(/\{\{(\w+)\}\}/g)].map((m) => m[1]))];
       devContent = devContent.replace(/\{\{(\w+)\}\}/g, (_, key) => promptVariables[key] ?? '');
     }
 
@@ -191,7 +196,7 @@ class EvalProvider {
       }
     }
 
-    return { devContent, assistant, toolSchemas, resolvedPromptId, presetModelConfig, resolvedPresetId };
+    return { devContent, assistant, toolSchemas, resolvedPromptId, presetModelConfig, resolvedPresetId, interpolatedVariables };
   }
 
   _modelConfig(presetModelConfig) {
@@ -204,7 +209,7 @@ class EvalProvider {
 
   _buildRequest({
     devContent, messages, promptVariables, toolSchemas, assistant, assistantId, presetId,
-    presetModelConfig = null,
+    presetModelConfig = null, interpolatedVariables = [],
   }) {
     const convertedMessages = this.provider._convertItemsToApiFormat(messages);
 
@@ -215,7 +220,11 @@ class EvalProvider {
       ? [{ role: 'developer', content: `Usa la siguiente información que conoces del paciente para personalizar tu respuesta. Adapta tu tono, contenido o recomendaciones según estas preferencias y contexto cuando sea relevante. No menciones que tienes esta información almacenada.\n\n${memoryParts.join('\n\n')}` }]
       : [];
 
-    const input = this.provider._sanitizeInput([...memoryMessage, ...convertedMessages]);
+    // The same runtime clinical block a live turn gets (context, symptoms, nutrition plan), unless the prompt interpolates it.
+    const clinicalMessage = typeof this.provider._buildClinicalMessage === 'function'
+      ? this.provider._buildClinicalMessage(promptVariables, interpolatedVariables)
+      : [];
+    const input = this.provider._sanitizeInput([...memoryMessage, ...clinicalMessage, ...convertedMessages]);
     const isContextOnly = this.mode === 'context-only';
 
     const shouldUsePromptRegistry = !presetId && this.provider.supportsPromptRegistry && !!assistantId;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peopl-health/nexus",
-  "version": "5.67.0-dev.11849",
+  "version": "5.67.0-dev.11889",
   "description": "Core messaging and assistant library for WhatsApp communication platforms",
   "keywords": [
     "whatsapp",