@oisincoveney/pipeline 2.11.1 → 2.11.2

package/dist/run-control/run-state-lock.js

@@ -0,0 +1,41 @@
+//#region src/run-control/run-state-lock.ts
+/**
+* Process-wide serialization for critical sections that read, write, or
+* temporarily relocate the `.pipeline/runs` run-state directory.
+*
+* The mechanical-check builtins (`lint`, `fallow`) hide `.pipeline/runs` for the
+* duration of their command so those tools do not scan supervisor run-state
+* (see builtins.ts `hidePipelineRunsDirectory`). Under the parallel
+* mechanical-checks fan-out that hide window overlapped the run-control
+* reporter's persistence of sibling node-status events, which then observed a
+* momentarily-missing run directory and failed the whole run with
+* "Run <id> does not exist". This lock makes the hide window and run-state
+* persistence mutually exclusive without serializing any unrelated parallel
+* work: only the run-state critical sections contend for it.
+*/
+let chain = Promise.resolve();
+/**
+* Acquire the lock, resolving with a release function once every previously
+* queued holder has released. Callers MUST invoke the returned release exactly
+* once (use `withRunStateLock` unless you must span non-promise boundaries, as
+* the builtin hide/restore does).
+*/
+function acquireRunStateLock() {
+	const previous = chain;
+	let release = () => {};
+	chain = new Promise((resolve) => {
+		release = resolve;
+	});
+	return previous.then(() => release);
+}
+/** Run `fn` while holding the run-state lock, releasing on success or failure. */
+async function withRunStateLock(fn) {
+	const release = await acquireRunStateLock();
+	try {
+		return await fn();
+	} finally {
+		release();
+	}
+}
+//#endregion
+export { acquireRunStateLock, withRunStateLock };

package/dist/run-control/runtime-reporter.js

@@ -1,3 +1,4 @@
+import { withRunStateLock } from "./run-state-lock.js";
 import { updateNodeSessionEffect, updateNodeStatusEffect, updateRunStatusEffect } from "./store.js";
 import { Effect } from "effect";
 import { isAbsolute, join } from "node:path";
@@ -23,7 +24,7 @@ function createRunStoreRuntimeReporterRuntime(input) {
 			activeHookPreviousStatuses,
 			observedNodeStatuses
 		});
-		writeChain = writeChain.then(() => Effect.runPromise(persistRuntimeEventEffect(input, event, projection, now)));
+		writeChain = writeChain.then(() => withRunStateLock(() => Effect.runPromise(persistRuntimeEventEffect(input, event, projection, now))));
 	};
 	const flushEffect = () => Effect.tryPromise({
 		catch: (error) => error,

package/dist/runtime/builtins/builtins.js

@@ -1,4 +1,5 @@
 import { parseJson } from "../../safe-json.js";
+import { acquireRunStateLock } from "../../run-control/run-state-lock.js";
 import { executeDrainMergeBuiltin } from "../drain-merge/drain-merge.js";
 import "../drain-merge/index.js";
 import { CommandExecutor, CommandExecutorLive } from "../services/command-executor-service.js";
@@ -95,7 +96,7 @@ function executeVisibleProjectCommand(command, context) {
 	});
 }
 function withHiddenPipelineRuns(worktreePath, effect) {
-	return Effect.acquireUseRelease(Effect.sync(() => hidePipelineRunsDirectory(worktreePath)), () => effect, (hiddenRuns) => Effect.sync(() => hiddenRuns?.restore()));
+	return Effect.acquireUseRelease(Effect.promise(() => acquireRunStateLock()), () => Effect.acquireUseRelease(Effect.sync(() => hidePipelineRunsDirectory(worktreePath)), () => effect, (hiddenRuns) => Effect.sync(() => hiddenRuns?.restore())), (release) => Effect.sync(() => release()));
 }
 function resolveRequiredScriptCommand(worktreePath, envName, scriptName) {
 	return Effect.gen(function* () {

package/package.json

@@ -127,7 +127,7 @@
     "prepack": "bun run build:cli"
   },
   "type": "module",
-  "version": "2.11.1",
+  "version": "2.11.2",
   "description": "Config-driven multi-agent pipeline runner for repository work",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",