polygram 0.38.1 → 0.38.3

package/lib/handlers/config-callback.js

@@ -605,19 +605,19 @@ function createHandleConfigCallback({
         for (const { scope, value } of cleared) scope[configField] = value;
       };
       // The user tapped one card; topics they never touched can change value
-      // with it. Record which ones and what they held, so a config that moved
-      // on its own is still explicable afterwards. Logged only once the sweep
-      // is persisted — an event for a rolled-back sweep is worse than none.
+      // with it. Record WHICH ones, by id, so a config that moved on its own
+      // is still explicable afterwards — the values they held are
+      // configuration content and the ids answer the question without them.
+      // Logged only once the sweep is persisted — an event for a rolled-back
+      // sweep is worse than none.
       const logTopicRichTextSweep = (cleared) => {
         if (!cleared.length) return;
         dbWrite(() => {
           db.logEvent('config-topic-override-swept', {
             chat_id: chatId,
             field: configField,
-            topics: cleared.map(({ threadId, value }) => ({
-              thread_id: threadId,
-              old_value: value,
-            })),
+            topic_ids: cleared.map(({ threadId }) => String(threadId)),
+            topics_cleared: cleared.length,
           });
         }, 'log topic-override sweep');
       };

package/lib/handlers/dispatcher.js

@@ -77,6 +77,7 @@ function createDispatcher({
   // Per-session in-flight handler count.
   const inFlightHandlers = new Map();
   const activeHandlers = new Set();
+  const activeHandlerTargets = new Set();
 
   function trackTask(task) {
     activeHandlers.add(task);
@@ -265,6 +266,15 @@ function createDispatcher({
   }
 
   function dispatchHandleMessage(sessionKey, chatId, msg, bot) {
+    const activeHandlerTarget = Object.freeze({
+      sessionKey: String(sessionKey),
+      chatId: String(chatId),
+      threadId: msg?.message_thread_id == null
+        ? null
+        : String(msg.message_thread_id),
+      telegramMessageId: String(msg?.message_id),
+    });
+    activeHandlerTargets.add(activeHandlerTarget);
     const count = (inFlightHandlers.get(sessionKey) || 0) + 1;
     inFlightHandlers.set(sessionKey, count);
     const warnAt = queueWarnThreshold();
@@ -305,13 +315,14 @@ function createDispatcher({
       logEvent('handler-error', {
         chat_id: chatId, session_key: sessionKey,
         msg_id: msg?.message_id,
-        error: err.message?.slice(0, 500),
+        error_class: err.name || 'Error',
+        error_len: err.message?.length ?? 0,
         code: err.code || undefined,
         cause_code: err.cause?.code || undefined,
-        stderr_tail: err.code === 'TMUX_SPAWN_FAILED' && typeof err.stderr === 'string'
-          ? err.stderr.replace(/[\x00-\x08\x0b-\x1f\x7f]/g, '').slice(-500)
-          : undefined,
-        stack: err.stack?.split('\n').slice(0, 5).join('\n'),
+        // A spawn failure's stderr is process output, not a field: its size
+        // says whether the child said anything at all, which is the signal
+        // that distinguishes "tmux refused" from "tmux said nothing".
+        stderr_len: typeof err.stderr === 'string' ? err.stderr.length : undefined,
         aborted: wasAborted || undefined,
         replay: isReplay || undefined,
         oom_shutdown: oomShutdown || undefined,
@@ -332,7 +343,7 @@ function createDispatcher({
           logger.error?.(`[${sessionKey}] failed to clear lost session: ${resetErr.message}`);
           logEvent('session-reset-after-process-loss-failed', {
             chat_id: chatId, session_key: sessionKey, msg_id: msg?.message_id,
-            error: resetErr.message?.slice(0, 200),
+            error_class: resetErr.name || 'Error',
           });
         }
       }
@@ -368,7 +379,8 @@ function createDispatcher({
         logger.error?.(`[auth] (${botName}) Claude access DISABLED by Anthropic — turn rejected immediately instead of wedging; check the account/billing.`);
         logEvent('auth-disabled', {
           chat_id: chatId, session_key: sessionKey, msg_id: msg?.message_id,
-          error: err.message?.slice(0, 500),
+          error_class: err.name || 'Error',
+        error_len: err.message?.length ?? 0,
         });
         // Never let the gate itself become a new failure mode — AUTH_DISABLED
         // is account-wide, so a bad gate would throw on every concurrent
@@ -486,7 +498,7 @@ function createDispatcher({
           autoResumeTracker.markAttempt(sessionKey);
           logEvent('auto-resume-attempted', {
             chat_id: chatId, session_key: sessionKey, msg_id: msg.message_id,
-            original_error: err.message?.slice(0, 200),
+            error_class: err.name || 'Error',
           });
           return attemptAutoResume(sessionKey, chatId, msg, bot, {
             provider,
@@ -502,7 +514,7 @@ function createDispatcher({
               logger.error?.(`[${sessionKey}] auto-resume failed: ${resumeErr?.message}`);
               logEvent('auto-resume-failed', {
                 chat_id: chatId, session_key: sessionKey, msg_id: msg.message_id,
-                error: resumeErr?.message?.slice(0, 200),
+                error_class: resumeErr?.name || 'Error',
               });
               // Music topic incident (2026-06-01): a channels session whose
               // context grew large enough to auto-/compact on resume loses its
@@ -562,6 +574,7 @@ function createDispatcher({
         }
       }
     }).finally(() => {
+      activeHandlerTargets.delete(activeHandlerTarget);
       const n = (inFlightHandlers.get(sessionKey) || 1) - 1;
       if (n <= 0) inFlightHandlers.delete(sessionKey);
       else inFlightHandlers.set(sessionKey, n);
@@ -575,6 +588,8 @@ function createDispatcher({
     errorReplyText,
     queueWarnThreshold,
     inFlightHandlers,  // exposed so polygram.js can introspect for shutdown drain
+    getActiveHandlerCount: () => activeHandlers.size,
+    getActiveHandlerTargets: () => [...activeHandlerTargets],
     awaitSettlement,
     trackTask,
   };