@bastani/atomic 0.9.13-alpha.1 → 0.9.13-alpha.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bastani/atomic",
-  "version": "0.9.13-alpha.1",
+  "version": "0.9.13-alpha.2",
   "description": "Atomic coding agent CLI with read, bash, edit, write tools and session management",
   "type": "module",
   "atomicConfig": {
@@ -29,6 +29,10 @@
       "import": "./dist/rpc-entry.js",
       "default": "./dist/rpc-entry.js"
     },
+    "./client": {
+      "types": "./dist/client/index.d.ts",
+      "import": "./dist/client/index.js"
+    },
     "./hooks": {
       "types": "./dist/core/hooks/index.d.ts",
       "import": "./dist/core/hooks/index.js"
@@ -62,6 +66,7 @@
   "scripts": {
     "clean": "shx rm -rf dist",
     "dev": "tsgo -p tsconfig.build.json --watch --preserveWatchOutput",
+    "typecheck": "tsgo -p tsconfig.build.json --noEmit",
     "build": "tsgo -p tsconfig.build.json && shx chmod +x dist/cli.js dist/rpc-entry.js && bun run copy-assets",
     "bundle:dev": "bun build src/cli.ts --target=bun --minify --external mupdf --define process.env.ATOMIC_BUNDLED_BUILD=\"'1'\" --outfile dist-dev/cli.js",
     "start:fast": "bun run bundle:dev && bun dist-dev/cli.js",
@@ -78,11 +83,13 @@
     "prepublishOnly": "bun run clean && bun run build && bun run shrinkwrap"
   },
   "dependencies": {
-    "@bastani/atomic-natives": "0.9.13-alpha.1",
+    "@bastani/atomic-natives": "0.9.13-alpha.2",
     "@dbos-inc/dbos-sdk": "4.24.16",
-    "@earendil-works/pi-agent-core": "^0.83.0",
-    "@earendil-works/pi-ai": "^0.83.0",
-    "@earendil-works/pi-tui": "^0.83.0",
+    "@earendil-works/pi-agent-core": "^0.84.1",
+    "@earendil-works/pi-ai": "^0.84.1",
+    "@earendil-works/pi-client": "^0.84.1",
+    "@earendil-works/pi-protocol": "^0.84.1",
+    "@earendil-works/pi-tui": "^0.84.1",
     "@modelcontextprotocol/ext-apps": "^1.7.5",
     "@modelcontextprotocol/sdk": "^1.30.0",
     "@mozilla/readability": "^0.6.0",
@@ -92,6 +99,7 @@
     "diff": "8.0.4",
     "embedded-postgres": "18.4.0-beta.17",
     "glob": "13.0.6",
+    "grok-mermaid": "0.2.2",
     "highlight.js": "10.7.3",
     "hosted-git-info": "9.0.3",
     "ignore": "7.0.5",
@@ -101,10 +109,9 @@
     "markit-ai": "0.5.3",
     "minimatch": "10.2.5",
     "open": "^11.0.0",
-    "p-limit": "^7.3.0",
+    "p-limit": "^7.3.1",
     "proper-lockfile": "4.1.2",
     "semver": "7.8.0",
-    "tsx": "^4.23.1",
     "turndown": "^7.2.0",
     "typebox": "1.3.7",
     "undici": "8.9.0",

package/dist/builtin/subagents/src/agents/chain-serializer.ts

@@ -1,257 +0,0 @@
-import { ChainOutputValidationError, validateChainOutputBindings } from "../runs/shared/chain-outputs.ts";
-import type { ChainStep } from "../shared/settings.ts";
-import type { ChainConfig, ChainStepConfig } from "./agents.ts";
-import { parseFrontmatter } from "./frontmatter.ts";
-import { buildRuntimeName, frontmatterNameForConfig, parsePackageName } from "./identity.ts";
-
-const REMOVED_CHAIN_CONFIG_FIELDS = new Set(["acceptance"]);
-
-function stripRemovedChainConfigFields(value: unknown): unknown {
-	if (!value || typeof value !== "object" || Array.isArray(value)) return value;
-	const input = value as Record<string, unknown>;
-	const output: Record<string, unknown> = {};
-	for (const [key, entry] of Object.entries(input)) {
-		if (REMOVED_CHAIN_CONFIG_FIELDS.has(key)) continue;
-		if (key === "parallel") {
-			output[key] = Array.isArray(entry)
-				? entry.map((item) => stripRemovedChainConfigFields(item))
-				: stripRemovedChainConfigFields(entry);
-			continue;
-		}
-		output[key] = entry;
-	}
-	return output;
-}
-
-function stripRemovedChainConfigSteps(steps: readonly unknown[]): ChainStepConfig[] {
-	return steps.map((step) => stripRemovedChainConfigFields(step) as ChainStepConfig);
-}
-
-function parseStepBody(agent: string, sectionBody: string): ChainStepConfig {
-	const lines = sectionBody.split("\n");
-	const blankIndex = lines.findIndex((line) => line.trim() === "");
-	const configLines = blankIndex === -1 ? lines : lines.slice(0, blankIndex);
-	const task = (blankIndex === -1 ? "" : lines.slice(blankIndex + 1).join("\n")).trim();
-
-	const step: ChainStepConfig = { agent, task };
-	for (const line of configLines) {
-		const match = line.match(/^([\w-]+):\s*(.*)$/);
-		if (!match) continue;
-		const key = match[1].trim().toLowerCase();
-		const rawValue = match[2].trim();
-
-		if (key === "output") {
-			if (rawValue === "false") step.output = false;
-			else if (rawValue) step.output = rawValue;
-			continue;
-		}
-		if (key === "phase") {
-			if (rawValue) step.phase = rawValue;
-			continue;
-		}
-		if (key === "label") {
-			if (rawValue) step.label = rawValue;
-			continue;
-		}
-		if (key === "as") {
-			if (rawValue) step.as = rawValue;
-			continue;
-		}
-		if (key === "outputschema") {
-			if (rawValue.startsWith("{") || rawValue.startsWith("[")) {
-				throw new Error("Inline outputSchema values are not supported in .chain.md files; use a schema file path.");
-			}
-			if (rawValue) step.outputSchema = rawValue;
-			continue;
-		}
-		if (key === "outputmode") {
-			if (rawValue === "inline" || rawValue === "file-only") step.outputMode = rawValue;
-			continue;
-		}
-		if (key === "reads") {
-			if (rawValue === "false") {
-				step.reads = false;
-			} else {
-				const reads = rawValue
-					.split(",")
-					.map((v) => v.trim())
-					.filter(Boolean);
-				step.reads = reads.length > 0 ? reads : false;
-			}
-			continue;
-		}
-		if (key === "model") {
-			if (rawValue) step.model = rawValue;
-			continue;
-		}
-		if (key === "skills") {
-			if (rawValue === "false") {
-				step.skills = false;
-			} else {
-				const skills = rawValue
-					.split(",")
-					.map((v) => v.trim())
-					.filter(Boolean);
-				step.skills = skills.length > 0 ? skills : false;
-			}
-			continue;
-		}
-		if (key === "progress") {
-			if (rawValue === "true") step.progress = true;
-			else if (rawValue === "false") step.progress = false;
-		}
-	}
-
-	return step;
-}
-
-export function parseChain(content: string, source: "user" | "project", filePath: string): ChainConfig {
-	const { frontmatter, body } = parseFrontmatter(content);
-	if (!frontmatter.name || !frontmatter.description) {
-		throw new Error("Chain frontmatter must include name and description");
-	}
-
-	const matches = [...body.matchAll(/^##\s+(.+)[^\S\n]*$/gm)];
-	const steps: ChainStepConfig[] = [];
-
-	for (let i = 0; i < matches.length; i++) {
-		const match = matches[i]!;
-		const agent = match[1]!.trim();
-		const lineEndOffset = body[match.index! + match[0].length] === "\n" ? 1 : 0;
-		const sectionStart = match.index! + match[0].length + lineEndOffset;
-		const sectionEnd = i + 1 < matches.length ? matches[i + 1]!.index! : body.length;
-		const sectionBody = body.slice(sectionStart, sectionEnd).trimEnd();
-		steps.push(parseStepBody(agent, sectionBody));
-	}
-
-	const localName = frontmatter.name;
-	const parsedPackage = parsePackageName(frontmatter.package, `Chain '${localName}' package`);
-	if (parsedPackage.error) throw new Error(parsedPackage.error);
-	const packageName = parsedPackage.packageName;
-	const extraFields: Record<string, string> = {};
-	for (const [key, value] of Object.entries(frontmatter)) {
-		if (key === "name" || key === "package" || key === "description") continue;
-		extraFields[key] = value;
-	}
-
-	return {
-		name: buildRuntimeName(localName, packageName),
-		localName,
-		packageName,
-		description: frontmatter.description,
-		source,
-		filePath,
-		steps,
-		extraFields: Object.keys(extraFields).length > 0 ? extraFields : undefined,
-	};
-}
-
-export function parseJsonChain(content: string, source: "user" | "project", filePath: string): ChainConfig {
-	let parsed: unknown;
-	try {
-		parsed = JSON.parse(content);
-	} catch (error) {
-		const message = error instanceof Error ? error.message : String(error);
-		throw new Error(`Invalid JSON chain '${filePath}': ${message}`);
-	}
-	if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
-		throw new Error(`JSON chain '${filePath}' must contain an object root.`);
-	}
-	const input = parsed as Record<string, unknown>;
-	if (typeof input.name !== "string" || !input.name.trim()) {
-		throw new Error(`JSON chain '${filePath}' must include string name.`);
-	}
-	if (typeof input.description !== "string" || !input.description.trim()) {
-		throw new Error(`JSON chain '${filePath}' must include string description.`);
-	}
-	if (!Array.isArray(input.chain)) {
-		throw new Error(`JSON chain '${filePath}' must include array chain.`);
-	}
-	for (let i = 0; i < input.chain.length; i++) {
-		const step = input.chain[i];
-		if (!step || typeof step !== "object" || Array.isArray(step)) {
-			throw new Error(`JSON chain '${filePath}' step ${i + 1} must be an object.`);
-		}
-	}
-	const chainSteps = stripRemovedChainConfigSteps(input.chain);
-	try {
-		validateChainOutputBindings(chainSteps as ChainStep[], { maxItems: Number.MAX_SAFE_INTEGER });
-	} catch (error) {
-		if (error instanceof ChainOutputValidationError)
-			throw new Error(`Invalid JSON chain '${filePath}': ${error.message}`);
-		throw error;
-	}
-	const parsedPackage = parsePackageName(
-		typeof input.package === "string" ? input.package : undefined,
-		`Chain '${input.name}' package`,
-	);
-	if (parsedPackage.error) throw new Error(parsedPackage.error);
-	const extraFields: Record<string, string> = {};
-	for (const [key, value] of Object.entries(input)) {
-		if (key === "name" || key === "package" || key === "description" || key === "chain") continue;
-		if (typeof value === "string") extraFields[key] = value;
-	}
-	return {
-		name: buildRuntimeName(input.name.trim(), parsedPackage.packageName),
-		localName: input.name.trim(),
-		packageName: parsedPackage.packageName,
-		description: input.description.trim(),
-		source,
-		filePath,
-		steps: chainSteps,
-		extraFields: Object.keys(extraFields).length > 0 ? extraFields : undefined,
-	};
-}
-
-export function serializeJsonChain(config: ChainConfig): string {
-	const root: Record<string, unknown> = {
-		name: frontmatterNameForConfig(config),
-		description: config.description,
-		chain: stripRemovedChainConfigSteps(config.steps),
-	};
-	if (config.packageName) root.package = config.packageName;
-	if (config.extraFields) {
-		for (const [key, value] of Object.entries(config.extraFields)) {
-			if (key !== "name" && key !== "description" && key !== "package" && key !== "chain") root[key] = value;
-		}
-	}
-	return `${JSON.stringify(root, null, 2)}\n`;
-}
-
-export function serializeChain(config: ChainConfig): string {
-	const lines: string[] = [];
-	lines.push("---");
-	lines.push(`name: ${frontmatterNameForConfig(config)}`);
-	if (config.packageName) lines.push(`package: ${config.packageName}`);
-	lines.push(`description: ${config.description}`);
-	if (config.extraFields) {
-		for (const [key, value] of Object.entries(config.extraFields)) {
-			lines.push(`${key}: ${value}`);
-		}
-	}
-	lines.push("---");
-	lines.push("");
-
-	for (let i = 0; i < config.steps.length; i++) {
-		const step = config.steps[i]!;
-		lines.push(`## ${step.agent}`);
-		if (step.output === false) lines.push("output: false");
-		else if (step.output) lines.push(`output: ${step.output}`);
-		if (step.phase) lines.push(`phase: ${step.phase}`);
-		if (step.label) lines.push(`label: ${step.label}`);
-		if (step.as) lines.push(`as: ${step.as}`);
-		if (step.outputSchema) lines.push(`outputSchema: ${step.outputSchema}`);
-		if (step.outputMode) lines.push(`outputMode: ${step.outputMode}`);
-		if (step.reads === false) lines.push("reads: false");
-		else if (Array.isArray(step.reads) && step.reads.length > 0) lines.push(`reads: ${step.reads.join(", ")}`);
-		if (step.model) lines.push(`model: ${step.model}`);
-		if (step.skills === false) lines.push("skills: false");
-		else if (Array.isArray(step.skills) && step.skills.length > 0) lines.push(`skills: ${step.skills.join(", ")}`);
-		if (step.progress !== undefined) lines.push(`progress: ${step.progress ? "true" : "false"}`);
-		lines.push("");
-		lines.push(step.task ?? "");
-		if (i < config.steps.length - 1) lines.push("");
-	}
-
-	return `${lines.join("\n")}\n`;
-}

package/dist/builtin/subagents/src/runs/foreground/chain-execution-details.ts

@@ -1,38 +0,0 @@
-import type { Details } from "../../shared/types.ts";
-import { compactForegroundDetails } from "../../shared/utils.ts";
-import { buildWorkflowGraphSnapshot } from "../shared/workflow-graph.ts";
-import type { ChainExecutionDetailsInput, ChainExecutionResult } from "./chain-execution-types.ts";
-
-export function buildChainExecutionDetails(input: ChainExecutionDetailsInput): Details {
-	return compactForegroundDetails({
-		mode: "chain",
-		results: input.results,
-		progress: input.includeProgress ? input.allProgress : undefined,
-		artifacts: input.allArtifactPaths.length ? { dir: input.artifactsDir, files: input.allArtifactPaths } : undefined,
-		chainAgents: input.chainAgents,
-		totalSteps: input.totalSteps,
-		currentStepIndex: input.currentStepIndex,
-		outputs: input.outputs,
-		workflowGraph: buildWorkflowGraphSnapshot({
-			runId: input.runId,
-			mode: "chain",
-			steps: input.chainSteps,
-			results: input.results,
-			currentStepIndex: input.currentStepIndex,
-			currentFlatIndex: input.currentFlatIndex,
-			dynamicChildren: input.dynamicChildren,
-			dynamicGroupStatuses: input.dynamicGroupStatuses,
-		}),
-	});
-}
-
-export function buildChainExecutionErrorResult(
-	message: string,
-	input: ChainExecutionDetailsInput,
-): ChainExecutionResult {
-	return {
-		content: [{ type: "text", text: message }],
-		isError: true,
-		details: buildChainExecutionDetails(input),
-	};
-}

package/dist/builtin/subagents/src/runs/foreground/chain-execution-dynamic-step.ts

@@ -1,280 +0,0 @@
-import * as path from "node:path";
-import { buildChainSummary } from "../../shared/formatters.ts";
-import {
-	aggregateParallelOutputs,
-	createParallelDirs,
-	type DynamicParallelStep,
-	type ParallelStep,
-	type ParallelTaskResult,
-	resolveParallelBehaviors,
-	suppressProgressForReadOnlyTask,
-} from "../../shared/settings.ts";
-import { getSingleResultOutput } from "../../shared/utils.ts";
-import {
-	collectDynamicResults,
-	type DynamicCollectedResult,
-	DynamicFanoutError,
-	materializeDynamicParallelStep,
-	validateDynamicCollection,
-} from "../shared/dynamic-fanout.ts";
-import { validateFileOnlyOutputMode } from "../shared/single-output.ts";
-import { buildChainExecutionDetails, buildChainExecutionErrorResult } from "./chain-execution-details.ts";
-import { ensureParallelProgressFile, runParallelChainTasks } from "./chain-execution-parallel-runner.ts";
-import type { ChainExecutionMutableState, ChainExecutionResult, ChainRuntimeContext } from "./chain-execution-types.ts";
-
-export async function runDynamicParallelChainStep(input: {
-	context: ChainRuntimeContext;
-	state: ChainExecutionMutableState;
-	step: DynamicParallelStep;
-	stepIndex: number;
-}): Promise<ChainExecutionResult | undefined> {
-	const { context, state, step, stepIndex } = input;
-	let materialized: ReturnType<typeof materializeDynamicParallelStep>;
-	try {
-		materialized = materializeDynamicParallelStep(step, state.outputs, stepIndex, {
-			maxItems: context.params.dynamicFanoutMaxItems,
-		});
-	} catch (error) {
-		const message =
-			error instanceof DynamicFanoutError ? error.message : error instanceof Error ? error.message : String(error);
-		state.dynamicGroupStatuses[stepIndex] = { status: "failed", error: message };
-		return buildChainExecutionErrorResult(
-			message,
-			context.makeDetailsInput({ currentStepIndex: stepIndex, currentFlatIndex: state.globalTaskIndex }),
-		);
-	}
-
-	state.dynamicChildren[stepIndex] = materialized.items.map((item, itemIndex) => ({
-		agent: step.parallel.agent,
-		label: materialized.parallel[itemIndex]?.label,
-		flatIndex: state.globalTaskIndex + itemIndex,
-		itemKey: item.key,
-		structured: Boolean(step.parallel.outputSchema),
-	}));
-
-	if (materialized.parallel.length === 0) {
-		const collection: DynamicCollectedResult[] = [];
-		try {
-			validateDynamicCollection(step.collect.outputSchema, collection);
-		} catch (error) {
-			const message =
-				error instanceof DynamicFanoutError
-					? error.message
-					: error instanceof Error
-						? error.message
-						: String(error);
-			state.dynamicGroupStatuses[stepIndex] = { status: "failed", error: message };
-			return buildChainExecutionErrorResult(
-				message,
-				context.makeDetailsInput({ currentStepIndex: stepIndex, currentFlatIndex: state.globalTaskIndex }),
-			);
-		}
-		state.outputs[step.collect.as] = {
-			text: JSON.stringify(collection),
-			structured: collection,
-			agent: step.parallel.agent,
-			stepIndex,
-		};
-		state.dynamicGroupStatuses[stepIndex] = { status: "completed" };
-		state.prev = "Dynamic fanout produced 0 results.";
-		return undefined;
-	}
-
-	const dynamicParallelStep: ParallelStep = {
-		parallel: materialized.parallel,
-		concurrency: step.concurrency,
-		failFast: step.failFast,
-		group: step.group,
-	};
-	const parallelTemplates = materialized.parallel.map((task) => task.task ?? "{previous}");
-	const parallelBehaviors = resolveParallelBehaviors(
-		dynamicParallelStep.parallel,
-		context.agents,
-		stepIndex,
-		context.chainSkills,
-	).map((behavior, taskIndex) =>
-		suppressProgressForReadOnlyTask(
-			behavior,
-			parallelTemplates[taskIndex] ?? dynamicParallelStep.parallel[taskIndex]?.task,
-			context.originalTask,
-		),
-	);
-
-	for (let taskIndex = 0; taskIndex < dynamicParallelStep.parallel.length; taskIndex++) {
-		const behavior = parallelBehaviors[taskIndex]!;
-		const outputPath =
-			typeof behavior.output === "string"
-				? path.isAbsolute(behavior.output)
-					? behavior.output
-					: path.join(context.chainDir, behavior.output)
-				: undefined;
-		const validationError = validateFileOnlyOutputMode(
-			behavior.outputMode,
-			outputPath,
-			`Dynamic chain step ${stepIndex + 1} item ${taskIndex + 1} (${dynamicParallelStep.parallel[taskIndex]!.agent})`,
-		);
-		if (validationError) {
-			state.dynamicGroupStatuses[stepIndex] = { status: "failed", error: validationError };
-			return buildChainExecutionErrorResult(
-				validationError,
-				context.makeDetailsInput({
-					currentStepIndex: stepIndex,
-					currentFlatIndex: state.globalTaskIndex + taskIndex,
-				}),
-			);
-		}
-	}
-
-	state.progressCreated = ensureParallelProgressFile(context.chainDir, state.progressCreated, parallelBehaviors);
-	createParallelDirs(
-		context.chainDir,
-		stepIndex,
-		dynamicParallelStep.parallel.length,
-		dynamicParallelStep.parallel.map((task) => task.agent),
-	);
-	const parallelResults = await runParallelChainTasks({
-		step: dynamicParallelStep,
-		chainIntercomGroup: context.params.group,
-		parallelTemplates,
-		parallelBehaviors,
-		agents: context.agents,
-		stepIndex,
-		availableModels: context.availableModels,
-		knownModelProviders: context.knownModelProviders,
-		chainDir: context.chainDir,
-		prev: state.prev,
-		originalTask: context.originalTask,
-		ctx: context.ctx,
-		intercomEvents: context.intercomEvents,
-		cwd: context.cwd,
-		runId: context.runId,
-		globalTaskIndex: state.globalTaskIndex,
-		sessionDirForIndex: context.sessionDirForIndex,
-		sessionFileForIndex: context.sessionFileForIndex,
-		shareEnabled: context.shareEnabled,
-		artifactConfig: context.artifactConfig,
-		artifactsDir: context.artifactsDir,
-		signal: context.signal,
-		onUpdate: context.onUpdate,
-		results: state.results,
-		allProgress: state.allProgress,
-		outputs: state.outputs,
-		chainAgents: context.chainAgents,
-		chainSteps: context.chainSteps,
-		totalSteps: context.totalSteps,
-		dynamicChildren: state.dynamicChildren,
-		dynamicGroupStatuses: state.dynamicGroupStatuses,
-		controlConfig: context.controlConfig,
-		onControlEvent: context.onControlEvent,
-		childIntercomTarget: context.childIntercomTarget,
-		orchestratorIntercomTarget: context.orchestratorIntercomTarget,
-		foregroundControl: context.foregroundControl,
-		nestedRoute: context.params.nestedRoute,
-		maxSubagentDepth: context.params.maxSubagentDepth,
-		workflowStageSubagentGuard: context.params.workflowStageSubagentGuard,
-		runSync: context.executeRunSync,
-		onDetachedExit: context.onDetachedExit,
-	});
-	state.globalTaskIndex += dynamicParallelStep.parallel.length;
-	for (const result of parallelResults) {
-		state.results.push(result);
-		if (result.progress) state.allProgress.push(result.progress);
-		if (result.artifactPaths) state.allArtifactPaths.push(result.artifactPaths);
-	}
-	const collected = collectDynamicResults(step, materialized.items, parallelResults);
-	const baseFlatIndex = state.globalTaskIndex - dynamicParallelStep.parallel.length;
-
-	const interruptedIndexInStep = parallelResults.findIndex((result) => result.interrupted);
-	const interrupted = interruptedIndexInStep >= 0 ? parallelResults[interruptedIndexInStep] : undefined;
-	if (interrupted) {
-		return {
-			content: [
-				{
-					type: "text",
-					text: `Chain paused after interrupt at step ${stepIndex + 1} (${interrupted.agent}). Waiting for explicit next action.`,
-				},
-			],
-			details: buildChainExecutionDetails(
-				context.makeDetailsInput({
-					currentStepIndex: stepIndex,
-					currentFlatIndex: baseFlatIndex + interruptedIndexInStep,
-				}),
-			),
-		};
-	}
-	const detachedIndexInStep = parallelResults.findIndex((result) => result.detached);
-	const detached = detachedIndexInStep >= 0 ? parallelResults[detachedIndexInStep] : undefined;
-	if (detached) {
-		return {
-			content: [
-				{
-					type: "text",
-					text: `Chain detached for intercom coordination at step ${stepIndex + 1} (${detached.agent}). Reply to the supervisor request first. After the child exits, start a fresh follow-up if needed.`,
-				},
-			],
-			details: buildChainExecutionDetails(
-				context.makeDetailsInput({
-					currentStepIndex: stepIndex,
-					currentFlatIndex: baseFlatIndex + detachedIndexInStep,
-				}),
-			),
-		};
-	}
-	const failures = parallelResults
-		.map((result, originalIndex) => ({ ...result, originalIndex }))
-		.filter((result) => result.status === "error");
-	if (failures.length > 0) {
-		const failureSummary = failures
-			.map(
-				(failure) =>
-					`- Item ${failure.originalIndex + 1} (${failure.agent}, key ${materialized.items[failure.originalIndex]?.key ?? failure.originalIndex}): ${failure.error || "failed"}`,
-			)
-			.join("\n");
-		const errorMsg = `Dynamic step ${stepIndex + 1} failed:\n${failureSummary}`;
-		state.dynamicGroupStatuses[stepIndex] = { status: "failed", error: errorMsg };
-		const summary = buildChainSummary(context.chainSteps, state.results, context.chainDir, "failed", {
-			index: stepIndex,
-			error: errorMsg,
-		});
-		return {
-			content: [{ type: "text", text: summary }],
-			isError: true,
-			details: buildChainExecutionDetails(
-				context.makeDetailsInput({
-					currentStepIndex: stepIndex,
-					currentFlatIndex: baseFlatIndex + failures[0]!.originalIndex,
-				}),
-			),
-		};
-	}
-	try {
-		validateDynamicCollection(step.collect.outputSchema, collected);
-	} catch (error) {
-		const message =
-			error instanceof DynamicFanoutError ? error.message : error instanceof Error ? error.message : String(error);
-		state.dynamicGroupStatuses[stepIndex] = { status: "failed", error: message };
-		return buildChainExecutionErrorResult(
-			message,
-			context.makeDetailsInput({ currentStepIndex: stepIndex, currentFlatIndex: baseFlatIndex }),
-		);
-	}
-	state.outputs[step.collect.as] = {
-		text: JSON.stringify(collected),
-		structured: collected,
-		agent: step.parallel.agent,
-		stepIndex,
-	};
-	state.dynamicGroupStatuses[stepIndex] = { status: "completed" };
-	const taskResults: ParallelTaskResult[] = parallelResults.map((result, index) => ({
-		agent: result.agent,
-		taskIndex: index,
-		output: getSingleResultOutput(result),
-		status: result.status,
-		error: result.error,
-	}));
-	state.prev = aggregateParallelOutputs(
-		taskResults,
-		(index, agent) => `=== Dynamic Item ${index + 1} (${agent}, key ${materialized.items[index]?.key ?? index}) ===`,
-	);
-	return undefined;
-}