@automagik/omni 2.260430.11 → 2.260430.13

package/db/drizzle/0035_unstick_redirected_sac_terminals.sql

@@ -0,0 +1,47 @@
+-- Backfill: chats that auto-promoted to a hard terminal under the old
+-- `redirected_sac` escalation rule (escalationThreshold=2 over 7 days) get
+-- `chats.settings.closed` flipped back to false. Companion to the config
+-- fix in `packages/api/src/routes/v2/_close-contact-config.ts` which drops
+-- the escalation going forward — an active customer ending every
+-- conversation with a support-channel redirect is normal account activity,
+-- not a signal worth silencing the reactive agent over.
+--
+-- `closeUntil`, `closeOutcome`, and `close_contact_logs` are deliberately
+-- left intact so reporting and downstream BI consumers see no change.
+-- Hard terminals from `won`/`lost` outcomes are not touched.
+--
+-- Hand-written (not `drizzle-kit generate`) because this is a JSONB merge
+-- with a filter predicate that `drizzle-kit` cannot emit directly.
+--
+-- Performance note: there is no index on `chats.settings->>'closed'` or
+-- `closeOutcome`, so the predicate is evaluated via a sequential scan.
+-- The expected blast radius is small (only chats that previously hit
+-- `redirected_sac` twice within a 7-day window), but on very large
+-- `chats` tables a single UPDATE would still hold row locks for the
+-- entire matched set in one transaction. Batched into 1000-row chunks so
+-- each iteration acquires a bounded number of row locks; once a chat is
+-- flipped to `closed: false` it stops matching the predicate, so
+-- subsequent iterations naturally narrow.
+
+DO $$
+DECLARE
+  rows_updated INT;
+BEGIN
+  LOOP
+    WITH candidates AS (
+      SELECT "id"
+      FROM "chats"
+      WHERE ("settings"->>'closed')::boolean IS TRUE
+        AND "settings"->>'closeOutcome' = 'redirected_sac'
+      LIMIT 1000
+    )
+    UPDATE "chats" AS c
+    SET "settings" = c."settings" - 'closed' || jsonb_build_object('closed', false)
+    FROM candidates
+    WHERE c."id" = candidates."id";
+
+    GET DIAGNOSTICS rows_updated = ROW_COUNT;
+    EXIT WHEN rows_updated = 0;
+  END LOOP;
+END
+$$;

package/db/drizzle/meta/_journal.json

@@ -246,6 +246,13 @@
       "when": 1777680000000,
       "tag": "0034_instances_require_genie_signature",
       "breakpoints": true
+    },
+    {
+      "idx": 35,
+      "version": "7",
+      "when": 1777770000000,
+      "tag": "0035_unstick_redirected_sac_terminals",
+      "breakpoints": true
     }
   ]
 }

package/dist/index.js

@@ -114079,7 +114079,7 @@ import { fileURLToPath } from "url";
 // package.json
 var package_default = {
   name: "@automagik/omni",
-  version: "2.260430.11",
+  version: "2.260430.13",
   description: "LLM-optimized CLI for Omni",
   type: "module",
   bin: {

package/dist/server/index.js

@@ -224556,7 +224556,7 @@ var init_sentry_scrub = __esm(() => {
 var require_package8 = __commonJS((exports, module) => {
   module.exports = {
     name: "@omni/api",
-    version: "2.260430.11",
+    version: "2.260430.13",
     type: "module",
     exports: {
       ".": {
@@ -280868,6 +280868,21 @@ var init_events2 = __esm(() => {
   init_drizzle_orm();
 });
 
+// ../api/src/lib/close-contact-state.ts
+function isChatInActiveCloseState(settings) {
+  if (!settings)
+    return false;
+  if (settings.closed === true)
+    return true;
+  const closeUntil = settings.closeUntil;
+  if (typeof closeUntil === "string" && closeUntil.length > 0) {
+    const ms = new Date(closeUntil).getTime();
+    if (Number.isFinite(ms) && Date.now() < ms)
+      return true;
+  }
+  return false;
+}
+
 // ../api/src/services/follow-up-lifecycle.ts
 function readAgentFollowUpConfig(row) {
   return row?.followUpConfig;
@@ -280907,6 +280922,8 @@ class FollowUpLifecycleService {
   async armForOutbound(input) {
     if (!this.eventBus)
       return;
+    if (await this.isInActiveCloseState(input.chatId, input.instanceId))
+      return;
     const config4 = input.config ?? await this.resolveConfig(input.chatId, input.instanceId, input.agentId ?? null);
     if (!config4 || config4.enabled === false)
       return;
@@ -280925,22 +280942,8 @@ class FollowUpLifecycleService {
         return;
       }
     }
-    const existing = await this.readExistingRow(input.chatId, input.instanceId);
-    if (existing?.disarmReason && TERMINAL_DISARM_REASONS.has(existing.disarmReason) && existing.disarmedAt) {
-      const lastInbound = existing.lastInboundCustomerMessageAt?.getTime() ?? 0;
-      const disarmedAt = existing.disarmedAt.getTime();
-      if (lastInbound <= disarmedAt) {
-        this.logger.info("follow-up lifecycle: refusing to arm \u2014 terminal disarm awaiting customer return", {
-          chatId: input.chatId,
-          instanceId: input.instanceId,
-          disarmReason: existing.disarmReason,
-          disarmedAt: existing.disarmedAt.toISOString(),
-          lastAgentMessageAt: input.lastAgentMessageAt.toISOString(),
-          lastInboundCustomerMessageAt: existing.lastInboundCustomerMessageAt?.toISOString() ?? null
-        });
-        return;
-      }
-    }
+    if (await this.shouldRefuseForTerminalDisarm(input))
+      return;
     try {
       await armSequence({ repo: this.repo, eventBus: this.eventBus, logger: this.logger }, {
         chatId: input.chatId,
@@ -280982,6 +280985,40 @@ class FollowUpLifecycleService {
       });
     }
   }
+  async shouldRefuseForTerminalDisarm(input) {
+    const existing = await this.readExistingRow(input.chatId, input.instanceId);
+    if (!existing?.disarmReason || !existing.disarmedAt)
+      return false;
+    if (!TERMINAL_DISARM_REASONS.has(existing.disarmReason))
+      return false;
+    const lastInbound = existing.lastInboundCustomerMessageAt?.getTime() ?? 0;
+    if (lastInbound > existing.disarmedAt.getTime())
+      return false;
+    this.logger.info("follow-up lifecycle: refusing to arm \u2014 terminal disarm awaiting customer return", {
+      chatId: input.chatId,
+      instanceId: input.instanceId,
+      disarmReason: existing.disarmReason,
+      disarmedAt: existing.disarmedAt.toISOString(),
+      lastAgentMessageAt: input.lastAgentMessageAt.toISOString(),
+      lastInboundCustomerMessageAt: existing.lastInboundCustomerMessageAt?.toISOString() ?? null
+    });
+    return true;
+  }
+  async isInActiveCloseState(chatId, instanceId) {
+    const [chatRow] = await this.db.select({ settings: chats.settings }).from(chats).where(eq(chats.id, chatId)).limit(1);
+    const settings = chatRow?.settings;
+    if (!isChatInActiveCloseState(settings))
+      return false;
+    const closeOutcome = settings?.closeOutcome;
+    this.logger.info("follow-up lifecycle: refusing to arm \u2014 chat in active close-contact state", {
+      chatId,
+      instanceId,
+      closed: settings?.closed === true,
+      closeUntil: settings?.closeUntil ?? null,
+      closeOutcome: closeOutcome ?? null
+    });
+    return true;
+  }
   async upsertArmed(input) {
     const values2 = {
       chatId: input.chatId,
@@ -284814,13 +284851,13 @@ async function applyCloseContactGate(services, chatRecordId, instanceId, chatId,
   if (!Number.isFinite(closeUntilMs))
     return "pass";
   if (Date.now() < closeUntilMs) {
-    log95.debug("Chat in close cooldown, skipping dispatch", {
+    log95.debug("Chat in soft close cooldown, follow-up disarmed but agent reactive", {
       instanceId,
       chatId,
       closeUntil: chatSettings.closeUntil,
       outcome: chatSettings.closeOutcome ?? null
     });
-    return "skip";
+    return "pass";
   }
   if (!chatRecordId)
     return "pass";
@@ -284833,7 +284870,7 @@ async function applyCloseContactGate(services, chatRecordId, instanceId, chatId,
         agentResumedAt: new Date().toISOString()
       }
     });
-    log95.info("Close cooldown expired, agent reopened", {
+    log95.info("Close cooldown expired, state cleaned", {
       instanceId,
       chatId,
       closeOutcome: chatSettings.closeOutcome ?? null
@@ -300283,7 +300320,7 @@ var init__close_contact_config = __esm(() => {
   DEFAULT_CLOSE_CONTACT_CONFIG = {
     won: { cooldownMs: null, escalationThreshold: null, escalationWindowMs: null },
     lost: { cooldownMs: null, escalationThreshold: null, escalationWindowMs: null },
-    redirected_sac: { cooldownMs: 24 * HOUR, escalationThreshold: 2, escalationWindowMs: 7 * DAY },
+    redirected_sac: { cooldownMs: 24 * HOUR, escalationThreshold: null, escalationWindowMs: null },
     unqualified: { cooldownMs: 7 * DAY, escalationThreshold: 3, escalationWindowMs: 30 * DAY },
     no_response: { cooldownMs: 48 * HOUR, escalationThreshold: 3, escalationWindowMs: 30 * DAY },
     other: { cooldownMs: 24 * HOUR, escalationThreshold: 2, escalationWindowMs: 14 * DAY }
@@ -301417,10 +301454,11 @@ var init_messages5 = __esm(() => {
       }
     }).returning();
     const { terminal, escalated, closeUntil } = await computeCloseContactTerminalState(db2, data.chatId, outcome, auditRow?.id ?? null);
+    const shouldPauseAgent = outcome === "lost";
     const closedAt = new Date;
     await services.chats.update(data.chatId, {
       settings: {
-        agentPaused: true,
+        ...shouldPauseAgent ? { agentPaused: true } : {},
         closed: terminal,
         closeUntil: closeUntil?.toISOString() ?? null,
         closeOutcome: outcome

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@automagik/omni",
-  "version": "2.260430.11",
+  "version": "2.260430.13",
   "description": "LLM-optimized CLI for Omni",
   "type": "module",
   "bin": {
@@ -51,15 +51,15 @@
     "qrcode-terminal": "^0.12.0"
   },
   "devDependencies": {
-    "@omni/api": "2.260430.10",
-    "@omni/channel-discord": "2.260430.10",
-    "@omni/channel-gupshup": "2.260430.10",
-    "@omni/channel-sdk": "2.260430.10",
-    "@omni/channel-slack": "2.260430.10",
-    "@omni/channel-telegram": "2.260430.10",
-    "@omni/channel-whatsapp": "2.260430.10",
-    "@omni/core": "2.260430.10",
-    "@omni/sdk": "2.260430.10",
+    "@omni/api": "2.260430.12",
+    "@omni/channel-discord": "2.260430.12",
+    "@omni/channel-gupshup": "2.260430.12",
+    "@omni/channel-sdk": "2.260430.12",
+    "@omni/channel-slack": "2.260430.12",
+    "@omni/channel-telegram": "2.260430.12",
+    "@omni/channel-whatsapp": "2.260430.12",
+    "@omni/core": "2.260430.12",
+    "@omni/sdk": "2.260430.12",
     "@types/node": "^22.10.3",
     "@types/qrcode-terminal": "^0.12.2",
     "typescript": "^5.7.3"