@poncho-ai/cli 0.33.2 → 0.33.4

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/cli@0.33.2 build /home/runner/work/poncho-ai/poncho-ai/packages/cli
+> @poncho-ai/cli@0.33.4 build /home/runner/work/poncho-ai/poncho-ai/packages/cli
 > tsup src/index.ts src/cli.ts --format esm --dts
 
 CLI Building entry: src/cli.ts, src/index.ts
@@ -9,10 +9,10 @@
 ESM Build start
 ESM dist/cli.js                          94.00 B
 ESM dist/index.js                        917.00 B
-ESM dist/run-interactive-ink-VS35YSBB.js 56.86 KB
-ESM dist/chunk-QAUWCAWU.js               528.49 KB
-ESM ⚡️ Build success in 67ms
+ESM dist/run-interactive-ink-JO343QM2.js 56.86 KB
+ESM dist/chunk-QVCJIXPQ.js               529.65 KB
+ESM ⚡️ Build success in 75ms
 DTS Build start
-DTS ⚡️ Build success in 4144ms
+DTS ⚡️ Build success in 4620ms
 DTS dist/cli.d.ts   20.00 B
 DTS dist/index.d.ts 7.07 KB

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @poncho-ai/cli
 
+## 0.33.4
+
+### Patch Changes
+
+- [#73](https://github.com/cesr/poncho-ai/pull/73) [`f72f202`](https://github.com/cesr/poncho-ai/commit/f72f202d839dbbb8240336ec76eb6340aba20f06) Thanks [@cesr](https://github.com/cesr)! - Fix Telegram approval message ordering: send accumulated assistant text before approval buttons so the conversation reads naturally. Skip empty bridge replies when text was already sent at checkpoint.
+
+- Updated dependencies [[`f72f202`](https://github.com/cesr/poncho-ai/commit/f72f202d839dbbb8240336ec76eb6340aba20f06)]:
+  - @poncho-ai/messaging@0.7.10
+
+## 0.33.3
+
+### 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.
+
+- Updated dependencies [[`3e5bf7e`](https://github.com/cesr/poncho-ai/commit/3e5bf7e527e394c5f823beac90712756e57cd491)]:
+  - @poncho-ai/messaging@0.7.9
+
 ## 0.33.2
 
 ### Patch Changes

package/dist/{chunk-QAUWCAWU.js → chunk-QVCJIXPQ.js}

@@ -10178,6 +10178,7 @@ ${resultBody}`,
       let latestRunId = "";
       const draft = createTurnDraftState();
       let checkpointedRun = false;
+      let checkpointTextAlreadySent = false;
       let runContextTokens = 0;
       let runContextWindow = 0;
       let runContinuation2 = false;
@@ -10268,6 +10269,19 @@ ${resultBody}`,
               if (conv?.channelMeta?.platform === "telegram") {
                 const tgAdapter = messagingAdapters.get("telegram");
                 if (tgAdapter) {
+                  const threadRef = {
+                    channelId: conv.channelMeta.channelId,
+                    platformThreadId: conv.channelMeta.platformThreadId
+                  };
+                  const pendingText = draft.assistantResponse.trim();
+                  if (pendingText) {
+                    try {
+                      await tgAdapter.sendReply(threadRef, pendingText);
+                      checkpointTextAlreadySent = true;
+                    } catch (err) {
+                      console.error("[messaging-runner] failed to send pre-approval text:", err instanceof Error ? err.message : err);
+                    }
+                  }
                   const approvals = event.approvals.map((a) => ({
                     approvalId: a.approvalId,
                     tool: a.tool,
@@ -10351,8 +10365,8 @@ ${resultBody}`,
         runOwners.delete(latestRunId);
         runConversations.delete(latestRunId);
       }
-      console.log("[messaging-runner] run complete, response length:", draft.assistantResponse.length, runContinuation2 ? "(continuation)" : "");
-      const response = draft.assistantResponse;
+      const response = checkpointTextAlreadySent ? "" : draft.assistantResponse;
+      console.log("[messaging-runner] run complete, response length:", response.length, checkpointTextAlreadySent ? "(text sent at checkpoint)" : "", runContinuation2 ? "(continuation)" : "");
       return {
         response,
         continuation: runContinuation2,
@@ -10905,26 +10919,30 @@ ${resultBody}`,
           }
           const found = await findPendingApproval(approvalId, "local-owner");
           let foundConversation = found?.conversation;
-          let foundApproval = found?.approval;
+          const foundApproval = found?.approval;
           if (!foundConversation || !foundApproval) {
             console.warn("[telegram-approval] approval not found:", approvalId);
             return;
           }
-          foundApproval = normalizeApprovalCheckpoint(foundApproval, foundConversation.messages);
-          await adapter.updateApprovalMessage(approvalId, approved ? "approved" : "denied", foundApproval.tool);
-          foundApproval.decision = approved ? "approved" : "denied";
+          const approvalDecision = approved ? "approved" : "denied";
+          await adapter.updateApprovalMessage(approvalId, approvalDecision, foundApproval.tool);
+          foundConversation.pendingApprovals = (foundConversation.pendingApprovals ?? []).map(
+            (approval) => approval.approvalId === approvalId ? { ...normalizeApprovalCheckpoint(approval, foundConversation.messages), decision: approvalDecision } : normalizeApprovalCheckpoint(approval, foundConversation.messages)
+          );
+          await conversationStore.update(foundConversation);
           broadcastEvent(
             foundConversation.conversationId,
             approved ? { type: "tool:approval:granted", approvalId } : { type: "tool:approval:denied", approvalId }
           );
-          const allApprovals = (foundConversation.pendingApprovals ?? []).map(
-            (approval) => normalizeApprovalCheckpoint(approval, foundConversation.messages)
+          const refreshedConversation = await conversationStore.get(foundConversation.conversationId);
+          const allApprovals = (refreshedConversation?.pendingApprovals ?? []).map(
+            (approval) => normalizeApprovalCheckpoint(approval, refreshedConversation.messages)
           );
           const allDecided = allApprovals.length > 0 && allApprovals.every((a) => a.decision != null);
           if (!allDecided) {
-            await conversationStore.update(foundConversation);
             return;
           }
+          foundConversation = refreshedConversation;
           const conversationId = foundConversation.conversationId;
           const checkpointRef = allApprovals[0];
           foundConversation.pendingApprovals = [];
@@ -13071,7 +13089,7 @@ var runInteractive = async (workingDir, params) => {
   await harness.initialize();
   const identity = await ensureAgentIdentity2(workingDir);
   try {
-    const { runInteractiveInk } = await import("./run-interactive-ink-VS35YSBB.js");
+    const { runInteractiveInk } = await import("./run-interactive-ink-JO343QM2.js");
     await runInteractiveInk({
       harness,
       params,

package/dist/cli.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   main
-} from "./chunk-QAUWCAWU.js";
+} from "./chunk-QVCJIXPQ.js";
 
 // src/cli.ts
 void main();

package/dist/index.js

@@ -24,7 +24,7 @@ import {
   runTests,
   startDevServer,
   updateAgentGuidance
-} from "./chunk-QAUWCAWU.js";
+} from "./chunk-QVCJIXPQ.js";
 export {
   __internalRunOrchestration,
   addSkill,

package/dist/{run-interactive-ink-VS35YSBB.js → run-interactive-ink-JO343QM2.js}

@@ -2,7 +2,7 @@ import {
   consumeFirstRunIntro,
   inferConversationTitle,
   resolveHarnessEnvironment
-} from "./chunk-QAUWCAWU.js";
+} from "./chunk-QVCJIXPQ.js";
 
 // src/run-interactive-ink.ts
 import * as readline from "readline";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/cli",
-  "version": "0.33.2",
+  "version": "0.33.4",
   "description": "CLI for building and deploying AI agents",
   "repository": {
     "type": "git",
@@ -27,8 +27,8 @@
     "react": "^19.2.4",
     "react-devtools-core": "^6.1.5",
     "yaml": "^2.8.1",
+    "@poncho-ai/messaging": "0.7.10",
     "@poncho-ai/harness": "0.32.1",
-    "@poncho-ai/messaging": "0.7.8",
     "@poncho-ai/sdk": "1.7.1"
   },
   "devDependencies": {

package/src/index.ts

@@ -3385,6 +3385,7 @@ export const createRequestHandler = async (options?: {
       let latestRunId = "";
       const draft = createTurnDraftState();
       let checkpointedRun = false;
+      let checkpointTextAlreadySent = false;
       let runContextTokens = 0;
       let runContextWindow = 0;
       let runContinuation = false;
@@ -3479,11 +3480,27 @@ export const createRequestHandler = async (options?: {
             });
             checkpointedRun = true;
 
-            // Send inline keyboard approval buttons to Telegram
             const conv = await conversationStore.get(conversationId);
             if (conv?.channelMeta?.platform === "telegram") {
               const tgAdapter = messagingAdapters.get("telegram") as TelegramAdapter | undefined;
               if (tgAdapter) {
+                const threadRef: import("@poncho-ai/messaging").ThreadRef = {
+                  channelId: conv.channelMeta.channelId,
+                  platformThreadId: conv.channelMeta.platformThreadId,
+                };
+
+                // Send accumulated text BEFORE approval buttons so Telegram
+                // shows them in the natural order (text → approval request).
+                const pendingText = draft.assistantResponse.trim();
+                if (pendingText) {
+                  try {
+                    await tgAdapter.sendReply(threadRef, pendingText);
+                    checkpointTextAlreadySent = true;
+                  } catch (err: unknown) {
+                    console.error("[messaging-runner] failed to send pre-approval text:", err instanceof Error ? err.message : err);
+                  }
+                }
+
                 const approvals = event.approvals.map(a => ({
                   approvalId: a.approvalId,
                   tool: a.tool,
@@ -3571,8 +3588,8 @@ export const createRequestHandler = async (options?: {
         runConversations.delete(latestRunId);
       }
 
-      console.log("[messaging-runner] run complete, response length:", draft.assistantResponse.length, runContinuation ? "(continuation)" : "");
-      const response = draft.assistantResponse;
+      const response = checkpointTextAlreadySent ? "" : draft.assistantResponse;
+      console.log("[messaging-runner] run complete, response length:", response.length, checkpointTextAlreadySent ? "(text sent at checkpoint)" : "", runContinuation ? "(continuation)" : "");
 
       return {
         response,
@@ -4188,17 +4205,22 @@ export const createRequestHandler = async (options?: {
           // Regular (non-subagent) approval
           const found = await findPendingApproval(approvalId, "local-owner");
           let foundConversation = found?.conversation;
-          let foundApproval = found?.approval;
+          const foundApproval = found?.approval;
 
           if (!foundConversation || !foundApproval) {
             console.warn("[telegram-approval] approval not found:", approvalId);
             return;
           }
-          foundApproval = normalizeApprovalCheckpoint(foundApproval, foundConversation.messages);
 
-          await adapter.updateApprovalMessage(approvalId, approved ? "approved" : "denied", foundApproval.tool);
+          const approvalDecision = approved ? "approved" as const : "denied" as const;
+          await adapter.updateApprovalMessage(approvalId, approvalDecision, foundApproval.tool);
 
-          foundApproval.decision = approved ? "approved" : "denied";
+          foundConversation.pendingApprovals = (foundConversation.pendingApprovals ?? []).map((approval) =>
+            approval.approvalId === approvalId
+              ? { ...normalizeApprovalCheckpoint(approval, foundConversation!.messages), decision: approvalDecision }
+              : normalizeApprovalCheckpoint(approval, foundConversation!.messages),
+          );
+          await conversationStore.update(foundConversation);
 
           broadcastEvent(foundConversation.conversationId,
             approved
@@ -4206,15 +4228,16 @@ export const createRequestHandler = async (options?: {
               : { type: "tool:approval:denied", approvalId },
           );
 
-          const allApprovals = (foundConversation.pendingApprovals ?? []).map((approval) =>
-            normalizeApprovalCheckpoint(approval, foundConversation!.messages),
+          const refreshedConversation = await conversationStore.get(foundConversation.conversationId);
+          const allApprovals = (refreshedConversation?.pendingApprovals ?? []).map((approval) =>
+            normalizeApprovalCheckpoint(approval, refreshedConversation!.messages),
           );
           const allDecided = allApprovals.length > 0 && allApprovals.every(a => a.decision != null);
 
           if (!allDecided) {
-            await conversationStore.update(foundConversation);
             return;
           }
+          foundConversation = refreshedConversation!;
 
           // All decided — resume the run
           const conversationId = foundConversation.conversationId;