@inetafrica/open-claudia 2.6.42 → 2.6.44

package/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v2.6.43
+- **Fix the `🧠 Recall this turn` banner rendering raw HTML.** The recall debug banner (from `/recall on`) is built with `<b>` tags and HTML-escaped names, but its two `send()` calls in `runner.js` omitted the Telegram `parseMode: "HTML"` opts that every normal reply already passes via `telegramHtmlOpts()` — so the tags and `&amp;` entities showed up literally in chat instead of rendering. Both recall sends now pass `telegramHtmlOpts()`, matching the rest of the reply path. Display-only; no change to recall behaviour itself.
+
 ## v2.6.42
 - **Read-time `📖 Recalled my notes on …` now fires for direct file reads, not just the CLI.** The read-side recall banner — and the Hebbian co-use signal that reinforces the recall graph — was wired only to shell commands, watching for `open-claudia pack show <dir>` / `entity show <slug>`. When the agent opened one of its own notes by reading the raw `…/packs/<dir>/PACK.md` or `…/entities/<slug>.md` straight through the Read tool, the detector never saw it: no banner, and the open never counted as co-use. A new `core/recall/read-signal.js` resolves a file path to its memory node (mirroring the existing Write/Edit path-detection), and `runner.js` calls it on Read tool-uses across the Claude and Cursor backends — deduped against the CLI path via the shared notify key, so reading the same node both ways still announces once. Adds `test-read-signal.js`.
 

package/channels/telegram/adapter.js

@@ -19,11 +19,16 @@ class TelegramAdapter {
     this.token = token;
     this.ownerChatId = ownerChatId;
     this.chatIds = chatIds || [];
+    // Own the HTTP(S) agent so we can drop dead keep-alive sockets on reconnect.
+    this._agent = new https.Agent({ keepAlive: true });
     this.bot = new TelegramBot(token, {
       polling: { autoStart: false, params: { timeout: 30 } },
+      request: { agent: this._agent },
     });
     this._listeners = { message: new Set(), action: new Set() };
     this._reconnectTimer = null;
+    this._recoveryTimer = null;
+    this._reconnectAttempts = 0;
     this._wireInbound();
   }
 
@@ -45,9 +50,49 @@ class TelegramAdapter {
   }
 
   async stop() {
+    if (this._reconnectTimer) { clearTimeout(this._reconnectTimer); this._reconnectTimer = null; }
+    if (this._recoveryTimer) { clearTimeout(this._recoveryTimer); this._recoveryTimer = null; }
     try { await this.bot.stopPolling(); } catch (e) {}
   }
 
+  // Recover from a wedged poll loop. The library reuses one keep-alive socket;
+  // after a sleep/wake or network blip that socket dies and every poll then
+  // ETIMEDOUTs forever — restarting polling alone reuses the same dead socket.
+  // So we destroy the agent (forcing a fresh connection), back off, and if we
+  // still can't recover after several tries we exit so launchd (KeepAlive)
+  // relaunches a fully clean process.
+  _scheduleReconnect() {
+    if (this._reconnectTimer) return; // one cycle in flight; swallow the flood
+    if (this._recoveryTimer) { clearTimeout(this._recoveryTimer); this._recoveryTimer = null; }
+    this._reconnectAttempts += 1;
+    const MAX = 6;
+    if (this._reconnectAttempts > MAX) {
+      console.error(`Polling unrecoverable after ${MAX} attempts — exiting for supervisor restart.`);
+      process.exit(1);
+    }
+    const delay = Math.min(30000, 1000 * 2 ** this._reconnectAttempts); // 2s,4s,8s,16s,30s,30s
+    console.log(`Network lost. Reconnecting in ${Math.round(delay / 1000)}s (attempt ${this._reconnectAttempts}/${MAX})...`);
+    this._reconnectTimer = setTimeout(async () => {
+      this._reconnectTimer = null;
+      try {
+        try { await this.bot.stopPolling(); } catch (e) {}
+        try { this._agent.destroy(); } catch (e) {} // drop the dead pooled socket
+        await new Promise((r) => setTimeout(r, 1000));
+        await this.bot.startPolling();
+        console.log("Polling restarted.");
+        // No fresh error within 30s ⇒ genuinely recovered; reset the counter.
+        this._recoveryTimer = setTimeout(() => {
+          this._recoveryTimer = null;
+          if (this._reconnectAttempts) console.log("Connection healthy again.");
+          this._reconnectAttempts = 0;
+        }, 30000);
+      } catch (e) {
+        console.error("Reconnect attempt failed:", e.message);
+        this._scheduleReconnect(); // back off and retry; don't die on first miss
+      }
+    }, delay);
+  }
+
   _wireInbound() {
     this.bot.on("polling_error", (err) => {
       const msg = err.message || "";
@@ -56,21 +101,8 @@ class TelegramAdapter {
         console.error("Another instance is polling. Exiting.");
         process.exit(1);
       }
-      if (msg.includes("ETIMEDOUT") || msg.includes("ECONNRESET") || msg.includes("ENOTFOUND") || msg.includes("EFATAL")) {
-        if (this._reconnectTimer) return;
-        console.log("Network lost. Reconnecting in 10s...");
-        this._reconnectTimer = setTimeout(async () => {
-          this._reconnectTimer = null;
-          try {
-            await this.bot.stopPolling();
-            await new Promise((r) => setTimeout(r, 2000));
-            await this.bot.startPolling();
-            console.log("Reconnected.");
-          } catch (e) {
-            console.error("Reconnect failed:", e.message);
-            process.exit(1);
-          }
-        }, 10000);
+      if (/ETIMEDOUT|ECONNRESET|ENOTFOUND|ENETUNREACH|EAI_AGAIN|EFATAL|socket hang up/i.test(msg)) {
+        this._scheduleReconnect();
       }
     });
 

package/core/runner.js

@@ -934,8 +934,8 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
       const esc = (s) => String(s).replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
       const fmt = (arr, icon) => arr.map((x) => (x.why ? `${icon} <b>${esc(x.name)}</b> — ${esc(x.why)}` : `${icon} <b>${esc(x.name)}</b>`));
       const lines = [...fmt(r.packs || [], "📦"), ...fmt(r.entities || [], "👤")];
-      if (lines.length) send(`🧠 <b>Recall this turn</b> (${esc(r.engine)})\n${lines.join("\n")}`).catch(() => {});
-      else if (r.gated) send(`🧠 <b>Recall</b> (${esc(r.engine)}): skipped by pre-gate — trivial turn.`).catch(() => {});
+      if (lines.length) send(`🧠 <b>Recall this turn</b> (${esc(r.engine)})\n${lines.join("\n")}`, telegramHtmlOpts()).catch(() => {});
+      else if (r.gated) send(`🧠 <b>Recall</b> (${esc(r.engine)}): skipped by pre-gate — trivial turn.`, telegramHtmlOpts()).catch(() => {});
     }
   } catch (e) { /* best-effort */ }
   // /stop landed during the pre-spawn window (recall/compaction): bail before

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inetafrica/open-claudia",
-  "version": "2.6.42",
+  "version": "2.6.44",
   "description": "Your always-on AI coding assistant — Claude Code, Cursor Agent, and OpenAI Codex via Telegram or Kazee Chat",
   "main": "bot.js",
   "bin": {