@jitsusama/agentic-harness.core 0.6.2 → 0.6.4

package/dist/review/ask/store.js

@@ -11,8 +11,10 @@
  * council needs that council still to be there, or nothing can say
  * what it consolidated.
  */
+import { randomUUID } from "node:crypto";
 import { mkdir, readdir, readFile, rename, writeFile } from "node:fs/promises";
 import { join } from "node:path";
+import { withFileLock } from "../../internal/file-lock.js";
 import { changeKey } from "../keys.js";
 /** Runs on disk, one file per change. */
 export function createRunStore(root) {
@@ -71,15 +73,29 @@ export function createRunStore(root) {
         // whose whole premise is that the session may not survive it,
         // so the window stopped being theoretical. A rename within one
         // directory is atomic: a reader sees the old ledger or the new
-        // one.
-        const pending = `${path}.${process.pid}.tmp`;
+        // one. Named per write, not per process: two writes in one
+        // process sharing a name tore the file, the shorter write landing
+        // over the start of the longer.
+        const pending = `${path}.${process.pid}.${randomUUID()}.tmp`;
         await writeFile(pending, JSON.stringify(ledger, null, 2), "utf8");
         await rename(pending, path);
     }
+    /**
+     * Read, change and write one change's ledger as a single step. Every
+     * write is a whole ledger built from the one read before it, so two
+     * of these at once, from parallel tool calls or two sessions on one
+     * change, would each lay its version over the other's and drop a
+     * round without an error.
+     */
+    async function mutate(change, next) {
+        await mkdir(root, { recursive: true });
+        await withFileLock(join(root, fileFor(change)), async () => {
+            await write(change, next(await read(change)));
+        });
+    }
     return {
         async record(change, run) {
-            const ledger = await read(change);
-            await write(change, { runs: [...ledger.runs, run] });
+            await mutate(change, (ledger) => ({ runs: [...ledger.runs, run] }));
         },
         async list(change) {
             return (await read(change)).runs;
@@ -176,24 +192,26 @@ export function createRunStore(root) {
             return (await read(change)).runs.find((r) => r.id === runId);
         },
         async keep(change, run) {
-            const ledger = await read(change);
-            const at = ledger.runs.findIndex((held) => held.id === run.id);
-            await write(change, {
-                runs: at === -1
-                    ? [...ledger.runs, run]
-                    : ledger.runs.map((held, index) => (index === at ? run : held)),
+            await mutate(change, (ledger) => {
+                const at = ledger.runs.findIndex((held) => held.id === run.id);
+                return {
+                    runs: at === -1
+                        ? [...ledger.runs, run]
+                        : ledger.runs.map((held, index) => (index === at ? run : held)),
+                };
             });
         },
         async replace(change, run) {
-            const ledger = await read(change);
-            const at = ledger.runs.findIndex((held) => held.id === run.id);
-            if (at === -1) {
-                // Adding it silently would make a retry look like it
-                // patched something when it invented a round instead.
-                throw new Error(`No run "${run.id}" is held against this change, so there is nothing to replace. Record it first, or check the id.`);
-            }
-            await write(change, {
-                runs: ledger.runs.map((held, index) => (index === at ? run : held)),
+            await mutate(change, (ledger) => {
+                const at = ledger.runs.findIndex((held) => held.id === run.id);
+                if (at === -1) {
+                    // Adding it silently would make a retry look like it
+                    // patched something when it invented a round instead.
+                    throw new Error(`No run "${run.id}" is held against this change, so there is nothing to replace. Record it first, or check the id.`);
+                }
+                return {
+                    runs: ledger.runs.map((held, index) => (index === at ? run : held)),
+                };
             });
         },
     };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@jitsusama/agentic-harness.core",
-	"version": "0.6.2",
+	"version": "0.6.4",
 	"description": "Pi-agnostic business logic for agentic-harness: state machines, guardian decisions, quest/TDD domain model.",
 	"license": "MIT",
 	"type": "module",
@@ -45,6 +45,12 @@
 		"./quest/verbs/queries": "./dist/quest/verbs/queries.js",
 		"./quest/verbs/lifecycle": "./dist/quest/verbs/lifecycle.js",
 		"./quest/verbs/tree-ops": "./dist/quest/verbs/tree-ops.js",
+		"./quest/bash-write": "./dist/internal/quest/bash-write.js",
+		"./quest/write-classifier": "./dist/internal/quest/write-classifier.js",
+		"./quest/record": "./dist/internal/quest/record.js",
+		"./quest/workspace": "./dist/internal/quest/workspace.js",
+		"./quest/record-gate": "./dist/internal/quest/record-gate.js",
+		"./quest/record-audit": "./dist/internal/quest/record-audit.js",
 		"./tree": "./dist/tree/index.js",
 		"./lsp": "./dist/lsp/index.js",
 		"./result": "./dist/result/index.js",