pullfrog 0.1.37 → 0.1.38

package/dist/cli.mjs

@@ -103001,7 +103001,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.37",
+  version: "0.1.38",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -104375,6 +104375,7 @@ async function finalizeAgentResult(params) {
 }
 var REFLECTION_SKIP_MODES = /* @__PURE__ */ new Set(["IncrementalReview"]);
 function shouldRunReflection(mode) {
+  if (process.env.PULLFROG_DISABLE_LEARNINGS_REFLECTION === "1") return false;
   if (!mode) return true;
   return !REFLECTION_SKIP_MODES.has(mode);
 }
@@ -158384,6 +158385,14 @@ var PushBranch = type({
     "cross-repo runs only: the writable secondary repo whose checkout to push from (bare name, from list_repos). omit for the primary repo."
   )
 });
+function assertWritableMode(ctx, toolName) {
+  const mode = ctx.toolState?.selectedMode;
+  if (mode && NON_COMMITTING_MODES.has(mode)) {
+    throw new Error(
+      `${toolName} is blocked in ${mode} mode \u2014 review and plan runs must not push or commit code. finish by submitting your review (create_pull_request_review) or plan. if this PR genuinely needs a code change, that is a separate task from reviewing it.`
+    );
+  }
+}
 function assertPushTarget(ctx, params) {
   const branch = params.branch;
   const pushDest = params.pushDest;
@@ -158508,6 +158517,7 @@ function PushBranchTool(ctx) {
       if (pushPermission === "disabled") {
         throw new Error("Push is disabled. This repository is configured for read-only access.");
       }
+      assertWritableMode(ctx, "push_branch");
       const rc = resolveRepoCtx(ctx, repo);
       if (rc.access === "read") {
         throw new Error(
@@ -158634,6 +158644,7 @@ function CommitChangesTool(ctx) {
       if (pushPermission === "disabled") {
         throw new Error("Push is disabled. This repository is configured for read-only access.");
       }
+      assertWritableMode(ctx, "commit_changes");
       const branch = $2("git", ["rev-parse", "--abbrev-ref", "HEAD"], { log: false }).trim();
       if (branch === "HEAD") {
         throw new Error(
@@ -158972,6 +158983,7 @@ function DeleteBranchTool(ctx) {
           "Branch deletion requires push: enabled permission. Current mode only allows pushing to non-protected branches."
         );
       }
+      assertWritableMode(ctx, "delete_branch");
       rejectSpecialRef(params.branchName, "branchName");
       if (params.branchName === defaultBranch) {
         throw new Error(
@@ -159005,6 +159017,7 @@ function PushTagsTool(ctx) {
           "Tag pushing requires push: enabled permission. Current mode only allows pushing branches."
         );
       }
+      assertWritableMode(ctx, "push_tags");
       validateTagName(params.tag);
       const pushArgs = [...params.force ? ["-f"] : [], "origin", `refs/tags/${params.tag}`];
       await pushWithRetry(pushArgs, ctx.gitToken, ctx.refreshGitToken);
@@ -168461,7 +168474,7 @@ async function runCli4(input) {
 }
 
 // cli.ts
-var VERSION10 = "0.1.37";
+var VERSION10 = "0.1.38";
 var bin = basename2(process.argv[1] || "");
 var PROG = bin === "pf" || bin === "pullfrog" ? bin : "pullfrog";
 var rawArgs = process.argv.slice(2);

package/dist/index.js

@@ -101072,7 +101072,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.37",
+  version: "0.1.38",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -102446,6 +102446,7 @@ async function finalizeAgentResult(params) {
 }
 var REFLECTION_SKIP_MODES = /* @__PURE__ */ new Set(["IncrementalReview"]);
 function shouldRunReflection(mode) {
+  if (process.env.PULLFROG_DISABLE_LEARNINGS_REFLECTION === "1") return false;
   if (!mode) return true;
   return !REFLECTION_SKIP_MODES.has(mode);
 }
@@ -156497,6 +156498,14 @@ var PushBranch = type({
     "cross-repo runs only: the writable secondary repo whose checkout to push from (bare name, from list_repos). omit for the primary repo."
   )
 });
+function assertWritableMode(ctx, toolName) {
+  const mode = ctx.toolState?.selectedMode;
+  if (mode && NON_COMMITTING_MODES.has(mode)) {
+    throw new Error(
+      `${toolName} is blocked in ${mode} mode \u2014 review and plan runs must not push or commit code. finish by submitting your review (create_pull_request_review) or plan. if this PR genuinely needs a code change, that is a separate task from reviewing it.`
+    );
+  }
+}
 function assertPushTarget(ctx, params) {
   const branch = params.branch;
   const pushDest = params.pushDest;
@@ -156621,6 +156630,7 @@ function PushBranchTool(ctx) {
       if (pushPermission === "disabled") {
         throw new Error("Push is disabled. This repository is configured for read-only access.");
       }
+      assertWritableMode(ctx, "push_branch");
       const rc = resolveRepoCtx(ctx, repo);
       if (rc.access === "read") {
         throw new Error(
@@ -156747,6 +156757,7 @@ function CommitChangesTool(ctx) {
       if (pushPermission === "disabled") {
         throw new Error("Push is disabled. This repository is configured for read-only access.");
       }
+      assertWritableMode(ctx, "commit_changes");
       const branch = $("git", ["rev-parse", "--abbrev-ref", "HEAD"], { log: false }).trim();
       if (branch === "HEAD") {
         throw new Error(
@@ -157085,6 +157096,7 @@ function DeleteBranchTool(ctx) {
           "Branch deletion requires push: enabled permission. Current mode only allows pushing to non-protected branches."
         );
       }
+      assertWritableMode(ctx, "delete_branch");
       rejectSpecialRef(params.branchName, "branchName");
       if (params.branchName === defaultBranch) {
         throw new Error(
@@ -157118,6 +157130,7 @@ function PushTagsTool(ctx) {
           "Tag pushing requires push: enabled permission. Current mode only allows pushing branches."
         );
       }
+      assertWritableMode(ctx, "push_tags");
       validateTagName(params.tag);
       const pushArgs = [...params.force ? ["-f"] : [], "origin", `refs/tags/${params.tag}`];
       await pushWithRetry(pushArgs, ctx.gitToken, ctx.refreshGitToken);

package/package.json

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