@peopl-health/nexus 2.5.2-fix → 2.5.3

package/lib/controllers/assistantController.js

@@ -3,8 +3,8 @@ const { Config_ID } = require('../config/airtableConfig');
 const { Thread } = require('../models/threadModel');
 
 const { getRecordByFilter } = require('../services/airtableService');
-const { createAssistant, addMsgAssistant, addInsAssistant } = require('../services/assistantService');
-const { getThreadInfo, switchAssistant } = require('../services/assistantService');
+const { createAssistant, addMsgAssistant, addInsAssistant, switchAssistant } = require('../services/assistantService');
+const { getThreadInfo } = require('../helpers/threadHelper');
 const { sendMessage } = require('../core/NexusMessaging');
 const { logger } = require('../utils/logger');
 
@@ -16,7 +16,7 @@ const activeAssistantController = async (req, res) => {
     await Thread.updateOne({ code }, { $set: { active: !!active } });
     return res.status(200).send({ message: 'Active assistant' });
   } catch (error) {
-    logger.error(error);
+    logger.error('[activeAssistantController] Error', { error: error.message, code });
     return res.status(500).send({ message: 'Failed to active assistant', error });
   }
 };
@@ -32,7 +32,7 @@ const addInsAssistantController = async (req, res) => {
     if (ans) await sendMessage({code, body: ans, fileType: 'text', origin: 'assistant'});
     return res.status(200).send({ message: 'Add instruction to the assistant' });
   } catch (error) {
-    logger.error(error);
+    logger.error('[addInsAssistantController] Error', { error: error.message, code });
     res.status(500).send({ message: 'Failed to add instruction to assistant', error });
   }
 };
@@ -45,7 +45,7 @@ const addMsgAssistantController = async (req, res) => {
     if (ans) await sendMessage({code, body: ans, fileType: 'text', origin: 'assistant'});
     return res.status(200).send({ message: 'Add message to the assistant' });
   } catch (error) {
-    logger.error(error);
+    logger.error('[addMsgAssistantController] Error', { error: error.message, code, role });
     res.status(500).send({ message: 'Failed to add message assistant', error });
   }
 };
@@ -59,18 +59,18 @@ const createAssistantController = async (req, res) => {
   messages = Array.isArray(messages) ? messages : [];
 
   try {
-    logger.info('[createAssistantController] codes', codes);
+    logger.info('[createAssistantController] codes', { codes });
     for (const code of codes) {
       await createAssistant(code, assistant_id, [...instrucciones, ...messages], force);
-      logger.info('[createAssistantController] messages', messages);
+      logger.info('[createAssistantController] messages', { messages });
       for (const message of messages) {
-        logger.info('[createAssistantController] message', message);
+        logger.info('[createAssistantController] message', { message });
         await sendMessage({code, body: message, fileType: 'text'});
       }
     }
     res.status(200).send({ message: 'Create the assistant' });
   } catch (error) {
-    logger.error(error);
+    logger.error('[createAssistantController] Error creating assistant', { error: error.message, stack: error.stack });
     res.status(500).send({ message: 'Failed to create assistant', error });
   }
 };
@@ -82,7 +82,7 @@ const getInfoAssistantController = async (req, res) => {
     let threadInfo = await getThreadInfo(code);
     return res.status(200).send({ message: 'Send assistant info' , threadInfo});
   } catch (error) {
-    logger.error(error);
+    logger.error('[getInfoAssistantController] Error', { error: error.message, code });
     res.status(500).send({ message: 'Failed to receive assistant info', error });
   }
 };
@@ -97,7 +97,7 @@ const listAssistantController = async (req, res) => {
     const assistants = await getRecordByFilter(Config_ID, tableName, `status="${airtableStatus}"`);
     return res.status(200).send({ message: 'List assistants' , assistants});
   } catch (error) {
-    logger.error(error);
+    logger.error('[listAssistantController] Error', { error: error.message });
     res.status(500).send({ message: 'Failed to list assistants', error });
   }
 };
@@ -109,7 +109,7 @@ const switchAssistantController = async (req, res) => {
     await switchAssistant(code, assistant_id);
     return res.status(200).send({ message: 'Switch assistant' });
   } catch (error) {
-    logger.error(error);
+    logger.error('[switchAssistantController] Error', { error: error.message, code, assistant_id });
     res.status(500).send({ message: 'Failed to switch assistant', error });
   }
 };
@@ -121,7 +121,7 @@ const stopAssistantController = async (req, res) => {
     await Thread.updateOne({ code }, { $set: { stopped: !!stop } });
     return res.status(200).send({ message: 'Stop assistant' });
   } catch (error) {
-    logger.error(error);
+    logger.error('[stopAssistantController] Error', { error: error.message, code, stop });
     return res.status(500).send({ message: 'Failed to stop assistant', error });
   }
 };

package/lib/services/assistantServiceCore.js

@@ -5,6 +5,7 @@ const { createProvider } = require('../providers/createProvider');
 const { Thread } = require('../models/threadModel.js');
 const { PredictionMetrics } = require('../models/predictionMetricsModel');
 const { insertMessage } = require('../models/messageModel');
+const { Historial_Clinico_ID } = require('../config/airtableConfig');
 
 const { getCurRow, runAssistantWithRetries } = require('../helpers/assistantHelper.js');
 const { getThread } = require('../helpers/threadHelper.js');
@@ -19,14 +20,14 @@ const createAssistantCore = async (code, assistant_id, getAssistantById) => {
   if (!thread) return null;
 
   const assistant = getAssistantById(assistant_id, thread);
-  const curRow = await getCurRow(code);
+  const curRow = await getCurRow(Historial_Clinico_ID, code);
   const context = { curRow };
 
   try {
     await assistant.create(code, context);
     return { success: true, assistant_id };
   } catch (error) {
-    logger.error('[createAssistantCore] Error:', error);
+    logger.error('[createAssistantCore] Error creating assistant', { error: error.message, assistant_id, code });
     return { success: false, error: error.message };
   }
 };
@@ -43,7 +44,7 @@ const addMsgAssistantCore = async (code, message, role = 'user') => {
     await insertMessage({ code, message, role });
     return { success: true };
   } catch (error) {
-    logger.error('[addMsgAssistantCore] Error:', error);
+    logger.error('[addMsgAssistantCore] Error adding message', { error: error.message, code, role });
     return { success: false, error: error.message };
   }
 };
@@ -59,7 +60,7 @@ const addInstructionCore = async (code, instruction, role = 'user') => {
     await provider.addMessage({ threadId, messages: [{ role, content: instruction }] });
     return { success: true };
   } catch (error) {
-    logger.error('[addInstructionCore] Error:', error);
+    logger.error('[addInstructionCore] Error adding instruction', { error: error.message, code, role });
     return { success: false, error: error.message };
   }
 };
@@ -175,12 +176,18 @@ const replyAssistantCore = async (code, message_ = null, thread_ = null, runOpti
         retry_count: retries,
         completed: completed,
         timing_breakdown: timings
-      }).catch(err => logger.error('[replyAssistantCore] Failed to store metrics:', err));
+      }).catch(err => logger.error('[replyAssistantCore] Failed to store metrics', { error: err.message }));
     }
 
     return { output, tools_executed };
   } catch (error) {
-    logger.error('[replyAssistantCore] Error:', { error: error.message });
+    logger.error('[replyAssistantCore] Error in reply', { 
+      error: error.message,
+      stack: error.stack,
+      code: code ? `${code.substring(0, 3)}***${code.slice(-4)}` : 'unknown',
+      hasCustomThread: !!thread_,
+      hasMessage: !!message_
+    });
     throw error;
   }
 };
@@ -189,7 +196,8 @@ const switchAssistantCore = async (code, assistant_id) => {
   try {
     const thread = await Thread.findOne({ code });
     if (!thread) {
-      return null;
+      logger.warn('[switchAssistantCore] Thread not found', { code, assistant_id });
+      return { success: false, error: 'Thread not found' };
     }
 
     const updateFields = {
@@ -199,10 +207,16 @@ const switchAssistantCore = async (code, assistant_id) => {
     };
 
     await Thread.updateOne({ code }, { $set: updateFields });
+    logger.info('[switchAssistantCore] Assistant switched', { code, assistant_id });
     return { success: true, assistant_id };
   } catch (error) {
-    logger.info(error);
-    return null;
+    logger.error('[switchAssistantCore] Error switching assistant', { 
+      error: error.message, 
+      stack: error.stack,
+      code, 
+      assistant_id 
+    });
+    return { success: false, error: error.message };
   }
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peopl-health/nexus",
-  "version": "2.5.2-fix",
+  "version": "2.5.3",
   "description": "Core messaging and assistant library for WhatsApp communication platforms",
   "keywords": [
     "whatsapp",