pullfrog 0.1.87 → 0.1.89

package/dist/cli.mjs

@@ -107103,7 +107103,7 @@ var import_picocolors = __toESM(require_picocolors(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.87",
+  version: "0.1.89",
   cliContractVersion: "1",
   type: "module",
   bin: {
@@ -111130,6 +111130,11 @@ import { spawnSync as spawnSync2 } from "node:child_process";
 import { chmodSync, createWriteStream, existsSync as existsSync3, mkdirSync } from "node:fs";
 import { join as join2 } from "node:path";
 import { pipeline } from "node:stream/promises";
+function resolveNpm() {
+  const npmCli = process.env.npm_execpath;
+  if (npmCli?.endsWith("npm-cli.js")) return { command: process.execPath, args: [npmCli] };
+  return { command: "npm", args: [] };
+}
 async function installFromNpmTarball(params) {
   const tempDir = process.env.PULLFROG_TEMP_DIR;
   if (!tempDir) throw new Error("PULLFROG_TEMP_DIR is not set");
@@ -111192,7 +111197,8 @@ async function installFromNpmTarball(params) {
   }
   if (params.installDependencies) {
     log2.debug(`\xBB installing dependencies for ${params.packageName}...`);
-    const installResult = spawnSync2("npm", ["install", "--production"], {
+    const npm2 = resolveNpm();
+    const installResult = spawnSync2(npm2.command, [...npm2.args, "install", "--production"], {
       cwd: extractedDir,
       stdio: "pipe",
       encoding: "utf-8",
@@ -186308,9 +186314,8 @@ function jitter(delayMs) {
 var DEFAULT_RETRY_AFTER_CAP_MS = 2e4;
 function getErrorResponseHeaders(error49) {
   if (error49 === null || typeof error49 !== "object") return void 0;
-  const response = error49.response;
-  if (response === null || typeof response !== "object") return void 0;
-  const headers = response.headers;
+  const source = error49;
+  const headers = source.response?.headers ?? source.headers;
   if (headers === null || typeof headers !== "object") return void 0;
   return headers;
 }
@@ -192410,7 +192415,7 @@ async function countOutstandingPullfrogThreads(ctx, pullNumber) {
 // utils/isTransientNetworkError.ts
 function isTransientNetworkError(error49, extraPatterns = []) {
   if (!(error49 instanceof Error)) return false;
-  if (error49.name === "AbortError") return true;
+  if (error49.name === "AbortError" || error49.name === "TimeoutError") return true;
   const patterns = ["fetch failed", "ECONNRESET", "ETIMEDOUT", ...extraPatterns];
   return patterns.some((p) => error49.message.includes(p));
 }
@@ -200099,6 +200104,23 @@ ${input.raw}
 \`\`\``
   ].join("\n");
 }
+function isRunTimeoutError(message) {
+  return message.startsWith("agent run timed out after");
+}
+function formatRunTimeoutSummary(input) {
+  const settingsUrl = `${getApiUrl()}/console/${input.owner}/${input.name}`;
+  return [
+    "**This run hit its time limit while still working, so it stopped before finishing.**",
+    "",
+    "Give runs longer with `--timeout=2h` (or `--notimeout`) in this repo's instructions \u2014 or raise the workflow's `timeout` input if it sets one, since that input wins. A smaller PR or a lower effort also finishes sooner.",
+    "",
+    `[Repo settings \u2192](${settingsUrl}) \xB7 [Flags \u2192](https://docs.pullfrog.com/flags)`,
+    "",
+    `\`\`\`
+${input.raw}
+\`\`\``
+  ].join("\n");
+}
 function formatContextOverflowSummary(input) {
   const settingsUrl = `${getApiUrl()}/console/${input.owner}/${input.name}`;
   return [
@@ -200348,6 +200370,16 @@ ${body}`, comment: body };
     });
     return { summary: `### \u274C Pullfrog failed
 
+${body}`, comment: body };
+  }
+  if (isRunTimeoutError(input.errorMessage)) {
+    const body = formatRunTimeoutSummary({
+      owner: input.repo.owner,
+      name: input.repo.name,
+      raw: input.errorMessage
+    });
+    return { summary: `### \u274C Pullfrog failed
+
 ${body}`, comment: body };
   }
   if (isTransientUpstreamError(input.errorMessage)) {
@@ -202193,7 +202225,7 @@ function resolveTarget2(repo) {
   return { owner: match3[1], repo: match3[2] };
 }
 function cliVersion() {
-  const parts = /^(\d+)\.(\d+)\.(\d+)$/.exec("0.1.87");
+  const parts = /^(\d+)\.(\d+)\.(\d+)$/.exec("0.1.89");
   if (!parts) return "0.0.0";
   return `${Number(parts[1])}.${Number(parts[2])}.${Number(parts[3])}`;
 }
@@ -202360,7 +202392,7 @@ async function runCli8(input) {
 }
 
 // cli.ts
-var VERSION10 = "0.1.87";
+var VERSION10 = "0.1.89";
 var bin = basename2(process.argv[1] || "");
 var PROG = bin === "pf" || bin === "pullfrog" ? bin : "pullfrog";
 var rawArgs = process.argv.slice(2);

package/dist/index.js

@@ -105416,6 +105416,11 @@ import { spawnSync } from "node:child_process";
 import { chmodSync, createWriteStream, existsSync as existsSync2, mkdirSync } from "node:fs";
 import { join } from "node:path";
 import { pipeline } from "node:stream/promises";
+function resolveNpm() {
+  const npmCli = process.env.npm_execpath;
+  if (npmCli?.endsWith("npm-cli.js")) return { command: process.execPath, args: [npmCli] };
+  return { command: "npm", args: [] };
+}
 async function installFromNpmTarball(params) {
   const tempDir = process.env.PULLFROG_TEMP_DIR;
   if (!tempDir) throw new Error("PULLFROG_TEMP_DIR is not set");
@@ -105478,7 +105483,8 @@ async function installFromNpmTarball(params) {
   }
   if (params.installDependencies) {
     log.debug(`\xBB installing dependencies for ${params.packageName}...`);
-    const installResult = spawnSync("npm", ["install", "--production"], {
+    const npm2 = resolveNpm();
+    const installResult = spawnSync(npm2.command, [...npm2.args, "install", "--production"], {
       cwd: extractedDir,
       stdio: "pipe",
       encoding: "utf-8",
@@ -105723,7 +105729,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.87",
+  version: "0.1.89",
   cliContractVersion: "1",
   type: "module",
   bin: {
@@ -182801,9 +182807,8 @@ function jitter(delayMs) {
 var DEFAULT_RETRY_AFTER_CAP_MS = 2e4;
 function getErrorResponseHeaders(error49) {
   if (error49 === null || typeof error49 !== "object") return void 0;
-  const response = error49.response;
-  if (response === null || typeof response !== "object") return void 0;
-  const headers = response.headers;
+  const source = error49;
+  const headers = source.response?.headers ?? source.headers;
   if (headers === null || typeof headers !== "object") return void 0;
   return headers;
 }
@@ -188903,7 +188908,7 @@ async function countOutstandingPullfrogThreads(ctx, pullNumber) {
 // utils/isTransientNetworkError.ts
 function isTransientNetworkError(error49, extraPatterns = []) {
   if (!(error49 instanceof Error)) return false;
-  if (error49.name === "AbortError") return true;
+  if (error49.name === "AbortError" || error49.name === "TimeoutError") return true;
   const patterns = ["fetch failed", "ECONNRESET", "ETIMEDOUT", ...extraPatterns];
   return patterns.some((p) => error49.message.includes(p));
 }
@@ -196592,6 +196597,23 @@ ${input.raw}
 \`\`\``
   ].join("\n");
 }
+function isRunTimeoutError(message) {
+  return message.startsWith("agent run timed out after");
+}
+function formatRunTimeoutSummary(input) {
+  const settingsUrl = `${getApiUrl()}/console/${input.owner}/${input.name}`;
+  return [
+    "**This run hit its time limit while still working, so it stopped before finishing.**",
+    "",
+    "Give runs longer with `--timeout=2h` (or `--notimeout`) in this repo's instructions \u2014 or raise the workflow's `timeout` input if it sets one, since that input wins. A smaller PR or a lower effort also finishes sooner.",
+    "",
+    `[Repo settings \u2192](${settingsUrl}) \xB7 [Flags \u2192](https://docs.pullfrog.com/flags)`,
+    "",
+    `\`\`\`
+${input.raw}
+\`\`\``
+  ].join("\n");
+}
 function formatContextOverflowSummary(input) {
   const settingsUrl = `${getApiUrl()}/console/${input.owner}/${input.name}`;
   return [
@@ -196841,6 +196863,16 @@ ${body}`, comment: body };
     });
     return { summary: `### \u274C Pullfrog failed
 
+${body}`, comment: body };
+  }
+  if (isRunTimeoutError(input.errorMessage)) {
+    const body = formatRunTimeoutSummary({
+      owner: input.repo.owner,
+      name: input.repo.name,
+      raw: input.errorMessage
+    });
+    return { summary: `### \u274C Pullfrog failed
+
 ${body}`, comment: body };
   }
   if (isTransientUpstreamError(input.errorMessage)) {

package/dist/internal.js

@@ -8,7 +8,7 @@ var __export = (target, all) => {
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.87",
+  version: "0.1.89",
   cliContractVersion: "1",
   type: "module",
   bin: {

package/dist/utils/isTransientNetworkError.d.ts

@@ -3,7 +3,8 @@
  * on explicit HTTP status / validation errors that carry their own message).
  *
  * Matches errors from `fetch` (`fetch failed`), TCP-level interruption
- * (`ECONNRESET`, `ETIMEDOUT`), and `AbortSignal.timeout` (`AbortError`).
+ * (`ECONNRESET`, `ETIMEDOUT`), and aborts — `AbortSignal.timeout` rejects with
+ * `TimeoutError`, a manual abort with `AbortError` (#1354).
  * Callers may pass `extraPatterns` to whitelist additional transient
  * substrings on `error.message` (e.g. provider-specific framings).
  */

package/dist/utils/runErrorRenderer.d.ts

@@ -40,6 +40,9 @@
  *      `maximum context length is N tokens`. Actionable, so it renders on
  *      both surfaces rather than collapsing to the one-line comment.
  *
+ *   4b'. Run time limit — the agent was still working when the run's own
+ *      timeout fired. Actionable (raise the limit), so both surfaces.
+ *
  *   4c. Transient upstream failure (#1173) — Anthropic `529 Overloaded`,
  *      OpenRouter `provider_unavailable` / `timeout`, Zen `Streaming response
  *      failed: [5xx]`. Last of the classified branches so every more specific
@@ -63,8 +66,8 @@
  *      the hang case, since the raw internal string helps nobody on the PR.
  *
  * Net: the actionable classifications (billing, API-key, model-not-found,
- * no-provider-available, context-overflow, transient-upstream) render identical
- * bodies on both surfaces; the non-actionable ones (unexplained hang,
+ * no-provider-available, context-overflow, run time limit, transient-upstream)
+ * render identical bodies on both surfaces; the non-actionable ones (unexplained hang,
  * generic) keep the forensics in the Actions job summary and show a calm
  * one-liner in the PR comment, whose footer already carries Pullfrog
  * branding + rerun links.

package/dist/yes/index.d.ts

@@ -6,7 +6,8 @@ export declare function range(n: number, fn: (i: number) => number): number[];
 export declare function schedule(fn: (i: number) => number, count: number): number[];
 /**
  * read a retry delay (ms) from a thrown HTTP error's response headers, honoring
- * GitHub's rate-limit backoff hints on octokit errors (`error.response.headers`):
+ * GitHub's rate-limit backoff hints on octokit errors (`error.response.headers`, or
+ * `error.headers` on a GraphQL response error):
  *   - `retry-after` (RFC 9110 delta-seconds or an HTTP-date) — an explicit
  *     backoff directive whenever present (covers secondary rate limits, which
  *     carry it with `x-ratelimit-remaining > 0`).

package/dist/yes.js

@@ -1034,9 +1034,8 @@ function jitter(delayMs) {
 var DEFAULT_RETRY_AFTER_CAP_MS = 2e4;
 function getErrorResponseHeaders(error) {
   if (error === null || typeof error !== "object") return void 0;
-  const response = error.response;
-  if (response === null || typeof response !== "object") return void 0;
-  const headers = response.headers;
+  const source = error;
+  const headers = source.response?.headers ?? source.headers;
   if (headers === null || typeof headers !== "object") return void 0;
   return headers;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pullfrog",
-  "version": "0.1.87",
+  "version": "0.1.89",
   "cliContractVersion": "1",
   "type": "module",
   "bin": {