@thehammer/danx-dashboard-mcp 0.1.210 → 0.1.212

package/dist/listen.js

@@ -122,6 +122,10 @@ export function invalidEventReason(value) {
     if (typeof value.origin !== "string" || !ACTIVITY_ORIGINS.has(value.origin)) {
         return "origin is not one of operator, agent, machine";
     }
+    // DX-4246 — the key snapshot is part of every event the dashboard sends, `null` when no key wrote it.
+    if (value.viaKey !== null && !(isRecord(value.viaKey) && typeof value.viaKey.id === "number" && typeof value.viaKey.name === "string")) {
+        return "viaKey is not null or {id, name}";
+    }
     const d = value.detail;
     if (!isRecord(d))
         return "detail is not an object";
@@ -187,7 +191,9 @@ export function invalidEventReason(value) {
  * reader would have to cross-reference.
  */
 function actorLabel(event) {
-    return `${event.actor} (${event.origin})`;
+    // DX-4246 — a write made through an API key names the user AND the key ("dan via claude-laptop (agent)").
+    const via = event.viaKey === null ? "" : ` via ${event.viaKey.name}`;
+    return `${event.actor}${via} (${event.origin})`;
 }
 /** Only ever called on an event `invalidEventReason` accepted, so its casts are checked. */
 function describe(event) {
@@ -323,6 +329,32 @@ export function formatNudgeLine(event) {
         "Work these cards; any you cannot work, update to its true state: open a problem, add depends_on or " +
         "conflict_on, block it with the real reason, or rollback_pickup.");
 }
+/** Why a reminder cannot be read, or `null` when it can. */
+export function invalidReminderReason(value) {
+    if (!isRecord(value))
+        return "the reminder is not an object";
+    if (value.kind !== "policy_reminder")
+        return "kind is not policy_reminder";
+    if (typeof value.text !== "string" || value.text.trim() === "")
+        return "text is not a non-empty string";
+    return null;
+}
+/**
+ * The label every reminder carries, so the session can never read one as the operator speaking
+ * (DX-2955: "labelled unmistakably as automated system reminders, never phrased as the operator").
+ */
+export const REMINDER_LABEL = "AUTOMATED SYSTEM REMINDER from the danxbot dashboard monitor (not from the operator):";
+/**
+ * The text of a reminder: the label, then the reminder as the dashboard wrote it. Unlike an
+ * activity line the body keeps its line breaks, because a reminder with something off is the
+ * numbered checklist plus what is off. Throws, naming the fault, on a malformed reminder.
+ */
+export function formatReminderLine(event) {
+    const invalid = invalidReminderReason(event);
+    if (invalid !== null)
+        throw new Error(`malformed reminder event: ${invalid}`);
+    return `${LINE_PREFIX} ${REMINDER_LABEL}\n${event.text.trim()}`;
+}
 /**
  * Incremental SSE parser. Comment lines (keep-alives, the connect marker) and
  * field-less blocks produce no message, which is what keeps them out of the output.
@@ -463,6 +495,25 @@ export async function runListener(options, deps) {
             deps.write({ type: "event", id: null, text, origin: null });
             return null;
         }
+        if (message.event === "reminder") {
+            // DX-2955 - like a nudge: no usable id, never replayed, never deduplicated, and
+            // `origin: null` so `bridge.ts` delivers it immediately instead of folding it
+            // into a digest (the generic null-origin exemption in `routeEvent`).
+            let parsed;
+            let invalid;
+            try {
+                parsed = JSON.parse(message.data);
+                invalid = invalidReminderReason(parsed);
+            }
+            catch (err) {
+                invalid = `not JSON: ${err.message}`;
+            }
+            const text = invalid === null
+                ? formatReminderLine(parsed)
+                : `${LINE_PREFIX} could not read reminder (${invalid}): ${oneLine(message.data, 300)}`;
+            deps.write({ type: "event", id: null, text, origin: null });
+            return null;
+        }
         if (message.event !== "activity")
             return null;
         // `Number(null)` is 0, so a frame with no id must never be read as event 0: that id

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-dashboard-mcp",
-  "version": "0.1.210",
+  "version": "0.1.212",
   "description": "Stdio MCP server fronting danxbot's dashboard API for agents (operator sessions and dispatched workers alike): the generic danxbot_api/danxbot_api_list/danxbot_api_spec tools reach any route the caller's credential may call, plus dedicated agent-specific tools (plan_connect, danxbot_complete, and other per-dispatch lifecycle callbacks) (DX-3465, PLN-11 R-25).",
   "license": "MIT",
   "type": "module",