polygram 0.10.0-rc.40 → 0.10.0-rc.41

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://anthropic.com/claude-code/plugin.schema.json",
   "name": "polygram",
-  "version": "0.10.0-rc.40",
+  "version": "0.10.0-rc.41",
   "description": "Telegram integration for Claude Code that preserves the OpenClaw per-chat session model. Migration target for OpenClaw users. Multi-bot, multi-chat, per-topic isolation; SQLite transcripts; inline-keyboard approvals. Bundles /polygram:status|logs|pair-code|approvals admin commands plus history (transcript queries) and polygram-send (out-of-turn IPC sends with file-upload validation) skills.",
   "keywords": [
     "telegram",

package/lib/process/tmux-process.js

@@ -202,6 +202,14 @@ const DEFAULT_QUIESCE_MS       = 500; // require READY for this long before decl
 //                              wedge-detection delay is bounded.
 const DEFAULT_HARD_BACKSTOP_MS = 4 * 60 * 60_000;     // 4 hours
 const IDLE_POLL_INTERVAL_MS    = 30_000;              // 30 s
+// 0.10.0 H4 — `Stop` hook as authoritative turn-done.
+//   The Stop hook fires when claude finishes responding — same
+//   semantic as the JSONL `result` event. Both should land within
+//   ms of each other; this grace gives JSONL a chance to win (full
+//   result data: subtype, stopReason, all the metadata) before the
+//   Stop hook synthesizes a fallback settle. If the JSONL stream is
+//   broken or stuck, Stop carries the turn to completion alone.
+const DEFAULT_STOP_GRACE_MS    = 2_000;               // 2 s
 
 // B8 (slow-MCP readiness): how long the claude `--debug-file` log must
 // have had NO new bytes appended before the startup is considered
@@ -251,6 +259,7 @@ class TmuxProcess extends Process {
     readyTimeoutMs = DEFAULT_READY_TIMEOUT_MS,
     turnTimeoutMs = DEFAULT_TURN_TIMEOUT_MS,
     hardBackstopMs = DEFAULT_HARD_BACKSTOP_MS,
+    stopGraceMs = DEFAULT_STOP_GRACE_MS,
     pollMs = DEFAULT_POLL_MS,
     quiesceMs = DEFAULT_QUIESCE_MS,
     lateGraceMs = 1500,
@@ -295,6 +304,7 @@ class TmuxProcess extends Process {
     this.readyTimeoutMs = readyTimeoutMs;
     this.turnTimeoutMs = turnTimeoutMs;
     this.hardBackstopMs = hardBackstopMs;
+    this.stopGraceMs = stopGraceMs;
     this.pollMs = pollMs;
     this.quiesceMs = quiesceMs;
     this.readyDebugQuietMs = readyDebugQuietMs;
@@ -1601,7 +1611,7 @@ class TmuxProcess extends Process {
   }
 
   /**
-   * Hook-event handler. Three roles, layered over time:
+   * Hook-event handler. Four roles, layered over time:
    *
    *   H1 (rc.36) — emit `hook-event` so polygram persists each event
    *     to the events DB; observer-only.
@@ -1615,24 +1625,63 @@ class TmuxProcess extends Process {
    *     the structural fix for the msg-884 incident (49-min
    *     SoundCloud subagent killed at the 30-min wall-clock while
    *     demonstrably alive).
-   *
-   *   H4 — `Stop` as authoritative completion (still pending).
+   *   H4 (rc.41) — `Stop` hook is an authoritative turn-done signal.
+   *     If JSONL `result` doesn't fire within `stopGraceMs`,
+   *     synthesize a settle from the Stop payload so a broken or
+   *     stuck JSONL stream can't strand a finished turn. Promise-
+   *     resolve idempotency means JSONL still wins when both fire.
    *
    * Parse errors and unknown event shapes are intentionally still
    * forwarded — observer-only metrics for stream-reliability soak.
    */
   _handleHookEvent(ev) {
     // H3: every hook event (except the diagnostic types) is liveness
-    // evidence. Heartbeat every turn in the active group so the
-    // idle-ceiling poller resets. We don't differentiate by event
+    // evidence. Heartbeat every turn we can identify as in-flight so
+    // the idle-ceiling poller resets. We don't differentiate by event
     // type — even Notification or UserPromptSubmit prove claude is
     // active in this session.
+    //
+    // Two scopes are searched (deduped via Set): active group turns
+    // (the steady state once `user-message` has landed) AND the
+    // pendingQueue head (the PRE-active window between turn start
+    // and the first `user-message`). Hook events can fire in either
+    // window — e.g. `UserPromptSubmit` arrives just after claude
+    // receives the paste but BEFORE the `user-message` is echoed
+    // back into the JSONL. Without the pendingQueue fallback, that
+    // window leaves the turn un-heartbeated and the idle poller
+    // could fire on a turn that's actively starting up.
     if (ev?.type && ev.type !== 'parse-error' && ev.type !== 'unknown') {
-      const turns = this._activeGroup?.turns || [];
+      const turns = new Set(this._activeGroup?.turns || []);
+      const head = this.pendingQueue[0];
+      if (head) turns.add(head);
       for (const t of turns) {
         this._heartbeat(t, `hook:${ev.type}`);
       }
     }
+    // H4: Stop hook → synthesize a settle for the primary turn after
+    // a grace, so JSONL `result` (which carries richer metadata)
+    // wins when both fire. If JSONL never arrives — broken stream,
+    // stuck parser — the Stop synth settles the turn instead of
+    // stranding it. Idempotent: a later JSONL settleResult call is
+    // a no-op once the promise has resolved.
+    if (ev?.type === 'Stop') {
+      const primary = (this._activeGroup?.turns || [])
+        .find((t) => t.kind === 'primary');
+      if (primary && typeof primary.settleResult === 'function') {
+        const synth = {
+          text: primary.text || ev.lastAssistantMessage || '',
+          subtype: 'success',
+          stopReason: 'stop_hook',
+          sessionId: this.claudeSessionId,
+          via: 'stop-hook',
+        };
+        const timer = setTimeout(
+          () => primary.settleResult(synth),
+          this.stopGraceMs,
+        );
+        timer.unref?.();
+      }
+    }
     this.emit('hook-event', ev);
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polygram",
-  "version": "0.10.0-rc.40",
+  "version": "0.10.0-rc.41",
   "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": {