polygram 0.17.7 → 0.17.10

package/lib/process/cli-process.js

@@ -350,6 +350,10 @@ class CliProcess extends Process {
     // Monotonic count of work hooks (all but the terminal Stop) — the rung-2
     // no-reply backstop snapshots it at Stop capture to detect a later resume.
     this._workHookSeq = 0;
+    // In-flight sub-agent starts (pushed on Agent PreToolUse, spliced on SubagentStop).
+    // Initialized here so a SubagentStop arriving before any Agent start (a lagged/orphan
+    // teardown on a fresh proc) reads a length safely instead of throwing.
+    this._pendingSubagentStarts = [];
     // 0.13 D2: the InputLedger — every user-shaped input written to the bridge
     // gets an observable lifecycle: written → seen → resolved | dropped |
     // superseded | fold-suspected. Pre-P3, injectUserMessage minted a turn_id
@@ -2040,12 +2044,32 @@ class CliProcess extends Process {
    *     sibling's text; otherwise leave the status (nothing more to send).
    */
   _resolveTurnDelivery(pending, turnId) {
+    const out = this._computeTurnDelivery(pending, turnId);
+    // 0.17.8 characterize-first: the channels double-delivery (shumorobot Music,
+    // 2026-06-28 — reply tool sent #2147, then the daemon re-sent result.text as
+    // #2149) is a turn that resolved with alreadyDelivered=false despite a reply tool
+    // delivery. Log the chosen branch + counts so the next occurrence pins WHY (the
+    // leading hypothesis: an interrupted turn loses its recorded reply → the
+    // zero-reply Stop-fallback re-delivers last_assistant_message).
+    this._logEvent('cli-resolve-delivery', {
+      turn_id: turnId, session_key: this.sessionKey, backend: this.backend,
+      branch: out.branch,
+      already_delivered: out.alreadyDelivered,
+      reply_count: pending.replies.length,
+      interim_count: pending._interimReplyCount || 0,
+      has_stop_data: !!pending._stopHookData,
+      text_len: (out.text || '').length,
+    });
+    return { text: out.text, alreadyDelivered: out.alreadyDelivered };
+  }
+
+  _computeTurnDelivery(pending, turnId) {
     const norm = (s) => (s || '').trim();
     const interimText = pending.replies.join('\n\n');
     const fallbackText = pending._stopHookData?.lastAssistantMessage || '';
 
     if (this._turnHasFinalReply(pending)) {
-      return { text: interimText, alreadyDelivered: true };
+      return { text: interimText, alreadyDelivered: true, branch: 'final-reply' };
     }
     if (pending.replies.length === 0) {
       // 0.12 Phase 1.7 fallback: no reply tool call landed — use the Stop hook's
@@ -2068,7 +2092,7 @@ class CliProcess extends Process {
           rescued_len: text.length, ack_len: norm(pending._consumedByText).length,
         });
       }
-      return { text, alreadyDelivered };
+      return { text, alreadyDelivered, branch: 'zero-reply' };
     }
     // Interim-only: the turn delivered ONLY status/progress promises ("give me a
     // couple min") and never a final reply. If claude produced a substantive final
@@ -2084,9 +2108,9 @@ class CliProcess extends Process {
         turn_id: turnId, session_key: this.sessionKey, backend: this.backend,
         rescued_len: fallbackText.length, interim_count: pending.replies.length,
       });
-      return { text: fallbackText, alreadyDelivered: false };
+      return { text: fallbackText, alreadyDelivered: false, branch: 'interim-rescue' };
     }
-    return { text: interimText, alreadyDelivered: true };
+    return { text: interimText, alreadyDelivered: true, branch: 'interim-noop' };
   }
 
   _finalizeTurn(turnId) {
@@ -2930,15 +2954,26 @@ class CliProcess extends Process {
 
     // 0.13 D1: every hook event is same-session ACTIVITY for the finalizer
     // ladder (generalizes the 2026-06-08 WA-topic fix, which only extended on
-    // Pre/PostToolUse) — EXCEPT Stop, which is a terminal signal, not work:
-    // noting it as activity would cancel its own attribution grace. parse-error
-    // and unknown are excluded too (stream noise is not evidence of work).
-    if (ev.type === 'Stop') {
+    // Pre/PostToolUse) — EXCEPT terminal signals, which are not work: noting them
+    // as activity would cancel a live attribution grace. parse-error and unknown
+    // are excluded too (stream noise is not evidence of work).
+    //
+    // Stop is always terminal. SubagentStop is terminal ONLY when it's an ORPHAN —
+    // a late/lagged/foreign teardown hook with no matching in-flight sub-agent start.
+    // Such an orphan trails the main Stop (or arrives from a prior cycle) and must not
+    // bump the work-hook counter (the rung-2 no-reply backstop reads a bump as "claude
+    // resumed", withdrawing the captured Stop's delivery) nor count as activity. A
+    // MATCHED SubagentStop (a sub-agent this cycle actually started is finishing) IS
+    // work-relevant — it keeps withdrawing rung-2 eligibility on a boundary Stop, incl.
+    // a tool-less sub-agent whose only post-boundary signal is its SubagentStop.
+    const orphanSubagentStop = ev.type === 'SubagentStop'
+      && !(this._pendingSubagentStarts && this._pendingSubagentStarts.length);
+    if (ev.type === 'Stop' || orphanSubagentStop) {
       this._lastHookEventAt = Date.now();
     } else if (ev.type && ev.type !== 'parse-error' && ev.type !== 'unknown') {
       this._lastHookEventAt = Date.now();
-      // Monotonic count of WORK hooks (everything but the terminal Stop). The rung-2
-      // no-reply backstop snapshots this at Stop capture; a later increment means
+      // Monotonic count of WORK hooks (all but terminal Stop / orphan SubagentStop). The
+      // rung-2 no-reply backstop snapshots this at Stop capture; a later increment means
       // claude resumed work, withdrawing the stale Stop's finalize eligibility.
       this._workHookSeq = (this._workHookSeq || 0) + 1;
       this._noteActivity(`hook:${ev.type}`);

package/lib/session-key.js

@@ -128,7 +128,19 @@ function getConfigWriteScope(chatConfig, threadId) {
   // runs), and report threadId:null so the audit row reflects chat-level reach.
   if (tid && chatConfig?.isolateTopics === true) {
     chatConfig.topics = chatConfig.topics || {};
-    chatConfig.topics[tid] = chatConfig.topics[tid] || {};
+    // A topic can be stored in the legacy / hand-edited form where the entry
+    // is the bare name string (topics["329"] = "Advertising") rather than the
+    // canonical object ({ name: "Advertising" }). getTopicConfig / getTopicName
+    // already tolerate that; the write path must too — assigning a property to
+    // a string throws under strict mode, which silently swallowed every /model
+    // + /effort button tap in such a topic. Normalise to the object form,
+    // preserving the name.
+    const existing = chatConfig.topics[tid];
+    if (typeof existing === 'string') {
+      chatConfig.topics[tid] = { name: existing };
+    } else if (!existing || typeof existing !== 'object') {
+      chatConfig.topics[tid] = {};
+    }
     return { scope: chatConfig.topics[tid], threadId: tid };
   }
   return { scope: chatConfig, threadId: null };

package/package.json

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