@peopl-health/nexus 1.3.0 → 1.3.1

package/lib/controllers/assistantController.js

@@ -1,6 +1,6 @@
 const { Config_ID } = require('../config/airtableConfig');
 
-const { updateThreadActive, updateThreadStop, Thread } = require('../models/threadModel');
+const { Thread } = require('../models/threadModel');
 
 const { getRecordByFilter } = require('../services/airtableService');
 const { createAssistant, addMsgAssistant, addInsAssistant } = require('../services/assistantService');
@@ -12,7 +12,7 @@ const activeAssistantController = async (req, res) => {
   const { code, active } = req.body;
 
   try {
-    await updateThreadActive(code, active);
+    await Thread.updateOne({ code }, { $set: { active: !!active } });
     return res.status(200).send({ message: 'Active assistant' });
   } catch (error) {
     console.log(error);
@@ -114,7 +114,7 @@ const stopAssistantController = async (req, res) => {
   const { code, stop } = req.body;
 
   try {
-    await updateThreadStop(code, stop);
+    await Thread.updateOne({ code }, { $set: { stopped: !!stop } });
     return res.status(200).send({ message: 'Stop assistant' });
   } catch (error) {
     console.log(error);

package/lib/controllers/messageController.js

@@ -3,17 +3,19 @@ const { Message } = require('../models/messageModel.js');
 // Import from Nexus core
 const { sendMessage } = require('../core/NexusMessaging');
 
-// Stub for missing model
-const ScheduledMessage = {
-  create: () => Promise.resolve({ success: false, error: 'Model not available' }),
-  find: () => Promise.resolve([]),
-  findById: () => Promise.resolve(null),
-  deleteOne: () => Promise.resolve({ success: false, error: 'Model not available' })
+// Injectable dependencies with safe defaults
+let injected = {
+  ScheduledMessage: null,
+  getRecordByFilter: null,
+  sendScheduledMessage: null
 };
 
-// Stub functions for missing services
-const getRecordByFilter = () => Promise.resolve(null);
-const sendScheduledMessage = () => Promise.resolve({ success: false, error: 'Service not available' });
+// Allow consumers to inject their model and services
+const configureMessageController = ({ ScheduledMessage, getRecordByFilter, sendScheduledMessage } = {}) => {
+  if (ScheduledMessage) injected.ScheduledMessage = ScheduledMessage;
+  if (getRecordByFilter) injected.getRecordByFilter = getRecordByFilter;
+  if (sendScheduledMessage) injected.sendScheduledMessage = sendScheduledMessage;
+};
 
 const moment = require('moment-timezone');
 
@@ -33,6 +35,9 @@ const sendMessageController = async (req, res) => {
   const sendMoment = sendTime ? moment.tz(sendTime, timeZone) + 2500 : new Date();
 
   try {
+    if (!injected.ScheduledMessage || typeof injected.ScheduledMessage.create !== 'function') {
+      return res.status(500).json({ success: false, error: 'ScheduledMessage model not configured. Call configureMessageController() to inject it.' });
+    }
     const messageData = {
       fileUrl,
       message,
@@ -46,7 +51,7 @@ const sendMessageController = async (req, res) => {
       extraDelay: 0,
       variables
     };
-    await ScheduledMessage.create(messageData);
+    await injected.ScheduledMessage.create(messageData);
     console.log('Sending message with data:', messageData);
     
     const result = await sendMessage(messageData);
@@ -78,12 +83,18 @@ const sendBulkMessageController = async (req, res) => {
   const sendMoment = sendTime ? moment.tz(sendTime, timeZone) + 20*1000 : new Date();
 
   try {
+    if (!injected.ScheduledMessage || typeof injected.ScheduledMessage.create !== 'function') {
+      return res.status(500).json({ success: false, error: 'ScheduledMessage model not configured. Call configureMessageController() to inject it.' });
+    }
+    if (!injected.sendScheduledMessage || typeof injected.sendScheduledMessage !== 'function') {
+      return res.status(500).json({ success: false, error: 'sendScheduledMessage not configured. Call configureMessageController() to inject it.' });
+    }
     let numSend = 0;
     let extraDelay = 0;
     let curMessage = message;
     const scheduledMessages = [];
     for (const code of codes) {
-      const scheduledMessage = new ScheduledMessage({
+      const scheduledMessage = new injected.ScheduledMessage({
         fileUrl,
         message: curMessage,
         fileType,
@@ -108,8 +119,8 @@ const sendBulkMessageController = async (req, res) => {
     // Schedule all messages with Agenda in parallel
     const sentMessages = await Promise.all(
       scheduledMessages.map(async (message) => {
-        const savedMessage = await ScheduledMessage.create(message);
-        return sendScheduledMessage(savedMessage);
+        const savedMessage = await injected.ScheduledMessage.create(message);
+        return injected.sendScheduledMessage(savedMessage);
       })
     );
 
@@ -155,7 +166,16 @@ const sendBulkMessageAirtableController = async (req, res) => {
   }
 
   try {
-    const rows = await getRecordByFilter(baseId, tableName, condition);
+    if (!injected.getRecordByFilter || typeof injected.getRecordByFilter !== 'function') {
+      return res.status(500).json({ success: false, error: 'Airtable getRecordByFilter not configured. Call configureMessageController() to inject it.' });
+    }
+    if (!injected.ScheduledMessage || typeof injected.ScheduledMessage.create !== 'function') {
+      return res.status(500).json({ success: false, error: 'ScheduledMessage model not configured. Call configureMessageController() to inject it.' });
+    }
+    if (!injected.sendScheduledMessage || typeof injected.sendScheduledMessage !== 'function') {
+      return res.status(500).json({ success: false, error: 'sendScheduledMessage not configured. Call configureMessageController() to inject it.' });
+    }
+    const rows = await injected.getRecordByFilter(baseId, tableName, condition);
     let extraDelay = 0;
     let curMessage = message;
     const sentPhones = new Set();
@@ -194,18 +214,14 @@ const sendBulkMessageAirtableController = async (req, res) => {
         variables
       };
             
-      // Add to scheduledMessages for both saving and scheduling
       scheduledMessages.push(scheduledMessage);
-
-      // Increment delay and message counter
       extraDelay += Math.floor(Math.random() * 5001) + 5000;
     }
 
-    // Schedule all messages with Agenda in parallel
     const sentMessages = await Promise.all(
       scheduledMessages.map(async (message) => {
-        const savedMessage = await ScheduledMessage.create(message);
-        return sendScheduledMessage(savedMessage);
+        const savedMessage = await injected.ScheduledMessage.create(message);
+        return injected.sendScheduledMessage(savedMessage);
       })
     );
 
@@ -256,5 +272,6 @@ module.exports = {
   sendMessageController,
   sendBulkMessageController,
   sendBulkMessageAirtableController,
-  getLastInteractionController
+  getLastInteractionController,
+  configureMessageController
 };

package/lib/controllers/templateController.js

@@ -17,33 +17,21 @@ const checkTemplateSupport = () => {
     throw new Error('Template operations are only supported with Twilio provider');
   }
 };
-const mongoose = require('mongoose');
+
 const { handleApiError } = require('../utils/errorHandler');
 
 const { Template } = require('../templates/templateStructure');
 const predefinedTemplates = require('../templates/predefinedTemplates');
 
-// Use mongoose models to avoid conflicts
+
 const getTemplateModel = () => {
-  if (mongoose.models.Template) {
-    return mongoose.models.Template;
-  }
-  
-  // If not in mongoose.models, require and return the model
+  // Require the concrete model; enforce presence instead of stubbing
   try {
+    // If already registered in mongoose.models, require still returns the same model
     const TemplateModel = require('../models/templateModel');
     return TemplateModel;
   } catch (error) {
-    console.error('Failed to load Template model:', error);
-    // Return a stub model with required methods
-    return {
-      deleteMany: () => Promise.resolve({ deletedCount: 0 }),
-      find: () => ({ sort: () => ({ limit: () => ({ lean: () => Promise.resolve([]) }) }) }),
-      findOne: () => Promise.resolve(null),
-      create: () => Promise.resolve({}),
-      updateOne: () => Promise.resolve({}),
-      deleteOne: () => Promise.resolve({})
-    };
+    throw new Error('Template model not available. Ensure models are loaded before using template controllers.');
   }
 };
 

package/lib/models/agendaMessageModel.js

@@ -0,0 +1,24 @@
+const mongoose = require('mongoose');
+
+const scheduledMessageSchema = new mongoose.Schema({
+  fileUrl: String,
+  message: String,
+  fileType: String,
+  sendTime: Date,
+  hidePreview: Boolean,
+  contentSid: String,
+  code: String,
+  author: String,
+  variables: Object,
+  extraDelay: Number,
+  timeZone: { type: String, default: 'Etc/GMT'},
+  status: { type: String, default: 'pending' },
+  wa_id: { type: String, default: null },
+  createdAt: { type: Date, default: Date.now }
+});
+
+const ScheduledMessage = mongoose.model('ScheduledMessage', scheduledMessageSchema);
+
+module.exports = {
+  ScheduledMessage
+};

package/lib/services/assistantService.js

@@ -40,7 +40,6 @@ const getAssistantById = (assistant_id, thread) => {
   }
 
   if (!assistantConfig) {
-    // Provide a more permissive default: allow registry usage without full config
     assistantConfig = {};
   }
   

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peopl-health/nexus",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Core messaging and assistant library for WhatsApp communication platforms",
   "publishConfig": {
     "access": "public"

package/lib/services/appointmentService.js

@@ -1,55 +0,0 @@
-const { Calendar_ID } = require('../config/airtableConfig.js');
-
-const { ISO_DATE, parseStartTime, addDays } = require('../utils/dateUtils.js');
-const { dateAndTimeFromStart } = require('../utils/dateUtils.js');
-
-const { getRecordByFilter, updateRecordByFilter } = require('../services/airtableService.js');
-
-
-const getNextAppointmentForPatient = async (name, after = new Date()) => {
-  const filter = `AND(patient_name = "${name}", start_time >= '${ISO_DATE(after)}')`;
-  const rows   = await getRecordByFilter(Calendar_ID, 'calendar_quimio', filter);
-  if (!rows?.length) return null;
-  return rows.sort((a, b) => new Date(a.start_time) - new Date(b.start_time))[0];
-};
-
-const getAppointmentsBetween = async (start, end) => {
-  const filter = `AND(start_time >= '${ISO_DATE(start)}', start_time < '${ISO_DATE(end)}')`;
-  return getRecordByFilter(Calendar_ID, 'calendar_quimio', filter);
-};
-
-const updateAppointmentById = async (recordId, data) => {
-  return updateRecordByFilter(
-    Calendar_ID,
-    'calendar_quimio',
-    `REGEX_MATCH({record_id}, '${recordId}')`,
-    data
-  );
-};
-
-const buildAvailabilityWindow = async originalDate => {
-  const start = addDays(originalDate, 1);
-  const end = addDays(originalDate, 7);
-  
-  const allSlots = await getAppointmentsBetween(start, addDays(end, 1));
-  const availableSlots = allSlots.filter(row => !row.patient);
-  
-  const result = {};
-  availableSlots.forEach(row => {
-    const { date, time } = dateAndTimeFromStart(row.start_time);
-    if (!result[date]) result[date] = [];
-    result[date].push({ time, availableSpots: 1 });
-  });
-  
-  return result;
-};
-
-const parseStart = parseStartTime;
-
-module.exports = {
-  getNextAppointmentForPatient,
-  getAppointmentsBetween,
-  updateAppointmentById,
-  parseStart,
-  buildAvailabilityWindow
-};