@gonzih/cc-discord 0.2.42 → 0.2.44

package/dist/meta-agent-manager.js

@@ -223,53 +223,24 @@ function wireStdoutToRedis(proc, ns, wire) {
                     result: resultText,
                     is_error: parsed.is_error ?? false,
                 };
-                if (resultText && !parsed.is_error) {
-                    const msg = {
-                        id: crypto.randomUUID(),
-                        source: "claude",
-                        role: "assistant",
-                        content: resultText,
-                        timestamp: new Date().toISOString(),
-                        chatId: 0,
-                    };
-                    // Publish text first (logs to chat history), then signal done for immediate finalization
-                    wire.discord.publishOutgoing(ns, msg).then(() => {
-                        rawRedis.publish(discordChatOutgoing(ns), JSON.stringify({
-                            id: crypto.randomUUID(), source: "claude", role: "assistant",
-                            content: "", event: "done", timestamp: new Date().toISOString(), chatId: 0,
-                        })).catch(() => { });
-                    }).catch((err) => {
-                        console.warn(`[meta-agent-manager] publishOutgoing (result) failed (ns=${ns}):`, err.message);
-                        // Still signal done even if text publish failed
-                        rawRedis.publish(discordChatOutgoing(ns), JSON.stringify({
-                            id: crypto.randomUUID(), source: "claude", role: "assistant",
-                            content: "", event: "done", timestamp: new Date().toISOString(), chatId: 0,
-                        })).catch(() => { });
-                    });
-                }
-                else if (parsed.is_error) {
-                    // Error result: publish error text and signal done
+                if (parsed.is_error && resultText) {
+                    // Error case: assistant event may not carry the error message — publish it directly
                     const errMsg = {
                         id: crypto.randomUUID(),
                         source: "claude",
                         role: "assistant",
-                        content: `⚠️ Error: ${resultText || "unknown error"}`,
+                        content: `⚠️ Error: ${resultText}`,
                         timestamp: new Date().toISOString(),
                         chatId: 0,
                     };
                     wire.discord.publishOutgoing(ns, errMsg).catch(() => { });
-                    rawRedis.publish(discordChatOutgoing(ns), JSON.stringify({
-                        id: crypto.randomUUID(), source: "claude", role: "assistant",
-                        content: "", event: "done", timestamp: new Date().toISOString(), chatId: 0,
-                    })).catch(() => { });
-                }
-                else {
-                    // Empty result — just signal done
-                    rawRedis.publish(discordChatOutgoing(ns), JSON.stringify({
-                        id: crypto.randomUUID(), source: "claude", role: "assistant",
-                        content: "", event: "done", timestamp: new Date().toISOString(), chatId: 0,
-                    })).catch(() => { });
                 }
+                // Signal turn completion — text was already published via the assistant event above.
+                // Do NOT re-publish resultText here: that would double-deliver the same content.
+                rawRedis.publish(discordChatOutgoing(ns), JSON.stringify({
+                    id: crypto.randomUUID(), source: "claude", role: "assistant",
+                    content: "", event: "done", timestamp: new Date().toISOString(), chatId: 0,
+                })).catch(() => { });
             }
             else {
                 structuredEvent = parsed;
@@ -551,13 +522,19 @@ export function createMetaAgentManager() {
                 return; // already running
             wireRef = wire;
             startWatchdog();
+            // Grace period: don't run stale-instance check until 20s after startup.
+            // The instance key write (in index.ts "ready" handler) is async — if the poll
+            // loop fires before that write completes, this instance sees the OLD key,
+            // incorrectly thinks it's stale, and self-destructs (SIGTERM to all sessions).
+            const pollStartTime = Date.now();
+            const STALE_CHECK_GRACE_MS = 20_000;
             pollInterval = setInterval(() => {
                 const namespaces = getNamespaces();
                 if (namespaces.length === 0)
                     return;
                 for (const { namespace: ns, repoUrl } of namespaces) {
-                    // Stale-instance check
-                    if (instanceId) {
+                    // Stale-instance check — skip during grace period
+                    if (instanceId && Date.now() - pollStartTime > STALE_CHECK_GRACE_MS) {
                         wire._redis.get(DISCORD_INSTANCE_KEY).then((current) => {
                             if (current && current !== instanceId) {
                                 console.log(`[meta-agent-manager] stale instance detected (current=${current}, ours=${instanceId}) — exiting`);

package/package.json

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