@peopl-health/nexus 3.13.6 → 3.13.8

package/examples/basic-usage.js

@@ -81,7 +81,7 @@ async function startServer() {
     apiVersion: process.env.META_API_VERSION || 'v21.0'
   });
 
-  // Example: postBatch hooks for running classifiers after each batch completes
+  // Example: postProcess hooks for running classifiers after each batch completes
   async function classifySymptoms({ chatId }) {
     console.log(`[symptomClassifier] Running for ${chatId}`);
     // Replace with actual classifier logic: check eligibility, fetch messages, classify via LLM, persist results
@@ -94,7 +94,7 @@ async function startServer() {
     console.log(`[escalationRouting] Done for ${chatId}`);
   }
 
-  // Example: prePromptBuild hook for per-turn context injection. Runs right
+  // Example: preProcess hook for per-turn context injection. Runs right
   // before the assistant prompt is sent to the model. Use this to inject
   // per-turn instructions, escalation acknowledgments, runtime feature flags, etc.
   // Return { additionalInstructions, additionalMessages } to inject context,
@@ -104,7 +104,7 @@ async function startServer() {
   // salience option (developer message at the end of history). To test the
   // others, comment the active `return` and uncomment one of the alternatives.
   async function injectPromptContext({ chatId, runId }) {
-    console.log(`[prePromptBuild] Running for ${chatId} (runId=${runId})`);
+    console.log(`[preProcess] Running for ${chatId} (runId=${runId})`);
     // Replace with your own lookup — e.g. query a pending escalations store,
     // check feature flags, fetch session state, etc.
 
@@ -114,7 +114,7 @@ async function startServer() {
     //return {
       //additionalMessages: [{
         //role: 'developer',
-        //content: 'TEST: prePromptBuild hook is active. End your response with the marker [hook-ok].',
+        //content: 'TEST: preProcess hook is active. End your response with the marker [hook-ok].',
       //}],
     //};
 
@@ -155,8 +155,8 @@ async function startServer() {
         enabled: true,           // Enable check-after processing
         checkDelayMs: 100,       // Delay before checking for new messages (ms)
         hooks: {
-          prePromptBuild: [injectPromptContext],
-          postBatch: [classifySymptoms, classifyEscalation],
+          preProcess: [injectPromptContext],
+          postProcess: [classifySymptoms, classifyEscalation],
         }
       }
     }

package/lib/controllers/messageStatusController.js

@@ -2,12 +2,14 @@ const { logger } = require('../utils/logger');
 
 const { handleStatusCallback, getMessageStatus } = require('../helpers/messageStatusHelper');
 
+const { getEventBus } = require('../core/NexusMessaging');
+
 async function messageStatusCallbackController(req, res) {
   try {
     const { MessageSid, MessageStatus } = req.body;
     logger.debug('[MessageStatusController] Status callback received', { messageSid: MessageSid, status: MessageStatus });
-    
-    const updated = await handleStatusCallback(req.body);
+
+    const updated = await handleStatusCallback(req.body, { eventBus: getEventBus() });
     
     if (!updated) {
       logger.warn('[MessageStatusController] Message not found', { messageSid: MessageSid });

package/lib/core/NexusMessaging.js

@@ -9,7 +9,6 @@ const { logger } = require('../utils/logger');
 const { Message, insertMessage } = require('../models/messageModel');
 const { Thread } = require('../models/threadModel');
 
-const { setEventBus: setStatusEventBus } = require('../helpers/messageStatusHelper');
 const { ensureThreadExists } = require('../helpers/threadHelper');
 const { storeRunMetrics } = require('../helpers/metricsHelper');
 
@@ -44,7 +43,6 @@ class NexusMessaging {
       onFlow: null
     };
     this.events = new EventEmitter();
-    setStatusEventBus(this.events);
     this.middleware = {
       any: [],
       message: [],
@@ -497,7 +495,6 @@ class NexusMessaging {
     if (this.provider) await this.provider.disconnect();
     if (this.queueAdapter) await this.queueAdapter.shutdown();
     this.events.removeAllListeners();
-    setStatusEventBus(null);
   }
 }
 

package/lib/core/ProcessingPipeline.js

@@ -2,13 +2,6 @@ const { logger } = require('../utils/logger');
 
 const VALID_HOOKS = ['preProcess', 'postProcess'];
 
-const LEGACY_MAP = {
-  preBatch: 'preProcess',
-  prePromptBuild: 'preProcess',
-  postBatch: 'postProcess',
-  onBatchError: 'postProcess',
-};
-
 class ProcessingPipeline {
   constructor(hooks = {}) {
     this._hooks = {
@@ -16,8 +9,6 @@ class ProcessingPipeline {
       postProcess: [],
     };
 
-    this._hookMeta = new WeakMap();
-
     if (hooks && typeof hooks === 'object') {
       for (const [name, handlers] of Object.entries(hooks)) {
         const fns = Array.isArray(handlers) ? handlers : [handlers];
@@ -29,27 +20,38 @@ class ProcessingPipeline {
   }
 
   addHook(name, fn) {
-    const resolved = LEGACY_MAP[name] || name;
-
-    if (!VALID_HOOKS.includes(resolved)) {
+    if (!VALID_HOOKS.includes(name)) {
       throw new Error(`[ProcessingPipeline] Unknown hook: "${name}". Valid hooks: ${VALID_HOOKS.join(', ')}`);
     }
     if (typeof fn !== 'function') {
       throw new Error(`[ProcessingPipeline] Hook "${name}" must be a function`);
     }
 
-    this._hooks[resolved].push(fn);
-    this._hookMeta.set(fn, { originalName: name });
+    this._hooks[name].push(fn);
     return this;
   }
 
   async run(context, executeFn, shouldContinue = () => true) {
     const ctx = { metadata: {}, ...context };
+    const preProcessCount = this._hooks.preProcess.length;
+    const postProcessCount = this._hooks.postProcess.length;
+
+    logger.info('[ProcessingPipeline] Running', { chatId: ctx.chatId, type: ctx.type, preProcessHooks: preProcessCount, postProcessHooks: postProcessCount });
 
     try {
       const preProcessResult = await this._runPreProcessHooks(ctx);
+
+      logger.debug('[ProcessingPipeline] preProcess complete', {
+        chatId: ctx.chatId,
+        type: ctx.type,
+        skip: preProcessResult.skip,
+        hasAdditionalInstructions: !!preProcessResult.additionalInstructions,
+        additionalMessagesCount: preProcessResult.additionalMessages.length,
+        hasToolChoice: preProcessResult.toolChoice != null,
+      });
+
       if (preProcessResult.skip) {
-        logger.debug('[ProcessingPipeline] Skipped by preProcess hook', { chatId: ctx.chatId, type: ctx.type });
+        logger.info('[ProcessingPipeline] Skipped by preProcess hook', { chatId: ctx.chatId, type: ctx.type });
         this._runPostProcessHooks(ctx, null, null);
         return null;
       }
@@ -119,15 +121,7 @@ class ProcessingPipeline {
       if (result) postContext.result = result;
       if (error) postContext.error = error;
 
-      // onBatchError: error only. postBatch: always (original used finally block).
-      // postProcess and untagged: always.
-      const applicable = hooks.filter(fn => {
-        const meta = this._hookMeta.get(fn);
-        if (meta?.originalName === 'onBatchError') return !!error;
-        return true;
-      });
-
-      Promise.allSettled(applicable.map(fn => fn(postContext)))
+      Promise.allSettled(hooks.map(fn => fn(postContext)))
         .then(results => {
           for (const r of results) {
             if (r.status === 'rejected') {

package/lib/helpers/messageStatusHelper.js

@@ -9,12 +9,6 @@ const { handle24HourWindowError } = require('../helpers/templateRecoveryHelper')
 
 const { addLinkedRecord, updateRecordByFilter } = require('../services/airtableService');
 
-let _eventBus = null;
-
-function setEventBus(eventBus) {
-  _eventBus = eventBus;
-}
-
 async function updateMessageStatus(messageSid, status, errorCode = null, errorMessage = null) {
   try {
     const updateData = {
@@ -72,7 +66,7 @@ async function updateMessageStatus(messageSid, status, errorCode = null, errorMe
   }
 }
 
-async function handleStatusCallback(twilioStatusData) {
+async function handleStatusCallback(twilioStatusData, { eventBus } = {}) {
   const { MessageSid, MessageStatus, ErrorCode, ErrorMessage } = twilioStatusData;
 
   if (!MessageSid || !MessageStatus) {
@@ -82,8 +76,8 @@ async function handleStatusCallback(twilioStatusData) {
 
   const updated = await updateMessageStatus(MessageSid, MessageStatus.toLowerCase(), ErrorCode, ErrorMessage);
 
-  if (updated && _eventBus) {
-    safeEmit(_eventBus, 'message:status', createEvent('message:status', {
+  if (updated && eventBus) {
+    safeEmit(eventBus, 'message:status', createEvent('message:status', {
       messageId: updated.message_id,
       to: updated.numero,
       status: MessageStatus.toLowerCase(),
@@ -120,6 +114,5 @@ async function getMessageStatus(messageSid) {
 module.exports = {
   updateMessageStatus,
   handleStatusCallback,
-  getMessageStatus,
-  setEventBus
+  getMessageStatus
 };

package/package.json

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