polygram 0.8.0-rc.2 → 0.8.0-rc.21

package/lib/status-reactions.js

@@ -29,19 +29,30 @@
 // are progressively safer. All endings in this list are in Telegram's
 // default available reactions as of 2026-04.
 const STATES = {
-  QUEUED:   { label: 'queued',   chain: ['👀', '🤔']       },
-  THINKING: { label: 'thinking', chain: ['🤔']             },
-  CODING:   { label: 'coding',   chain: ['👨‍💻', '✍', '🤔'] },
-  WEB:      { label: 'web',      chain: ['⚡', '🔥', '🤔']  },
-  TOOL:     { label: 'tool',     chain: ['🔥', '🤔']       },
-  WRITING:  { label: 'writing',  chain: ['✍', '🤔']        },
-  DONE:     { label: 'done',     chain: ['👍']             },
-  ERROR:    { label: 'error',    chain: ['🤯', '🤔']       },
-  STALL:    { label: 'stall',    chain: ['🥱', '🤔']       },
-  TIMEOUT:  { label: 'timeout',  chain: ['😨', '🤯']       },
+  QUEUED:      { label: 'queued',      chain: ['👀', '🤔']       },
+  THINKING:    { label: 'thinking',    chain: ['🤔']             },
+  CODING:      { label: 'coding',      chain: ['👨‍💻', '✍', '🤔'] },
+  WEB:         { label: 'web',         chain: ['⚡', '🔥', '🤔']  },
+  TOOL:        { label: 'tool',        chain: ['🔥', '🤔']       },
+  WRITING:     { label: 'writing',     chain: ['✍', '🤔']        },
+  // 0.8.0-rc.11: terminal "your follow-up was incorporated into the
+  // in-flight turn" state. Used by polygram's autosteer block when a
+  // mid-turn user message is buffered for the next PostToolBatch
+  // injection.
+  AUTOSTEERED: { label: 'autosteered', chain: ['✍', '👀']        },
+  DONE:        { label: 'done',        chain: ['👍']             },
+  ERROR:       { label: 'error',       chain: ['🤯', '🤔']       },
+  STALL:       { label: 'stall',       chain: ['🥱', '🤔']       },
+  TIMEOUT:     { label: 'timeout',     chain: ['😨', '🤯']       },
 };
 
-const TERMINAL_STATES = new Set(['DONE', 'ERROR', 'TIMEOUT']);
+// Terminal states bypass throttle, disarm stall promotion, and the
+// reactor stays at this emoji until explicitly cleared. AUTOSTEERED
+// is included so setState('AUTOSTEERED') flushes immediately
+// (matters because the autosteer code path returns from
+// handleMessage right after — we don't want the apply to be
+// scheduled-and-cancelled by reactor.stop in the outer finally).
+const TERMINAL_STATES = new Set(['DONE', 'ERROR', 'TIMEOUT', 'AUTOSTEERED']);
 const DEFAULT_THROTTLE_MS = 800;
 // 0.7.4 (item A): after this long with no setState() call (Claude is
 // silently chugging on a long tool / model latency), auto-flip to STALL
@@ -132,24 +143,40 @@ function createReactionManager({
   let stallTimer = null;
   let freezeTimer = null;
   let stopped = false;
+  // 0.8.0-rc.11: serialize Telegram setMessageReaction calls. Without
+  // this, multiple flush()es race at the network layer because each
+  // calls `await apply(emoji)` from a separate stack — Telegram
+  // processes them in arbitrary order and the FINAL visible state is
+  // whichever apply landed last. Symptom: 👀 stuck on autosteered
+  // messages when the QUEUED apply landed AFTER our explicit ✍ apply.
+  // Chaining all applies through `applyChain` guarantees they're sent
+  // to Telegram in setState() invocation order.
+  let applyChain = Promise.resolve();
   // States the auto-stall path may transition to. Once we've already
   // shown STALL or TIMEOUT we don't downgrade or rearm — only an
   // explicit setState() call (Claude resumed) can move us forward.
   const STALL_PROMOTABLE = new Set(['THINKING', 'CODING', 'WEB', 'TOOL', 'WRITING']);
 
   const flush = async (stateName) => {
-    if (stopped) return;
+    if (stopped && !TERMINAL_STATES.has(stateName)) return;
     const spec = STATES[stateName];
     if (!spec) return;
     const emoji = resolveEmoji(spec.chain, availableEmojis);
     if (emoji === currentEmoji) return;
     currentEmoji = emoji;
     lastFlushTs = Date.now();
-    try {
-      await apply(emoji);
-    } catch (err) {
-      logError(`reaction apply failed (${stateName} → ${emoji}): ${err?.message || err}`);
-    }
+    // Chain through applyChain so concurrent flushes are sent to
+    // Telegram serially in invocation order. Returning the chain
+    // promise lets callers await this specific flush completing.
+    const myApply = applyChain.then(async () => {
+      try {
+        await apply(emoji);
+      } catch (err) {
+        logError(`reaction apply failed (${stateName} → ${emoji}): ${err?.message || err}`);
+      }
+    });
+    applyChain = myApply;
+    return myApply;
   };
 
   const clearStallTimers = () => {
@@ -217,8 +244,16 @@ function createReactionManager({
     clearStallTimers();
     if (currentEmoji == null) return;
     currentEmoji = null;
-    try { await apply(null); }
-    catch (err) { logError(`reaction clear failed: ${err?.message || err}`); }
+    // Same applyChain serialization as flush — clear() is a state
+    // transition, just to "no emoji". Without chaining, a clear()
+    // racing with a pending apply (e.g. THINKING flush in flight)
+    // could land BEFORE that apply, leaving the emoji visible.
+    const myApply = applyChain.then(async () => {
+      try { await apply(null); }
+      catch (err) { logError(`reaction clear failed: ${err?.message || err}`); }
+    });
+    applyChain = myApply;
+    return myApply;
   };
 
   const stop = () => {
@@ -227,10 +262,26 @@ function createReactionManager({
     clearStallTimers();
   };
 
+  // 0.8.0-rc.16: heartbeat — re-arm stall/freeze timers without
+  // changing the visible emoji. Used by SDK pm's onStreamChunk
+  // callback so long text generation doesn't unfairly trip STALL
+  // (🥱) / TIMEOUT (😨) promotions after 10/30s of no explicit
+  // setState calls. Each text chunk is "I'm still here" evidence.
+  // Pre-rc.16, prolonged streaming with no tool use was reliably
+  // promoted to 🥱 within 10s even though the bot was producing
+  // output the whole time.
+  const heartbeat = () => {
+    if (stopped) return;
+    if (!STALL_PROMOTABLE.has(currentState)) return;
+    lastSetStateTs = Date.now();
+    armStallTimers();
+  };
+
   return {
     setState,
     clear,
     stop,
+    heartbeat,
     // Introspection for tests:
     get currentState() { return currentState; },
     get currentEmoji() { return currentEmoji; },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polygram",
-  "version": "0.8.0-rc.2",
+  "version": "0.8.0-rc.21",
   "description": "Telegram daemon for Claude Code that preserves the OpenClaw per-chat session model. Migration path for OpenClaw users moving to Claude Code.",
   "main": "lib/ipc-client.js",
   "bin": {