@delt/claude-alarm 1.3.0 → 1.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@delt/claude-alarm",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "Monitor and get notifications from multiple Claude Code sessions via MCP Channels",
   "type": "module",
   "main": "./dist/index.js",
@@ -28,7 +28,7 @@
     "dev": "tsup --watch",
     "start:hub": "node dist/cli.js hub start",
     "start:channel": "node dist/channel/server.js",
-    "test": "node --import tsx --import ./test/isolate-home.ts --test --test-timeout=30000 \"test/**/*.test.ts\""
+    "test": "node --import tsx --import ./test/isolate-home.ts --test --test-timeout=60000 \"test/**/*.test.ts\""
   },
   "keywords": [
     "claude",

package/src/dashboard/index.html

@@ -1286,13 +1286,20 @@
   }
 
   function showUploadRejected(msg) {
-    state.waitingReply[msg.sessionId] = false;
-    const label = msg.withText ? 'Image and message not delivered' : 'Image not delivered';
-    if (state.selectedSession === msg.sessionId) {
+    showNotDelivered(msg.sessionId, msg.withText ? 'Image and message not delivered' : 'Image not delivered', msg.reason);
+  }
+
+  function showMessageRejected(msg) {
+    showNotDelivered(msg.sessionId, 'Message not delivered', msg.reason);
+  }
+
+  function showNotDelivered(sessionId, label, reason) {
+    state.waitingReply[sessionId] = false;
+    if (state.selectedSession === sessionId) {
       renderMessages();
-      showMentionError(`${label}: ${msg.reason}`);
+      showMentionError(`${label}: ${reason}`);
     }
-    state.notifications.unshift({ sessionId: msg.sessionId, title: label, message: msg.reason, level: 'warning', time: Date.now() });
+    state.notifications.unshift({ sessionId, title: label, message: reason, level: 'warning', time: Date.now() });
     renderNotifications();
   }
 
@@ -1462,32 +1469,15 @@
       case 'upload_rejected':
         showUploadRejected(msg);
         break;
+      case 'message_rejected':
+        showMessageRejected(msg);
+        break;
       case 'notification':
         state.notifications.unshift({ sessionId: msg.sessionId, title: msg.title, message: msg.message, level: msg.level || 'info', time: msg.timestamp });
         renderNotifications();
         break;
       case 'permission_request':
-        if (!state.permissionRequests[msg.sessionId]) state.permissionRequests[msg.sessionId] = [];
-        state.permissionRequests[msg.sessionId].unshift({
-          requestId: msg.requestId,
-          toolName: msg.toolName,
-          description: msg.description,
-          inputPreview: msg.inputPreview,
-          timestamp: msg.timestamp,
-          resolved: false,
-          behavior: null,
-          choices: msg.choices || null,
-          sent: null,
-          outcome: null,
-        });
-        // Auto-select session if none selected
-        if (!state.selectedSession && state.sessions[msg.sessionId]) selectSession(msg.sessionId);
-        if (state.selectedSession === msg.sessionId) { renderMessages(); renderPermissionBar(); }
-        // Also show in notifications with permission info
-        state.notifications.unshift({ sessionId: msg.sessionId, title: 'Permission Request', message: `${msg.toolName}: ${msg.description}`, level: 'warning', time: msg.timestamp, permRequestId: msg.requestId });
-        renderNotifications();
-        // Play notification sound effect via title flash
-        flashTitle('Permission Request');
+        showPermissionRequest(msg);
         break;
       case 'permission_response':
         // Another source (Telegram, other dashboard) resolved a permission request
@@ -1501,11 +1491,7 @@
         }
         break;
       case 'permission_pending':
-        for (const m of applyPendingChoices(state.permissionRequests, msg.requests || [])) {
-          handleMessage({ type: 'permission_request', ...m });
-        }
-        renderPermissionBar();
-        renderNotifications();
+        restorePendingRequests(msg.requests || []);
         break;
       case 'codex_adapters': {
         state.codexAdapters = msg.adapters || [];
@@ -2128,6 +2114,46 @@
     return missing;
   }
 
+  function addPermissionRequest(msg) {
+    if (!state.permissionRequests[msg.sessionId]) state.permissionRequests[msg.sessionId] = [];
+    state.permissionRequests[msg.sessionId].unshift({
+      requestId: msg.requestId,
+      toolName: msg.toolName,
+      description: msg.description,
+      inputPreview: msg.inputPreview,
+      timestamp: msg.timestamp,
+      resolved: false,
+      behavior: null,
+      choices: msg.choices || null,
+      sent: null,
+      outcome: null,
+    });
+    if (!state.selectedSession && state.sessions[msg.sessionId]) selectSession(msg.sessionId);
+    if (state.selectedSession === msg.sessionId) { renderMessages(); renderPermissionBar(); }
+  }
+
+  function showPermissionRequest(msg) {
+    addPermissionRequest(msg);
+    state.notifications.unshift({ sessionId: msg.sessionId, title: 'Permission Request', message: `${msg.toolName}: ${msg.description}`, level: 'warning', time: msg.timestamp, permRequestId: msg.requestId });
+    renderNotifications();
+    flashTitle('Permission Request');
+  }
+
+  // A reconnect can bring back many requests at once; one row and one flash keep them from burying the notifications.
+  function restorePendingRequests(requests) {
+    const restored = applyPendingChoices(state.permissionRequests, requests);
+    for (const m of restored) addPermissionRequest(m);
+    if (restored.length) {
+      const label = `${restored.length} approval request(s) waiting`;
+      const listed = restored.slice(0, 3).map(m => `${m.toolName}: ${m.description}`).join(' · ');
+      const message = restored.length > 3 ? `${listed} …and ${restored.length - 3} more` : listed;
+      state.notifications.unshift({ sessionId: restored[0].sessionId, title: label, message, level: 'warning', time: Date.now() });
+      flashTitle(label);
+    }
+    renderPermissionBar();
+    renderNotifications();
+  }
+
   function sendPermissionChoice(sessionId, requestId, choiceId) {
     if (!state.ws || state.ws.readyState !== WebSocket.OPEN) return;
     state.ws.send(JSON.stringify({ type: 'permission_response', sessionId, requestId, choiceId }));
@@ -2211,23 +2237,28 @@
   }, true);
 
   // Flash title for attention
-  let flashInterval = null;
+  let flash = null;
   function flashTitle(msg) {
-    const original = document.title;
-    let on = true;
-    clearInterval(flashInterval);
-    flashInterval = setInterval(() => {
-      document.title = on ? `** ${msg} **` : original;
-      on = !on;
-    }, 500);
-    const stopFlash = () => {
-      clearInterval(flashInterval);
-      flashInterval = null;
-      document.title = original;
-      window.removeEventListener('focus', stopFlash);
+    // While flashing, document.title may hold the flash text, so only a flash that starts from rest reads the real title.
+    if (flash) {
+      flash.msg = msg;
+      return;
+    }
+    const current = { msg, original: document.title, on: true, interval: null, timeout: null };
+    const stop = () => {
+      clearInterval(current.interval);
+      clearTimeout(current.timeout);
+      window.removeEventListener('focus', stop);
+      document.title = current.original;
+      flash = null;
     };
-    window.addEventListener('focus', stopFlash);
-    setTimeout(stopFlash, 30000);
+    current.interval = setInterval(() => {
+      document.title = current.on ? `** ${current.msg} **` : current.original;
+      current.on = !current.on;
+    }, 500);
+    current.timeout = setTimeout(stop, 30000);
+    window.addEventListener('focus', stop);
+    flash = current;
   }
 
   // Clear all notifications