@hilbras/omninode 1.0.0 → 2.0.0-alpha.2

package/dist/cli/index.js

@@ -6,7 +6,7 @@ import { realpathSync } from "fs";
 import { Command } from "commander";
 
 // src/version.ts
-var OMNINODE_VERSION = "1.0.0";
+var OMNINODE_VERSION = "2.0.0-alpha.2";
 
 // src/errors/index.ts
 var OmniNodeError = class extends Error {
@@ -1037,7 +1037,19 @@ var ProcessAgent = class {
               new AgentError(
                 "AGENT_TIMEOUT",
                 `Agent "${this.config.name}" timed out after ${timeoutMs}ms and was killed.`,
-                { details: { taskId: input.taskId, stdout: stdout.slice(0, 4e3), stderr: stderr.slice(0, 2e3) } }
+                {
+                  details: {
+                    operation: "agent.run",
+                    agent: this.config.name,
+                    durationMs: timeoutMs,
+                    // The process was started and may have performed side
+                    // effects before being killed — the outcome is unknown,
+                    // not failed (roadmap §6.3).
+                    dispatched: true,
+                    stdout: stdout.slice(0, 4e3),
+                    stderr: stderr.slice(0, 2e3)
+                  }
+                }
               )
             );
             return;
@@ -1302,40 +1314,54 @@ function registerRoleCommands(program) {
   });
 }
 
-// src/tasks/store.ts
+// src/persistence/json-file-store.ts
 import { mkdir, readFile, rename, writeFile } from "fs/promises";
 import { dirname } from "path";
-var FileTaskStore = class {
+var JsonFileStore = class {
   filePath;
-  /**
-   * Read-modify-write operations are serialized: parallel pipeline tasks
-   * would otherwise race on the JSON file (last writer wins).
-   */
+  schemaVersion;
+  onCorrupt;
+  log;
   queue = Promise.resolve();
-  constructor(directory = `${process.cwd()}/.omninode`) {
-    this.filePath = `${directory}/tasks.json`;
+  constructor(options) {
+    this.filePath = `${options.directory ?? `${process.cwd()}/.omninode`}/${options.fileName}`;
+    this.schemaVersion = options.schemaVersion ?? 1;
+    this.onCorrupt = options.onCorrupt;
+    this.log = logger.child({ component: "persistence", file: options.fileName });
   }
-  async save(task) {
-    return this.synchronized(async () => {
-      const tasks = await this.load();
-      const index = tasks.findIndex((t) => t.id === task.id);
-      if (index >= 0) tasks[index] = task;
-      else tasks.push(task);
-      await this.write(tasks);
-    });
+  get path() {
+    return this.filePath;
   }
-  async get(id) {
-    return this.synchronized(async () => {
-      const tasks = await this.load();
-      return tasks.find((t) => t.id === id);
-    });
+  /** All items, or [] when missing/corrupt. */
+  async readAll() {
+    let raw;
+    try {
+      raw = await readFile(this.filePath, "utf8");
+    } catch {
+      return [];
+    }
+    try {
+      const parsed = JSON.parse(raw);
+      if (Array.isArray(parsed)) return parsed;
+      if (parsed !== null && typeof parsed === "object" && Array.isArray(parsed.items)) {
+        return parsed.items;
+      }
+      throw new Error("persisted file has an unrecognized shape");
+    } catch (error) {
+      this.log.warn(`Treating ${this.filePath} as empty \u2014 it could not be parsed.`);
+      this.onCorrupt?.(this.filePath, error);
+      return [];
+    }
   }
-  async list(filter = {}) {
-    return this.synchronized(async () => {
-      const tasks = await this.load();
-      return tasks.filter((task) => filter.status ? task.status === filter.status : true).filter((task) => filter.project ? task.project === filter.project : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
-    });
+  async writeAll(items) {
+    const document = { schemaVersion: this.schemaVersion, items };
+    const tmp = `${this.filePath}.tmp`;
+    await mkdir(dirname(this.filePath), { recursive: true });
+    await writeFile(tmp, `${JSON.stringify(document, null, 2)}
+`, "utf8");
+    await rename(tmp, this.filePath);
   }
+  /** Serializes an async operation against this file. */
   synchronized(operation) {
     const result = this.queue.then(operation, operation);
     this.queue = result.then(
@@ -1344,21 +1370,37 @@ var FileTaskStore = class {
     );
     return result;
   }
-  async load() {
-    try {
-      const raw = await readFile(this.filePath, "utf8");
-      const parsed = JSON.parse(raw);
-      return Array.isArray(parsed) ? parsed : [];
-    } catch {
-      return [];
-    }
+  /** Read-modify-write helper: applies `mutate` to the current items. */
+  async mutate(mutate) {
+    return this.synchronized(async () => {
+      const items = await this.readAll();
+      const [next, result] = await mutate(items);
+      await this.writeAll(next);
+      return result;
+    });
   }
-  async write(tasks) {
-    const tmp = `${this.filePath}.tmp`;
-    await mkdir(dirname(this.filePath), { recursive: true });
-    await writeFile(tmp, `${JSON.stringify(tasks, null, 2)}
-`, "utf8");
-    await rename(tmp, this.filePath);
+};
+
+// src/tasks/store.ts
+var FileTaskStore = class extends JsonFileStore {
+  constructor(directory) {
+    super({ ...directory !== void 0 ? { directory } : {}, fileName: "tasks.json" });
+  }
+  save(task) {
+    return this.mutate((tasks) => {
+      const index = tasks.findIndex((t) => t.id === task.id);
+      if (index >= 0) tasks[index] = task;
+      else tasks.push(task);
+      return [tasks, void 0];
+    });
+  }
+  async get(id) {
+    const tasks = await this.readAll();
+    return tasks.find((t) => t.id === id);
+  }
+  async list(filter = {}) {
+    const tasks = await this.readAll();
+    return tasks.filter((task) => filter.status ? task.status === filter.status : true).filter((task) => filter.project ? task.project === filter.project : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
   }
 };
 
@@ -1367,12 +1409,18 @@ import { randomBytes } from "crypto";
 var ALLOWED_TRANSITIONS = {
   created: ["queued", "running", "cancelled"],
   queued: ["running", "cancelled"],
-  running: ["completed", "failed", "cancelled"],
+  running: ["completed", "failed", "timed_out", "unknown", "cancelled", "partially_completed"],
   waiting: ["running", "cancelled"],
   completed: [],
-  failed: [],
-  cancelled: []
+  // failed → running is the only re-entry: bounded automatic retry (§6.4).
+  failed: ["running"],
+  timed_out: [],
+  cancelled: [],
+  unknown: [],
+  partially_completed: []
 };
+var RETRYABLE_ERROR_CODES = ["AGENT_FAILED"];
+var EXECUTION_HISTORY_LIMIT = 10;
 var TaskEngine = class {
   constructor(options) {
     this.options = options;
@@ -1404,6 +1452,7 @@ var TaskEngine = class {
       id: generateTaskId(),
       objective: input.objective.trim(),
       status: "created",
+      attempt: 0,
       createdAt: now,
       updatedAt: now,
       ...input.project !== void 0 ? { project: input.project } : {},
@@ -1420,8 +1469,10 @@ var TaskEngine = class {
     const task = await this.mustGet(id);
     return this.transition(task, "queued");
   }
-  /** Runs the task through its assigned agent and records the result. */
-  async run(id) {
+  /** Runs the task through its assigned agent and records the result.
+   *  `maxAttempts` enables bounded automatic retries of *retryable* errors
+   *  with linear backoff; unknown/timed_out outcomes are never auto-retried. */
+  async run(id, options = {}) {
     const task = await this.mustGet(id);
     if (task.status !== "created" && task.status !== "queued") {
       throw new TaskError("TASK_INVALID", `Task ${id} is "${task.status}" and cannot be run.`);
@@ -1434,49 +1485,84 @@ var TaskEngine = class {
       throw new TaskError("TASK_INVALID", `Agent "${task.agent}" is not registered anymore.`);
     }
     const role = task.role ? this.options.roles.get(task.role) : void 0;
-    const running = await this.transition(task, "running");
     this.log.info(`Running task ${task.id} on agent "${task.agent}".`);
-    let background = running.context?.background;
+    let background = task.context?.background;
     if (this.options.memory) {
       const memoryContext = await this.options.memory.gatherContext({
-        objective: running.objective,
-        ...running.role ? { role: running.role } : {},
-        ...running.project ? { project: running.project } : {}
+        objective: task.objective,
+        ...task.role ? { role: task.role } : {},
+        ...task.project ? { project: task.project } : {}
       });
       if (memoryContext.length > 0) {
         background = [memoryContext, background].filter((part) => part && part.length > 0).join("\n\n");
       }
     }
-    let output;
-    try {
-      output = await agent.run({
-        taskId: running.id,
-        objective: running.objective,
-        ...role ? { role } : {},
-        ...background !== void 0 ? { context: background } : {}
-      });
-    } catch (error) {
-      const message = error instanceof Error ? error.message : String(error);
-      this.log.error(`Task ${task.id} failed: ${message}`);
-      return this.transition(running, "failed", {
-        result: { error: message, finishedAt: (/* @__PURE__ */ new Date()).toISOString() }
+    const maxAttempts = Math.max(1, options.maxAttempts ?? 1);
+    const backoffMs = options.retryBackoffMs ?? 0;
+    let current = task;
+    for (let i = 0; i < maxAttempts; i += 1) {
+      const attempt = current.attempt + 1;
+      const executionId = `exec-${Date.now().toString(36)}-${randomBytes(3).toString("hex")}`;
+      const startedAt = (/* @__PURE__ */ new Date()).toISOString();
+      current = await this.transition(current, "running", { attempt });
+      let output;
+      let thrown;
+      try {
+        output = await agent.run({
+          taskId: current.id,
+          objective: current.objective,
+          ...role ? { role } : {},
+          ...background !== void 0 ? { context: background } : {}
+        });
+      } catch (error) {
+        thrown = error;
+      }
+      const finishedAt = (/* @__PURE__ */ new Date()).toISOString();
+      if (thrown !== void 0 || output === void 0) {
+        const outcome = classifyThrown(thrown);
+        const message = thrown instanceof Error ? thrown.message : String(thrown);
+        const retryable = outcome === "failed" && RETRYABLE_ERROR_CODES.includes(errorCode(thrown)) && i < maxAttempts - 1;
+        current = await this.transition(current, outcome, {
+          attempt,
+          lastError: message,
+          execution: {
+            executionId,
+            attempt,
+            startedAt,
+            finishedAt,
+            outcome,
+            error: message
+          }
+        });
+        this.log.warn(`Task ${current.id} attempt ${attempt} \u2192 ${outcome}: ${message}`);
+        if (retryable) {
+          if (backoffMs > 0) await sleep(backoffMs * attempt);
+          continue;
+        }
+        if (this.options.memory) await this.options.memory.recordTaskOutcome(current);
+        return current;
+      }
+      const result = {
+        ...output.summary !== void 0 ? { summary: output.summary } : {},
+        ...output.reports !== void 0 && output.reports.length > 0 ? { reports: output.reports } : {},
+        // Keep the verbatim output (bounded) for report extraction and audit.
+        ...output.rawOutput !== void 0 && output.rawOutput.length > 0 ? { rawOutput: output.rawOutput.slice(0, 1e4) } : {},
+        ...output.error !== void 0 ? { error: output.error } : {},
+        finishedAt
+      };
+      const finalStatus = output.status === "completed" ? "completed" : "failed";
+      current = await this.transition(current, finalStatus, {
+        attempt,
+        result,
+        execution: { executionId, attempt, startedAt, finishedAt, outcome: finalStatus }
       });
+      this.log.info(`Task ${current.id} finished: ${finalStatus}.`);
+      break;
     }
-    const result = {
-      ...output.summary !== void 0 ? { summary: output.summary } : {},
-      ...output.reports !== void 0 && output.reports.length > 0 ? { reports: output.reports } : {},
-      // Keep the verbatim output (bounded) for report extraction and audit.
-      ...output.rawOutput !== void 0 && output.rawOutput.length > 0 ? { rawOutput: output.rawOutput.slice(0, 1e4) } : {},
-      ...output.error !== void 0 ? { error: output.error } : {},
-      finishedAt: (/* @__PURE__ */ new Date()).toISOString()
-    };
-    const finalStatus = output.status === "completed" ? "completed" : "failed";
-    const finished = await this.transition(running, finalStatus, { result });
-    this.log.info(`Task ${task.id} finished: ${finalStatus}.`);
     if (this.options.memory) {
-      await this.options.memory.recordTaskOutcome(finished);
+      await this.options.memory.recordTaskOutcome(current);
     }
-    return finished;
+    return current;
   }
   async cancel(id) {
     const task = await this.mustGet(id);
@@ -1489,13 +1575,14 @@ var TaskEngine = class {
     }
     return this.transition(task, "cancelled");
   }
-  /** Error recovery (§27 Phase 9): reset a failed/cancelled task so it can run again. */
+  /** Error recovery: reset a failed/cancelled/unknown/timed_out task so it can run again. */
   async retry(id) {
     const task = await this.mustGet(id);
-    if (task.status !== "failed" && task.status !== "cancelled") {
+    const retryableStates = ["failed", "cancelled", "unknown", "timed_out"];
+    if (!retryableStates.includes(task.status)) {
       throw new TaskError(
         "TASK_INVALID",
-        `Only failed or cancelled tasks can be retried \u2014 task ${id} is "${task.status}".`
+        `Only failed, timed out, unknown or cancelled tasks can be retried \u2014 task ${id} is "${task.status}".`
       );
     }
     const { result: _discarded, ...rest } = task;
@@ -1527,10 +1614,14 @@ var TaskEngine = class {
     if (!allowed.includes(to)) {
       throw new TaskError("TASK_INVALID", `Task ${task.id} cannot move from "${task.status}" to "${to}".`);
     }
+    const executions = extra.execution ? [...task.executions ?? [], extra.execution].slice(-EXECUTION_HISTORY_LIMIT) : task.executions;
     const updated = {
       ...task,
       status: to,
       updatedAt: (/* @__PURE__ */ new Date()).toISOString(),
+      ...extra.attempt !== void 0 ? { attempt: extra.attempt } : {},
+      ...extra.lastError !== void 0 ? { lastError: extra.lastError } : {},
+      ...executions !== void 0 ? { executions } : {},
       ...extra.result !== void 0 ? { result: { ...task.result, ...extra.result } } : {}
     };
     await this.options.store.save(updated);
@@ -1551,6 +1642,19 @@ var TaskEngine = class {
 function generateTaskId() {
   return `task-${Date.now().toString(36)}-${randomBytes(3).toString("hex")}`;
 }
+function errorCode(error) {
+  return error?.code ?? "";
+}
+function classifyThrown(error) {
+  if (error instanceof AgentError && error.code === "AGENT_TIMEOUT") {
+    const dispatched = error.details?.dispatched;
+    return dispatched === false ? "timed_out" : "unknown";
+  }
+  return "failed";
+}
+function sleep(ms) {
+  return new Promise((resolve) => setTimeout(resolve, ms));
+}
 
 // src/audit/index.ts
 import { appendFile, mkdir as mkdir2 } from "fs/promises";
@@ -1575,10 +1679,10 @@ var FileAuditLog = class {
   }
   /** Recent events, oldest first; used by the CLI. */
   async recent(limit = 20) {
-    const { readFile: readFile6 } = await import("fs/promises");
+    const { readFile: readFile2 } = await import("fs/promises");
     let raw;
     try {
-      raw = await readFile6(this.filePath, "utf8");
+      raw = await readFile2(this.filePath, "utf8");
     } catch {
       return [];
     }
@@ -1594,8 +1698,6 @@ var FileAuditLog = class {
 };
 
 // src/memory/local.ts
-import { mkdir as mkdir3, readFile as readFile2, rename as rename2, writeFile as writeFile2 } from "fs/promises";
-import { dirname as dirname3 } from "path";
 var STOPWORDS = /* @__PURE__ */ new Set([
   "the",
   "and",
@@ -1652,29 +1754,24 @@ function relevanceScore(queryKeywords, entry2) {
   }
   return overlap * 2 + tagMatches * 1.5;
 }
-var LocalMemoryProvider = class {
+var LocalMemoryProvider = class extends JsonFileStore {
   name = "local";
-  filePath;
-  queue = Promise.resolve();
-  constructor(directory = `${process.cwd()}/.omninode`) {
-    this.filePath = `${directory}/memory.json`;
+  constructor(directory) {
+    super({ ...directory !== void 0 ? { directory } : {}, fileName: "memory.json" });
   }
   async query(query) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
-      const filtered = entries.filter((entry2) => query.scope ? entry2.scope === query.scope : true).filter(
-        (entry2) => query.tags && query.tags.length > 0 ? query.tags.every((tag) => (entry2.tags ?? []).includes(tag)) : true
-      );
-      if (!query.text || query.text.trim().length === 0) {
-        return filtered.sort((a, b) => (b.updatedAt ?? b.createdAt ?? "").localeCompare(a.updatedAt ?? a.createdAt ?? "")).slice(0, query.limit ?? 10);
-      }
-      const queryKeywords = keywords(query.text);
-      return filtered.map((entry2) => ({ entry: entry2, score: relevanceScore(queryKeywords, entry2) })).filter(({ score }) => score > 0).sort((a, b) => b.score - a.score).slice(0, query.limit ?? 10).map(({ entry: entry2 }) => entry2);
-    });
+    const entries = await this.readAll();
+    const filtered = entries.filter((entry2) => query.scope ? entry2.scope === query.scope : true).filter(
+      (entry2) => query.tags && query.tags.length > 0 ? query.tags.every((tag) => (entry2.tags ?? []).includes(tag)) : true
+    );
+    if (!query.text || query.text.trim().length === 0) {
+      return filtered.sort((a, b) => (b.updatedAt ?? b.createdAt ?? "").localeCompare(a.updatedAt ?? a.createdAt ?? "")).slice(0, query.limit ?? 10);
+    }
+    const queryKeywords = keywords(query.text);
+    return filtered.map((entry2) => ({ entry: entry2, score: relevanceScore(queryKeywords, entry2) })).filter(({ score }) => score > 0).sort((a, b) => b.score - a.score).slice(0, query.limit ?? 10).map(({ entry: entry2 }) => entry2);
   }
-  async write(entry2) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
+  write(entry2) {
+    return this.mutate((entries) => {
       const index = entries.findIndex((e) => e.key === entry2.key);
       const stamped = {
         ...entry2,
@@ -1683,35 +1780,11 @@ var LocalMemoryProvider = class {
       };
       if (index >= 0) entries[index] = stamped;
       else entries.push(stamped);
-      await this.saveAll(entries);
+      return [entries, void 0];
     });
   }
   async count() {
-    return (await this.load()).length;
-  }
-  synchronized(operation) {
-    const result = this.queue.then(operation, operation);
-    this.queue = result.then(
-      () => void 0,
-      () => void 0
-    );
-    return result;
-  }
-  async load() {
-    try {
-      const raw = await readFile2(this.filePath, "utf8");
-      const parsed = JSON.parse(raw);
-      return Array.isArray(parsed) ? parsed : [];
-    } catch {
-      return [];
-    }
-  }
-  async saveAll(entries) {
-    const tmp = `${this.filePath}.tmp`;
-    await mkdir3(dirname3(this.filePath), { recursive: true });
-    await writeFile2(tmp, `${JSON.stringify(entries, null, 2)}
-`, "utf8");
-    await rename2(tmp, this.filePath);
+    return (await this.readAll()).length;
   }
 };
 
@@ -1981,7 +2054,7 @@ function registerTaskCommands(program) {
     const cancelled = await engine.cancel(taskId);
     console.log(`Task ${cancelled.id} is now "${cancelled.status}".`);
   });
-  task.command("retry <task-id>").description("Reset a failed/cancelled task to created; combine with --run to execute it again.").option("--run", "Run the task again immediately after resetting it.").action(async (taskId, options) => {
+  task.command("retry <task-id>").description("Reset a failed, timed out, unknown or cancelled task to created; combine with --run to execute it again.").option("--run", "Run the task again immediately after resetting it.").action(async (taskId, options) => {
     const config = loadConfig();
     const engine = createTaskEngine(config);
     const reset = await engine.retry(taskId);
@@ -2004,6 +2077,15 @@ function printOutcome(task, options = {}) {
     if (task.context?.files) console.log(`  files:   ${task.context.files.join(", ")}`);
     console.log(`  objective: ${task.objective}`);
   }
+  if (options.verbose && task.attempt > 0) {
+    console.log(`  attempt: ${task.attempt}`);
+  }
+  if (task.lastError) console.log(`  error:   ${task.lastError}`);
+  if (options.verbose && task.executions && task.executions.length > 0) {
+    console.log(
+      `  executions: ${task.executions.map((e) => `#${e.attempt}:${e.outcome}`).join(", ")}`
+    );
+  }
   const result = task.result;
   if (!result) return;
   if (result.summary) console.log(`  summary: ${result.summary}`);
@@ -2228,84 +2310,39 @@ var ReportService = class {
 };
 
 // src/reports/store.ts
-import { mkdir as mkdir4, readFile as readFile3, rename as rename3, writeFile as writeFile3 } from "fs/promises";
-import { dirname as dirname4 } from "path";
-var FileReportStore = class {
-  filePath;
-  queue = Promise.resolve();
-  constructor(directory = `${process.cwd()}/.omninode`) {
-    this.filePath = `${directory}/reports.json`;
+var FileReportStore = class extends JsonFileStore {
+  constructor(directory) {
+    super({ ...directory !== void 0 ? { directory } : {}, fileName: "reports.json" });
   }
   async saveReports(reports) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
+    await this.mutate((entries) => {
       for (const report of reports) {
-        const index = entries.findIndex(
-          (e) => e.kind === "report" && e.report.id === report.id
-        );
+        const index = entries.findIndex((e) => e.kind === "report" && e.report.id === report.id);
         const entry2 = { kind: "report", report };
         if (index >= 0) entries[index] = entry2;
         else entries.push(entry2);
       }
-      await this.write(entries);
+      return [entries, void 0];
     });
   }
   async listReports(filter = {}) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
-      return entries.filter((e) => e.kind === "report").map((e) => e.report).filter((report) => filter.taskId ? report.taskId === filter.taskId : true).filter((report) => filter.agent ? report.agent === filter.agent : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
-    });
+    return (await this.readAll()).filter((e) => e.kind === "report").map((e) => e.report).filter((report) => filter.taskId ? report.taskId === filter.taskId : true).filter((report) => filter.agent ? report.agent === filter.agent : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
   }
   async saveCombined(combined) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
-      const index = entries.findIndex(
-        (e) => e.kind === "combined" && e.combined.id === combined.id
-      );
+    await this.mutate((entries) => {
+      const index = entries.findIndex((e) => e.kind === "combined" && e.combined.id === combined.id);
       const entry2 = { kind: "combined", combined };
       if (index >= 0) entries[index] = entry2;
       else entries.push(entry2);
-      await this.write(entries);
+      return [entries, void 0];
     });
   }
   async getCombined(id) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
-      const found = entries.find((e) => e.kind === "combined" && e.combined.id === id);
-      return found?.kind === "combined" ? found.combined : void 0;
-    });
+    const found = (await this.readAll()).find((e) => e.kind === "combined" && e.combined.id === id);
+    return found?.kind === "combined" ? found.combined : void 0;
   }
   async listCombined(filter = {}) {
-    return this.synchronized(async () => {
-      const entries = await this.load();
-      return entries.filter((e) => e.kind === "combined").map((e) => e.combined).filter(
-        (combined) => filter.pipelineRunId ? combined.pipelineRunId === filter.pipelineRunId : true
-      ).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
-    });
-  }
-  synchronized(operation) {
-    const result = this.queue.then(operation, operation);
-    this.queue = result.then(
-      () => void 0,
-      () => void 0
-    );
-    return result;
-  }
-  async load() {
-    try {
-      const raw = await readFile3(this.filePath, "utf8");
-      const parsed = JSON.parse(raw);
-      return Array.isArray(parsed) ? parsed : [];
-    } catch {
-      return [];
-    }
-  }
-  async write(entries) {
-    const tmp = `${this.filePath}.tmp`;
-    await mkdir4(dirname4(this.filePath), { recursive: true });
-    await writeFile3(tmp, `${JSON.stringify(entries, null, 2)}
-`, "utf8");
-    await rename3(tmp, this.filePath);
+    return (await this.readAll()).filter((e) => e.kind === "combined").map((e) => e.combined).filter((combined) => filter.pipelineRunId ? combined.pipelineRunId === filter.pipelineRunId : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
   }
 };
 
@@ -2789,7 +2826,7 @@ ${contribution}`);
         }
         const stepRun = {
           stepId: step.id,
-          status: outcome.status === "skipped" ? "cancelled" : outcome.status === "failed" ? "failed" : "completed",
+          status: outcome.status === "skipped" ? "cancelled" : outcome.status === "failed" ? "failed" : outcome.status === "partial" ? "partially_completed" : "completed",
           ...outcome.taskIds.length > 0 ? { taskIds: outcome.taskIds } : {},
           ...outcome.planId !== void 0 ? { planId: outcome.planId } : {},
           ...outcome.error !== void 0 ? { error: outcome.error } : {}
@@ -2800,8 +2837,10 @@ ${contribution}`);
       run2.reports = [...allReports];
       await this.options.store.save(run2);
     }
-    const failed = [...outcomes.values()].some((o) => o.status === "failed");
-    const finalStatus = failed ? "failed" : "completed";
+    const allOutcomes = [...outcomes.values()];
+    const failed = allOutcomes.some((o) => o.status === "failed");
+    const partial = !failed && allOutcomes.some((o) => o.status === "partial");
+    const finalStatus = failed ? "failed" : partial ? "partial" : "completed";
     const planId = [...outcomes.values()].find((o) => o.planId !== void 0)?.planId;
     let resultSummary;
     for (let i = def.steps.length - 1; i >= 0; i -= 1) {
@@ -2907,6 +2946,7 @@ ${contribution}`);
       )
     );
     const results = await Promise.all(created.map((task) => this.options.tasks.run(task.id)));
+    const completedTasks = results.filter((r) => r.status === "completed");
     const failedTasks = results.filter((r) => r.status !== "completed");
     const lines = results.map((result, index) => {
       const agentName = agentNames[index] ?? "unknown";
@@ -2916,12 +2956,15 @@ ${contribution}`);
     const reports = results.flatMap((r) => r.result?.reports ?? []);
     const summary = lines.join("\n");
     if (failedTasks.length > 0) {
-      const firstError = failedTasks[0]?.result?.error ?? "unknown error";
+      const firstError = failedTasks[0]?.result?.error ?? failedTasks[0]?.lastError ?? "unknown outcome";
+      const states = failedTasks.map((r) => r.status).join(", ");
       return {
-        status: "failed",
+        // Some agents succeeded → the step produced usable but incomplete
+        // intelligence: partial, not failed (roadmap §9.3).
+        status: completedTasks.length > 0 ? "partial" : "failed",
         summary,
         taskIds: created.map((t) => t.id),
-        error: `${failedTasks.length}/${results.length} task(s) failed \u2014 ${firstError}`,
+        error: `${failedTasks.length}/${results.length} task(s) incomplete (${states}) \u2014 ${firstError}`,
         reports
       };
     }
@@ -2937,7 +2980,9 @@ ${contribution}`);
     const result = await this.options.tasks.run(task.id);
     if (result.status !== "completed") {
       return {
-        status: "failed",
+        // Unknown/timed_out outcomes are not failures — the work may have
+        // happened remotely; the step is incomplete, not broken.
+        status: result.status === "unknown" || result.status === "timed_out" ? "partial" : "failed",
         summary: result.result?.summary ?? "",
         taskIds: [task.id],
         error: result.result?.error ?? `task ${task.id} did not complete`,
@@ -3053,114 +3098,44 @@ function createDefaultChatFn(providers) {
 }
 
 // src/pipelines/store.ts
-import { mkdir as mkdir5, readFile as readFile4, rename as rename4, writeFile as writeFile4 } from "fs/promises";
-import { dirname as dirname5 } from "path";
-var FilePipelineRunStore = class {
-  filePath;
-  queue = Promise.resolve();
-  constructor(directory = `${process.cwd()}/.omninode`) {
-    this.filePath = `${directory}/pipelines.json`;
+var FilePipelineRunStore = class extends JsonFileStore {
+  constructor(directory) {
+    super({ ...directory !== void 0 ? { directory } : {}, fileName: "pipelines.json" });
   }
-  async save(run2) {
-    return this.synchronized(async () => {
-      const runs = await this.load();
+  save(run2) {
+    return this.mutate((runs) => {
       const index = runs.findIndex((r) => r.id === run2.id);
       if (index >= 0) runs[index] = run2;
       else runs.push(run2);
-      await this.write(runs);
+      return [runs, void 0];
     });
   }
   async get(id) {
-    return this.synchronized(async () => {
-      const runs = await this.load();
-      return runs.find((r) => r.id === id);
-    });
+    return (await this.readAll()).find((r) => r.id === id);
   }
   async list(filter = {}) {
-    return this.synchronized(async () => {
-      const runs = await this.load();
-      return runs.filter((run2) => filter.pipelineId ? run2.pipelineId === filter.pipelineId : true).filter((run2) => filter.status ? run2.status === filter.status : true).sort((a, b) => (a.startedAt ?? "").localeCompare(b.startedAt ?? ""));
-    });
-  }
-  synchronized(operation) {
-    const result = this.queue.then(operation, operation);
-    this.queue = result.then(
-      () => void 0,
-      () => void 0
-    );
-    return result;
-  }
-  async load() {
-    try {
-      const raw = await readFile4(this.filePath, "utf8");
-      const parsed = JSON.parse(raw);
-      return Array.isArray(parsed) ? parsed : [];
-    } catch {
-      return [];
-    }
-  }
-  async write(runs) {
-    const tmp = `${this.filePath}.tmp`;
-    await mkdir5(dirname5(this.filePath), { recursive: true });
-    await writeFile4(tmp, `${JSON.stringify(runs, null, 2)}
-`, "utf8");
-    await rename4(tmp, this.filePath);
+    return (await this.readAll()).filter((run2) => filter.pipelineId ? run2.pipelineId === filter.pipelineId : true).filter((run2) => filter.status ? run2.status === filter.status : true).sort((a, b) => (a.startedAt ?? "").localeCompare(b.startedAt ?? ""));
   }
 };
 
 // src/planner/store.ts
-import { mkdir as mkdir6, readFile as readFile5, rename as rename5, writeFile as writeFile5 } from "fs/promises";
-import { dirname as dirname6 } from "path";
-var FilePlanStore = class {
-  filePath;
-  queue = Promise.resolve();
-  constructor(directory = `${process.cwd()}/.omninode`) {
-    this.filePath = `${directory}/plans.json`;
+var FilePlanStore = class extends JsonFileStore {
+  constructor(directory) {
+    super({ ...directory !== void 0 ? { directory } : {}, fileName: "plans.json" });
   }
-  async save(plan) {
-    return this.synchronized(async () => {
-      const plans = await this.load();
+  save(plan) {
+    return this.mutate((plans) => {
       const index = plans.findIndex((p) => p.id === plan.id);
       if (index >= 0) plans[index] = plan;
       else plans.push(plan);
-      await this.write(plans);
+      return [plans, void 0];
     });
   }
   async get(id) {
-    return this.synchronized(async () => {
-      const plans = await this.load();
-      return plans.find((p) => p.id === id);
-    });
+    return (await this.readAll()).find((p) => p.id === id);
   }
   async list(filter = {}) {
-    return this.synchronized(async () => {
-      const plans = await this.load();
-      return plans.filter((plan) => filter.pipelineRunId ? plan.pipelineRunId === filter.pipelineRunId : true).filter((plan) => filter.taskId ? plan.taskId === filter.taskId : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
-    });
-  }
-  synchronized(operation) {
-    const result = this.queue.then(operation, operation);
-    this.queue = result.then(
-      () => void 0,
-      () => void 0
-    );
-    return result;
-  }
-  async load() {
-    try {
-      const raw = await readFile5(this.filePath, "utf8");
-      const parsed = JSON.parse(raw);
-      return Array.isArray(parsed) ? parsed : [];
-    } catch {
-      return [];
-    }
-  }
-  async write(plans) {
-    const tmp = `${this.filePath}.tmp`;
-    await mkdir6(dirname6(this.filePath), { recursive: true });
-    await writeFile5(tmp, `${JSON.stringify(plans, null, 2)}
-`, "utf8");
-    await rename5(tmp, this.filePath);
+    return (await this.readAll()).filter((plan) => filter.pipelineRunId ? plan.pipelineRunId === filter.pipelineRunId : true).filter((plan) => filter.taskId ? plan.taskId === filter.taskId : true).sort((a, b) => a.createdAt.localeCompare(b.createdAt));
   }
 };