@neriros/ralphy 2.16.2 → 2.16.4

package/dist/cli/index.js

@@ -35029,8 +35029,8 @@ import { readFileSync as readFileSync2 } from "fs";
 import { resolve } from "path";
 function getVersion() {
   try {
-    if ("2.16.2")
-      return "2.16.2";
+    if ("2.16.4")
+      return "2.16.4";
   } catch {}
   const dirsToTry = [];
   try {
@@ -59481,6 +59481,9 @@ var MarkerSchema, GetIndicatorSchema, SetIndicatorSchema, IndicatorsSchema, Ralp
   // Seconds between CI status polls.
   "ciPollIntervalSeconds": 30,
 
+  // CI check names to ignore when polling PR status (case-insensitive).
+  // "ignoreCiChecks": ["Vercel", "codeql"],
+
   // Underlying engine: "claude" or "codex".
   "engine": "claude",
 
@@ -59591,6 +59594,7 @@ var init_config = __esm(() => {
     fixCiOnFailure: exports_external.boolean().default(false),
     maxCiFixAttempts: exports_external.number().int().positive().default(5),
     ciPollIntervalSeconds: exports_external.number().int().positive().default(30),
+    ignoreCiChecks: exports_external.array(exports_external.string()).default([]),
     engine: exports_external.enum(["claude", "codex"]).default("claude"),
     model: exports_external.enum(["haiku", "sonnet", "opus"]).default("opus"),
     linear: exports_external.object({
@@ -60450,7 +60454,7 @@ ${e.stdout ?? ""}`;
   }
   throw lastErr;
 }
-async function getPrChecksStatus(prRef, runner, cwd2, onTransientRetry) {
+async function getPrChecksStatus(prRef, runner, cwd2, onTransientRetry, ignoreCiChecks = []) {
   let out;
   try {
     out = await runGhWithRetry(["gh", "pr", "checks", prRef, "--json", PR_CHECKS_FIELDS], runner, cwd2, onTransientRetry);
@@ -60463,7 +60467,8 @@ ${e.stdout ?? ""}`;
       return { bucket: "pass", failedRunIds: [] };
     throw err;
   }
-  const checks = JSON.parse(out.stdout || "[]").filter((c) => c.bucket !== "skipping");
+  const ignoredLower = ignoreCiChecks.map((n) => n.toLowerCase());
+  const checks = JSON.parse(out.stdout || "[]").filter((c) => !ignoredLower.includes(c.name.toLowerCase())).filter((c) => c.bucket !== "skipping");
   if (checks.some((c) => c.bucket === "pending")) {
     return { bucket: "pending", failedRunIds: [] };
   }
@@ -60602,57 +60607,9 @@ ${pe.stderr ?? ""}`;
     }
   }
 }
-async function commitResidualChanges(ctx, maxAttempts) {
-  let hookFixAttempt = 0;
-  while (true) {
-    ctx.emit("committing", "git status");
-    let dirty = "";
-    try {
-      const status = await ctx.cmd.run(["git", "status", "--porcelain"], ctx.cwd);
-      dirty = status.stdout.trim();
-    } catch (err) {
-      ctx.log(`! git status failed for ${ctx.changeName}: ${err.message}`, "yellow");
-      break;
-    }
-    if (!dirty)
-      break;
-    try {
-      ctx.emit("committing", "git add -A");
-      await ctx.cmd.run(["git", "add", "-A"], ctx.cwd);
-      ctx.emit("committing", "git commit");
-      await ctx.cmd.run(["git", "commit", "-m", `chore(ralph): residual changes for ${ctx.changeName}`], ctx.cwd);
-      ctx.log(`  committed residual changes for ${ctx.changeName}`, "gray");
-      break;
-    } catch (err) {
-      const e = err;
-      const detail = e.stderr?.trim() || e.message;
-      const combined = `${e.stdout ?? ""}
-${e.stderr ?? ""}`;
-      if (/nothing to commit/i.test(combined) || /empty git commit/i.test(combined))
-        break;
-      if (hookFixAttempt >= maxAttempts) {
-        ctx.log(`! commit rejected for ${ctx.changeName} after ${hookFixAttempt} hook-fix attempts (host pre-commit hook still failing) \u2014 worktree preserved at ${ctx.cwd}`, "red");
-        ctx.log(`    detail: ${detail}`, "red");
-        return { gaveUp: true, hookFixAttempt };
-      }
-      hookFixAttempt += 1;
-      ctx.emit("commit-retry", `${hookFixAttempt}/${maxAttempts}`);
-      ctx.log(`! commit rejected for ${ctx.changeName} \u2014 prepending fix task and re-running loop (attempt ${hookFixAttempt}/${maxAttempts})`, "yellow");
-      ctx.log(`    detail: ${detail}`, "yellow");
-      const retryCode = await runWorkerWithFixTask(ctx, "Fix host pre-commit hook rejection", `Committing residual changes was rejected by the host repo's pre-commit hook. ` + `Fix the underlying problem, then the commit will be retried.
-
-` + combined.trim());
-      if (retryCode !== 0) {
-        ctx.log(`! worker re-run after commit rejection exited code ${retryCode} \u2014 giving up`, "red");
-        return { gaveUp: true, hookFixAttempt };
-      }
-    }
-  }
-  return { gaveUp: false, hookFixAttempt };
-}
-async function createPrWithRetry(ctx, issue, initialHookFixAttempt) {
+async function createPrWithRetry(ctx, issue) {
   const maxAttempts = ctx.cfg.maxCiFixAttempts;
-  let hookFixAttempt = initialHookFixAttempt;
+  let hookFixAttempt = 0;
   let nonFfRebaseAttempted = false;
   let pr = null;
   while (true) {
@@ -60688,12 +60645,16 @@ ${re.stderr ?? ""}`;
           ctx.emit("rebasing", "conflicts detected \u2014 aborting + queueing fix task");
           try {
             await ctx.cmd.run(["git", "rebase", "--abort"], ctx.cwd);
-          } catch {}
+          } catch (err2) {
+            ctx.log(`! git rebase --abort failed (worktree may already be clean): ${err2.message}`, "yellow");
+          }
           let conflictedFiles = "";
           try {
             const r = await ctx.cmd.run(["git", "diff", "--name-only", `HEAD..origin/${ctx.branch}`], ctx.cwd);
             conflictedFiles = r.stdout.trim();
-          } catch {}
+          } catch (err2) {
+            ctx.log(`! could not list conflicted files: ${err2.message}`, "yellow");
+          }
           if (hookFixAttempt >= maxAttempts) {
             ctx.log(`! merge conflict on rebase of ${ctx.branch} after ${hookFixAttempt} attempts \u2014 worktree preserved at ${ctx.cwd}`, "red");
             ctx.log(`    detail: ${reBlob.trim().split(`
@@ -60789,7 +60750,7 @@ async function fixConflictsAndCiLoop(ctx, prUrl, wantFixCi, checkPrConflict) {
       ctx.emit("ci-poll", "starting");
       const result2 = await fixCiUntilGreen({
         onPhase: (p, d) => ctx.emit(p, d),
-        getStatus: () => getPrChecksStatus(prUrl, ctx.cmd, ctx.cwd, (n, ms, why) => ctx.log(`  gh transient (try ${n}) \u2014 retry in ${Math.round(ms / 1000)}s \xB7 ${why}`, "yellow")),
+        getStatus: () => getPrChecksStatus(prUrl, ctx.cmd, ctx.cwd, (n, ms, why) => ctx.log(`  gh transient (try ${n}) \u2014 retry in ${Math.round(ms / 1000)}s \xB7 ${why}`, "yellow"), ctx.cfg.ignoreCiChecks),
         getFailedLogs: (ids) => fetchFailedRunLogs(ids, ctx.cmd, ctx.cwd),
         runTaskWithSteering: (steering) => runWorkerWithFixTask(ctx, "Fix failing CI checks", steering),
         pushBranch: async () => {
@@ -60855,11 +60816,16 @@ async function runPostTask(input, deps) {
         emit,
         respawnWorker
       };
-      const { gaveUp: commitGaveUp, hookFixAttempt } = await commitResidualChanges(ctx, cfg.maxCiFixAttempts);
-      if (commitGaveUp) {
-        effectiveCode = PR_FAILED_EXIT;
-      } else {
-        const { pr, gaveUp: prGaveUp } = await createPrWithRetry(ctx, issue, hookFixAttempt);
+      try {
+        const status = await cmd.run(["git", "status", "--porcelain"], cwd2);
+        if (status.stdout.trim()) {
+          log2(`! ${changeName} has uncommitted changes after worker exit \u2014 the agent should commit everything before finishing. These changes will not be included in the PR.`, "yellow");
+        }
+      } catch (err) {
+        log2(`! git status check failed for ${changeName}: ${err.message}`, "yellow");
+      }
+      {
+        const { pr, gaveUp: prGaveUp } = await createPrWithRetry(ctx, issue);
         if (prGaveUp) {
           effectiveCode = PR_FAILED_EXIT;
         } else if (!pr) {
@@ -60882,7 +60848,9 @@ async function runPostTask(input, deps) {
     emit("teardown", cfg.teardownScript);
     try {
       await runScript("teardown", cfg.teardownScript, cwd2);
-    } catch {}
+    } catch (err) {
+      log2(`! teardown script threw: ${err.message}`, "yellow");
+    }
   }
   if (useWorktree && cwd2 !== projectRoot) {
     emit("cleanup", "checking worktree safety");
@@ -61385,7 +61353,8 @@ PR: ${prUrl}` : ""
           prBaseBranch: cfg.prBaseBranch,
           maxCiFixAttempts: cfg.maxCiFixAttempts,
           ciPollIntervalSeconds: cfg.ciPollIntervalSeconds,
-          cleanupWorktreeOnSuccess: cfg.cleanupWorktreeOnSuccess
+          cleanupWorktreeOnSuccess: cfg.cleanupWorktreeOnSuccess,
+          ignoreCiChecks: cfg.ignoreCiChecks
         },
         respawnWorker: respawn
       }, {
@@ -71880,6 +71849,8 @@ function buildTaskPrompt(state, taskDir) {
 
 `;
   prompt += `Run \`bunx openspec validate ${state.name}\` before committing.
+`;
+  prompt += `Commit all changed files yourself before finishing \u2014 stage files individually (e.g. \`git add path/to/file\`), never \`git add -A\` or \`git commit -am\`. Nothing is committed automatically after you exit.
 `;
   return prompt;
 }
@@ -72551,8 +72522,6 @@ function phaseColor(phase) {
       return "cyan";
     case "scaffolding":
       return "magenta";
-    case "committing":
-    case "commit-retry":
     case "pushing":
     case "push-retry":
     case "rebasing":
@@ -72577,8 +72546,6 @@ function workerBorderColor(phase) {
     case "working":
     case "scaffolding":
       return "cyan";
-    case "committing":
-    case "commit-retry":
     case "pushing":
     case "push-retry":
     case "rebasing":
@@ -72796,7 +72763,9 @@ function AgentMode({ args, projectRoot, statesDir, tasksDir }) {
               const json = await file.json();
               meta.iter = json.iteration ?? meta.iter;
             }
-          } catch {}
+          } catch (err) {
+            console.error(`Failed to read state file for worker '${changeName}' (may not exist yet):`, err);
+          }
           if (meta.changeDir) {
             try {
               const tasksFile = Bun.file(join16(meta.changeDir, "tasks.md"));
@@ -72805,7 +72774,9 @@ function AgentMode({ args, projectRoot, statesDir, tasksDir }) {
                 const match = text.match(/^- \[ \] (.+)$/m);
                 meta.currentTask = match?.[1]?.trim() ?? null;
               }
-            } catch {}
+            } catch (err) {
+              console.error(`Failed to read tasks.md for worker '${changeName}' (may not exist yet):`, err);
+            }
           }
         }
         if (!cancelled)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neriros/ralphy",
-  "version": "2.16.2",
+  "version": "2.16.4",
   "description": "An iterative AI task execution framework. Orchestrates multi-phase autonomous work using Claude or Codex engines.",
   "keywords": [
     "agent",