@neriros/ralphy 2.7.8 → 2.7.9

package/README.md

@@ -140,6 +140,8 @@ Use `setupScript` (run inside the worktree right after scaffolding) to install d
 
 **`fixCiOnFailure`** (or `--fix-ci`) watches the PR's checks via `gh pr checks` and, on failure, fetches the failed-run logs (`gh run view --log-failed`), appends them to `proposal.md` under `## Steering`, re-spawns the task loop in the worktree, and pushes the new commits — repeating until checks go green or `maxCiFixAttempts` is hit (default 5, polling interval `ciPollIntervalSeconds` defaults to 30s). Requires `--create-pr`.
 
+When `fixCiOnFailure` is enabled, the issue is **not** moved to `doneStatus` (and `doneLabel` is not applied, and the issue is not marked processed in `.ralph/agent-state.json`) until CI actually goes green. If the fix loop exhausts its attempts the worker is treated as failed for completion-marking purposes and the issue will be re-picked-up on the next poll (the resume-in-progress filter ensures that).
+
 Every CLI flag is also configurable in `ralphy.config.json`; CLI values override config when both are set. The agent forwards `maxRuntimeMinutesPerTask` / `maxConsecutiveFailuresPerTask` / `iterationDelaySeconds` / `logRawStream` / `taskVerbose` to each spawned `ralph task` worker.
 
 Failed workers (non-zero exit) are not marked processed, so they'll be retried on the next poll. SIGINT/SIGTERM cleanly stops polling and kills active workers. All Linear side effects are best-effort — failures log a warning but never block the task loop.

package/dist/cli/index.js

@@ -70448,12 +70448,14 @@ function AgentMode({ args, projectRoot, statesDir, tasksDir }) {
             stdin: "ignore"
           });
           const wantPr = args.createPr || cfg.createPrOnSuccess;
+          const CI_FAILED_EXIT = 70;
           const wrapped = proc.exited.then(async (code) => {
             if (cfg.teardownScript) {
               try {
                 await runScript("teardown", cfg.teardownScript, cwd2);
               } catch {}
             }
+            let effectiveCode = code;
             const ok = code === 0;
             if (ok && wantPr) {
               const branch = branchByChange.get(changeName);
@@ -70513,7 +70515,8 @@ ${stamped}
                         pollIntervalSeconds: cfg.ciPollIntervalSeconds
                       });
                       if (!result2.success) {
-                        appendLog(`! CI fix loop gave up after ${result2.attempts} attempts (${result2.reason ?? "unknown"})`, "red");
+                        appendLog(`! CI fix loop gave up after ${result2.attempts} attempts (${result2.reason ?? "unknown"}) \u2014 withholding done-status until CI passes`, "red");
+                        effectiveCode = CI_FAILED_EXIT;
                       }
                     }
                   }
@@ -70523,7 +70526,7 @@ ${stamped}
               }
             }
             if (useWorktree && cwd2 !== projectRoot) {
-              if (ok && cfg.cleanupWorktreeOnSuccess) {
+              if (effectiveCode === 0 && cfg.cleanupWorktreeOnSuccess) {
                 try {
                   await removeWorktree(projectRoot, cwd2, bunGitRunner);
                   appendLog(`  removed worktree ${cwd2}`, "gray");
@@ -70536,7 +70539,7 @@ ${stamped}
             statesDirByChange.delete(changeName);
             branchByChange.delete(changeName);
             issueByChange.delete(changeName);
-            return code;
+            return effectiveCode;
           });
           return { exited: wrapped, kill: () => proc.kill() };
         },

package/package.json

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