omni-notify-mcp 1.3.11 → 1.3.12

package/dist/index.js

@@ -33,7 +33,33 @@ import { fileURLToPath } from "url";
 import { z } from "zod";
 const PORT = process.env.NOTIFY_MCP_PORT ? parseInt(process.env.NOTIFY_MCP_PORT) : 3737;
 const BASE = `http://localhost:${PORT}`;
-const VSC_ID = (process.env.NOTIFY_MCP_TAG || basename(process.cwd())).toLowerCase().replace(/[^a-z0-9_-]/g, "");
+const CLEAN_ID = (s) => s.toLowerCase().replace(/[^a-z0-9_-]/g, "");
+// NOTIFY_MCP_TAG is the explicit per-window name; otherwise use the nearest
+// meaningful working-dir folder, skipping generic launcher/tool/system dirs so a
+// bridge spawned from an editor's launch dir names itself after the project
+// (e.g. "bullseyenotify"), never the host ("claude-code").
+function deriveVscId() {
+    const explicit = CLEAN_ID(process.env.NOTIFY_MCP_TAG ?? "");
+    if (explicit)
+        return explicit;
+    const generic = new Set([
+        "claude-code", "claude", "code", "cursor", "vscode", "windsurf",
+        "bin", "dist", "build", "src", "out", "node_modules",
+        "windows", "system32", "users", "appdata", "roaming", "local", "programs", "temp", "tmp",
+    ]);
+    let dir = process.cwd();
+    for (let i = 0; i < 5; i++) {
+        const name = CLEAN_ID(basename(dir));
+        if (name && !generic.has(name))
+            return name;
+        const parent = dirname(dir);
+        if (!parent || parent === dir)
+            break;
+        dir = parent;
+    }
+    return CLEAN_ID(basename(process.cwd())) || "agent";
+}
+const VSC_ID = deriveVscId();
 const SESSION_TAG = `${hostname().toLowerCase().replace(/[^a-z0-9_-]/g, "")}-${VSC_ID}`;
 const CLIENT_NAME = "claude-channel-bridge";
 // ── 1. Ensure the HTTP server is up ───────────────────────────────────────────

package/dist/ui/server.js

@@ -1659,18 +1659,34 @@ async function slackPost(text) {
     catch { /* webhook post is best-effort */ }
 }
 function slackClientTags() {
+    pruneDeadSessions();
     const tags = new Set();
     for (const sess of listActiveSessions())
         if (sess.tag)
             tags.add(sess.tag);
-    for (const c of inboxStreamClients)
+    for (const c of inboxStreamClients) {
+        if (c.res.destroyed || c.res.writableEnded || !c.res.writable)
+            continue;
         if (c.tag)
             tags.add(c.tag);
+    }
     for (const [, w] of inboxWaiters)
         if (w.tag)
             tags.add(w.tag);
     return [...tags].sort();
 }
+// How many live agents would actually receive a message with this tag (undefined
+// = broadcast). Counts live SSE subscribers, parked long-poll waiters, and MCP
+// sessions. Used to gate the Slack ack: a "ack" posted when nobody is connected
+// is a lie — the message only sits queued for the next connector.
+function liveListenerCount(tag) {
+    pruneDeadSessions();
+    let waiters = 0;
+    for (const [, w] of inboxWaiters)
+        if (!tag || w.tag === tag)
+            waiters++;
+    return sseSubscribersForTag(tag) + waiters + sessionsMatchingTag(tag).length;
+}
 function slackClientsNumbered() {
     const tags = slackClientTags();
     return tags.length ? tags.map((t, i) => `${i + 1}. ${t}`).join("\n") : "(none connected)";
@@ -1724,11 +1740,13 @@ async function pollSlackOnce(token, channel) {
                 continue;
             }
             ingestInboxEntry({ text: msg, ts: new Date().toISOString(), tag, origin: "slack" }, "slack");
-            await slackPost(sessionBusyNote(tag) || "ack");
+            if (liveListenerCount(tag) > 0)
+                await slackPost(sessionBusyNote(tag) || "ack");
         }
         else {
             ingestInboxEntry({ text, ts: new Date().toISOString(), origin: "slack" }, "slack");
-            await slackPost("ack");
+            if (liveListenerCount(undefined) > 0)
+                await slackPost("ack");
         }
     }
     const newest = all.reduce((acc, m) => (Number(m.ts) > Number(acc || 0) ? String(m.ts) : acc), slackCursor);
@@ -2433,8 +2451,11 @@ const httpServer = app.listen(PORT, "0.0.0.0", () => {
     else {
         console.log("  MCP endpoint (remote)    → disabled (set ENABLE_MCP=1 to enable)\n");
     }
-    startTelegramListener();
-    startSlackListener();
+    // Live Slack/Telegram pollers hit real external channels — never under test.
+    if (process.env.NOTIFY_MCP_TEST_ENDPOINTS !== "1") {
+        startTelegramListener();
+        startSlackListener();
+    }
     open(`http://localhost:${PORT}`).catch(() => { });
 });
 // TCP-level keepalive on every incoming socket. Without this, a client that

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omni-notify-mcp",
-  "version": "1.3.11",
+  "version": "1.3.12",
   "description": "An MCP server that lets AI agents (Claude, Cursor, etc.) reach you on any channel — desktop, Telegram, SMS, email — with two-way ask/reply, real-time inbox push, Do Not Disturb, idle gating, multi-session routing, and a one-page web UI for setup. Zero config code; configure once, agents call notify/ask.",
   "main": "dist/index.js",
   "type": "module",