@integrity-labs/agt-cli 0.28.612 → 0.28.614

package/dist/mcp/direct-chat-channel.js

@@ -33932,7 +33932,14 @@ var AGENT_DIRECTED_NOTICE_KINDS = [
   // written for the machine. The user learns the same fact from the connect card
   // in that conversation flipping to its connected state, so showing this row as
   // well would say it twice, the second time in prose aimed at an agent.
-  "integration_connected"
+  "integration_connected",
+  // ENG-8745. The offer RESULT receipt (not the offer card — that one is
+  // written for the human and is how they decided). Its content opens "Your
+  // offer was ACCEPTED", carries an `[offer_result]` machine tag, and closes by
+  // instructing the agent what not to do ("do not re-raise it yourself"). The
+  // human needs none of it: they are the person who tapped the button, and the
+  // card swapped under their finger to say so.
+  "offer_result"
 ];
 var AGENT_DIRECTED_KIND_SET = new Set(AGENT_DIRECTED_NOTICE_KINDS);
 
@@ -34082,6 +34089,17 @@ var CONDITIONAL_DELIVERY_BODY = [
 var FRESH_HEARTBEAT_THRESHOLD_MS = 2 * 60 * 1e3;
 
 // ../core/dist/claude-code-usage/banner-parser.js
+var SESSION_QUALIFIER = /^(?:session|\d{1,2}\s*-?\s*hours?|\d{1,2}h)$/i;
+function classifyLimitScope(qualifier) {
+  const q = (qualifier ?? "").trim();
+  if (!q)
+    return "weekly";
+  if (SESSION_QUALIFIER.test(q))
+    return "session";
+  if (/^weekly$/i.test(q))
+    return "weekly";
+  return "unknown";
+}
 var SEP = /[\s·\-–—]+/.source;
 var SUBJECT = /(?:You(?:['’]ve|\s+have)?\s+)?/.source;
 var TIME_OF_DAY = /\d{1,2}(?::\d{2})?\s*(?:am|pm)(?:\s*\(?UTC\)?)?/.source;
@@ -34095,7 +34113,27 @@ var BANNER_PATTERNS = [
   // Wrapped to keep the per-pattern shape consistent with the percentage form:
   // the parser checks group 1 for a digit string and treats a missing one as
   // pct=100.
-  new RegExp(`${SUBJECT}hit\\s+your\\s+(?:[a-z0-9-]+\\s+)?limit${SEP}resets\\s+(${RESET_DATE})`, "i")
+  // ENG-9007: the qualifier is CAPTURED now, by name. It used to be
+  // `(?:[a-z0-9-]+\s+)?` - permissive and non-capturing - so the parser
+  // recognised "session" and "5-hour", matched on them, and threw the word away.
+  // Named groups keep the numbered capture indices stable for the percentage
+  // form above, which still reads groups 1 and 2 positionally.
+  new RegExp(
+    // CodeRabbit on #4621: the qualifier was ONE token, so `classifyLimitScope`
+    // accepted "5 hours" while this pattern could never deliver it — the whole
+    // banner failed to match, no observation was recorded, and the agent went
+    // dark rather than merely being misclassified. Multi-token qualifiers now
+    // match the duration shapes SESSION_QUALIFIER already accepts.
+    //
+    // Deliberately NOT the naive `[a-z0-9]+(?:\s*-?\s*[a-z0-9]+)*`: the
+    // separator there can match empty, which reduces to `(a+)+` and backtracks
+    // catastrophically on a long alphanumeric run that is never followed by
+    // "limit". This parser runs over agent-produced pane.log tails, so that is
+    // reachable input. Requiring at least one separator per extra token, and
+    // bounding the count, keeps it linear.
+    `${SUBJECT}hit\\s+your\\s+(?<qualifier>[a-z0-9]+(?:[\\s-]+[a-z0-9]+){0,3}\\s+)?limit${SEP}resets\\s+(?<reset>${RESET_DATE})`,
+    "i"
+  )
 ];
 function parseUsageBanner(text, now = /* @__PURE__ */ new Date()) {
   let bestIndex = -1;
@@ -34108,24 +34146,33 @@ function parseUsageBanner(text, now = /* @__PURE__ */ new Date()) {
         pattern.lastIndex++;
       let pct;
       let resetStr;
+      let qualifier;
       if (i === 0) {
         pct = Number.parseInt(match[1], 10);
         resetStr = match[2];
+        qualifier = "weekly";
       } else {
         pct = 100;
-        resetStr = match[1];
+        resetStr = match.groups?.["reset"] ?? "";
+        qualifier = match.groups?.["qualifier"]?.trim() || null;
       }
       if (!Number.isFinite(pct) || pct < 0 || pct > 100)
         continue;
       const nextChar = text[match.index + match[0].length];
       if (nextChar !== void 0 && /[A-Za-z0-9]/.test(nextChar))
         continue;
-      const weekResetsAt = parseResetDateTime(resetStr, now);
-      if (!weekResetsAt)
+      const reset = parseResetDateTime(resetStr, now);
+      if (!reset)
         continue;
       if (match.index >= bestIndex) {
         bestIndex = match.index;
-        best = { pct, weekResetsAt };
+        best = {
+          pct,
+          weekResetsAt: reset.at,
+          resetPrecision: reset.precision,
+          limitScope: classifyLimitScope(qualifier),
+          limitQualifier: qualifier
+        };
       }
     }
   }
@@ -34169,7 +34216,10 @@ function parseResetDateTime(humanDate, now) {
     if (!hm)
       return null;
     const todayAt = Date.UTC(now.getUTCFullYear(), now.getUTCMonth(), now.getUTCDate(), hm.hour, hm.minute);
-    return new Date(todayAt <= now.getTime() ? todayAt + MS_PER_DAY : todayAt);
+    return {
+      at: new Date(todayAt <= now.getTime() ? todayAt + MS_PER_DAY : todayAt),
+      precision: "hour_only"
+    };
   }
   const timeMatch = trimmed.match(TIME_TAIL);
   const dateOnly = timeMatch ? trimmed.slice(0, timeMatch.index).trim() : trimmed;
@@ -34202,7 +34252,7 @@ function parseResetDateTime(humanDate, now) {
       resolved = candidate;
     }
   }
-  return resolved;
+  return resolved ? { at: resolved, precision: "dated" } : null;
 }
 
 // ../core/dist/claude-code-usage/run-marker.js

package/dist/mcp/origami.js

@@ -40266,7 +40266,14 @@ var AGENT_DIRECTED_NOTICE_KINDS = [
   // written for the machine. The user learns the same fact from the connect card
   // in that conversation flipping to its connected state, so showing this row as
   // well would say it twice, the second time in prose aimed at an agent.
-  "integration_connected"
+  "integration_connected",
+  // ENG-8745. The offer RESULT receipt (not the offer card — that one is
+  // written for the human and is how they decided). Its content opens "Your
+  // offer was ACCEPTED", carries an `[offer_result]` machine tag, and closes by
+  // instructing the agent what not to do ("do not re-raise it yourself"). The
+  // human needs none of it: they are the person who tapped the button, and the
+  // card swapped under their finger to say so.
+  "offer_result"
 ];
 var AGENT_DIRECTED_KIND_SET = new Set(AGENT_DIRECTED_NOTICE_KINDS);
 
@@ -40373,7 +40380,27 @@ var BANNER_PATTERNS = [
   // Wrapped to keep the per-pattern shape consistent with the percentage form:
   // the parser checks group 1 for a digit string and treats a missing one as
   // pct=100.
-  new RegExp(`${SUBJECT}hit\\s+your\\s+(?:[a-z0-9-]+\\s+)?limit${SEP}resets\\s+(${RESET_DATE})`, "i")
+  // ENG-9007: the qualifier is CAPTURED now, by name. It used to be
+  // `(?:[a-z0-9-]+\s+)?` - permissive and non-capturing - so the parser
+  // recognised "session" and "5-hour", matched on them, and threw the word away.
+  // Named groups keep the numbered capture indices stable for the percentage
+  // form above, which still reads groups 1 and 2 positionally.
+  new RegExp(
+    // CodeRabbit on #4621: the qualifier was ONE token, so `classifyLimitScope`
+    // accepted "5 hours" while this pattern could never deliver it — the whole
+    // banner failed to match, no observation was recorded, and the agent went
+    // dark rather than merely being misclassified. Multi-token qualifiers now
+    // match the duration shapes SESSION_QUALIFIER already accepts.
+    //
+    // Deliberately NOT the naive `[a-z0-9]+(?:\s*-?\s*[a-z0-9]+)*`: the
+    // separator there can match empty, which reduces to `(a+)+` and backtracks
+    // catastrophically on a long alphanumeric run that is never followed by
+    // "limit". This parser runs over agent-produced pane.log tails, so that is
+    // reachable input. Requiring at least one separator per extra token, and
+    // bounding the count, keeps it linear.
+    `${SUBJECT}hit\\s+your\\s+(?<qualifier>[a-z0-9]+(?:[\\s-]+[a-z0-9]+){0,3}\\s+)?limit${SEP}resets\\s+(?<reset>${RESET_DATE})`,
+    "i"
+  )
 ];
 var MS_PER_DAY = 24 * 60 * 60 * 1e3;
 

package/dist/mcp/slack-channel.js

@@ -34317,7 +34317,14 @@ var AGENT_DIRECTED_NOTICE_KINDS = [
   // written for the machine. The user learns the same fact from the connect card
   // in that conversation flipping to its connected state, so showing this row as
   // well would say it twice, the second time in prose aimed at an agent.
-  "integration_connected"
+  "integration_connected",
+  // ENG-8745. The offer RESULT receipt (not the offer card — that one is
+  // written for the human and is how they decided). Its content opens "Your
+  // offer was ACCEPTED", carries an `[offer_result]` machine tag, and closes by
+  // instructing the agent what not to do ("do not re-raise it yourself"). The
+  // human needs none of it: they are the person who tapped the button, and the
+  // card swapped under their finger to say so.
+  "offer_result"
 ];
 var AGENT_DIRECTED_KIND_SET = new Set(AGENT_DIRECTED_NOTICE_KINDS);
 
@@ -34411,6 +34418,17 @@ var CONDITIONAL_DELIVERY_BODY = [
 var FRESH_HEARTBEAT_THRESHOLD_MS = 2 * 60 * 1e3;
 
 // ../core/dist/claude-code-usage/banner-parser.js
+var SESSION_QUALIFIER = /^(?:session|\d{1,2}\s*-?\s*hours?|\d{1,2}h)$/i;
+function classifyLimitScope(qualifier) {
+  const q = (qualifier ?? "").trim();
+  if (!q)
+    return "weekly";
+  if (SESSION_QUALIFIER.test(q))
+    return "session";
+  if (/^weekly$/i.test(q))
+    return "weekly";
+  return "unknown";
+}
 var SEP = /[\s·\-–—]+/.source;
 var SUBJECT = /(?:You(?:['’]ve|\s+have)?\s+)?/.source;
 var TIME_OF_DAY = /\d{1,2}(?::\d{2})?\s*(?:am|pm)(?:\s*\(?UTC\)?)?/.source;
@@ -34424,7 +34442,27 @@ var BANNER_PATTERNS = [
   // Wrapped to keep the per-pattern shape consistent with the percentage form:
   // the parser checks group 1 for a digit string and treats a missing one as
   // pct=100.
-  new RegExp(`${SUBJECT}hit\\s+your\\s+(?:[a-z0-9-]+\\s+)?limit${SEP}resets\\s+(${RESET_DATE})`, "i")
+  // ENG-9007: the qualifier is CAPTURED now, by name. It used to be
+  // `(?:[a-z0-9-]+\s+)?` - permissive and non-capturing - so the parser
+  // recognised "session" and "5-hour", matched on them, and threw the word away.
+  // Named groups keep the numbered capture indices stable for the percentage
+  // form above, which still reads groups 1 and 2 positionally.
+  new RegExp(
+    // CodeRabbit on #4621: the qualifier was ONE token, so `classifyLimitScope`
+    // accepted "5 hours" while this pattern could never deliver it — the whole
+    // banner failed to match, no observation was recorded, and the agent went
+    // dark rather than merely being misclassified. Multi-token qualifiers now
+    // match the duration shapes SESSION_QUALIFIER already accepts.
+    //
+    // Deliberately NOT the naive `[a-z0-9]+(?:\s*-?\s*[a-z0-9]+)*`: the
+    // separator there can match empty, which reduces to `(a+)+` and backtracks
+    // catastrophically on a long alphanumeric run that is never followed by
+    // "limit". This parser runs over agent-produced pane.log tails, so that is
+    // reachable input. Requiring at least one separator per extra token, and
+    // bounding the count, keeps it linear.
+    `${SUBJECT}hit\\s+your\\s+(?<qualifier>[a-z0-9]+(?:[\\s-]+[a-z0-9]+){0,3}\\s+)?limit${SEP}resets\\s+(?<reset>${RESET_DATE})`,
+    "i"
+  )
 ];
 function parseUsageBanner(text, now = /* @__PURE__ */ new Date()) {
   let bestIndex = -1;
@@ -34437,24 +34475,33 @@ function parseUsageBanner(text, now = /* @__PURE__ */ new Date()) {
         pattern.lastIndex++;
       let pct;
       let resetStr;
+      let qualifier;
       if (i === 0) {
         pct = Number.parseInt(match[1], 10);
         resetStr = match[2];
+        qualifier = "weekly";
       } else {
         pct = 100;
-        resetStr = match[1];
+        resetStr = match.groups?.["reset"] ?? "";
+        qualifier = match.groups?.["qualifier"]?.trim() || null;
       }
       if (!Number.isFinite(pct) || pct < 0 || pct > 100)
         continue;
       const nextChar = text[match.index + match[0].length];
       if (nextChar !== void 0 && /[A-Za-z0-9]/.test(nextChar))
         continue;
-      const weekResetsAt = parseResetDateTime(resetStr, now);
-      if (!weekResetsAt)
+      const reset = parseResetDateTime(resetStr, now);
+      if (!reset)
         continue;
       if (match.index >= bestIndex) {
         bestIndex = match.index;
-        best = { pct, weekResetsAt };
+        best = {
+          pct,
+          weekResetsAt: reset.at,
+          resetPrecision: reset.precision,
+          limitScope: classifyLimitScope(qualifier),
+          limitQualifier: qualifier
+        };
       }
     }
   }
@@ -34498,7 +34545,10 @@ function parseResetDateTime(humanDate, now) {
     if (!hm)
       return null;
     const todayAt = Date.UTC(now.getUTCFullYear(), now.getUTCMonth(), now.getUTCDate(), hm.hour, hm.minute);
-    return new Date(todayAt <= now.getTime() ? todayAt + MS_PER_DAY : todayAt);
+    return {
+      at: new Date(todayAt <= now.getTime() ? todayAt + MS_PER_DAY : todayAt),
+      precision: "hour_only"
+    };
   }
   const timeMatch = trimmed.match(TIME_TAIL);
   const dateOnly = timeMatch ? trimmed.slice(0, timeMatch.index).trim() : trimmed;
@@ -34531,7 +34581,7 @@ function parseResetDateTime(humanDate, now) {
       resolved = candidate;
     }
   }
-  return resolved;
+  return resolved ? { at: resolved, precision: "dated" } : null;
 }
 
 // ../core/dist/claude-code-usage/run-marker.js
@@ -39394,6 +39444,91 @@ async function runOrRetry(fn, opts) {
   }
 }
 
+// src/slack-socket-reconnect.ts
+var RECONNECT_BASE_DELAY_MS = 1e3;
+var RECONNECT_MAX_DELAY_MS = 3e4;
+var SOCKET_OPEN_TIMEOUT_MS = 3e4;
+var CONNECT_FETCH_TIMEOUT_MS = 15e3;
+var SOCKET_DOWN_ALERT_AFTER_MS = 3 * 6e4;
+function reconnectDelayMs(attempt, opts = {}) {
+  const base = opts.baseMs ?? RECONNECT_BASE_DELAY_MS;
+  const max = opts.maxMs ?? RECONNECT_MAX_DELAY_MS;
+  const random = opts.random ?? Math.random;
+  const safeAttempt = Math.max(1, Math.min(Math.floor(attempt), 30));
+  const ceiling = Math.min(max, base * 2 ** (safeAttempt - 1));
+  const floor = Math.min(base, ceiling);
+  return Math.round(floor + random() * (ceiling - floor));
+}
+function redactSocketUrl(text) {
+  return text.replace(
+    /([?&])(ticket|token|app_id_key)=[^&\s"']*/gi,
+    (_m, sep2, key2) => `${sep2}${key2}=<redacted>`
+  );
+}
+function describeWebSocketError(err) {
+  const parts = [];
+  const e = err ?? {};
+  if (typeof e.message === "string" && e.message) parts.push(e.message);
+  const cause = e.error;
+  if (cause && typeof cause === "object") {
+    if (typeof cause.message === "string" && cause.message && !parts.includes(cause.message)) {
+      parts.push(cause.message);
+    }
+    if (typeof cause.code === "string" && cause.code) parts.push(`errno=${cause.code}`);
+  }
+  if (typeof e.code === "number") parts.push(`code=${e.code}`);
+  if (typeof e.reason === "string" && e.reason) parts.push(`reason=${e.reason}`);
+  if (typeof e.wasClean === "boolean") parts.push(`wasClean=${e.wasClean}`);
+  if (parts.length === 0) {
+    const type = typeof e.type === "string" && e.type ? e.type : typeof err;
+    const ctor = err && typeof err === "object" ? err.constructor?.name ?? "Object" : typeof err;
+    return redactSocketUrl(`no detail available (${ctor}, type=${type})`);
+  }
+  return redactSocketUrl(parts.join(" "));
+}
+function isWebSocketOpen(ws) {
+  return ws?.readyState === 1;
+}
+function createReconnectScheduler(deps) {
+  const now = deps.now ?? Date.now;
+  const setTimer = deps.setTimer ?? ((fn, ms) => setTimeout(fn, ms));
+  let pending = false;
+  let attempt = 0;
+  let downSince = now();
+  return {
+    schedule(reason) {
+      if (deps.shouldStop() || pending) return;
+      pending = true;
+      if (downSince === null) downSince = now();
+      attempt += 1;
+      const opts = {};
+      if (deps.baseMs !== void 0) opts.baseMs = deps.baseMs;
+      if (deps.maxMs !== void 0) opts.maxMs = deps.maxMs;
+      if (deps.random !== void 0) opts.random = deps.random;
+      const delay = reconnectDelayMs(attempt, opts);
+      deps.log(
+        `slack-channel: Socket Mode reconnecting in ${delay}ms (attempt ${attempt}, ${reason})`
+      );
+      const timer = setTimer(() => {
+        pending = false;
+        if (deps.shouldStop()) return;
+        deps.connect();
+      }, delay);
+      timer.unref?.();
+    },
+    noteOpen() {
+      attempt = 0;
+      downSince = null;
+    },
+    downForMs(connected) {
+      if (connected) return null;
+      return downSince === null ? null : now() - downSince;
+    },
+    attempts: () => attempt,
+    isPending: () => pending
+  };
+}
+
 // src/turn-initiator-marker.ts
 import { writeFileSync as writeFileSync7, readFileSync as readFileSync9, mkdirSync as mkdirSync5, renameSync as renameSync2 } from "fs";
 import { dirname as dirname4, join as join12 } from "path";
@@ -43163,10 +43298,12 @@ function formatLastActivity() {
   return `${kindLabel} ${ago}`;
 }
 function buildAgentStatusReply(codeName) {
-  const connected = currentWs != null && !isShuttingDown;
+  const connected = isSocketConnected();
   const dot = connected ? "\u{1F7E2}" : "\u{1F534}";
   const state = connected ? "online" : "offline";
-  const connectivityLine = `${dot} *${state}* \u2014 Socket Mode ${connected ? "connected" : "disconnected"}. Last activity: ${formatLastActivity()}.`;
+  const downMs = socketDownForMs();
+  const downSuffix = !connected && downMs !== null ? ` for ${Math.floor(downMs / 6e4)}m \u2014 inbound is dead` : "";
+  const connectivityLine = `${dot} *${state}* \u2014 Socket Mode ${connected ? "connected" : `disconnected${downSuffix}`}. Last activity: ${formatLastActivity()}.`;
   const sessionState = readAgentSessionState(SLACK_AGENT_DIR);
   return buildAgentConfigReport({ codeName, connectivityLine, state: sessionState });
 }
@@ -43177,9 +43314,16 @@ function statusRuntimeConfig() {
 }
 async function computeSlackOwnChannelLiveStatus() {
   const checked_at = (/* @__PURE__ */ new Date()).toISOString();
-  const connected = currentWs != null && !isShuttingDown;
+  const connected = isSocketConnected();
   if (!connected) {
-    return { level: "down", checked_at, reason: "Socket Mode disconnected" };
+    const downMs = socketDownForMs();
+    const forSuffix = downMs === null ? "" : ` for ${Math.floor(downMs / 1e3)}s`;
+    const sustained = downMs !== null && downMs >= SOCKET_DOWN_ALERT_AFTER_MS;
+    return {
+      level: "down",
+      checked_at,
+      reason: `Socket Mode disconnected${forSuffix}${sustained ? " - inbound Slack is DEAD (outbound still works, so this is invisible from outside)" : ""}`
+    };
   }
   const { id, authFailed } = await getBotUserId();
   if (id) {
@@ -43706,7 +43850,7 @@ async function handleSlashCommandEnvelope(payload) {
       );
       return;
     }
-    const connected = currentWs != null && !isShuttingDown;
+    const connected = isSocketConnected();
     const dot = connected ? "\u{1F7E2}" : "\u{1F534}";
     const state = connected ? "online" : "offline";
     const pong = `${dot} pong - \`${codeName}\` is ${state} (manager \u2022 ${(/* @__PURE__ */ new Date()).toISOString()}). Last activity: ${formatLastActivity()}.`;
@@ -45778,6 +45922,47 @@ async function resolveUserEmail(userId) {
 }
 var currentWs = null;
 var isShuttingDown = false;
+var clearOpenWatchdog = null;
+function clearOpenWatchdogFor(ws) {
+  if (currentWs !== ws) return;
+  clearOpenWatchdog?.();
+  clearOpenWatchdog = null;
+}
+var reconnectScheduler = createReconnectScheduler({
+  connect: () => connectSocketModeSafely(),
+  log: (line) => process.stderr.write(`${line}
+`),
+  shouldStop: () => isShuttingDown
+});
+function isSocketConnected() {
+  return !isShuttingDown && isWebSocketOpen(currentWs);
+}
+function scheduleReconnect(reason) {
+  reconnectScheduler.schedule(reason);
+}
+function armOpenWatchdog(ws) {
+  clearOpenWatchdog?.();
+  const timer = setTimeout(() => {
+    if (currentWs !== ws) return;
+    if (isShuttingDown || isWebSocketOpen(ws)) return;
+    process.stderr.write(
+      `slack-channel: Socket Mode did not open within ${SOCKET_OPEN_TIMEOUT_MS}ms - abandoning it
+`
+    );
+    currentWs = null;
+    clearOpenWatchdog = null;
+    try {
+      ws.close();
+    } catch {
+    }
+    scheduleReconnect("open timeout");
+  }, SOCKET_OPEN_TIMEOUT_MS);
+  timer.unref?.();
+  clearOpenWatchdog = () => clearTimeout(timer);
+}
+function socketDownForMs() {
+  return reconnectScheduler.downForMs(isSocketConnected());
+}
 var acquiredLockPath = null;
 var stopLockHeartbeat = null;
 async function seedMutedChannelsFromApi() {
@@ -45828,14 +46013,27 @@ async function connectSocketMode() {
   if (isShuttingDown) return;
   const res = await fetch("https://slack.com/api/apps.connections.open", {
     method: "POST",
-    headers: { Authorization: `Bearer ${APP_TOKEN}` }
+    headers: { Authorization: `Bearer ${APP_TOKEN}` },
+    // CodeRabbit, #4655. Without a deadline a hanging fetch is the one failure
+    // this whole change does not cover: it never rejects, so the retry chain is
+    // never entered, and no socket exists yet so the open watchdog does not
+    // apply either. The process ends up with nothing pending and no attempt in
+    // flight — the incident's shape, reached a different way. A timeout turns
+    // the hang into a rejection, which the retry chain already handles.
+    signal: AbortSignal.timeout(CONNECT_FETCH_TIMEOUT_MS)
+    // CodeRabbit, #4655. Without a deadline a hanging fetch is the one failure
+    // this whole change does not cover: it never rejects, so the retry chain is
+    // never entered, and no socket exists yet so the open watchdog does not
+    // apply either. The process ends up with nothing pending and no attempt in
+    // flight — the incident's shape, reached a different way. A timeout turns
+    // the hang into a rejection, which the retry chain already handles.
   });
   const data = await res.json();
   if (isShuttingDown) return;
   if (!data.ok || !data.url) {
     process.stderr.write(`slack-channel: Socket Mode connection failed: ${data.error}
 `);
-    if (!isShuttingDown) setTimeout(connectSocketModeSafely, 1e4).unref?.();
+    scheduleReconnect(`apps.connections.open ${data.error ?? "failed"}`);
     return;
   }
   const ws = new WebSocket(data.url);
@@ -45847,7 +46045,17 @@ async function connectSocketMode() {
     return;
   }
   currentWs = ws;
+  armOpenWatchdog(ws);
   ws.onopen = () => {
+    if (currentWs !== ws) {
+      try {
+        ws.close();
+      } catch {
+      }
+      return;
+    }
+    clearOpenWatchdogFor(ws);
+    reconnectScheduler.noteOpen();
     process.stderr.write("slack-channel: Socket Mode connected\n");
     recordActivity("connect");
     void setBotStatus(":large_green_circle:", "Active");
@@ -46493,15 +46701,35 @@ async function connectSocketMode() {
 `);
     }
   };
-  ws.onclose = () => {
-    if (currentWs === ws) currentWs = null;
+  ws.onclose = (evt) => {
+    if (currentWs !== ws) return;
+    clearOpenWatchdogFor(ws);
+    currentWs = null;
     if (isShuttingDown) return;
-    process.stderr.write("slack-channel: Socket Mode disconnected, reconnecting...\n");
-    setTimeout(connectSocketModeSafely, 5e3).unref?.();
+    process.stderr.write(
+      `slack-channel: Socket Mode disconnected (${describeWebSocketError(evt)})
+`
+    );
+    scheduleReconnect("close");
   };
   ws.onerror = (err) => {
-    process.stderr.write(`slack-channel: WebSocket error: ${err}
+    process.stderr.write(`slack-channel: WebSocket error: ${describeWebSocketError(err)}
 `);
+    if (currentWs !== ws) {
+      try {
+        ws.close();
+      } catch {
+      }
+      return;
+    }
+    clearOpenWatchdogFor(ws);
+    currentWs = null;
+    try {
+      ws.close();
+    } catch {
+    }
+    if (isShuttingDown) return;
+    scheduleReconnect("error");
   };
 }
 function shutdown(reason, exitCode = 0) {

package/dist/mcp/telegram-channel.js

@@ -34476,7 +34476,14 @@ var AGENT_DIRECTED_NOTICE_KINDS = [
   // written for the machine. The user learns the same fact from the connect card
   // in that conversation flipping to its connected state, so showing this row as
   // well would say it twice, the second time in prose aimed at an agent.
-  "integration_connected"
+  "integration_connected",
+  // ENG-8745. The offer RESULT receipt (not the offer card — that one is
+  // written for the human and is how they decided). Its content opens "Your
+  // offer was ACCEPTED", carries an `[offer_result]` machine tag, and closes by
+  // instructing the agent what not to do ("do not re-raise it yourself"). The
+  // human needs none of it: they are the person who tapped the button, and the
+  // card swapped under their finger to say so.
+  "offer_result"
 ];
 var AGENT_DIRECTED_KIND_SET = new Set(AGENT_DIRECTED_NOTICE_KINDS);
 
@@ -34570,6 +34577,17 @@ var CONDITIONAL_DELIVERY_BODY = [
 var FRESH_HEARTBEAT_THRESHOLD_MS = 2 * 60 * 1e3;
 
 // ../core/dist/claude-code-usage/banner-parser.js
+var SESSION_QUALIFIER = /^(?:session|\d{1,2}\s*-?\s*hours?|\d{1,2}h)$/i;
+function classifyLimitScope(qualifier) {
+  const q = (qualifier ?? "").trim();
+  if (!q)
+    return "weekly";
+  if (SESSION_QUALIFIER.test(q))
+    return "session";
+  if (/^weekly$/i.test(q))
+    return "weekly";
+  return "unknown";
+}
 var SEP = /[\s·\-–—]+/.source;
 var SUBJECT = /(?:You(?:['’]ve|\s+have)?\s+)?/.source;
 var TIME_OF_DAY = /\d{1,2}(?::\d{2})?\s*(?:am|pm)(?:\s*\(?UTC\)?)?/.source;
@@ -34583,7 +34601,27 @@ var BANNER_PATTERNS = [
   // Wrapped to keep the per-pattern shape consistent with the percentage form:
   // the parser checks group 1 for a digit string and treats a missing one as
   // pct=100.
-  new RegExp(`${SUBJECT}hit\\s+your\\s+(?:[a-z0-9-]+\\s+)?limit${SEP}resets\\s+(${RESET_DATE})`, "i")
+  // ENG-9007: the qualifier is CAPTURED now, by name. It used to be
+  // `(?:[a-z0-9-]+\s+)?` - permissive and non-capturing - so the parser
+  // recognised "session" and "5-hour", matched on them, and threw the word away.
+  // Named groups keep the numbered capture indices stable for the percentage
+  // form above, which still reads groups 1 and 2 positionally.
+  new RegExp(
+    // CodeRabbit on #4621: the qualifier was ONE token, so `classifyLimitScope`
+    // accepted "5 hours" while this pattern could never deliver it — the whole
+    // banner failed to match, no observation was recorded, and the agent went
+    // dark rather than merely being misclassified. Multi-token qualifiers now
+    // match the duration shapes SESSION_QUALIFIER already accepts.
+    //
+    // Deliberately NOT the naive `[a-z0-9]+(?:\s*-?\s*[a-z0-9]+)*`: the
+    // separator there can match empty, which reduces to `(a+)+` and backtracks
+    // catastrophically on a long alphanumeric run that is never followed by
+    // "limit". This parser runs over agent-produced pane.log tails, so that is
+    // reachable input. Requiring at least one separator per extra token, and
+    // bounding the count, keeps it linear.
+    `${SUBJECT}hit\\s+your\\s+(?<qualifier>[a-z0-9]+(?:[\\s-]+[a-z0-9]+){0,3}\\s+)?limit${SEP}resets\\s+(?<reset>${RESET_DATE})`,
+    "i"
+  )
 ];
 function parseUsageBanner(text, now = /* @__PURE__ */ new Date()) {
   let bestIndex = -1;
@@ -34596,24 +34634,33 @@ function parseUsageBanner(text, now = /* @__PURE__ */ new Date()) {
         pattern.lastIndex++;
       let pct;
       let resetStr;
+      let qualifier;
       if (i === 0) {
         pct = Number.parseInt(match[1], 10);
         resetStr = match[2];
+        qualifier = "weekly";
       } else {
         pct = 100;
-        resetStr = match[1];
+        resetStr = match.groups?.["reset"] ?? "";
+        qualifier = match.groups?.["qualifier"]?.trim() || null;
       }
       if (!Number.isFinite(pct) || pct < 0 || pct > 100)
         continue;
       const nextChar = text[match.index + match[0].length];
       if (nextChar !== void 0 && /[A-Za-z0-9]/.test(nextChar))
         continue;
-      const weekResetsAt = parseResetDateTime(resetStr, now);
-      if (!weekResetsAt)
+      const reset = parseResetDateTime(resetStr, now);
+      if (!reset)
         continue;
       if (match.index >= bestIndex) {
         bestIndex = match.index;
-        best = { pct, weekResetsAt };
+        best = {
+          pct,
+          weekResetsAt: reset.at,
+          resetPrecision: reset.precision,
+          limitScope: classifyLimitScope(qualifier),
+          limitQualifier: qualifier
+        };
       }
     }
   }
@@ -34657,7 +34704,10 @@ function parseResetDateTime(humanDate, now) {
     if (!hm)
       return null;
     const todayAt = Date.UTC(now.getUTCFullYear(), now.getUTCMonth(), now.getUTCDate(), hm.hour, hm.minute);
-    return new Date(todayAt <= now.getTime() ? todayAt + MS_PER_DAY : todayAt);
+    return {
+      at: new Date(todayAt <= now.getTime() ? todayAt + MS_PER_DAY : todayAt),
+      precision: "hour_only"
+    };
   }
   const timeMatch = trimmed.match(TIME_TAIL);
   const dateOnly = timeMatch ? trimmed.slice(0, timeMatch.index).trim() : trimmed;
@@ -34690,7 +34740,7 @@ function parseResetDateTime(humanDate, now) {
       resolved = candidate;
     }
   }
-  return resolved;
+  return resolved ? { at: resolved, precision: "dated" } : null;
 }
 
 // ../core/dist/claude-code-usage/run-marker.js