@oisincoveney/pipeline 1.15.3 → 1.16.0

package/dist/config.d.ts

@@ -9,6 +9,7 @@ declare const NODE_KINDS: readonly ["agent", "command", "builtin", "group", "par
 declare const HOOK_EVENTS: readonly ["workflow.start", "workflow.success", "workflow.failure", "workflow.complete", "node.start", "node.success", "node.error", "node.finish", "gate.failure"];
 declare const GATE_KINDS: readonly ["acceptance", "artifact", "builtin", "changed_files", "command", "json_schema", "verdict"];
 declare const SCHEDULE_BASELINES: readonly ["epic", "pipe"];
+declare const SCHEDULING_ROLES: readonly ["coverage", "implementation"];
 type PipelineConfigErrorCode = "PIPELINE_CONFIG_LEGACY_UNSUPPORTED" | "PIPELINE_CONFIG_MISSING" | "PIPELINE_CONFIG_PARSE_ERROR" | "PIPELINE_CONFIG_VALIDATION_ERROR";
 interface PipelineConfigIssue {
   message: string;
@@ -291,6 +292,10 @@ declare const configSchema: z.ZodObject<{
     }, z.core.$strict>>;
     rules: z.ZodOptional<z.ZodArray<z.ZodString>>;
     runner: z.ZodString;
+    scheduling_roles: z.ZodOptional<z.ZodArray<z.ZodEnum<{
+      coverage: "coverage";
+      implementation: "implementation";
+    }>>>;
     skills: z.ZodOptional<z.ZodArray<z.ZodString>>;
     tools: z.ZodOptional<z.ZodArray<z.ZodEnum<{
       read: "read";
@@ -379,6 +384,7 @@ type WorkflowNodeKind = (typeof NODE_KINDS)[number];
 type HookEvent = (typeof HOOK_EVENTS)[number];
 type GateKind = (typeof GATE_KINDS)[number];
 type ScheduleBaseline = (typeof SCHEDULE_BASELINES)[number];
+type SchedulingRole = (typeof SCHEDULING_ROLES)[number];
 interface PipelineConfigParts {
   pipeline: string;
   profiles: string;
@@ -393,4 +399,4 @@ declare function parsePipelineConfigYaml(source: string, sourcePath?: string, pr
 declare function parsePipelineConfigParts(sources: PipelineConfigParts, projectRoot?: string, sourcePaths?: PipelineConfigParts, options?: PipelineConfigValidationOptions): PipelineConfig;
 declare function validatePipelineConfig(rawConfig: PipelineConfig, projectRoot?: string, options?: PipelineConfigValidationOptions): PipelineConfig;
 //#endregion
-export { GateKind, HookEvent, PIPELINE_CONFIG_PATH, PROFILES_CONFIG_PATH, PipelineConfig, PipelineConfigError, PipelineConfigErrorCode, PipelineConfigIssue, PipelineConfigParts, PipelineConfigValidationOptions, RUNNERS_CONFIG_PATH, RunnerType, ScheduleBaseline, WorkflowNodeKind, loadPipelineConfig, parsePipelineConfigParts, parsePipelineConfigYaml, tryLoadPipelineConfig, validatePipelineConfig, workflowSchema };
+export { GateKind, HookEvent, PIPELINE_CONFIG_PATH, PROFILES_CONFIG_PATH, PipelineConfig, PipelineConfigError, PipelineConfigErrorCode, PipelineConfigIssue, PipelineConfigParts, PipelineConfigValidationOptions, RUNNERS_CONFIG_PATH, RunnerType, ScheduleBaseline, SchedulingRole, WorkflowNodeKind, loadPipelineConfig, parsePipelineConfigParts, parsePipelineConfigYaml, tryLoadPipelineConfig, validatePipelineConfig, workflowSchema };

package/dist/config.js

@@ -55,6 +55,7 @@ const RETRY_REASONS = [
 	"timeout"
 ];
 const SCHEDULE_BASELINES = ["epic", "pipe"];
+const SCHEDULING_ROLES = ["coverage", "implementation"];
 const PIPELINE_GATEWAY_SERVER_ID = "pipeline-gateway";
 var PipelineConfigError = class extends Error {
 	code;
@@ -227,6 +228,7 @@ const profileSchema = z.object({
 	output: outputSchema.optional(),
 	rules: z.array(z.string()).optional(),
 	runner: z.string(),
+	scheduling_roles: z.array(z.enum(SCHEDULING_ROLES)).optional(),
 	skills: z.array(z.string()).optional(),
 	tools: z.array(z.enum(TOOL_NAMES)).optional()
 }).strict();

package/dist/pipeline-init.js

@@ -462,6 +462,7 @@ profiles:
         max_attempts: 1
   pipeline-code-writer:
     runner: codex
+    scheduling_roles: [implementation]
     description: Implement production code until the failing tests pass.
     instructions:
       path: .pipeline/prompts/code-writer.md
@@ -479,6 +480,7 @@ profiles:
       format: text
   pipeline-acceptance-reviewer:
     runner: codex
+    scheduling_roles: [coverage]
     description: Audit the finished change against every acceptance criterion.
     instructions:
       path: .pipeline/prompts/acceptance-reviewer.md
@@ -500,6 +502,7 @@ profiles:
         max_attempts: 1
   pipeline-thermo-nuclear-reviewer:
     runner: codex
+    scheduling_roles: [coverage]
     description: Perform the final thermo-nuclear code quality review of the integration branch.
     instructions:
       path: .agents/skills/critique/SKILL.md
@@ -520,6 +523,7 @@ profiles:
         max_attempts: 1
   pipeline-verifier:
     runner: codex
+    scheduling_roles: [coverage]
     description: Verify checks, implementation fit, and final evidence.
     instructions:
       path: .pipeline/prompts/verifier.md
@@ -720,7 +724,7 @@ const SCAFFOLD_FILES = {
 		"",
 		"Generate exactly one workflow named `root`. Do not embed `default`, `epic-drain`, `infra`, `track`, or other configured workflow copies. Use explicit generated agent, builtin, command, parallel, or group nodes. Do not use `kind: workflow`.",
 		"",
-		"Use the provided backlog work units as the source of truth when present. Assign each work unit to explicit generated agent nodes with only its `task_context.id`, use only allowed configured profiles, and ensure implementation work has downstream acceptance, verification, or review coverage. Do not invent profiles or node-level skill overrides.",
+		"Use the provided backlog work units as the source of truth when present. Assign each work unit to explicit generated agent nodes with only its `task_context.id`, use only allowed configured profiles, and ensure profiles with the `implementation` scheduling role have downstream profiles with the `coverage` scheduling role. Do not invent profiles or node-level skill overrides.",
 		"Do not copy backlog descriptions or acceptance criteria into output; the scheduler hydrates them from the assigned `task_context.id` after parsing.",
 		"Preserve Backlog dependency ids as schedule needs edges. A node assigned a dependent work unit must depend on the nodes assigned its prerequisite work units, directly or through an explicit path.",
 		"",

package/dist/schedule-planner.js

@@ -376,7 +376,7 @@ function plannerPrompt(entrypointId, task, baseline, config, planningContext) {
 		"Every agent node must declare one configured profile id. Do not invent profile ids or node-level skill overrides.",
 		"Assign each backlog work unit to explicit generated agent nodes with task_context.id. The scheduler hydrates title, description, and acceptance_criteria after parsing.",
 		"Do not copy backlog descriptions or acceptance criteria into task_context output.",
-		"Implementation work must have downstream acceptance, verification, or review coverage in the generated DAG.",
+		"Profiles with the implementation scheduling role must have downstream profiles with the coverage scheduling role in the generated DAG.",
 		"Preserve Backlog dependency ids as schedule needs edges. A node assigned a dependent work unit must depend on the nodes assigned its prerequisite work units, directly or through an explicit path.",
 		"Shape the graph by intent, not by ticket count. Do not create a full RED/GREEN/ACCEPTANCE/VERIFY chain for each backlog ticket unless each step needs ticket-specific evidence.",
 		"Only add needs edges for real dependencies, shared constraints, or verification/review fan-in.",
@@ -385,7 +385,7 @@ function plannerPrompt(entrypointId, task, baseline, config, planningContext) {
 		"Only serialize ticket nodes when the backlog, a shared migration/schema/API dependency, or implementation risk requires it.",
 		"",
 		"Allowed profiles:",
-		...Object.keys(config.profiles).sort().map((id) => `- ${id}`),
+		...Object.keys(config.profiles).sort().map((id) => allowedProfilePromptLine(config, id)),
 		"",
 		"Gate recipes:",
 		"- Prefer preserving valid gates from the baseline workflows instead of recreating them.",
@@ -403,6 +403,13 @@ function plannerPrompt(entrypointId, task, baseline, config, planningContext) {
 		stringify(baseline)
 	].join("\n");
 }
+function allowedProfilePromptLine(config, id) {
+	const roles = effectiveSchedulingRoles(config, id);
+	return roles.length > 0 ? `- ${id} (scheduling_roles: ${roles.join(", ")})` : `- ${id}`;
+}
+function effectiveSchedulingRoles(config, profileId) {
+	return [...new Set(config.profiles[profileId]?.scheduling_roles ?? [])].sort();
+}
 function canonicalizeGeneratedScheduleIds(artifact) {
 	return {
 		...artifact,
@@ -451,9 +458,9 @@ function validateScheduleArtifact(config, artifact, planningContext) {
 		...workflowReferenceNodeIssues(artifact),
 		...workflowAssignedWorkUnitIssues(artifact, planningContext.workUnits),
 		...missingAssignedWorkUnitIssues(artifact, planningContext.workUnits),
-		...workUnitDependencyIssues(artifact, planningContext.workUnits),
+		...workUnitDependencyIssues(config, artifact, planningContext.workUnits),
 		...invalidWorkflowPrimitiveIssues(config, artifact),
-		...implementationCoverageIssues(artifact)
+		...implementationCoverageIssues(config, artifact)
 	];
 	if (issues.length > 0) throw new ScheduleArtifactError(["Invalid generated schedule:", ...issues.map((issue) => `- ${issue}`)].join("\n"));
 }
@@ -514,7 +521,7 @@ function missingAssignedWorkUnitIssues(artifact, workUnits) {
 	const missing = workUnits.map((unit) => unit.id).filter((id) => !assigned.has(id));
 	return missing.length > 0 ? [`missing assigned backlog work units: ${missing.join(", ")}`] : [];
 }
-function workUnitDependencyIssues(artifact, workUnits) {
+function workUnitDependencyIssues(config, artifact, workUnits) {
 	if (workUnits.length === 0) return [];
 	const workUnitIds = new Set(workUnits.map((unit) => unit.id));
 	const dependenciesByUnit = new Map(workUnits.map((unit) => [unit.id, (unit.dependencies ?? []).filter((id) => workUnitIds.has(id))]));
@@ -522,7 +529,7 @@ function workUnitDependencyIssues(artifact, workUnits) {
 		const nodes = workflow.nodes.flatMap(flattenWorkflowNode);
 		const dependentsByNeed = workflowDependentsByNeed(nodes);
 		const nodesByWorkUnit = nodesByAssignedWorkUnit(nodes);
-		return nodes.filter(isImplementationNode).flatMap((node) => {
+		return nodes.filter((node) => isImplementationNode(config, node)).flatMap((node) => {
 			const dependentId = node.task_context?.id;
 			if (!dependentId) return [];
 			return (dependenciesByUnit.get(dependentId) ?? []).flatMap((prerequisiteId) => {
@@ -564,14 +571,15 @@ function invalidWorkflowPrimitiveIssues(config, artifact) {
 		return [];
 	});
 }
-function implementationCoverageIssues(artifact) {
+function implementationCoverageIssues(config, artifact) {
 	return Object.entries(artifact.workflows).flatMap(([workflowId, workflow]) => {
-		const dependentsByNeed = workflowDependentsByNeed(workflow.nodes);
-		return workflow.nodes.filter(isImplementationNode).filter((node) => !hasDownstreamCoverage(node.id, dependentsByNeed)).map((node) => `implementation node '${workflowId}.${node.id}' is without downstream verification or review`);
+		const nodes = workflow.nodes.flatMap(flattenWorkflowNode);
+		const dependentsByNeed = workflowDependentsByNeed(nodes);
+		return nodes.filter((node) => isImplementationNode(config, node)).filter((node) => !hasDownstreamCoverage(config, node.id, dependentsByNeed)).map((node) => `implementation node '${workflowId}.${node.id}' is without downstream verification or review`);
 	});
 }
-function isImplementationNode(node) {
-	return node.kind === "agent" && node.profile === "pipeline-code-writer";
+function isImplementationNode(config, node) {
+	return hasSchedulingRole(config, node, "implementation");
 }
 function workflowDependentsByNeed(nodes) {
 	const dependentsByNeed = /* @__PURE__ */ new Map();
@@ -582,25 +590,24 @@ function workflowDependentsByNeed(nodes) {
 	}
 	return dependentsByNeed;
 }
-function hasDownstreamCoverage(nodeId, dependentsByNeed) {
+function hasDownstreamCoverage(config, nodeId, dependentsByNeed) {
 	const queue = [...dependentsByNeed.get(nodeId) ?? []];
 	const seen = /* @__PURE__ */ new Set();
 	while (queue.length > 0) {
 		const node = queue.shift();
 		if (!node || seen.has(node.id)) continue;
 		seen.add(node.id);
-		if (isCoverageNode(node)) return true;
+		if (isCoverageNode(config, node)) return true;
 		queue.push(...dependentsByNeed.get(node.id) ?? []);
 	}
 	return false;
 }
-function isCoverageNode(node) {
+function isCoverageNode(config, node) {
+	return hasSchedulingRole(config, node, "coverage");
+}
+function hasSchedulingRole(config, node, role) {
 	if (node.kind !== "agent") return false;
-	return [
-		"pipeline-acceptance-reviewer",
-		"pipeline-thermo-nuclear-reviewer",
-		"pipeline-verifier"
-	].includes(node.profile);
+	return config.profiles[node.profile]?.scheduling_roles?.includes(role) ?? false;
 }
 function allWorkflowNodes(workflows) {
 	return Object.values(workflows).flatMap((workflow) => workflow.nodes.flatMap(flattenWorkflowNode));

package/package.json

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