@peopl-health/nexus 5.68.0-dev.12085 → 5.68.0-dev.12126

package/lib/clinical/services/clinicalAirtableService.js

@@ -49,7 +49,6 @@ const REMINDER_TYPE_TO_AIRTABLE = {
 const PENDING_ROW_FILTER = 'IF({is_active}, "true", "false") = "true", IF({is_sent}, "true", "false") = "false"';
 const ROUTINE_OUTREACH_FILTER = 'OR({type} = "seguimiento", LEFT({schedule_id}, 9) = "recovery_")';
 const RECOVERY_FILTER = 'LEFT({schedule_id}, 9) = "recovery_"';
-// The fire trigger opens 5 min either side of reminder_date; a row still pending after that was not sent.
 const FIRE_TRIGGER_GRACE_MS = 10 * 60 * 1000;
 
 // intake.source only offers self|doctor; team_relay carries "según médico" authority.
@@ -435,8 +434,6 @@ async function hasPendingReminderForPatient({ code, withinMinutes }) {
   return Array.isArray(rows) && rows.length > 0;
 }
 
-// A row whose fire time passed unsent was skipped and is never re-triggered, and one past its expiry
-// can never fire; neither is a plan the agent or the create guard should see.
 async function readPendingFollowUpsForPatient({ code, now = new Date() }) {
   if (!code) return [];
   const rows = await airtableService.getRecordByFilter(
@@ -460,13 +457,6 @@ function isAtOrBefore(when, now) {
   return Number.isFinite(at) && at <= new Date(now).getTime();
 }
 
-/*
- * Another check-in close to this one: a net's patient check-in or another routine follow-up that
- * was sent in the last `withinMinutes`, or is still pending and due within them. The pending lower
- * bound is the trigger's own 5-minute lead, so two rows firing together see each other; an unsent
- * row older than that, or expired, was missed and is not coming. Patient-requested reminders are
- * left out; they are what she asked for.
- */
 async function hasCheckInNear({ code, withinMinutes, excludeReminderId, fireAt = null }) {
   const window = Number(withinMinutes);
   if (!code || !Number.isFinite(window) || window <= 0) return false;
@@ -479,8 +469,6 @@ async function hasCheckInNear({ code, withinMinutes, excludeReminderId, fireAt =
     null,
     ['schedule_id', 'reminder_date', 'is_sent'],
   );
-  // A sent row or a net check-in always wins. Between two pending routine rows the earlier one,
-  // then the lower id, wins, so two jobs firing together on separate instances never both stand down.
   const ownAt = new Date(fireAt).getTime() || 0;
   return (rows || []).some((row) => {
     const id = String(row.schedule_id || '');
@@ -510,7 +498,6 @@ async function readPendingReminderForPatient({ code, reminderId }) {
   } : null;
 }
 
-// Patient activity retires only turn-loss recovery rows; a seguimiento stays for the agent to keep, update or cancel.
 async function cancelPendingFollowUps({ code, trace = null, reason = 'patient_replied', recoveryOnly = false }) {
   return standDownRow({
     code,

package/lib/clinical/services/pausedSafetyNetService.js

@@ -219,7 +219,6 @@ async function applyChoice(code, choice) {
   const queued = await fileUrgentRequest(code, choice);
   if (!queued) return { queued, pledge: null };
 
-  // The team now owns this patient's next contact; no turn ran to reassess a pending seguimiento.
   await cancelPendingFollowUps({ code, reason: 'safety_net_request' }).catch((error) =>
     logger.error('[PausedSafetyNet] Pending follow-ups were not stood down', { code, error: error.message })
   );

package/lib/core/NexusMessaging.js

@@ -608,7 +608,6 @@ class NexusMessaging {
         if (chatId) {
           // This is the common ingress for text, media, buttons, lists, and flows. Retire turn-loss
           // recovery here, before specialized handlers split off, so all patient activity counts.
-          // A seguimiento survives: the next turn shows it to the agent, who keeps, updates or cancels it.
           await cancelPendingFollowUps({ code: chatId, recoveryOnly: true }).catch(error =>
             logger.error('[processIncomingMessage] Follow-up stand-down failed', { code: chatId, error: error.message })
           );
@@ -1088,10 +1087,6 @@ class NexusMessaging {
         return result;
       }
 
-      // An open net alone does not hold a seguimiento: the two ask different things. Hold only when
-      // another check-in is about to reach the patient. Same-day activity does not hold an agent
-      // seguimiento: the agent saw it on every turn since and kept it. The fire trigger does not come
-      // back for a skipped row, so a skip retires it with its reason instead of leaving it armed.
       const holdReason = await hasCheckInNear({ code, withinMinutes: FOLLOW_UP_SPACING_MIN, excludeReminderId: reminderId, fireAt: plan.fireAt })
         ? 'check_in_near'
         : await followUpCollisionReason(code, reminderId, startedAt);

package/lib/models/escalationReviewModel.js

@@ -194,6 +194,7 @@ const escalationReviewSchema = new mongoose.Schema({
   followUpDecision: { type: Number, default: null },
   draftDueAt: { type: Date, default: null },
   draftAttempts: { type: Number, default: 0 },
+  draftFailedAt: { type: Date, default: null },
   doctorAlerts: { type: [doctorAlertSchema], default: [] },
   doctorAlertDueAt: { type: Date, default: null },
   doctorAlertKind: { type: String, enum: ['initial', 'realert', null], default: null },

package/lib/services/escalationReviewService.js

@@ -1118,7 +1118,7 @@ async function generateBotDraft({ id, lease = null } = {}) {
   };
   const saved = await getEscalationReview().findOneAndUpdate(
     { _id: review._id, status: { $in: DOCUMENTING_STATUSES }, draftDueAt: lease, drafts: { $size: (review.drafts ?? []).length } },
-    { $push: { drafts: entry }, $set: { status: 'draft_ready', draftDueAt: null }, $unset: { identityBlock: '' } },
+    { $push: { drafts: entry }, $set: { status: 'draft_ready', draftDueAt: null, draftFailedAt: null }, $unset: { identityBlock: '' } },
     { new: true },
   ).lean();
   if (saved) await audit(saved, DRAFT_OPERATOR, null, { action: 'draft_saved', draftNumber: saved.drafts.length });
@@ -1811,6 +1811,7 @@ async function syncEscalationReviewCase({ id, syncedBy, requestId = null } = {})
 
 module.exports = {
   BOOKKEEPING,
+  DOCUMENTING_STATUSES,
   PRE_DECISION_STATUSES,
   _resetEscalationReviewIndexes,
   addCallLog,

package/lib/services/reviewDraftSweepService.js

@@ -3,7 +3,7 @@ const { createIntervalSweeper } = require('../utils/intervalSweeper');
 const { withTimeout } = require('../utils/timeoutUtils');
 const { getEscalationReview } = require('../models/escalationReviewModel');
 
-const { BOOKKEEPING, generateBotDraft } = require('./escalationReviewService');
+const { BOOKKEEPING, DOCUMENTING_STATUSES, generateBotDraft } = require('./escalationReviewService');
 
 const SWEEP_LIMIT = 1;
 const DEFAULT_SWEEP_INTERVAL_MS = 30 * 1000;
@@ -34,6 +34,14 @@ async function draftReview(EscalationReview, { _id, draftDueAt, draftAttempts =
       escalationReviewId: String(_id), attempts: draftAttempts + 1,
     });
   }
+  if (retry && !willRetry) {
+    const marked = await EscalationReview.updateOne(
+      { _id, draftDueAt: lease, closedAt: null, status: { $in: DOCUMENTING_STATUSES } },
+      { $set: { draftDueAt: null, draftFailedAt: new Date() } },
+      BOOKKEEPING,
+    );
+    if (marked.matchedCount) return false;
+  }
   await EscalationReview.updateOne(
     { _id, draftDueAt: lease },
     willRetry

package/package.json

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