pullfrog 0.1.61 → 0.1.62

package/dist/cli.mjs

@@ -107293,6 +107293,9 @@ function isProviderNoRoutableEndpoints(text) {
 function isOpenRouterKeyLimitExceeded(text) {
   return /Key limit exceeded \(total limit\)/i.test(text) || /openrouter\.ai\/[^\s"')]*\/keys\//i.test(text);
 }
+function isProviderMissingCredential(text) {
+  return /API key is missing\. Pass it using the/i.test(text) || /Missing Authentication header/i.test(text);
+}
 function isTransientUpstreamError(text) {
   return /API Error:\s*5\d\d/i.test(text) || /\bOverloaded\b/.test(text) || /"error_type"\s*:\s*"(?:provider_unavailable|timeout)"/i.test(text) || /Streaming response failed:\s*\[5\d\d\]/i.test(text);
 }
@@ -107355,7 +107358,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.61",
+  version: "0.1.62",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -108446,7 +108449,6 @@ function getUnsubmittedReview(toolState, expectsReviewOutput2 = toolState.hadPro
 }
 function expectsReviewOutput(ctx) {
   if (ctx.toolState.hadProgressComment) return true;
-  if (ctx.payload.progressComments) return false;
   return ctx.payload.event.silent !== true && ctx.payload.event.issue_number !== void 0;
 }
 function buildStopHookPrompt(failure2) {
@@ -108661,6 +108663,7 @@ async function runPostRunRetryLoop(params) {
       result = preResume;
       break;
     }
+    if (onlySummaryStale) result = { ...result, output: preResume.output || result.output };
     gateResumeCount++;
   }
   if (gateResumeCount > 0 && result.success && hasPostRunIssues(finalIssues)) {
@@ -116005,24 +116008,23 @@ async function installOpencodeCli(params) {
 var AUTO_SELECT_WARNING = "select a model explicitly in the Pullfrog console (https://pullfrog.com/console) to avoid this \u2014 if the picker is locked, this run had no Pullfrog Router funding: add a card or top up your credit.";
 function autoSelectModel() {
   const authorized2 = getAuthorizedModels();
-  if (authorized2.size > 0) {
-    const servable = (a2) => {
-      if (!authorized2.has(a2.resolve)) return false;
-      const envVars = getModelEnvVars(a2.resolve);
-      return envVars.length === 0 || envVars.some((name) => process.env[name]);
-    };
-    const match3 = modelAliases.find((a2) => !a2.hidden && !a2.fallback && a2.preferred && servable(a2)) ?? modelAliases.find((a2) => !a2.hidden && !a2.fallback && servable(a2));
-    if (match3) {
-      log2.info(
-        `\xBB model: ${match3.resolve} (auto-selected${match3.preferred ? " \u2014 preferred" : ""} curated match)`
-      );
-      log2.warning(`\xBB model auto-selected. ${AUTO_SELECT_WARNING}`);
-      return match3.resolve;
-    }
+  const selectable = modelAliases.filter((a2) => !a2.hidden && !a2.fallback && !a2.routing);
+  const servable = selectable.filter((a2) => {
+    const envVars = getModelEnvVars(a2.resolve);
+    return envVars.length === 0 || envVars.some((name) => process.env[name]);
+  });
+  const pick4 = (list) => list.find((a2) => a2.preferred) ?? list[0];
+  const match3 = pick4(servable.filter((a2) => authorized2.has(a2.resolve))) ?? pick4(servable);
+  if (match3) {
     log2.info(
-      `\xBB opencode has ${authorized2.size} models but none match curated aliases \u2014 letting OpenCode auto-select`
+      `\xBB model: ${match3.resolve} (auto-selected${match3.preferred ? " \u2014 preferred" : ""} curated match)`
     );
+    log2.warning(`\xBB model auto-selected. ${AUTO_SELECT_WARNING}`);
+    return match3.resolve;
   }
+  log2.info(
+    `\xBB opencode has ${authorized2.size} models but none match curated aliases and none hold a credential \u2014 letting OpenCode auto-select: ${[...authorized2].join(", ")}`
+  );
   log2.warning(`\xBB no model resolved. letting OpenCode auto-select. ${AUTO_SELECT_WARNING}`);
   return void 0;
 }
@@ -188634,8 +188636,8 @@ function validatePushDestination(repoState, branch, cwd) {
     throw new Error(
       `Push blocked: destination does not match expected repository.
 Expected: ${pushUrl}
-Actual: ${dest.url}
-Git configuration may have been tampered with.`
+Actual: remote '${dest.remoteName}' -> ${dest.url}
+Git configuration may have been tampered with, or a url.*.insteadOf rewrite is in effect.`
     );
   }
   return dest;
@@ -190757,6 +190759,9 @@ async function configureRepoGit(params) {
   removeIncludeIfEntries(repoDir);
   const originUrl = `https://github.com/${params.owner}/${params.name}.git`;
   $("git", ["remote", "set-url", "origin", originUrl], { cwd: repoDir });
+  $("git", ["config", "--local", "--replace-all", "remote.origin.pushurl", originUrl], {
+    cwd: repoDir
+  });
   repoState.pushUrl = originUrl;
   $("git", ["config", "--local", "credential.helper", ""], { cwd: repoDir });
   repoState.initialHead = captureInitialHead(repoDir);
@@ -191095,6 +191100,9 @@ async function checkoutPrBranch(pr, params) {
       $("git", ["remote", "set-url", remoteName, forkUrl], { log: false });
       log2.debug(`\xBB updated remote '${remoteName}' for fork ${pr.headRepoFullName}`);
     }
+    $("git", ["config", "--local", "--replace-all", `remote.${remoteName}.pushurl`, forkUrl], {
+      log: false
+    });
     $("git", ["config", `branch.${localBranch}.pushRemote`, remoteName], { log: false });
     $("git", ["config", `branch.${localBranch}.merge`, `refs/heads/${pr.headRef}`], { log: false });
     log2.debug(`\xBB configured branch '${localBranch}' to push to '${remoteName}/${pr.headRef}'`);
@@ -196916,7 +196924,7 @@ async function handleAgentResult(ctx) {
   const mode = toolState.selectedMode;
   const isReviewMode = mode === "Review" || mode === "IncrementalReview";
   const payload = ctx.toolContext.payload;
-  const hasCommentTarget = toolState.hadProgressComment || !payload.progressComments && payload.event.issue_number !== void 0;
+  const hasCommentTarget = toolState.hadProgressComment || payload.event.issue_number !== void 0;
   if (!isReviewMode && !toolState.wasUpdated && hasCommentTarget && !ctx.silent) {
     const tracker = toolState.todoTracker;
     if (tracker) {
@@ -197350,6 +197358,19 @@ ${input.errorMessage}
 \`\`\``
   ].join("\n");
 }
+function formatProviderMissingCredential(input) {
+  return [
+    "**No credential reached the model provider.**",
+    "",
+    "The run started without a key for the provider it ended up using, so the request was refused before any model work happened. Nothing was billed, and no key of yours was rejected \u2014 add a provider key, or pick a model you already have one for.",
+    "",
+    `[Configure model \u2192](${getApiUrl()}/console/${input.owner}/${input.name})`,
+    "",
+    `\`\`\`
+${input.errorMessage}
+\`\`\``
+  ].join("\n");
+}
 function formatProviderModelNotFoundSummary(input) {
   return `The configured model is no longer available in OpenCode's catalog. Pick a different model in the Pullfrog console for \`${input.owner}/${input.name}\`, or contact support if this persists.
 
@@ -197395,6 +197416,16 @@ ${body}`, comment: body };
     });
     return { summary: `### \u274C Pullfrog failed
 
+${body}`, comment: body };
+  }
+  if (isProviderMissingCredential(input.errorMessage)) {
+    const body = formatProviderMissingCredential({
+      owner: input.repo.owner,
+      name: input.repo.name,
+      errorMessage: input.errorMessage
+    });
+    return { summary: `### \u274C Pullfrog failed
+
 ${body}`, comment: body };
   }
   const apiKeySource = hangBody ?? input.errorMessage;
@@ -199234,7 +199265,7 @@ async function runCli4(input) {
 }
 
 // cli.ts
-var VERSION10 = "0.1.61";
+var VERSION10 = "0.1.62";
 var bin = basename2(process.argv[1] || "");
 var PROG = bin === "pf" || bin === "pullfrog" ? bin : "pullfrog";
 var rawArgs = process.argv.slice(2);

package/dist/index.js

@@ -105126,6 +105126,9 @@ function isProviderNoRoutableEndpoints(text) {
 function isOpenRouterKeyLimitExceeded(text) {
   return /Key limit exceeded \(total limit\)/i.test(text) || /openrouter\.ai\/[^\s"')]*\/keys\//i.test(text);
 }
+function isProviderMissingCredential(text) {
+  return /API key is missing\. Pass it using the/i.test(text) || /Missing Authentication header/i.test(text);
+}
 function isTransientUpstreamError(text) {
   return /API Error:\s*5\d\d/i.test(text) || /\bOverloaded\b/.test(text) || /"error_type"\s*:\s*"(?:provider_unavailable|timeout)"/i.test(text) || /Streaming response failed:\s*\[5\d\d\]/i.test(text);
 }
@@ -105188,7 +105191,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.61",
+  version: "0.1.62",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -106279,7 +106282,6 @@ function getUnsubmittedReview(toolState, expectsReviewOutput2 = toolState.hadPro
 }
 function expectsReviewOutput(ctx) {
   if (ctx.toolState.hadProgressComment) return true;
-  if (ctx.payload.progressComments) return false;
   return ctx.payload.event.silent !== true && ctx.payload.event.issue_number !== void 0;
 }
 function buildStopHookPrompt(failure2) {
@@ -106494,6 +106496,7 @@ async function runPostRunRetryLoop(params) {
       result = preResume;
       break;
     }
+    if (onlySummaryStale) result = { ...result, output: preResume.output || result.output };
     gateResumeCount++;
   }
   if (gateResumeCount > 0 && result.success && hasPostRunIssues(finalIssues)) {
@@ -113885,24 +113888,23 @@ async function installOpencodeCli(params) {
 var AUTO_SELECT_WARNING = "select a model explicitly in the Pullfrog console (https://pullfrog.com/console) to avoid this \u2014 if the picker is locked, this run had no Pullfrog Router funding: add a card or top up your credit.";
 function autoSelectModel() {
   const authorized2 = getAuthorizedModels();
-  if (authorized2.size > 0) {
-    const servable = (a) => {
-      if (!authorized2.has(a.resolve)) return false;
-      const envVars = getModelEnvVars(a.resolve);
-      return envVars.length === 0 || envVars.some((name) => process.env[name]);
-    };
-    const match3 = modelAliases.find((a) => !a.hidden && !a.fallback && a.preferred && servable(a)) ?? modelAliases.find((a) => !a.hidden && !a.fallback && servable(a));
-    if (match3) {
-      log.info(
-        `\xBB model: ${match3.resolve} (auto-selected${match3.preferred ? " \u2014 preferred" : ""} curated match)`
-      );
-      log.warning(`\xBB model auto-selected. ${AUTO_SELECT_WARNING}`);
-      return match3.resolve;
-    }
+  const selectable = modelAliases.filter((a) => !a.hidden && !a.fallback && !a.routing);
+  const servable = selectable.filter((a) => {
+    const envVars = getModelEnvVars(a.resolve);
+    return envVars.length === 0 || envVars.some((name) => process.env[name]);
+  });
+  const pick4 = (list) => list.find((a) => a.preferred) ?? list[0];
+  const match3 = pick4(servable.filter((a) => authorized2.has(a.resolve))) ?? pick4(servable);
+  if (match3) {
     log.info(
-      `\xBB opencode has ${authorized2.size} models but none match curated aliases \u2014 letting OpenCode auto-select`
+      `\xBB model: ${match3.resolve} (auto-selected${match3.preferred ? " \u2014 preferred" : ""} curated match)`
     );
+    log.warning(`\xBB model auto-selected. ${AUTO_SELECT_WARNING}`);
+    return match3.resolve;
   }
+  log.info(
+    `\xBB opencode has ${authorized2.size} models but none match curated aliases and none hold a credential \u2014 letting OpenCode auto-select: ${[...authorized2].join(", ")}`
+  );
   log.warning(`\xBB no model resolved. letting OpenCode auto-select. ${AUTO_SELECT_WARNING}`);
   return void 0;
 }
@@ -186514,8 +186516,8 @@ function validatePushDestination(repoState, branch, cwd) {
     throw new Error(
       `Push blocked: destination does not match expected repository.
 Expected: ${pushUrl}
-Actual: ${dest.url}
-Git configuration may have been tampered with.`
+Actual: remote '${dest.remoteName}' -> ${dest.url}
+Git configuration may have been tampered with, or a url.*.insteadOf rewrite is in effect.`
     );
   }
   return dest;
@@ -188637,6 +188639,9 @@ async function configureRepoGit(params) {
   removeIncludeIfEntries(repoDir);
   const originUrl = `https://github.com/${params.owner}/${params.name}.git`;
   $("git", ["remote", "set-url", "origin", originUrl], { cwd: repoDir });
+  $("git", ["config", "--local", "--replace-all", "remote.origin.pushurl", originUrl], {
+    cwd: repoDir
+  });
   repoState.pushUrl = originUrl;
   $("git", ["config", "--local", "credential.helper", ""], { cwd: repoDir });
   repoState.initialHead = captureInitialHead(repoDir);
@@ -188975,6 +188980,9 @@ async function checkoutPrBranch(pr, params) {
       $("git", ["remote", "set-url", remoteName, forkUrl], { log: false });
       log.debug(`\xBB updated remote '${remoteName}' for fork ${pr.headRepoFullName}`);
     }
+    $("git", ["config", "--local", "--replace-all", `remote.${remoteName}.pushurl`, forkUrl], {
+      log: false
+    });
     $("git", ["config", `branch.${localBranch}.pushRemote`, remoteName], { log: false });
     $("git", ["config", `branch.${localBranch}.merge`, `refs/heads/${pr.headRef}`], { log: false });
     log.debug(`\xBB configured branch '${localBranch}' to push to '${remoteName}/${pr.headRef}'`);
@@ -194796,7 +194804,7 @@ async function handleAgentResult(ctx) {
   const mode = toolState.selectedMode;
   const isReviewMode = mode === "Review" || mode === "IncrementalReview";
   const payload = ctx.toolContext.payload;
-  const hasCommentTarget = toolState.hadProgressComment || !payload.progressComments && payload.event.issue_number !== void 0;
+  const hasCommentTarget = toolState.hadProgressComment || payload.event.issue_number !== void 0;
   if (!isReviewMode && !toolState.wasUpdated && hasCommentTarget && !ctx.silent) {
     const tracker = toolState.todoTracker;
     if (tracker) {
@@ -195230,6 +195238,19 @@ ${input.errorMessage}
 \`\`\``
   ].join("\n");
 }
+function formatProviderMissingCredential(input) {
+  return [
+    "**No credential reached the model provider.**",
+    "",
+    "The run started without a key for the provider it ended up using, so the request was refused before any model work happened. Nothing was billed, and no key of yours was rejected \u2014 add a provider key, or pick a model you already have one for.",
+    "",
+    `[Configure model \u2192](${getApiUrl()}/console/${input.owner}/${input.name})`,
+    "",
+    `\`\`\`
+${input.errorMessage}
+\`\`\``
+  ].join("\n");
+}
 function formatProviderModelNotFoundSummary(input) {
   return `The configured model is no longer available in OpenCode's catalog. Pick a different model in the Pullfrog console for \`${input.owner}/${input.name}\`, or contact support if this persists.
 
@@ -195275,6 +195296,16 @@ ${body}`, comment: body };
     });
     return { summary: `### \u274C Pullfrog failed
 
+${body}`, comment: body };
+  }
+  if (isProviderMissingCredential(input.errorMessage)) {
+    const body = formatProviderMissingCredential({
+      owner: input.repo.owner,
+      name: input.repo.name,
+      errorMessage: input.errorMessage
+    });
+    return { summary: `### \u274C Pullfrog failed
+
 ${body}`, comment: body };
   }
   const apiKeySource = hangBody ?? input.errorMessage;

package/dist/utils/providerErrors.d.ts

@@ -50,6 +50,12 @@ export declare function isProviderNoRoutableEndpoints(text: string): boolean;
  * …/settings/credits`) carries no `/keys/` path and correctly stays out.
  */
 export declare function isOpenRouterKeyLimitExceeded(text: string): boolean;
+/**
+ * the provider was reached with NO credential at all — the AI SDK's shared `loadApiKey`
+ * refusal, so it is provider-agnostic. NOT `isApiKeyAuthError`, which is a key we hold
+ * being rejected: "rotate your key" names a credential that does not exist here.
+ */
+export declare function isProviderMissingCredential(text: string): boolean;
 /**
  * The upstream is having a moment: Anthropic's `API Error: 529 Overloaded`,
  * OpenRouter's `provider_unavailable` / `timeout`, OpenCode Zen's `Streaming

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pullfrog",
-  "version": "0.1.61",
+  "version": "0.1.62",
   "type": "module",
   "bin": {
     "pullfrog": "dist/cli.mjs",