@poncho-ai/messaging 0.7.7 → 0.7.9

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/messaging@0.7.7 build /home/runner/work/poncho-ai/poncho-ai/packages/messaging
+> @poncho-ai/messaging@0.7.9 build /home/runner/work/poncho-ai/poncho-ai/packages/messaging
 > tsup src/index.ts --format esm --dts
 
 CLI Building entry: src/index.ts
@@ -7,8 +7,8 @@
 CLI tsup v8.5.1
 CLI Target: es2022
 ESM Build start
-ESM dist/index.js 52.41 KB
-ESM ⚡️ Build success in 89ms
+ESM dist/index.js 51.93 KB
+ESM ⚡️ Build success in 52ms
 DTS Build start
-DTS ⚡️ Build success in 5653ms
-DTS dist/index.d.ts 11.72 KB
+DTS ⚡️ Build success in 4732ms
+DTS dist/index.d.ts 11.66 KB

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @poncho-ai/messaging
 
+## 0.7.9
+
+### Patch Changes
+
+- [#71](https://github.com/cesr/poncho-ai/pull/71) [`3e5bf7e`](https://github.com/cesr/poncho-ai/commit/3e5bf7e527e394c5f823beac90712756e57cd491) Thanks [@cesr](https://github.com/cesr)! - Fix Telegram tool approval handler never persisting the approval decision, preventing the resume-from-checkpoint flow from triggering. Make answerCallbackQuery best-effort so transient fetch failures don't block approval processing.
+
+## 0.7.8
+
+### Patch Changes
+
+- [`30026c5`](https://github.com/cesr/poncho-ai/commit/30026c5eba3f714bb80c2402c5e8f32c6fd38d87) Thanks [@cesr](https://github.com/cesr)! - Fix Telegram conversation instability on serverless: use stable platformThreadId instead of in-memory session counter.
+
 ## 0.7.7
 
 ### Patch Changes

package/dist/index.d.ts

@@ -233,7 +233,6 @@ declare class TelegramAdapter implements MessagingAdapter {
     private handler;
     private resetHandler;
     private approvalDecisionHandler;
-    private readonly sessionCounters;
     private readonly approvalMessageIds;
     private lastUpdateId;
     constructor(options?: TelegramAdapterOptions);
@@ -252,7 +251,6 @@ declare class TelegramAdapter implements MessagingAdapter {
     updateApprovalMessage(approvalId: string, decision: "approved" | "denied", tool: string): Promise<void>;
     private handleRequest;
     private handleCallbackQuery;
-    private sessionKey;
     private extractFiles;
 }
 

package/dist/index.js

@@ -1218,7 +1218,6 @@ var TelegramAdapter = class {
   handler;
   resetHandler;
   approvalDecisionHandler;
-  sessionCounters = /* @__PURE__ */ new Map();
   approvalMessageIds = /* @__PURE__ */ new Map();
   lastUpdateId = 0;
   constructor(options = {}) {
@@ -1436,18 +1435,15 @@ ${inputSummary}` : "(no input)"
         res.end();
         return;
       }
-      const key2 = this.sessionKey(message);
-      const current = this.sessionCounters.get(key2) ?? 0;
-      this.sessionCounters.set(key2, current + 1);
       res.writeHead(200);
       res.end();
       if (this.resetHandler) {
         const topicId2 = message.message_thread_id;
-        const prevThreadId = topicId2 ? `${chatId}:${topicId2}:${current}` : `${chatId}:${current}`;
+        const threadId = topicId2 ? `${chatId}:${topicId2}:0` : `${chatId}:0`;
         try {
           await this.resetHandler("telegram", {
             channelId: chatId,
-            platformThreadId: prevThreadId
+            platformThreadId: threadId
           });
         } catch (err) {
           console.error("[telegram-adapter] reset handler error:", err instanceof Error ? err.message : err);
@@ -1481,10 +1477,8 @@ ${inputSummary}` : "(no input)"
     res.end();
     if (!this.handler) return;
     const files = await this.extractFiles(message);
-    const key = this.sessionKey(message);
-    const session = this.sessionCounters.get(key) ?? 0;
     const topicId = message.message_thread_id;
-    const platformThreadId = topicId ? `${chatId}:${topicId}:${session}` : `${chatId}:${session}`;
+    const platformThreadId = topicId ? `${chatId}:${topicId}:0` : `${chatId}:0`;
     const userId = String(message.from?.id ?? "unknown");
     const userName = [message.from?.first_name, message.from?.last_name].filter(Boolean).join(" ") || void 0;
     const ponchoMessage = {
@@ -1532,6 +1526,7 @@ ${inputSummary}` : "(no input)"
     const chatId = query.message?.chat.id ? String(query.message.chat.id) : void 0;
     await answerCallbackQuery(this.botToken, query.id, {
       text: approved ? "Approved" : "Denied"
+    }).catch(() => {
     });
     if (this.approvalDecisionHandler && chatId) {
       await this.approvalDecisionHandler(approvalId, approved, chatId);
@@ -1540,10 +1535,6 @@ ${inputSummary}` : "(no input)"
   // -----------------------------------------------------------------------
   // Helpers
   // -----------------------------------------------------------------------
-  sessionKey(message) {
-    const chatId = String(message.chat.id);
-    return message.message_thread_id ? `${chatId}:${message.message_thread_id}` : chatId;
-  }
   async extractFiles(message) {
     const files = [];
     try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/messaging",
-  "version": "0.7.7",
+  "version": "0.7.9",
   "description": "Messaging platform adapters for Poncho agents (Slack, Telegram, etc.)",
   "repository": {
     "type": "git",

package/src/adapters/telegram/index.ts

@@ -85,7 +85,6 @@ export class TelegramAdapter implements MessagingAdapter {
   private handler: IncomingMessageHandler | undefined;
   private resetHandler: ResetHandler | undefined;
   private approvalDecisionHandler: TelegramApprovalDecisionHandler | undefined;
-  private readonly sessionCounters = new Map<string, number>();
   private readonly approvalMessageIds = new Map<string, { chatId: string; messageId: number }>();
   private lastUpdateId = 0;
 
@@ -374,25 +373,19 @@ export class TelegramAdapter implements MessagingAdapter {
         return;
       }
 
-      const key = this.sessionKey(message);
-      const current = this.sessionCounters.get(key) ?? 0;
-      this.sessionCounters.set(key, current + 1);
-
       res.writeHead(200);
       res.end();
 
-      // Persist the reset so it survives serverless cold starts.
-      // The in-memory counter handles long-running processes; the bridge
-      // clears messages in the conversation store for serverless.
+      // Clear conversation in the store so the next message starts fresh.
       if (this.resetHandler) {
         const topicId = message.message_thread_id;
-        const prevThreadId = topicId
-          ? `${chatId}:${topicId}:${current}`
-          : `${chatId}:${current}`;
+        const threadId = topicId
+          ? `${chatId}:${topicId}:0`
+          : `${chatId}:0`;
         try {
           await this.resetHandler("telegram", {
             channelId: chatId,
-            platformThreadId: prevThreadId,
+            platformThreadId: threadId,
           });
         } catch (err) {
           console.error("[telegram-adapter] reset handler error:", err instanceof Error ? err.message : err);
@@ -440,12 +433,12 @@ export class TelegramAdapter implements MessagingAdapter {
     const files = await this.extractFiles(message);
 
     // -- Build thread ref -------------------------------------------------
-    const key = this.sessionKey(message);
-    const session = this.sessionCounters.get(key) ?? 0;
+    // Always use a fixed session component so the conversationId is stable
+    // across serverless cold starts. /new resets via the store instead.
     const topicId = message.message_thread_id;
     const platformThreadId = topicId
-      ? `${chatId}:${topicId}:${session}`
-      : `${chatId}:${session}`;
+      ? `${chatId}:${topicId}:0`
+      : `${chatId}:0`;
 
     const userId = String(message.from?.id ?? "unknown");
     const userName =
@@ -509,7 +502,7 @@ export class TelegramAdapter implements MessagingAdapter {
 
     await answerCallbackQuery(this.botToken, query.id, {
       text: approved ? "Approved" : "Denied",
-    });
+    }).catch(() => {});
 
     if (this.approvalDecisionHandler && chatId) {
       await this.approvalDecisionHandler(approvalId, approved, chatId);
@@ -520,13 +513,6 @@ export class TelegramAdapter implements MessagingAdapter {
   // Helpers
   // -----------------------------------------------------------------------
 
-  private sessionKey(message: TelegramMessage): string {
-    const chatId = String(message.chat.id);
-    return message.message_thread_id
-      ? `${chatId}:${message.message_thread_id}`
-      : chatId;
-  }
-
   private async extractFiles(
     message: TelegramMessage,
   ): Promise<FileAttachment[]> {