@cohortapp/agent-sdk 2.18.5 → 2.18.6

package/lib/org/inbound/directedness.mjs

@@ -939,4 +939,94 @@ export default {
   matchesMyName,
   taskIdFromFileKey,
   surfaceDef,
+  isMissedHumanMessage,
+  missedHumanRecord,
 };
+
+// ─────────────────────────────────────────────────────────────────────────────
+// A PERSON'S MESSAGE THAT THIS SEAT DROPPED
+// ─────────────────────────────────────────────────────────────────────────────
+
+/**
+ * Drop reasons that are NOT a person going unanswered, even when a human wrote
+ * the message. Each one says the room was never this seat's to read or the
+ * item was never a message at all, so counting it would bury the reasons that
+ * matter under noise every tick produces.
+ *
+ * `own_message` is here for the obvious reason; the membership/visibility ones
+ * because a room the seat cannot see is not a room it declined to answer in.
+ */
+const NOT_A_MISSED_PERSON = Object.freeze([
+  "own_message",
+  "channel_not_visible",
+  "not_a_member",
+  "membership_unknown",
+  "channel_kind_unknown",
+  "surface_disabled",
+]);
+
+/**
+ * PURE. Did this seat just drop a message a PERSON wrote in a room the seat
+ * belongs to?
+ *
+ * ── WHY THIS EXISTS ──────────────────────────────────────────────────────────
+ * `pullWideInbound` counts every drop into `stats.dropped[reason]` — a tally,
+ * by reason, with no identity and, unless something else happened that tick, no
+ * log line at all. That tally is why the 2026-09-21 #general roll-call took a
+ * reconstruction to explain: thirteen seats each incremented
+ * `dropped.no_mentions_not_threaded` by one and none of them said WHICH message
+ * or that a human had written it. A counter cannot be audited after the fact
+ * and cannot be correlated across seats.
+ *
+ * A person's message that this seat decided not to answer is a different class
+ * of event from ambient chatter it correctly ignored, and it is the only class
+ * anyone ever asks about afterwards. So it is named individually, once, at
+ * WARN — the same treatment `hydrate` already gives a body it could not read.
+ *
+ * Deliberately narrow, for the same reason the surface list is: this fires only
+ * for a MESSAGE-topic candidate, authored by a member the DIRECTORY says is
+ * HUMAN (never assumed from the absence of evidence), in a room whose
+ * membership this seat has PROVEN. Everything else is either not a person, not
+ * a room, or not knowable — and an over-broad warn line is a line operators
+ * learn to skip.
+ *
+ * @param {Candidate} cand
+ * @param {{directed:boolean, reason?:string}} verdict the resolved verdict
+ * @param {import("./facts.mjs").Facts} facts
+ * @param {string} me this seat's member id
+ * @returns {boolean}
+ */
+export function isMissedHumanMessage(cand, verdict, facts, me) {
+  if (!cand || !verdict || verdict.directed) return false;
+  if (cand.topic !== "message") return false;
+  const reason = String(verdict.reason || "");
+  if (NOT_A_MISSED_PERSON.includes(reason)) return false;
+  const channelId = cand.ids && cand.ids.channelId;
+  if (!channelId) return false;
+  // A room whose membership is PROVEN, not merely visible: a public channel the
+  // seat can read but has not joined does not address it.
+  const members = asSet(facts && facts.memberChannelIds);
+  if (!has(members, channelId)) return false;
+  const author = cand.actor;
+  if (!author || (me && String(author) === String(me))) return false;
+  // Proven human. `authorKindOf` returns "" when the directory read degraded,
+  // and an unknown author is NOT reported as a missed person — a degraded tick
+  // would otherwise warn about every message in every room.
+  return authorKindOf(facts, author) === "HUMAN";
+}
+
+/**
+ * PURE. The redaction-safe record of one missed person's message. No body, no
+ * prose: the ids an operator needs to go and look, plus the reason this seat
+ * gave. Mirrors the payload shape hq's `responder.silence` rows carry, so the
+ * two planes describe the same event in the same words.
+ */
+export function missedHumanRecord(cand, verdict) {
+  return {
+    seq: cand && cand.seq != null ? cand.seq : null,
+    messageId: (cand && cand.ids && cand.ids.messageId) || cand?.entityId || null,
+    channelId: (cand && cand.ids && cand.ids.channelId) || null,
+    author: (cand && cand.actor) || null,
+    reason: String((verdict && verdict.reason) || "unknown"),
+  };
+}

package/lib/org/inbound/index.mjs

@@ -55,7 +55,12 @@
 "use strict";
 
 import { read as clientRead } from "../client.mjs";
-import { classifyEvent, resolveDirected } from "./directedness.mjs";
+import {
+  classifyEvent,
+  resolveDirected,
+  isMissedHumanMessage,
+  missedHumanRecord,
+} from "./directedness.mjs";
 import { resolveFacts, DEFAULT_LIMITS } from "./facts.mjs";
 import { hydrate } from "./hydrate.mjs";
 import { toMessageEvent } from "./project.mjs";
@@ -106,6 +111,11 @@ export async function pullWideInbound(o = {}) {
     delivered: 0,
     bySurface: {},
     dropped: {},
+    // EVERY person's message this tick decided not to answer, named. See
+    // `directedness.mjs#isMissedHumanMessage`: `dropped` is a tally by reason
+    // and cannot say WHICH message or that a human wrote it, which is the only
+    // question anyone asks after a room goes quiet.
+    unanswered: [],
     degraded: [],
     calls: 0,
     reads: 0,
@@ -195,6 +205,18 @@ export async function pullWideInbound(o = {}) {
     const verdict = resolveDirected(cand, me, facts, { enabled, meAliases });
     if (!verdict.directed) {
       stats.dropped[verdict.reason] = (stats.dropped[verdict.reason] || 0) + 1;
+      // A PERSON's message this seat is dropping is not ambient chatter, and a
+      // counter is not a record of it. Name it — once, at WARN, with the ids —
+      // so "I posted and nobody replied" is answerable from this machine's own
+      // log instead of by reasoning backwards from thirteen tallies.
+      if (isMissedHumanMessage(cand, verdict, facts, me)) {
+        const rec = missedHumanRecord(cand, verdict);
+        stats.unanswered.push(rec);
+        log(
+          "warn",
+          `[inbound] NOT answering a person: message ${rec.messageId || cand.seq} in channel ${rec.channelId} from ${rec.author} — ${rec.reason}`
+        );
+      }
       continue;
     }
     directed.push({ cand, verdict });

package/lib/org/ui-parity.mjs

@@ -3317,7 +3317,19 @@ export function messagingSearch(params, o = {}) {
  * message ({ messageId }) or a channel over a window ({ channelId,
  * windowDays? }); one of the two is required. Each row carries the operator
  * reason (human_mentioned | addressed_to_human | nobody_elected |
- * no_responders) and its text. The agent-plane twin of the "Unanswered
+ * no_responders | handed_to_daemons) and its text; a `handed_to_daemons` row
+ * also carries `daemonDriven`, the seats hq stood down for — every eligible
+ * colleague in the room was answering from its OWN machine, so hq deliberately
+ * said nothing and a room that then heard nothing is a fault on the DAEMON
+ * plane, with the seat list already in hand.
+ *
+ * The same sentence is NOT repeated in `protocol.mjs`: that file is pinned by
+ * `protocol.checksum` to hq's vendored copy, so even a comment there is a
+ * cross-repo re-vendor (`scripts/sync-protocol.mjs`). The vocabulary's source
+ * of truth is hq `src/server/llm-responder/silence-verdict.ts`; this JSDoc is
+ * the agent-facing restatement of it.
+ *
+ * The agent-plane twin of the "Unanswered
  * messages" panel on /settings/ai — same reader, same decoder.
  * @param {object} params - { channelId?, messageId?, windowDays?, limit? }
  * @param {object} o - { base, token, fetchImpl? }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cohortapp/agent-sdk",
-  "version": "2.18.5",
+  "version": "2.18.6",
   "description": "Cohort Agent SDK — autonomous AI colleague runtime. Deploy senior AI colleagues on dedicated Mac minis, wired to the Cohort operating surface.",
   "type": "module",
   "bin": {