@neriros/ralphy 2.9.3 → 2.10.0

package/dist/cli/index.js

@@ -56407,7 +56407,7 @@ function log(msg) {
 // package.json
 var package_default = {
   name: "@neriros/ralphy",
-  version: "2.9.3",
+  version: "2.10.0",
   description: "An iterative AI task execution framework. Orchestrates multi-phase autonomous work using Claude or Codex engines.",
   keywords: [
     "agent",
@@ -70178,25 +70178,82 @@ async function addLabelToIssue(apiKey, issueId, labelId) {
 
 // apps/cli/src/agent/state.ts
 import { join as join10 } from "path";
+var TaskStateSchema = exports_external.enum(["started", "processed", "failed"]);
+var TaskEntrySchema = exports_external.object({
+  issueId: exports_external.string(),
+  identifier: exports_external.string(),
+  state: TaskStateSchema,
+  changeName: exports_external.string().optional(),
+  startedAt: exports_external.string().optional(),
+  finishedAt: exports_external.string().optional(),
+  exitCode: exports_external.number().optional(),
+  commentPosted: exports_external.boolean().optional()
+});
 var AgentStateSchema = exports_external.object({
+  tasks: exports_external.record(exports_external.string(), TaskEntrySchema).default({}),
+  lastPollAt: exports_external.string().nullable().default(null)
+});
+var LegacyAgentStateSchema = exports_external.object({
   processedIssueIds: exports_external.array(exports_external.string()).default([]),
   startedIssueIds: exports_external.array(exports_external.string()).default([]),
   failedIssueIds: exports_external.array(exports_external.string()).default([]),
   lastPollAt: exports_external.string().nullable().default(null),
-  changeMeta: exports_external.record(exports_external.string(), exports_external.object({
-    issueId: exports_external.string(),
-    identifier: exports_external.string()
-  })).default({})
-});
+  changeMeta: exports_external.record(exports_external.string(), exports_external.object({ issueId: exports_external.string(), identifier: exports_external.string() })).default({})
+}).partial();
+function migrateLegacy(raw) {
+  const parsed = LegacyAgentStateSchema.safeParse(raw);
+  if (!parsed.success)
+    return AgentStateSchema.parse({});
+  const legacy = parsed.data;
+  const tasks = {};
+  const byIssueId = new Map;
+  for (const [changeName, meta] of Object.entries(legacy.changeMeta ?? {})) {
+    byIssueId.set(meta.issueId, { identifier: meta.identifier, changeName });
+  }
+  const fold = (ids, state) => {
+    for (const issueId of ids ?? []) {
+      const found = byIssueId.get(issueId);
+      if (!found)
+        continue;
+      tasks[found.identifier] = {
+        issueId,
+        identifier: found.identifier,
+        state,
+        changeName: found.changeName
+      };
+    }
+  };
+  fold(legacy.startedIssueIds, "started");
+  fold(legacy.processedIssueIds, "processed");
+  fold(legacy.failedIssueIds, "failed");
+  for (const issueId of legacy.startedIssueIds ?? []) {
+    const found = byIssueId.get(issueId);
+    if (!found)
+      continue;
+    const entry = tasks[found.identifier];
+    if (entry)
+      entry.commentPosted = true;
+  }
+  return { tasks, lastPollAt: legacy.lastPollAt ?? null };
+}
 function statePath(projectRoot) {
   return join10(projectRoot, ".ralph", "agent-state.json");
 }
+function looksLegacy(raw) {
+  if (typeof raw !== "object" || raw === null)
+    return false;
+  const r = raw;
+  return "processedIssueIds" in r || "startedIssueIds" in r || "failedIssueIds" in r || "changeMeta" in r;
+}
 async function readAgentState(projectRoot) {
   const file = Bun.file(statePath(projectRoot));
   if (!await file.exists()) {
     return AgentStateSchema.parse({});
   }
-  return AgentStateSchema.parse(await file.json());
+  const raw = await file.json();
+  if (looksLegacy(raw))
+    return migrateLegacy(raw);
+  return AgentStateSchema.parse(raw);
 }
 async function writeAgentState(projectRoot, state) {
   await Bun.write(statePath(projectRoot), JSON.stringify(state, null, 2) + `
@@ -70371,15 +70428,19 @@ class AgentCoordinator {
       return { found: 0, added: 0 };
     }
     const state = this.state;
-    const seen = new Set(state.processedIssueIds);
-    const failed = new Set(state.failedIssueIds);
+    const tasksByIssueId = new Map;
+    for (const entry of Object.values(state.tasks)) {
+      tasksByIssueId.set(entry.issueId, entry);
+    }
+    const isProcessed = (id) => tasksByIssueId.get(id)?.state === "processed";
+    const isFailed = (id) => tasksByIssueId.get(id)?.state === "failed";
     const queued = new Set(this.queue.map((i) => i.id));
     const active = new Set(this.workers.map((w) => w.issueId));
     let added = 0;
     for (const issue of issues) {
-      if (seen.has(issue.id))
+      if (isProcessed(issue.id))
         continue;
-      if (failed.has(issue.id))
+      if (isFailed(issue.id))
         continue;
       if (queued.has(issue.id))
         continue;
@@ -70387,7 +70448,7 @@ class AgentCoordinator {
         continue;
       if (this.pendingIds.has(issue.id))
         continue;
-      const blocker = issue.blockedByIds.find((bid) => !seen.has(bid));
+      const blocker = issue.blockedByIds.find((bid) => !isProcessed(bid));
       if (blocker !== undefined) {
         this.deps.onLog(`  \u23F8 ${issue.identifier} skipped \u2014 blocked by unresolved dependency`, "yellow");
         continue;
@@ -70445,6 +70506,18 @@ class AgentCoordinator {
       this.launchWorker(issue);
     }
   }
+  upsertTask(issue, patch) {
+    if (!this.state)
+      return;
+    const existing = this.state.tasks[issue.identifier];
+    this.state.tasks[issue.identifier] = {
+      issueId: issue.id,
+      identifier: issue.identifier,
+      state: existing?.state ?? "started",
+      ...existing,
+      ...patch
+    };
+  }
   async launchWorker(issue) {
     let changeName;
     try {
@@ -70459,6 +70532,14 @@ class AgentCoordinator {
       this.pendingIds.delete(issue.id);
       return;
     }
+    if (this.state) {
+      this.upsertTask(issue, {
+        state: "started",
+        changeName,
+        startedAt: this.state.tasks[issue.identifier]?.startedAt ?? new Date().toISOString()
+      });
+      this.deps.saveState(this.state);
+    }
     this.deps.onLog(`\u25B6 ${issue.identifier} \u2192 ${changeName} (worker started)`, "cyan");
     const handle = this.deps.spawnWorker(changeName, issue);
     const worker = {
@@ -70479,12 +70560,12 @@ class AgentCoordinator {
         this.workers.splice(idx, 1);
       const ok = code === 0;
       this.deps.onLog(`${ok ? "\u2713" : "\u2717"} ${issue.identifier} \u2192 ${changeName} exited (code ${code})`, ok ? "green" : "red");
-      if (ok && this.state && !this.state.processedIssueIds.includes(issue.id)) {
-        this.state.processedIssueIds.push(issue.id);
-        this.deps.saveState(this.state);
-      }
-      if (!ok && this.state && !this.state.failedIssueIds.includes(issue.id)) {
-        this.state.failedIssueIds.push(issue.id);
+      if (this.state) {
+        this.upsertTask(issue, {
+          state: ok ? "processed" : "failed",
+          finishedAt: new Date().toISOString(),
+          exitCode: code
+        });
         this.deps.saveState(this.state);
       }
       this.notifyExited(issue, changeName, code);
@@ -70496,12 +70577,12 @@ class AgentCoordinator {
     const updater = this.deps.updater;
     if (!updater)
       return;
-    const alreadyStarted = this.state?.startedIssueIds.includes(issue.id) ?? false;
-    if (this.opts.postComments !== false && !alreadyStarted) {
+    const alreadyCommented = this.state?.tasks[issue.identifier]?.commentPosted === true;
+    if (this.opts.postComments !== false && !alreadyCommented) {
       try {
         await updater.postComment(issue, `\uD83E\uDD16 Ralph started working on this issue. Tracking change: \`${changeName}\``);
-        if (this.state && !this.state.startedIssueIds.includes(issue.id)) {
-          this.state.startedIssueIds.push(issue.id);
+        if (this.state) {
+          this.upsertTask(issue, { commentPosted: true });
           await this.deps.saveState(this.state);
         }
       } catch (err) {
@@ -70806,22 +70887,28 @@ function nextId() {
   return `${Date.now()}-${lineCounter}`;
 }
 var SPINNER_FRAMES = ["\u280B", "\u2819", "\u2839", "\u2838", "\u283C", "\u2834", "\u2826", "\u2827", "\u2807", "\u280F"];
-async function appendSteering(proposalPath, steering) {
-  const file = Bun.file(proposalPath);
-  const prev = await file.exists() ? await file.text() : "";
-  const stamped = `
-
-### Steering feedback (${new Date().toISOString()})
+async function injectFixSteering(changeDir, heading, steering) {
+  const steeringFile = Bun.file(join14(changeDir, "steering.md"));
+  const existing = await steeringFile.exists() ? await steeringFile.text() : "";
+  const stamped = `## ${heading} (${new Date().toISOString()})
 
 ${steering}
 `;
-  const next = prev.includes("## Steering") ? prev.replace(/## Steering\n+([\s\S]*?)$/, (_m, rest2) => `## Steering
-${stamped}
-${rest2}`) : prev + `
-## Steering
-${stamped}
+  const nextSteering = existing ? `${stamped}
+${existing.trimStart()}` : `${stamped}
+`;
+  await Bun.write(join14(changeDir, "steering.md"), nextSteering);
+  const tasksFile = Bun.file(join14(changeDir, "tasks.md"));
+  const tasks = await tasksFile.exists() ? await tasksFile.text() : "";
+  const taskSection = `
+## ${heading} (${new Date().toISOString()})
+
+` + `- [ ] ${heading}. The error output is recorded in steering.md \u2014 read it first, ` + `then fix the underlying problem (do not just retry the failing command).
 `;
-  await Bun.write(proposalPath, next);
+  const nextTasks = tasks.endsWith(`
+`) ? tasks + taskSection : tasks + `
+` + taskSection;
+  await Bun.write(join14(changeDir, "tasks.md"), nextTasks);
 }
 function fmtElapsed(ms) {
   const s = Math.floor(ms / 1000);
@@ -70930,13 +71017,6 @@ function AgentMode({ args, projectRoot, statesDir, tasksDir }) {
           issueByChange.set(changeName, issue);
           if (workerBranch)
             branchByChange.set(changeName, workerBranch);
-          try {
-            const s = await readAgentState(projectRoot);
-            s.changeMeta[changeName] = { issueId: issue.id, identifier: issue.identifier };
-            await writeAgentState(projectRoot, s);
-          } catch (err) {
-            appendLog(`! failed to record agent meta for ${changeName}: ${err.message}`, "yellow");
-          }
           if (cfg.setupScript) {
             await runScript("setup", cfg.setupScript, workerCwd);
           }
@@ -71005,13 +71085,13 @@ function AgentMode({ args, projectRoot, statesDir, tasksDir }) {
                 appendLog(`! createPr requested but no worktree branch is tracked for ${changeName} (use --worktree)`, "yellow");
                 effectiveCode = PR_FAILED_EXIT;
               } else {
-                const proposalPath = join14(statesDirByChange.get(changeName) ?? statesDir, "..", "..", "openspec", "changes", changeName, "proposal.md");
+                const changeDir = join14(statesDirByChange.get(changeName) ?? statesDir, "..", "..", "openspec", "changes", changeName);
                 const maxHookFixAttempts = cfg.maxCiFixAttempts;
-                const runWorkerWithSteering = async (steering) => {
+                const runWorkerWithFixSteering = async (heading, steering) => {
                   try {
-                    await appendSteering(proposalPath, steering);
+                    await injectFixSteering(changeDir, heading, steering);
                   } catch (steerErr) {
-                    appendLog(`! could not append steering: ${steerErr.message}`, "red");
+                    appendLog(`! could not inject steering: ${steerErr.message}`, "red");
                     return 1;
                   }
                   const rp = Bun.spawn({
@@ -71058,7 +71138,7 @@ ${e.stderr ?? ""}`;
                     commitFixAttempt += 1;
                     appendLog(`! commit rejected for ${changeName} \u2014 feeding error back to worker (attempt ${commitFixAttempt}/${maxHookFixAttempts})`, "yellow");
                     appendLog(`    detail: ${detail}`, "yellow");
-                    const retryCode = await runWorkerWithSteering(`Committing residual changes was rejected by the host repo's pre-commit hook. ` + `Fix the underlying problem reported below, then the commit will be retried.
+                    const retryCode = await runWorkerWithFixSteering("Fix host pre-commit hook rejection", `Committing residual changes was rejected by the host repo's pre-commit hook. ` + `Fix the underlying problem reported below, then the commit will be retried.
 
 ` + "```\n" + combined.trim() + "\n```");
                     if (retryCode !== 0) {
@@ -71096,7 +71176,7 @@ ${e.stderr ?? ""}`;
                     pushFixAttempt += 1;
                     appendLog(`! push rejected for ${changeName} \u2014 feeding error back to worker (attempt ${pushFixAttempt}/${maxHookFixAttempts})`, "yellow");
                     appendLog(`    detail: ${detail}`, "yellow");
-                    const retryCode = await runWorkerWithSteering(`Push to origin/${branch} was rejected by the host repo's pre-push hook. ` + `Fix the underlying problem reported below, then the push will be retried.
+                    const retryCode = await runWorkerWithFixSteering("Fix host pre-push hook rejection", `Push to origin/${branch} was rejected by the host repo's pre-push hook. ` + `Fix the underlying problem reported below, then the push will be retried.
 
 ` + "```\n" + combined.trim() + "\n```");
                     if (retryCode !== 0) {
@@ -71119,11 +71199,11 @@ ${e.stderr ?? ""}`;
                       getFailedLogs: (ids) => fetchFailedRunLogs(ids, bunCmdRunner, cwd2),
                       runTaskWithSteering: async (steering) => {
                         try {
-                          await appendSteering(proposalPath, `CI feedback:
+                          await injectFixSteering(changeDir, "Fix failing CI checks", `CI feedback:
 
 ${steering}`);
                         } catch (err) {
-                          appendLog(`! could not append steering: ${err.message}`, "red");
+                          appendLog(`! could not inject steering: ${err.message}`, "red");
                         }
                         const p = Bun.spawn({
                           cmd: buildTaskCmd(),
@@ -71684,14 +71764,11 @@ try {
     }
     try {
       const agentState = await readAgentState(projectRoot);
-      const meta = agentState.changeMeta[args.name];
-      if (meta) {
-        agentState.processedIssueIds = agentState.processedIssueIds.filter((id) => id !== meta.issueId);
-        agentState.startedIssueIds = agentState.startedIssueIds.filter((id) => id !== meta.issueId);
-        agentState.failedIssueIds = agentState.failedIssueIds.filter((id) => id !== meta.issueId);
-        delete agentState.changeMeta[args.name];
+      const entry = Object.values(agentState.tasks).find((t) => t.changeName === args.name);
+      if (entry) {
+        delete agentState.tasks[entry.identifier];
         await writeAgentState(projectRoot, agentState);
-        removed.push(`agent-state entry for ${meta.identifier} (${meta.issueId})`);
+        removed.push(`agent-state entry for ${entry.identifier} (${entry.issueId})`);
       }
     } catch {}
     if (removed.length === 0) {

package/package.json

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