polygram 0.12.0 → 0.12.2

package/lib/process/channels-bridge.mjs

@@ -322,7 +322,7 @@ mcp.setRequestHandler(ListToolsRequestSchema, async () => {
         // (P0 spike Q-B: claude echoes only the trigger id) — this array can.
         consumed_turn_ids: {
           type: 'array', items: { type: 'string' },
-          description: 'turn_id of EVERY <channel> message this reply answers or has absorbed since your last reply (including mid-turn follow-ups). Always set it.',
+          description: 'turn_id of EVERY <channel> message this reply answers or has absorbed since your last reply, including mid-turn follow-ups. Set it on EVERY reply, even short one-line ones; if you answered two messages in one reply, list BOTH turn_ids. Omitting a folded follow-up makes polygram treat it as dropped.',
         },
       },
       required: ['chat_id', 'text'],

package/lib/process/cli-process.js

@@ -754,11 +754,14 @@ class CliProcess extends Process {
       'as normal — only the FINAL user-visible message needs to go through',
       'the reply tool.',
       '',
-      'When you call `reply`, ALWAYS set `consumed_turn_ids` to the turn_id',
-      'attribute of EVERY <channel> message you are answering or have received',
-      'since your last reply — including mid-turn follow-ups you absorbed into',
-      'the current answer. polygram uses it to confirm follow-up delivery;',
-      'omitting it can cause a follow-up to be re-sent to you.',
+      'When you call `reply`, ALWAYS set `consumed_turn_ids` to the turn_id of',
+      'EVERY <channel> message this reply answers or folds in — every mid-turn',
+      'follow-up you absorbed since your last reply. This applies to EVERY reply,',
+      'including SHORT one-line ones: if two messages arrived and you answered both',
+      'in one reply, list BOTH turn_ids (e.g. consumed_turn_ids: ["<original-id>",',
+      '"<follow-up-id>"]). Omitting a folded follow-up makes polygram read it as',
+      'DROPPED — it gets re-sent to you or flagged as a lost message. When unsure,',
+      'include the id.',
       '',
       '### Staying responsive on a long task',
       '',
@@ -1159,7 +1162,10 @@ class CliProcess extends Process {
     // happens at the chat_id guard below.
     const chatIdMatches = this.chatId == null || String(args.chat_id) === String(this.chatId);
     if (chatIdMatches && Array.isArray(args.consumed_turn_ids) && args.consumed_turn_ids.length) {
-      this._ledgerAckConsumed(args.consumed_turn_ids.filter((x) => typeof x === 'string'));
+      this._ledgerAckConsumed(
+        args.consumed_turn_ids.filter((x) => typeof x === 'string'),
+        typeof args.text === 'string' ? args.text : '',
+      );
     } else if (chatIdMatches && msg.name === 'reply' && 'consumed_turn_ids' in args) {
       this._lastAckFieldAt = Date.now();   // field present but empty — contract observed
     }
@@ -1534,8 +1540,16 @@ class CliProcess extends Process {
     if (e._watchdogTimer) { clearTimeout(e._watchdogTimer); e._watchdogTimer = null; }
   }
 
-  /** Tier 2C: a reply carried consumed_turn_ids — acknowledge every known id. */
-  _ledgerAckConsumed(ids) {
+  /**
+   * Tier 2C: a reply carried consumed_turn_ids — acknowledge every known id.
+   * `consumingText` is the text the consuming reply actually delivered; it is
+   * recorded on each consumed pending so _finalizeTurn can tell a genuine fold
+   * (the answer rode the sibling reply) from an early ack that DIDN'T carry the
+   * answer (prod 2026-06-13: a 294-char "Researching now…" ack, then the real
+   * answer arrived as Stop-fallback text — which must NOT be suppressed).
+   * See docs/0.13-consumed-ack-stop-fallback-drop-spec.md.
+   */
+  _ledgerAckConsumed(ids, consumingText = '') {
     this._lastAckFieldAt = Date.now();
     for (const id of ids) {
       const e = this.inputLedger.get(id);
@@ -1552,6 +1566,14 @@ class CliProcess extends Process {
       const pending = this.pendingTurns.get(id);
       if (pending) {
         pending._consumedAcked = true;
+        // Remember WHAT the consuming reply delivered for this turn. The last
+        // ack wins; a longer subsequent ack is the safer record (more likely to
+        // actually contain the answer). _finalizeTurn only suppresses a
+        // Stop-fallback finalize when its text matches this.
+        if (typeof consumingText === 'string'
+            && consumingText.length > (pending._consumedByText?.length || 0)) {
+          pending._consumedByText = consumingText;
+        }
         // The ack itself flips rung-2 eligibility on — arm now. (The turn's
         // last _noteActivity ran BEFORE this flag was set, so without this
         // a quiet tail would never re-arm and the turn would sit until a
@@ -1890,10 +1912,33 @@ class CliProcess extends Process {
     // the turn without calling mcp__polygram-bridge__reply), use the Stop
     // hook's last_assistant_message as the text. Same rescue pattern rc.41
     // H4 uses on tmux backend when JSONL stream is broken.
-    if (!text && pending._stopHookData?.lastAssistantMessage) {
+    const usedStopFallback = !text && !!pending._stopHookData?.lastAssistantMessage;
+    if (usedStopFallback) {
       text = pending._stopHookData.lastAssistantMessage;
       this.logger.warn?.(`[${this.label}] cli: turn finalized via stop-hook fallback (no reply tool call); text_len=${text.length}`);
     }
+    // A _consumedAcked turn is "already delivered" ONLY when the consuming
+    // sibling reply actually carried THIS text — not merely an ack. Prod
+    // 2026-06-13 (Shumabit@UMI/37): the consuming reply was a 294-char
+    // "Researching now…" ack, then the real answer arrived as Stop-fallback
+    // text. Suppressing it (alreadyDelivered=true → polygram.js short-circuit)
+    // dropped the answer silently for 5h20m. Only suppress when the rescued
+    // text matches what the sibling delivered.
+    // docs/0.13-consumed-ack-stop-fallback-drop-spec.md
+    const norm = (s) => (s || '').trim();
+    const consumedCoversFallback = !usedStopFallback || norm(text) === norm(pending._consumedByText);
+    const alreadyDelivered = hadReplyToolCalls
+      || (pending._consumedAcked === true && consumedCoversFallback);
+    if (usedStopFallback && pending._consumedAcked === true && !consumedCoversFallback) {
+      this.logger.warn?.(`[${this.label}] cli: consumed-ack did NOT cover the Stop-fallback answer — delivering rescued text (len=${text.length})`);
+      this._logEvent('cli-consumed-ack-fallback-rescued', {
+        turn_id: turnId,
+        session_key: this.sessionKey,
+        backend: this.backend,
+        rescued_len: text.length,
+        ack_len: norm(pending._consumedByText).length,
+      });
+    }
     const duration = Date.now() - pending.startedAt;
     // Review AC4: cost=null + metrics-tokens=null signal "unmeasured-subscription"
     // (channels protocol doesn't expose per-turn cost or token breakdowns).
@@ -1910,9 +1955,11 @@ class CliProcess extends Process {
       // would resolve the turn silently and the user still sees nothing. So
       // only claim already-delivered when reply tool calls actually fired —
       // or when claude ACKED consuming this turn in a sibling reply
-      // (consumed_turn_ids; the fold-id-echo case): re-sending the Stop
-      // fallback there would duplicate the delivered answer.
-      alreadyDelivered: hadReplyToolCalls || pending._consumedAcked === true,
+      // (consumed_turn_ids; the fold-id-echo case) AND that sibling actually
+      // delivered this text — re-sending the Stop fallback there would
+      // duplicate. A consumed-ack whose ack did NOT carry the Stop-fallback
+      // answer must still deliver (see consumedCoversFallback above).
+      alreadyDelivered,
       sessionId: this.claudeSessionId,
       cost: null,             // Channels protocol doesn't expose per-turn cost
       duration,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polygram",
-  "version": "0.12.0",
+  "version": "0.12.2",
   "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": {