@oisincoveney/pipeline 3.11.4 → 3.11.6

package/dist/argo-graph.js

@@ -1,4 +1,5 @@
 import { uniqueStrings } from "./strings.js";
+import { resolveExecutableDependencyIds } from "./planning/dependency-refs.js";
 import { z } from "zod";
 import { Data } from "effect";
 //#region src/argo-graph.ts
@@ -88,26 +89,7 @@ var ArgoGraphCompiler = class {
 		this.compileNodes(node.children ?? [], [...inheritedNeeds, ...node.needs]);
 	}
 	resolveDependencyTaskNames(nodeIds) {
-		return uniqueStrings(nodeIds.flatMap((nodeId) => this.resolveDependencyNodeIds(nodeId).map((id) => argoTaskName(id))));
-	}
-	resolveDependencyNodeIds(nodeId) {
-		const node = this.nodeById.get(nodeId);
-		if (!node) return [];
-		const kind = node.kind;
-		switch (kind) {
-			case "agent":
-			case "builtin":
-			case "command": return [node.id];
-			case "group": return this.resolveGroupNodeIds(node);
-			case "parallel": return this.resolveParallelNodeIds(node);
-			default: throw new ArgoGraphCompilerError(String(kind), node.id);
-		}
-	}
-	resolveGroupNodeIds(node) {
-		return uniqueStrings([...node.nodes ?? [], ...node.needs].flatMap((id) => this.resolveDependencyNodeIds(id)));
-	}
-	resolveParallelNodeIds(node) {
-		return uniqueStrings((node.children ?? []).flatMap((child) => this.resolveDependencyNodeIds(child.id)));
+		return uniqueStrings(resolveExecutableDependencyIds(this.nodeById, nodeIds).map((id) => argoTaskName(id)));
 	}
 	terminalTasks() {
 		const dependedOn = new Set(this.tasks.flatMap((task) => task.dependencies));

package/dist/argo-workflow.js

@@ -13,7 +13,7 @@ const RUNNER_WORKFLOW_PAYLOAD_PATH = "/etc/pipeline/payload.json";
 const RUNNER_WORKFLOW_SCHEDULE_PATH = "/etc/pipeline/schedule.yaml";
 const RUNNER_GIT_CREDENTIALS_PATH = "/etc/pipeline/git-credentials";
 const RUNNER_RETRY_STRATEGY = {
-	expression: "lastRetry.status == 'Error' || asInt(lastRetry.exitCode) == 70",
+	expression: "lastRetry.status == 'Error' || lastRetry.exitCode == '70' || lastRetry.exitCode == '137'",
 	limit: "3",
 	retryPolicy: "Always"
 };

package/dist/moka-submit.d.ts

@@ -5,13 +5,13 @@ import { z } from "zod";
 //#region src/moka-submit.d.ts
 declare const mokaSubmitDirectHooksSchema: z.ZodRecord<z.ZodEnum<{
   "workflow.start": "workflow.start";
+  "node.finish": "node.finish";
+  "node.start": "node.start";
   "workflow.success": "workflow.success";
   "workflow.failure": "workflow.failure";
   "workflow.complete": "workflow.complete";
-  "node.start": "node.start";
   "node.success": "node.success";
   "node.error": "node.error";
-  "node.finish": "node.finish";
   "gate.failure": "gate.failure";
 }> & z.core.$partial, z.ZodDiscriminatedUnion<[z.ZodObject<{
   failure: z.ZodDefault<z.ZodEnum<{
@@ -94,13 +94,13 @@ declare const mokaSubmitOptionsSchema: z.ZodDiscriminatedUnion<[z.ZodObject<{
   }, z.core.$strict>>;
   hooks: z.ZodOptional<z.ZodRecord<z.ZodEnum<{
     "workflow.start": "workflow.start";
+    "node.finish": "node.finish";
+    "node.start": "node.start";
     "workflow.success": "workflow.success";
     "workflow.failure": "workflow.failure";
     "workflow.complete": "workflow.complete";
-    "node.start": "node.start";
     "node.success": "node.success";
     "node.error": "node.error";
-    "node.finish": "node.finish";
     "gate.failure": "gate.failure";
   }> & z.core.$partial, z.ZodDiscriminatedUnion<[z.ZodObject<{
     failure: z.ZodDefault<z.ZodEnum<{
@@ -207,13 +207,13 @@ declare const mokaSubmitOptionsSchema: z.ZodDiscriminatedUnion<[z.ZodObject<{
   }, z.core.$strict>>;
   hooks: z.ZodOptional<z.ZodRecord<z.ZodEnum<{
     "workflow.start": "workflow.start";
+    "node.finish": "node.finish";
+    "node.start": "node.start";
     "workflow.success": "workflow.success";
     "workflow.failure": "workflow.failure";
     "workflow.complete": "workflow.complete";
-    "node.start": "node.start";
     "node.success": "node.success";
     "node.error": "node.error";
-    "node.finish": "node.finish";
     "gate.failure": "gate.failure";
   }> & z.core.$partial, z.ZodDiscriminatedUnion<[z.ZodObject<{
     failure: z.ZodDefault<z.ZodEnum<{

package/dist/planning/dependency-refs.js

@@ -0,0 +1,43 @@
+import { uniqueStrings } from "../strings.js";
+//#region src/planning/dependency-refs.ts
+/**
+* Index every planned node — including the children nested inside `parallel`
+* containers — by id, so a dependency id can be resolved regardless of nesting.
+*/
+function indexPlannedNodesById(nodes, into = /* @__PURE__ */ new Map()) {
+	for (const node of nodes) {
+		into.set(node.id, node);
+		indexPlannedNodesById(node.children ?? [], into);
+	}
+	return into;
+}
+/**
+* Expand dependency ids to the executable (branch-producing) leaf node ids,
+* making `group`/`parallel` container nodes transparent.
+*
+* A container produces no output of its own: a `parallel` lowers to its children
+* and a `group` is a pure dependency anchor, so neither pushes a `nodes/<id>`
+* result branch. A downstream node that lists the container in `needs` actually
+* depends on the container's executable descendants. Both the Argo DAG ordering
+* and the runner's upstream-output (git ref) materialization resolve dependencies
+* through this one function so the two representations never diverge — divergence
+* is what made review nodes fetch a non-existent `nodes/mechanical-checks` branch.
+*/
+function resolveExecutableDependencyIds(nodeById, needs) {
+	const resolveOne = (nodeId) => {
+		const node = nodeById.get(nodeId);
+		if (!node) return [];
+		const kind = node.kind;
+		switch (kind) {
+			case "agent":
+			case "builtin":
+			case "command": return [node.id];
+			case "group": return uniqueStrings([...node.nodes ?? [], ...node.needs].flatMap(resolveOne));
+			case "parallel": return uniqueStrings((node.children ?? []).flatMap((child) => resolveOne(child.id)));
+			default: throw new Error(`resolveExecutableDependencyIds: unsupported node kind '${String(kind)}' on '${node.id}'`);
+		}
+	};
+	return uniqueStrings(needs.flatMap(resolveOne));
+}
+//#endregion
+export { indexPlannedNodesById, resolveExecutableDependencyIds };

package/dist/runner-command/run.js

@@ -1,3 +1,4 @@
+import { indexPlannedNodesById, resolveExecutableDependencyIds } from "../planning/dependency-refs.js";
 import { loadPipelineConfig } from "../config/load.js";
 import "../config.js";
 import { findPlannedNode } from "../planned-node.js";
@@ -120,20 +121,21 @@ function runRunnerCommandEffect(options, runtime) {
 			workflowId: compiled.workflowId
 		}, "schedule.compile finish");
 		const node = yield* resolveRunnerTargetNode(payload, compiled, descriptor);
+		const dependencyNodeIds = resolveExecutableDependencyIds(indexPlannedNodesById(compiled.plan.topologicalOrder), node.needs);
 		logger.info({
-			dependencyCount: node.needs.length,
+			dependencyCount: dependencyNodeIds.length,
 			nodeId: descriptor.nodeId,
 			phase: "dependency.merge",
 			status: "start"
 		}, "dependency.merge start");
 		yield* io.mergeDependencyRefs({
 			committer: compiled.config.runner_command.git.committer,
-			dependencyNodeIds: node.needs,
+			dependencyNodeIds,
 			payload,
 			worktreePath
 		});
 		logger.info({
-			dependencyCount: node.needs.length,
+			dependencyCount: dependencyNodeIds.length,
 			nodeId: descriptor.nodeId,
 			phase: "dependency.merge",
 			status: "finish"

package/dist/runner-event-schema.d.ts

@@ -11,8 +11,8 @@ declare const runnerEventRecordSchema: z.ZodUnion<readonly [z.ZodObject<{
   runId: z.ZodString;
   sequence: z.ZodNumber;
   type: z.ZodEnum<{
-    "workflow.start": "workflow.start";
     "workflow.planned": "workflow.planned";
+    "workflow.start": "workflow.start";
   }>;
   workflowPlan: z.ZodObject<{
     edges: z.ZodOptional<z.ZodArray<z.ZodObject<{
@@ -58,10 +58,10 @@ declare const runnerEventRecordSchema: z.ZodUnion<readonly [z.ZodObject<{
     }>;
   }, z.core.$strip>;
   type: z.ZodEnum<{
-    "node.start": "node.start";
-    "node.finish": "node.finish";
     "agent.finish": "agent.finish";
     "agent.start": "agent.start";
+    "node.finish": "node.finish";
+    "node.start": "node.start";
   }>;
 }, z.core.$strip>, z.ZodObject<{
   at: z.ZodOptional<z.ZodString>;
@@ -189,8 +189,8 @@ declare const runnerEventBatchSchema: z.ZodObject<{
     runId: z.ZodString;
     sequence: z.ZodNumber;
     type: z.ZodEnum<{
-      "workflow.start": "workflow.start";
       "workflow.planned": "workflow.planned";
+      "workflow.start": "workflow.start";
     }>;
     workflowPlan: z.ZodObject<{
       edges: z.ZodOptional<z.ZodArray<z.ZodObject<{
@@ -236,10 +236,10 @@ declare const runnerEventBatchSchema: z.ZodObject<{
       }>;
     }, z.core.$strip>;
     type: z.ZodEnum<{
-      "node.start": "node.start";
-      "node.finish": "node.finish";
       "agent.finish": "agent.finish";
       "agent.start": "agent.start";
+      "node.finish": "node.finish";
+      "node.start": "node.start";
     }>;
   }, z.core.$strip>, z.ZodObject<{
     at: z.ZodOptional<z.ZodString>;

package/package.json

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