@poncho-ai/cli 0.20.2 → 0.21.0

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/cli@0.20.2 build /home/runner/work/poncho-ai/poncho-ai/packages/cli
+> @poncho-ai/cli@0.21.0 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                        857.00 B
-ESM dist/run-interactive-ink-2YFB7AEJ.js 55.30 KB
-ESM dist/chunk-VRLHV23J.js               384.02 KB
-ESM ⚡️ Build success in 62ms
+ESM dist/run-interactive-ink-H55INI7H.js 55.30 KB
+ESM dist/chunk-54WK2ATG.js               384.68 KB
+ESM ⚡️ Build success in 57ms
 DTS Build start
-DTS ⚡️ Build success in 3793ms
+DTS ⚡️ Build success in 3701ms
 DTS dist/cli.d.ts   20.00 B
 DTS dist/index.d.ts 3.59 KB

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @poncho-ai/cli
 
+## 0.21.0
+
+### Minor Changes
+
+- [`c5d94c5`](https://github.com/cesr/poncho-ai/commit/c5d94c5854081ac23e91f7104047354ad1a415ef) Thanks [@cesr](https://github.com/cesr)! - Automatically add `@vercel/functions` to project dependencies when running `poncho build vercel`. This ensures `waitUntil` is available for keeping serverless functions alive during webhook processing and approval resume.
+
+## 0.20.3
+
+### Patch Changes
+
+- [`9b38474`](https://github.com/cesr/poncho-ai/commit/9b38474a25bfb54cec108d1d3eae664aaed37ccf) Thanks [@cesr](https://github.com/cesr)! - Make approval resume robust: wrap in try/catch to always clear runStatus, and await the work when waitUntil is unavailable so the resume completes before the response is sent.
+
 ## 0.20.2
 
 ### Patch Changes

package/dist/{chunk-VRLHV23J.js → chunk-54WK2ATG.js}

@@ -6862,7 +6862,7 @@ var UPLOAD_PROVIDER_DEPS = {
     { name: "@aws-sdk/s3-request-presigner", fallback: "^3.700.0" }
   ]
 };
-var ensureRuntimeCliDependency = async (projectDir, cliVersion, config) => {
+var ensureRuntimeCliDependency = async (projectDir, cliVersion, config, target) => {
   const packageJsonPath = resolve3(projectDir, "package.json");
   const content = await readFile3(packageJsonPath, "utf8");
   const parsed = JSON.parse(content);
@@ -6886,6 +6886,10 @@ var ensureRuntimeCliDependency = async (projectDir, cliVersion, config) => {
       }
     }
   }
+  if (target === "vercel" && !dependencies["@vercel/functions"]) {
+    dependencies["@vercel/functions"] = "^1.0.0";
+    addedDeps.push("@vercel/functions");
+  }
   parsed.dependencies = dependencies;
   await writeFile3(packageJsonPath, `${JSON.stringify(parsed, null, 2)}
 `, "utf8");
@@ -7094,7 +7098,8 @@ CMD ["node","server.js"]
   const { paths: packagePaths, addedDeps } = await ensureRuntimeCliDependency(
     projectDir,
     cliVersion,
-    config
+    config,
+    target
   );
   const depNote = addedDeps.length > 0 ? ` (added ${addedDeps.join(", ")})` : "";
   for (const p of packagePaths) {
@@ -8608,48 +8613,60 @@ data: ${JSON.stringify(data)}
       await conversationStore.update(foundConversation);
       const checkpointRef = allApprovals[0];
       const resumeWork = (async () => {
-        const toolContext = {
-          runId: checkpointRef.runId,
-          agentId: identity.id,
-          step: 0,
-          workingDir,
-          parameters: {}
-        };
-        const approvalToolCallIds = new Set(allApprovals.map((a) => a.toolCallId));
-        const callsToExecute = [];
-        const deniedResults = [];
-        for (const a of allApprovals) {
-          if (a.decision === "approved" && a.toolCallId) {
-            callsToExecute.push({ id: a.toolCallId, name: a.tool, input: a.input });
-          } else if (a.decision === "denied" && a.toolCallId) {
-            deniedResults.push({ callId: a.toolCallId, toolName: a.tool, error: "Tool execution denied by user" });
-          }
-        }
-        const pendingToolCalls = checkpointRef.pendingToolCalls ?? [];
-        for (const tc of pendingToolCalls) {
-          if (!approvalToolCallIds.has(tc.id)) {
-            callsToExecute.push(tc);
-          }
-        }
-        let toolResults = [...deniedResults];
-        if (callsToExecute.length > 0) {
-          const execResults = await harness.executeTools(callsToExecute, toolContext);
-          toolResults.push(...execResults.map((r) => ({
-            callId: r.callId,
-            toolName: r.tool,
-            result: r.output,
-            error: r.error
-          })));
-        }
-        await resumeRunFromCheckpoint(
-          conversationId,
-          foundConversation,
-          checkpointRef,
-          toolResults
-        );
+        try {
+          const toolContext = {
+            runId: checkpointRef.runId,
+            agentId: identity.id,
+            step: 0,
+            workingDir,
+            parameters: {}
+          };
+          const approvalToolCallIds = new Set(allApprovals.map((a) => a.toolCallId));
+          const callsToExecute = [];
+          const deniedResults = [];
+          for (const a of allApprovals) {
+            if (a.decision === "approved" && a.toolCallId) {
+              callsToExecute.push({ id: a.toolCallId, name: a.tool, input: a.input });
+            } else if (a.decision === "denied" && a.toolCallId) {
+              deniedResults.push({ callId: a.toolCallId, toolName: a.tool, error: "Tool execution denied by user" });
+            }
+          }
+          const pendingToolCalls = checkpointRef.pendingToolCalls ?? [];
+          for (const tc of pendingToolCalls) {
+            if (!approvalToolCallIds.has(tc.id)) {
+              callsToExecute.push(tc);
+            }
+          }
+          let toolResults = [...deniedResults];
+          if (callsToExecute.length > 0) {
+            const execResults = await harness.executeTools(callsToExecute, toolContext);
+            toolResults.push(...execResults.map((r) => ({
+              callId: r.callId,
+              toolName: r.tool,
+              result: r.output,
+              error: r.error
+            })));
+          }
+          await resumeRunFromCheckpoint(
+            conversationId,
+            foundConversation,
+            checkpointRef,
+            toolResults
+          );
+        } catch (err) {
+          console.error("[approval-resume] failed:", err instanceof Error ? err.message : err);
+          const conv = await conversationStore.get(conversationId);
+          if (conv) {
+            conv.runStatus = "idle";
+            conv.updatedAt = Date.now();
+            await conversationStore.update(conv);
+          }
+        }
       })();
       if (waitUntilHook) {
         waitUntilHook(resumeWork);
+      } else {
+        await resumeWork;
       }
       writeJson(response, 200, { ok: true, approvalId, approved, batchComplete: true });
       return;
@@ -9679,7 +9696,7 @@ var runInteractive = async (workingDir, params) => {
   await harness.initialize();
   const identity = await ensureAgentIdentity2(workingDir);
   try {
-    const { runInteractiveInk } = await import("./run-interactive-ink-2YFB7AEJ.js");
+    const { runInteractiveInk } = await import("./run-interactive-ink-H55INI7H.js");
     await runInteractiveInk({
       harness,
       params,

package/dist/cli.js

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

package/dist/index.js

@@ -23,7 +23,7 @@ import {
   runTests,
   startDevServer,
   updateAgentGuidance
-} from "./chunk-VRLHV23J.js";
+} from "./chunk-54WK2ATG.js";
 export {
   addSkill,
   buildCli,

package/dist/{run-interactive-ink-2YFB7AEJ.js → run-interactive-ink-H55INI7H.js}

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/cli",
-  "version": "0.20.2",
+  "version": "0.21.0",
   "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/harness": "0.20.0",
     "@poncho-ai/messaging": "0.2.7",
+    "@poncho-ai/harness": "0.20.0",
     "@poncho-ai/sdk": "1.4.0"
   },
   "devDependencies": {

package/src/index.ts

@@ -889,6 +889,7 @@ const ensureRuntimeCliDependency = async (
   projectDir: string,
   cliVersion: string,
   config?: PonchoConfig,
+  target?: string,
 ): Promise<{ paths: string[]; addedDeps: string[] }> => {
   const packageJsonPath = resolve(projectDir, "package.json");
   const content = await readFile(packageJsonPath, "utf8");
@@ -922,6 +923,11 @@ const ensureRuntimeCliDependency = async (
     }
   }
 
+  if (target === "vercel" && !dependencies["@vercel/functions"]) {
+    dependencies["@vercel/functions"] = "^1.0.0";
+    addedDeps.push("@vercel/functions");
+  }
+
   parsed.dependencies = dependencies;
   await writeFile(packageJsonPath, `${JSON.stringify(parsed, null, 2)}\n`, "utf8");
   return { paths: [relative(projectDir, packageJsonPath)], addedDeps };
@@ -1137,6 +1143,7 @@ CMD ["node","server.js"]
     projectDir,
     cliVersion,
     config,
+    target,
   );
   const depNote = addedDeps.length > 0 ? ` (added ${addedDeps.join(", ")})` : "";
   for (const p of packagePaths) {
@@ -2909,55 +2916,67 @@ export const createRequestHandler = async (options?: {
       const checkpointRef = allApprovals[0]!;
 
       const resumeWork = (async () => {
-        const toolContext = {
-          runId: checkpointRef.runId,
-          agentId: identity.id,
-          step: 0,
-          workingDir,
-          parameters: {},
-        };
+        try {
+          const toolContext = {
+            runId: checkpointRef.runId,
+            agentId: identity.id,
+            step: 0,
+            workingDir,
+            parameters: {},
+          };
 
-        // Collect tool calls to execute: approved approval-gated tools + auto-approved deferred tools
-        const approvalToolCallIds = new Set(allApprovals.map(a => a.toolCallId));
-        const callsToExecute: Array<{ id: string; name: string; input: Record<string, unknown> }> = [];
-        const deniedResults: Array<{ callId: string; toolName: string; error: string }> = [];
+          // Collect tool calls to execute: approved approval-gated tools + auto-approved deferred tools
+          const approvalToolCallIds = new Set(allApprovals.map(a => a.toolCallId));
+          const callsToExecute: Array<{ id: string; name: string; input: Record<string, unknown> }> = [];
+          const deniedResults: Array<{ callId: string; toolName: string; error: string }> = [];
 
-        for (const a of allApprovals) {
-          if (a.decision === "approved" && a.toolCallId) {
-            callsToExecute.push({ id: a.toolCallId, name: a.tool, input: a.input });
-          } else if (a.decision === "denied" && a.toolCallId) {
-            deniedResults.push({ callId: a.toolCallId, toolName: a.tool, error: "Tool execution denied by user" });
+          for (const a of allApprovals) {
+            if (a.decision === "approved" && a.toolCallId) {
+              callsToExecute.push({ id: a.toolCallId, name: a.tool, input: a.input });
+            } else if (a.decision === "denied" && a.toolCallId) {
+              deniedResults.push({ callId: a.toolCallId, toolName: a.tool, error: "Tool execution denied by user" });
+            }
           }
-        }
 
-        // Auto-approved tools that were deferred alongside the approval-needing ones
-        const pendingToolCalls = checkpointRef.pendingToolCalls ?? [];
-        for (const tc of pendingToolCalls) {
-          if (!approvalToolCallIds.has(tc.id)) {
-            callsToExecute.push(tc);
+          // Auto-approved tools that were deferred alongside the approval-needing ones
+          const pendingToolCalls = checkpointRef.pendingToolCalls ?? [];
+          for (const tc of pendingToolCalls) {
+            if (!approvalToolCallIds.has(tc.id)) {
+              callsToExecute.push(tc);
+            }
           }
-        }
 
-        let toolResults: Array<{ callId: string; toolName: string; result?: unknown; error?: string }> = [...deniedResults];
-        if (callsToExecute.length > 0) {
-          const execResults = await harness.executeTools(callsToExecute, toolContext);
-          toolResults.push(...execResults.map(r => ({
-            callId: r.callId,
-            toolName: r.tool,
-            result: r.output,
-            error: r.error,
-          })));
-        }
+          let toolResults: Array<{ callId: string; toolName: string; result?: unknown; error?: string }> = [...deniedResults];
+          if (callsToExecute.length > 0) {
+            const execResults = await harness.executeTools(callsToExecute, toolContext);
+            toolResults.push(...execResults.map(r => ({
+              callId: r.callId,
+              toolName: r.tool,
+              result: r.output,
+              error: r.error,
+            })));
+          }
 
-        await resumeRunFromCheckpoint(
-          conversationId,
-          foundConversation!,
-          checkpointRef,
-          toolResults,
-        );
+          await resumeRunFromCheckpoint(
+            conversationId,
+            foundConversation!,
+            checkpointRef,
+            toolResults,
+          );
+        } catch (err) {
+          console.error("[approval-resume] failed:", err instanceof Error ? err.message : err);
+          const conv = await conversationStore.get(conversationId);
+          if (conv) {
+            conv.runStatus = "idle";
+            conv.updatedAt = Date.now();
+            await conversationStore.update(conv);
+          }
+        }
       })();
       if (waitUntilHook) {
         waitUntilHook(resumeWork);
+      } else {
+        await resumeWork;
       }
 
       writeJson(response, 200, { ok: true, approvalId, approved, batchComplete: true });