@peopl-health/nexus 2.4.5-fix → 2.4.6

package/lib/helpers/processHelper.js

@@ -198,36 +198,43 @@ const processMediaFiles = async (code, reply, provider) => {
   return { messagesChat, url, tempFiles };
 };
 
-const processIndividualMessage = async (code, reply, provider, thread) => {  
-  let tempFiles = [];
-  try {
-    const isPatient = reply.origin === 'patient';
-    const textMessages = processTextMessage(reply);
-    
-    const { messagesChat: mediaMessages, url, tempFiles: mediaFiles } = await processMediaFiles(code, reply, provider);
-    tempFiles = mediaFiles;
-    
-    const allMessages = [...textMessages, ...mediaMessages];
-    
-    if (allMessages.length > 0) {
-      await addMessageToThread(reply, allMessages, provider, thread);
-      await updateMessageRecord(reply, thread);
-    }
-    
-    return { isPatient, url };
-  } catch (error) {
-    logger.error('processIndividualMessage', error, {
-      message_id: reply.message_id,
-      code: code ? `${code.substring(0, 3)}***${code.slice(-4)}` : 'unknown',
-      origin: reply.origin
-    });
-    await cleanupFiles(tempFiles);
-    return { isPatient: false, url: null };
-  } finally {
-    if (tempFiles.length > 0) {
-      await cleanupFiles(tempFiles);
-    }
-  }
+const processThreadMessage = async (code, replies, provider) => {
+  const replyArray = Array.isArray(replies) ? replies : [replies];
+  
+  const results = await Promise.all(
+    replyArray.map(async (reply, i) => {
+      let tempFiles = [];
+      try {
+        const isPatient = reply.origin === 'patient';
+        const [textMessages, mediaResult] = await Promise.all([
+          Promise.resolve(processTextMessage(reply)),
+          processMediaFiles(code, reply, provider)
+        ]);
+        
+        const { messagesChat: mediaMessages, url, tempFiles: mediaFiles } = mediaResult;
+        tempFiles = mediaFiles;
+        
+        const allMessages = [...textMessages, ...mediaMessages];
+        const role = reply.origin === 'patient' ? 'user' : 'assistant';
+        const messages = allMessages.map(content => ({ role, content }));
+        
+        logger.info('processThreadMessage', { 
+          index: i + 1, 
+          total: replyArray.length,
+          isPatient, 
+          hasUrl: !!url 
+        });
+        
+        return { isPatient, url, messages, reply, tempFiles };
+      } catch (error) {
+        logger.error('processThreadMessage', error, { message_id: reply.message_id, origin: reply.origin });
+        await cleanupFiles(tempFiles);
+        return { isPatient: false, url: null, messages: [], reply, tempFiles: [] };
+      }
+    })
+  );
+  
+  return results;
 };
 
 module.exports = {
@@ -235,5 +242,5 @@ module.exports = {
   processImageFile,
   processAudioFile,
   processMediaFiles,
-  processIndividualMessage
+  processThreadMessage
 };

package/lib/providers/OpenAIAssistantsProvider.js

@@ -82,22 +82,28 @@ class OpenAIAssistantsProvider {
     await this.client.beta.threads.del(this._ensureId(threadId));
   }
 
-  async addMessage({ threadId, role = 'user', content, metadata }) {
-    const formattedContent = this._normalizeContent(content);
-
-    const payload = {
-      role,
-      content: formattedContent,
-      metadata,
-    };
-
-    if (payload.metadata && Object.keys(payload.metadata).length === 0) {
-      delete payload.metadata;
+  async addMessage({ threadId, messages, role = 'user', content, metadata }) {
+    const id = this._ensureId(threadId);
+    const messagesToAdd = messages || [{ role, content, metadata }];
+    
+    const results = [];
+    for (const msg of messagesToAdd) {
+      const formattedContent = this._normalizeContent(msg.content);
+      const payload = {
+        role: msg.role || 'user',
+        content: formattedContent,
+      };
+      
+      if (msg.metadata && Object.keys(msg.metadata).length > 0) {
+        payload.metadata = msg.metadata;
+      }
+      
+      const result = await this._retryWithRateLimit(() => 
+        this.client.beta.threads.messages.create(id, payload)
+      );
+      results.push(result);
     }
-
-    return this._retryWithRateLimit(() => 
-      this.client.beta.threads.messages.create(this._ensureId(threadId), payload)
-    );
+    return messages ? results : results[0];
   }
 
   async listMessages({ threadId, runId, order = 'desc', limit } = {}) {

package/lib/providers/OpenAIResponsesProvider.js

@@ -197,23 +197,26 @@ class OpenAIResponsesProvider {
     return await this._delete(`/conversations/${id}`);
   }
 
-  async addMessage({ threadId, role = 'user', content, metadata }) {
+  async addMessage({ threadId, messages, role = 'user', content, metadata }) {
     const id = this._ensurethreadId(threadId);
+    const messagesToAdd = messages || [{ role, content, metadata }];
     
-    const payload = this._cleanObject({
-      role,
-      content: this._normalizeContent(role, content),
-      type: 'message',
-    });
+    const payloads = messagesToAdd.map(msg => 
+      this._cleanObject({
+        role: msg.role || 'user',
+        content: this._normalizeContent(msg.role || 'user', msg.content),
+        type: 'message',
+      })
+    ).filter(p => p.content);
 
-    if (payload.content) {
-      return this._retryWithRateLimit(async () => {
-        if (this.conversations?.items?.create) {
-          return await this.conversations.items.create(id, {items: [payload]});
-        }
-        return await this._post(`/conversations/${id}/items`, {items: [payload]});
-      });
-    }
+    if (payloads.length === 0) return null;
+
+    return this._retryWithRateLimit(async () => {
+      if (this.conversations?.items?.create) {
+        return await this.conversations.items.create(id, { items: payloads });
+      }
+      return await this._post(`/conversations/${id}/items`, { items: payloads });
+    });
   }
 
   async listMessages({ threadId, order = 'desc', limit } = {}) {

package/lib/services/assistantService.js

@@ -12,8 +12,8 @@ const { getCurRow } = require('../helpers/assistantHelper.js');
 const { runAssistantAndWait, runAssistantWithRetries } = require('../helpers/assistantHelper.js');
 const { getThread, getThreadInfo } = require('../helpers/threadHelper.js');
 const { withTracing } = require('../utils/tracingDecorator.js');
-const { processIndividualMessage } = require('../helpers/processHelper.js');
-const { getLastMessages } = require('../helpers/messageHelper.js');
+const { processThreadMessage } = require('../helpers/processHelper.js');
+const { getLastMessages, updateMessageRecord } = require('../helpers/messageHelper.js');
 const { combineImagesToPDF, cleanupFiles } = require('../helpers/filesHelper.js');
 const { logger } = require('../middleware/requestId');
 
@@ -297,19 +297,24 @@ const replyAssistantCore = async (code, message_ = null, thread_ = null, runOpti
   const provider = createProvider({ variant: process.env.VARIANT || 'assistants' });
   
   timings.processMessages = Date.now();
-  logger.log(`[replyAssistantCore] Processing ${patientReply.length} messages sequentially`);
+  logger.log(`[replyAssistantCore] Processing ${patientReply.length} messages in parallel`);
   
-  let patientMsg = false;
-  const urls = [];
+  const processResults = await processThreadMessage(code, patientReply, provider);
   
-  for (let i = 0; i < patientReply.length; i++) {
-    const reply = patientReply[i];
-    const { isPatient, url } = await processIndividualMessage(code, reply, provider, thread);
-    logger.log(`[replyAssistantCore] Message ${i + 1}/${patientReply.length}: isPatient=${isPatient}, hasUrl=${!!url}`);
-    patientMsg = patientMsg || isPatient;
-    if (url) urls.push({ url });
+  const patientMsg = processResults.some(r => r.isPatient);
+  const urls = processResults.filter(r => r.url).map(r => ({ url: r.url }));
+  const allMessagesToAdd = processResults.flatMap(r => r.messages || []);
+  const allTempFiles = processResults.flatMap(r => r.tempFiles || []);
+  
+  if (allMessagesToAdd.length > 0) {
+    const threadId = thread.getConversationId();
+    logger.log(`[replyAssistantCore] Adding ${allMessagesToAdd.length} messages to thread in batch`);
+    await provider.addMessage({ threadId, messages: allMessagesToAdd });
   }
   
+  await Promise.all(processResults.map(r => updateMessageRecord(r.reply, thread)));
+  await cleanupFiles(allTempFiles);
+  
   timings.processMessages = Date.now() - timings.processMessages;
   
   if (urls.length > 0) {

package/package.json

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