@neriros/ralphy 2.9.4 → 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.4",
+  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) {
@@ -70936,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);
           }
@@ -71690,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.4",
+  "version": "2.10.0",
   "description": "An iterative AI task execution framework. Orchestrates multi-phase autonomous work using Claude or Codex engines.",
   "keywords": [
     "agent",