@gonzih/cc-discord 0.2.23 → 0.2.24

package/dist/notifier.js

@@ -101,6 +101,9 @@ function shortenModelName(model, driver) {
  * Appends " cost: $X.XXX" if a numeric cost field is present.
  */
 export function parseNotification(raw) {
+    // Filter cron-fire noise before any parsing — catches plain-text and JSON-wrapped ⏰ cron notifications
+    if (raw.startsWith("⏰") || raw.includes('"⏰'))
+        return null;
     let text = raw;
     let driver;
     let model;
@@ -127,10 +130,8 @@ export function parseNotification(raw) {
             isCron = parsed.is_cron;
     }
     catch {
-        // non-JSON: fall through to text-based check below
+        // non-JSON: fall through
     }
-    if (text.startsWith("⏰"))
-        return null;
     // Parse eval_report if present — this field is non-standard and not in NotificationPayload type
     const evalReport = parseEvalReport(raw);
     if (!driver)
@@ -356,23 +357,24 @@ export function startNotifier(bot, notifyChannelId, namespace, redis, handleUser
             const notification = parseNotification(raw);
             if (notification === null)
                 continue; // routing excludes discord
-            // Dedup: skip if this notification was already forwarded recently
-            // Uses Redis when available; falls back to in-memory dedup on Redis errors.
+            // Dedup: skip if this notification was already forwarded recently.
+            // Check in-memory first (catches pub/sub-delivered notifications in the same process).
+            if (checkAndMarkSentSync(ns, raw)) {
+                log("info", `dedup: skipping already-sent notification (ns=${ns})`);
+                continue;
+            }
+            // Also check Redis for cross-restart/cross-process dedup.
             let isDup = false;
             try {
                 if (typeof redis.sadd === "function") {
                     isDup = await checkAndMarkSent(redis, ns, raw);
                 }
-                else {
-                    isDup = checkAndMarkSentSync(ns, raw);
-                }
             }
             catch (err) {
-                log("warn", `dedup check failed (ns=${ns}), falling back to in-memory:`, err.message);
-                isDup = checkAndMarkSentSync(ns, raw);
+                log("warn", `dedup Redis check failed (ns=${ns}):`, err.message);
             }
             if (isDup) {
-                log("info", `dedup: skipping already-sent notification (ns=${ns})`);
+                log("info", `dedup: skipping already-sent notification (ns=${ns}) [redis]`);
                 continue;
             }
             // Primary namespace: honour chatId-based per-channel routing via reverseSnowflakeLookup,
@@ -436,6 +438,8 @@ export function startNotifier(bot, notifyChannelId, namespace, redis, handleUser
                 log("info", `dedup: skipping already-sent pub/sub notification (ns=${ns})`);
                 return;
             }
+            // Also mark in Redis so list-poller dedup sees it (prevents cross-path duplicates)
+            checkAndMarkSent(redis, ns, message).catch(() => { });
             let mainChannelId;
             if (isPrimary) {
                 mainChannelId = resolveNotifyChannel(notification.chatId, notifyChannelId, getActiveChannelId, reverseSnowflakeLookup, ns, getChannelIdForNamespace);
@@ -483,10 +487,13 @@ export function startNotifier(bot, notifyChannelId, namespace, redis, handleUser
                 ? (notifyChannelId ?? getActiveChannelId?.())
                 : routedChannelIds.get(ns);
             if (targetChannelId !== undefined) {
-                // Echo to Discord so the user sees UI messages
-                bot.sendToChannelById(targetChannelId, `[from UI]: ${content}`).catch((err) => {
-                    log("warn", `sendToChannelById (UI echo) failed (ns=${ns}):`, err.message);
-                });
+                // Echo to Discord so the user sees UI messages (suppress cron-fire noise)
+                const isCronMessage = content.startsWith("⏰") || content.includes("[cron]");
+                if (!isCronMessage) {
+                    bot.sendToChannelById(targetChannelId, `[from UI]: ${content}`).catch((err) => {
+                        log("warn", `sendToChannelById (UI echo) failed (ns=${ns}):`, err.message);
+                    });
+                }
                 // Log the incoming message
                 const inMsg = {
                     id: crypto.randomUUID(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gonzih/cc-discord",
-  "version": "0.2.23",
+  "version": "0.2.24",
   "description": "Claude Code Discord bot — chat with Claude Code via Discord",
   "type": "module",
   "bin": {