omni-notify-mcp 1.1.5 → 1.1.7

package/README.md

@@ -86,7 +86,7 @@ Priority routing for `notify`:
 ## Features
 
 ### Channels
-- **Desktop** — native `node-notifier` (macOS/Windows/Linux). Per-channel **system-sound toggle** and optional **text-to-speech** (natural neural voice via `msedge-tts`, no API key).
+- **Desktop** — native `node-notifier` (macOS/Windows/Linux). Per-channel **system-sound toggle** and optional **text-to-speech** with a voice picker covering 30+ neural voices (US/UK/AU/CA/IN/IE/NZ/…) via `msedge-tts`, no API key.
 - **Telegram** — bidirectional. The bot **replies in-thread** to user messages and acknowledges every inbound message so the user knows it landed.
 - **SMS** — Twilio.
 - **Email** — Gmail App Password (one click) or any SMTP. `ask` over email sends a reply link the user clicks to answer.

package/dist/ui/server.js

@@ -190,10 +190,12 @@ async function speakText(text, voice) {
         spawn("aplay", [audioFilePath], { stdio: "ignore" });
     }
 }
-app.post("/api/test/tts", async (_req, res) => {
+app.post("/api/test/tts", async (req, res) => {
     try {
         const cfg = loadConfig();
-        const voice = cfg.desktop?.ttsVoice ?? "en-US-AndrewMultilingualNeural";
+        const voice = (typeof req.body?.voice === "string" && req.body.voice) ||
+            cfg.desktop?.ttsVoice ||
+            "en-US-AndrewMultilingualNeural";
         await speakText("Notification from Claude. This is a voice test.", voice);
         res.json({ ok: true, message: `TTS played (${voice})` });
     }
@@ -201,6 +203,26 @@ app.post("/api/test/tts", async (_req, res) => {
         res.status(500).json({ error: err instanceof Error ? err.message : String(err) });
     }
 });
+let voiceCache = null;
+app.get("/api/voices", async (_req, res) => {
+    try {
+        if (!voiceCache || Date.now() - voiceCache.ts > 24 * 60 * 60 * 1000) {
+            const mod = await import("msedge-tts");
+            const tts = new mod.MsEdgeTTS();
+            const all = await tts.getVoices();
+            voiceCache = {
+                ts: Date.now(),
+                voices: all
+                    .filter((v) => v.Locale.startsWith("en-") && v.ShortName.includes("Neural"))
+                    .map((v) => ({ shortName: v.ShortName, gender: v.Gender, locale: v.Locale })),
+            };
+        }
+        res.json({ voices: voiceCache.voices });
+    }
+    catch (err) {
+        res.status(500).json({ error: err instanceof Error ? err.message : String(err) });
+    }
+});
 app.post("/api/test/desktop", (_req, res) => {
     const time = new Date().toLocaleTimeString();
     const cfg = loadConfig();
@@ -948,6 +970,51 @@ BEHAVIORAL RULES for every client that connects:
 6. The 'get_idle_*' and 'get_dnd_status' tools are informational. You can
    inspect them if you want to explain a delivery decision, but they are NOT
    required pre-flights — the server gates server-side.
+
+7. If your tool call fails with "MCP server not connected" / "transport
+   closed" / similar — the SERVER IS ALMOST CERTAINLY FINE. Other clients are
+   connected to the same server right now. Only YOUR client's transport
+   broke (commonly: stdio child process died, or HTTP client gave up after a
+   transient slow response at startup). DO NOT tell the user "notify is down"
+   without first probing the server directly.
+
+   Probe before declaring the server dead:
+     curl -s -o /dev/null -w "%{http_code}\n" http://localhost:3737/mcp
+   - 406 (Not Acceptable) = server is healthy, doing MCP protocol
+     negotiation. Your client is the broken party. Use the bypass below.
+   - Connection refused / timeout = server is genuinely down. Tell the user.
+
+   HTTP-transport bypass when your MCP client is broken (works from any
+   shell that has curl). Three calls, stateful — the session id from step 1
+   must be passed in mcp-session-id on every subsequent call:
+
+     # 1) initialize and capture session id from response headers
+     SID=$(curl -s -i -X POST http://localhost:3737/mcp \
+       -H "Content-Type: application/json" \
+       -H "Accept: application/json, text/event-stream" \
+       -d '{"jsonrpc":"2.0","id":1,"method":"initialize","params":{"protocolVersion":"2024-11-05","capabilities":{},"clientInfo":{"name":"curl-bridge","version":"1.0"}}}' \
+       | grep -i "^mcp-session-id:" | awk '{print $2}' | tr -d '\r')
+
+     # 2) ack initialized (no response body, fire-and-forget)
+     curl -s -X POST http://localhost:3737/mcp \
+       -H "Content-Type: application/json" \
+       -H "Accept: application/json, text/event-stream" \
+       -H "mcp-session-id: $SID" \
+       -d '{"jsonrpc":"2.0","method":"notifications/initialized"}' >/dev/null
+
+     # 3) call any tool — successful response includes "Sent via: <channel>"
+     curl -s -X POST http://localhost:3737/mcp \
+       -H "Content-Type: application/json" \
+       -H "Accept: application/json, text/event-stream" \
+       -H "mcp-session-id: $SID" \
+       -d '{"jsonrpc":"2.0","id":2,"method":"tools/call","params":{"name":"notify","arguments":{"message":"YOUR MESSAGE","priority":"normal"}}}'
+
+   Same pattern works for 'ask', 'poll', 'get_idle_seconds', etc. — change
+   the "name" and "arguments" in step 3. Reuse the same $SID for all calls
+   in one bypass session.
+
+   Bottom line: rule #1 still applies. If you owe the user a notify and your
+   MCP client is broken, you owe them the bypass — not an excuse.
 `.trim();
 function createMcpServer(clientId, sessionTag) {
     const server = new McpServer({ name: "notify-mcp", version: "1.0.0" }, { instructions: MCP_INSTRUCTIONS });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omni-notify-mcp",
-  "version": "1.1.5",
+  "version": "1.1.7",
   "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",

package/ui/public/app.js

@@ -45,6 +45,8 @@ function populateForm() {
   $("desktop-enabled").checked = !!config.desktop?.enabled;
   $("desktop-sound").checked = config.desktop?.sound !== false; // default on
   $("desktop-tts").checked = !!config.desktop?.tts; // default off
+  updateTtsVoiceRow();
+  loadVoices().catch(() => {});
 
   // Email / Gmail
   const email = config.email ?? {};
@@ -115,14 +117,16 @@ function updateBadges() {
     email.connectedEmail ? "Connected" : email.clientId ? "Credentials saved" : "Not configured");
 
   const tg = config.telegram ?? {};
+  const tgReady = tg.token && tg.chatId;
   setBadge("telegram",
-    tg.enabled && tg.token && tg.chatId ? "ok" : tg.token ? "warn" : "idle",
-    tg.enabled && tg.token && tg.chatId ? "Configured" : tg.token ? "Incomplete" : "Not configured");
+    tg.enabled && tgReady ? "ok" : tgReady ? "warn" : tg.token ? "warn" : "idle",
+    tg.enabled && tgReady ? "Configured" : tgReady ? "Disabled" : tg.token ? "Incomplete" : "Not configured");
 
-const sms = config.sms ?? {};
+  const sms = config.sms ?? {};
+  const smsReady = sms.accountSid && sms.authToken;
   setBadge("sms",
-    sms.enabled && sms.accountSid && sms.authToken ? "ok" : sms.accountSid ? "warn" : "idle",
-    sms.enabled && sms.accountSid && sms.authToken ? "Configured" : sms.accountSid ? "Incomplete" : "Not configured");
+    sms.enabled && smsReady ? "ok" : smsReady ? "warn" : sms.accountSid ? "warn" : "idle",
+    sms.enabled && smsReady ? "Configured" : smsReady ? "Disabled" : sms.accountSid ? "Incomplete" : "Not configured");
 
   // DND badge: "Active" (red), "Scheduled" (warn), or "Off" (idle)
   const dnd = config.dnd ?? {};
@@ -151,15 +155,50 @@ function setBadge(channel, type, text) {
 // ── Save handlers ─────────────────────────────────────────────────────────
 
 function saveDesktop() {
+  updateTtsVoiceRow();
+  const ttsVoice = $("desktop-tts-voice").value || undefined;
   patch({
     desktop: {
       enabled: $("desktop-enabled").checked,
       sound: $("desktop-sound").checked,
       tts: $("desktop-tts").checked,
+      ttsVoice,
     },
   });
 }
 
+function updateTtsVoiceRow() {
+  const row = $("tts-voice-row");
+  row.style.display = $("desktop-tts").checked ? "" : "none";
+}
+
+let voicesLoaded = false;
+async function loadVoices() {
+  if (voicesLoaded) return;
+  const res = await fetch("/api/voices");
+  if (!res.ok) return;
+  const { voices } = await res.json();
+  const sel = $("desktop-tts-voice");
+  const current = config.desktop?.ttsVoice || "en-US-AndrewMultilingualNeural";
+  const byLocale = {};
+  for (const v of voices) (byLocale[v.locale] ??= []).push(v);
+  sel.innerHTML = "";
+  for (const locale of Object.keys(byLocale).sort()) {
+    const og = document.createElement("optgroup");
+    og.label = locale;
+    for (const v of byLocale[locale].sort((a, b) => a.shortName.localeCompare(b.shortName))) {
+      const opt = document.createElement("option");
+      opt.value = v.shortName;
+      const name = v.shortName.replace(locale + "-", "").replace(/Neural$/, "").replace(/Multilingual$/, " (Multi)");
+      opt.textContent = `${name} · ${v.gender}`;
+      if (v.shortName === current) opt.selected = true;
+      og.appendChild(opt);
+    }
+    sel.appendChild(og);
+  }
+  voicesLoaded = true;
+}
+
 async function saveEmail() {
   const to = $("gmail-to-connected").value.trim();
   const enabled = $("email-enabled").checked;
@@ -402,7 +441,12 @@ async function testSound() {
 
 async function testTts() {
   try {
-    const res = await fetch("/api/test/tts", { method: "POST" });
+    const voice = $("desktop-tts-voice").value || undefined;
+    const res = await fetch("/api/test/tts", {
+      method: "POST",
+      headers: { "Content-Type": "application/json" },
+      body: JSON.stringify({ voice }),
+    });
     const json = await res.json();
     if (!res.ok) throw new Error(json.error);
     toast(json.message, "ok");

package/ui/public/index.html

@@ -63,6 +63,10 @@
             <span class="toggle-lbl">Speak notification</span>
             <button class="btn btn-sm btn-ghost" onclick="testTts()">Test voice</button>
           </div>
+          <div id="tts-voice-row" class="fg" style="margin-top:8px; display:none">
+            <label>Voice</label>
+            <select id="desktop-tts-voice" onchange="saveDesktop()"></select>
+          </div>
           <span id="os-hint" class="os-tag hidden"></span>
         </div>
       </div>

package/ui/public/style.css

@@ -102,11 +102,11 @@ main {
   flex: 1;
   min-height: 0;
   height: 100%;
-  overflow: hidden;
-  display: flex;
-  flex-direction: column;
-  flex-wrap: wrap;
-  align-content: flex-start;
+  overflow: auto;
+  display: grid;
+  grid-template-columns: repeat(auto-fill, minmax(260px, 1fr));
+  grid-auto-rows: min-content;
+  align-content: start;
   gap: 12px;
 }
 
@@ -207,8 +207,9 @@ main {
 /* ── Cards ───────────────────────────────────────────────────────────────── */
 
 .card {
-  width: 280px;
+  width: auto;
   max-width: 100%;
+  min-width: 0;
   background: var(--surface);
   border: 2px solid #4a4a5a;
   border-radius: var(--r);
@@ -362,6 +363,27 @@ input[type="time"] {
   box-shadow: inset 0 1px 0 rgba(255,255,255,0.08);
 }
 input::placeholder { color: #cdcdda; opacity: 1; }
+
+select {
+  width: 100%;
+  max-width: 100%;
+  box-sizing: border-box;
+  padding: 8px 11px;
+  border: 2px solid #7a7a95;
+  border-radius: 7px;
+  font-size: 14px;
+  font-weight: 500;
+  color: #ffffff;
+  background: #55556a;
+  outline: none;
+  transition: border-color .15s, box-shadow .15s;
+  color-scheme: dark;
+  box-shadow: inset 0 1px 0 rgba(255,255,255,0.08);
+  cursor: pointer;
+  text-overflow: ellipsis;
+}
+select:hover { border-color: #9a9ab5; }
+select:focus { border-color: var(--accent); box-shadow: 0 0 0 2px rgba(124,109,250,.15); }
 input[type="text"]:hover,
 input[type="email"]:hover,
 input[type="tel"]:hover,