@goodandready/dsh-messenger-gateway 0.3.13 → 0.3.18

package/lib/telegram-errors.js

@@ -1,41 +1,51 @@
-// Network error classification for Telegram sends (mirrors Hermes adapter policy).
-//
-// The key distinction: some network failures mean the request NEVER left the
-// process (connect/pool timeout, ECONNRESET before send) → resending is safe.
-// Others (a generic timeout after the request may have reached Telegram) could
-// duplicate a message if we resend → do NOT retry.
-
-function rootCause(err) {
-  let e = err
-  let depth = 0
-  while (e && e.cause && e.cause !== e && depth < 10) {
-    e = e.cause
-    depth++
-  }
-  return e || err
-}
-
-// True when a resend cannot duplicate a message: the request did not reach Telegram.
-export function isResendSafeNetworkError(err) {
-  const c = rootCause(err)
-  const msg = String(c?.message || err?.message || '').toLowerCase()
-  if (/not sent to telegram|connect timeout|und_err_connect|econnreset|enotfound|econnrefused|ECONNRESET|ENOTFOUND|ECONNREFUSED/i.test(msg)) {
-    return true
-  }
-  // undici PoolTimeout message: "Request was *not* sent to Telegram."
-  if (/pool timeout|request was \*?not\*? sent/i.test(msg)) return true
-  return false
-}
-
-// True for a 409 from getUpdates: a second bot instance polls the same token.
-export function isPollingConflict(err) {
-  const msg = String(err?.message || '').toLowerCase()
-  return /terminated by other getupdates request|another bot instance is running|getupdates.*conflict|conflict.*getupdates/i.test(msg)
-}
-
-// True when the target chat/topic no longer exists (deleted/closed/upgraded).
-// Such sends should not be retried and any pending ask binding must be pruned.
-export function isTopicGoneError(err) {
-  const msg = String(err?.message || '').toLowerCase()
-  return /thread not found|message thread not found|topic[_ ]?(not found|closed|deleted)|chat (not found|was (upgraded|deleted))|group chat was (upgraded|deleted)/i.test(msg)
-}
+// Network error classification for Telegram sends (mirrors Hermes adapter policy).
+//
+// The key distinction: some network failures mean the request NEVER left the
+// process (connect/pool timeout, ECONNRESET before send) → resending is safe.
+// Others (a generic timeout after the request may have reached Telegram) could
+// duplicate a message if we resend → do NOT retry.
+
+function rootCause(err) {
+  let e = err
+  let depth = 0
+  while (e && e.cause && e.cause !== e && depth < 10) {
+    e = e.cause
+    depth++
+  }
+  return e || err
+}
+
+// True when a resend cannot duplicate a message: the request did not reach Telegram.
+export function isResendSafeNetworkError(err) {
+  const c = rootCause(err)
+  const msg = String(c?.message || err?.message || '').toLowerCase()
+  if (/not sent to telegram|connect timeout|und_err_connect|econnreset|enotfound|econnrefused|ECONNRESET|ENOTFOUND|ECONNREFUSED/i.test(msg)) {
+    return true
+  }
+  // undici PoolTimeout message: "Request was *not* sent to Telegram."
+  if (/pool timeout|request was \*?not\*? sent/i.test(msg)) return true
+  return false
+}
+
+// True for a 409 from getUpdates: a second bot instance polls the same token.
+export function isPollingConflict(err) {
+  const msg = String(err?.message || '').toLowerCase()
+  return /terminated by other getupdates request|another bot instance is running|getupdates.*conflict|conflict.*getupdates/i.test(msg)
+}
+
+// True when the target chat/topic no longer exists (deleted/closed/upgraded).
+// Such sends should not be retried and any pending ask binding must be pruned.
+export function isTopicGoneError(err) {
+  const msg = String(err?.message || '').toLowerCase()
+  return /thread not found|message thread not found|topic[_ ]?(not found|closed|deleted)|chat (not found|was (upgraded|deleted))|group chat was (upgraded|deleted)/i.test(msg)
+}
+
+/**
+ * Calculates exponential backoff with max cap for polling retries.
+ * errorCount is 1-indexed.
+ */
+export function computePollBackoffMs(baseIntervalMs, errorCount, maxDelayMs = 30_000) {
+  const count = Math.max(1, Math.min(10, Number(errorCount) || 1))
+  const base = Math.max(100, Number(baseIntervalMs) || 500)
+  return Math.min(maxDelayMs, base * Math.pow(2, count - 1))
+}

package/lib/voice-prefs.js

@@ -1,5 +1,19 @@
-import { readFileSync, writeFileSync, mkdirSync, existsSync } from "node:fs"
+import { readFileSync, writeFileSync, mkdirSync, existsSync, renameSync, unlinkSync } from "node:fs"
 import { dirname } from "node:path"
+import { randomUUID } from "node:crypto"
+
+function writeJsonAtomicSync(filePath, data) {
+  const dir = dirname(filePath)
+  mkdirSync(dir, { recursive: true })
+  const tmpPath = `${filePath}.${randomUUID().slice(0, 8)}.tmp`
+  try {
+    writeFileSync(tmpPath, JSON.stringify(data, null, 2), "utf8")
+    renameSync(tmpPath, filePath)
+  } catch (err) {
+    try { unlinkSync(tmpPath) } catch {}
+    throw err
+  }
+}
 
 export function createVoicePrefs(filePath) {
   /** @type {Record<string, boolean>} */
@@ -12,8 +26,9 @@ export function createVoicePrefs(filePath) {
   }
   const persist = () => {
     if (!filePath) return
-    mkdirSync(dirname(filePath), { recursive: true })
-    writeFileSync(filePath, JSON.stringify(state, null, 2))
+    try {
+      writeJsonAtomicSync(filePath, state)
+    } catch {}
   }
   const key = (userId) => String(Number(userId) || userId || "")
   return {
@@ -43,4 +58,4 @@ export function shouldSpeakReply({ globalTts, voiceMode, inboundWasVoice, userPr
   if (voiceMode === "off") return false
   // mirror: speak if inbound was voice
   return Boolean(inboundWasVoice)
-}
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-messenger-gateway",
-  "version": "0.3.13",
+  "version": "0.3.18",
   "description": "Telegram messenger bridge for DeepSeek Harness: sessions, steer, homes, inline asks, notify bridge, and optional TTS voice notes.",
   "license": "MIT",
   "type": "module",
@@ -42,10 +42,7 @@
     },
     "client": {
       "platform": "web",
-      "inject": [
-        "@deepseek-ai/dsh-client-locale",
-        "@deepseek-ai/dsh-client-ui-settings"
-      ]
+      "inject": []
     }
   },
   "peerDependencies": {