pi-crew 0.9.16 → 0.9.17

package/src/utils/visual.ts

@@ -6,39 +6,66 @@ const widthCache = new Map<string, number>();
 /** Code-point ranges that render as width 2 in most terminals (CJK + emoji). */
 const WIDE_RANGES: Array<[number, number]> = [
 	// CJK Unified Ideographs
-	[0x4E00, 0x9FFF],
+	[0x4e00, 0x9fff],
 	// CJK Extension A
-	[0x3400, 0x4DBF],
+	[0x3400, 0x4dbf],
 	// CJK Compatibility Ideographs
-	[0xF900, 0xFAFF],
+	[0xf900, 0xfaff],
 	// Hangul Syllables
-	[0xAC00, 0xD7AF],
+	[0xac00, 0xd7af],
 	// CJK Symbols and Punctuation, Hiragana, Katakana
-	[0x3000, 0x33FF],
+	[0x3000, 0x33ff],
 	// Fullwidth forms
-	[0xFF01, 0xFF60],
+	[0xff01, 0xff60],
 	// Emoji blocks
 	// Emoji-presentation codepoints in 0x2600-0x27BF (narrow chars like ✓✗★ excluded)
-	[0x2615, 0x2615], [0x2648, 0x2653], [0x267F, 0x267F], [0x2693, 0x2693],
-	[0x26A1, 0x26A1], [0x26AA, 0x26AB], [0x26BD, 0x26BE], [0x26C4, 0x26C5],
-	[0x26CE, 0x26CE], [0x26D4, 0x26D4], [0x26EA, 0x26EA], [0x26F2, 0x26F3],
-	[0x26F5, 0x26F5], [0x26FA, 0x26FA], [0x26FD, 0x26FD], [0x2702, 0x2702],
-	[0x2705, 0x2705], [0x2708, 0x270D], [0x270F, 0x270F], [0x2712, 0x2712],
-	[0x2714, 0x2714], [0x2716, 0x2716], [0x271D, 0x271D], [0x2721, 0x2721],
-	[0x2728, 0x2728], [0x2733, 0x2734], [0x2744, 0x2744], [0x2747, 0x2747],
-	[0x274C, 0x274C], [0x274E, 0x274E], [0x2753, 0x2755], [0x2757, 0x2757],
-	[0x2763, 0x2764], [0x2795, 0x2797], [0x27A1, 0x27A1], [0x27B0, 0x27B0],
-	[0x27BF, 0x27BF],
+	[0x2615, 0x2615],
+	[0x2648, 0x2653],
+	[0x267f, 0x267f],
+	[0x2693, 0x2693],
+	[0x26a1, 0x26a1],
+	[0x26aa, 0x26ab],
+	[0x26bd, 0x26be],
+	[0x26c4, 0x26c5],
+	[0x26ce, 0x26ce],
+	[0x26d4, 0x26d4],
+	[0x26ea, 0x26ea],
+	[0x26f2, 0x26f3],
+	[0x26f5, 0x26f5],
+	[0x26fa, 0x26fa],
+	[0x26fd, 0x26fd],
+	[0x2702, 0x2702],
+	[0x2705, 0x2705],
+	[0x2708, 0x270d],
+	[0x270f, 0x270f],
+	[0x2712, 0x2712],
+	[0x2714, 0x2714],
+	[0x2716, 0x2716],
+	[0x271d, 0x271d],
+	[0x2721, 0x2721],
+	[0x2728, 0x2728],
+	[0x2733, 0x2734],
+	[0x2744, 0x2744],
+	[0x2747, 0x2747],
+	[0x274c, 0x274c],
+	[0x274e, 0x274e],
+	[0x2753, 0x2755],
+	[0x2757, 0x2757],
+	[0x2763, 0x2764],
+	[0x2795, 0x2797],
+	[0x27a1, 0x27a1],
+	[0x27b0, 0x27b0],
+	[0x27bf, 0x27bf],
 	// Geometric Shapes / Misc Symbols-Arrows emoji that pi-tui upstream counts as
 	// width=2 (RGI emoji). Mismatch here caused the "Rendered line N exceeds
 	// terminal width (160 > 159)" TUI crash: pi-crew truncated to width 159 by
 	// its own (mismatched) measure, then Box padded to 159 chars, but pi-tui
 	// re-measured the padded line at 160 because ⬜ counts as 2 upstream.
-	[0x2B1B, 0x2B1C], // ⬛ BLACK LARGE SQUARE, ⬜ WHITE LARGE SQUARE
-	[0x1F300, 0x1F9FF], // Misc Symbols, Emoticons, Transport, Map, Supplement
-	[0x1FA00, 0x1FAFF], // Symbols Extended-A
-	[0x1F000, 0x1F02F], // Mahjong, Dominos
-	[0xFE00, 0xFE0F],   // Variation Selectors (emoji presentation)
+	[0x2b1b, 0x2b1c], // ⬛ BLACK LARGE SQUARE, ⬜ WHITE LARGE SQUARE
+	[0x1f300, 0x1f9ff], // Misc Symbols, Emoticons, Transport, Map, Supplement
+	[0x1fa00, 0x1faff], // Symbols Extended-A
+	[0x1f000, 0x1f02f], // Mahjong, Dominos
+	[0xfe00, 0xfe0f], // Variation Selectors (emoji presentation)
 ];
 // NOTE: U+200D (Zero Width Joiner, ZWJ) is intentionally NOT listed as wide.
 // ZWJ has zero advance width by Unicode definition; miscounting it as 2 caused
@@ -113,7 +140,7 @@ export function truncateToWidth(value: string, width: number, ellipsis = "…"):
 			continue;
 		}
 		const char = value[i] as string;
-		const nextIndex = ((char.codePointAt(0) ?? 0) > 0xFFFF) ? i + 2 : i + 1;
+		const nextIndex = (char.codePointAt(0) ?? 0) > 0xffff ? i + 2 : i + 1;
 		const segment = value.slice(i, nextIndex);
 		const charWidth = visibleWidth(segment);
 		if (renderedWidth + charWidth > width - ellipsis.length) {
@@ -183,7 +210,7 @@ function readAnsiCode(input: string, index: number): string | undefined {
 function takeCodePoint(input: string, index: number): { chunk: string; nextIndex: number } {
 	const code = input.codePointAt(index);
 	if (code === undefined) return { chunk: "", nextIndex: index + 1 };
-	if (code >= 0xD800 && code <= 0xDBFF && index + 1 < input.length) {
+	if (code >= 0xd800 && code <= 0xdbff && index + 1 < input.length) {
 		return { chunk: input.slice(index, index + 2), nextIndex: index + 2 };
 	}
 	return { chunk: input[index] ?? "", nextIndex: index + 1 };
@@ -231,20 +258,13 @@ export interface VisualTruncateResult {
 	skippedCount: number;
 }
 
-export function truncateToVisualLines(
-	text: string,
-	maxVisualLines: number,
-	width: number,
-	paddingX = 0,
-): VisualTruncateResult {
+export function truncateToVisualLines(text: string, maxVisualLines: number, width: number, paddingX = 0): VisualTruncateResult {
 	if (!text) {
 		return { visualLines: [], skippedCount: 0 };
 	}
 	const effectiveWidth = Math.max(1, width - paddingX * 2);
 	const limit = Math.max(1, maxVisualLines);
-	const visualLines = text
-		.split("\n")
-		.flatMap((line) => wrapHard(pad(line, Math.max(0, effectiveWidth)).trimEnd(), effectiveWidth));
+	const visualLines = text.split("\n").flatMap((line) => wrapHard(pad(line, Math.max(0, effectiveWidth)).trimEnd(), effectiveWidth));
 	if (visualLines.length <= limit) return { visualLines, skippedCount: 0 };
 	const truncated = visualLines.slice(-limit);
 	return { visualLines: truncated, skippedCount: visualLines.length - limit };

package/src/workflows/discover-workflows.ts

@@ -64,57 +64,38 @@ function parseStepSection(id: string, body: string): WorkflowStep | undefined {
 		reads: config.reads === "false" ? false : parseCsv(config.reads),
 		model: config.model || undefined,
 		skills: config.skills === "false" ? false : parseCsv(config.skills),
-		progress:
-			config.progress === "true"
-				? true
-				: config.progress === "false"
-					? false
-					: undefined,
-		worktree:
-			config.worktree === "true"
-				? true
-				: config.worktree === "false"
-					? false
-					: undefined,
-		verify:
-			config.verify === "true"
-				? true
-				: config.verify === "false"
-					? false
-					: undefined,
+		progress: config.progress === "true" ? true : config.progress === "false" ? false : undefined,
+		worktree: config.worktree === "true" ? true : config.worktree === "false" ? false : undefined,
+		verify: config.verify === "true" ? true : config.verify === "false" ? false : undefined,
 		seedPaths: parseCsv(config.seedPaths) || undefined,
 		preStepScript: config.preStepScript || undefined,
 		preStepArgs: parseCsv(config.preStepArgs) || undefined,
-		preStepTimeout:
-			parseOptionalInteger(config.preStepTimeout) ?? undefined,
-		preStepOptional:
-			config.preStepOptional === "true" || config.preStepOptional === "1",
+		preStepTimeout: parseOptionalInteger(config.preStepTimeout) ?? undefined,
+		preStepOptional: config.preStepOptional === "true" || config.preStepOptional === "1",
 	};
 }
 
-const parseOptionalInteger = (
-	value: string | undefined,
-): number | undefined => {
-	if (!value) return undefined;
-	const parsed = Number.parseInt(value, 10);
-	if (!Number.isFinite(parsed) || parsed < 1) return undefined;
-	return Math.trunc(parsed);
+const parseOptionalInteger = (value: string | undefined): number | undefined => {
+	const trimmed = value?.trim();
+	if (!trimmed) return undefined;
+	const n = Number.parseInt(trimmed, 10);
+	return Number.isNaN(n) ? undefined : n;
+};
+
+const parseOptionalBoolean = (value: string | undefined): boolean | undefined => {
+	const trimmed = value?.trim().toLowerCase();
+	if (!trimmed) return undefined;
+	if (trimmed === "true" || trimmed === "yes" || trimmed === "1") return true;
+	if (trimmed === "false" || trimmed === "no" || trimmed === "0") return false;
+	return undefined;
 };
 
 /** Parse frontmatter `topology:` field. Validates against allowed enum; bad values are silently
  *  dropped (fall through to auto-classification in topology-analyzer). */
-const parseTopology = (
-	value: string | undefined,
-): WorkflowConfig["topology"] => {
+const parseTopology = (value: string | undefined): WorkflowConfig["topology"] => {
 	if (!value) return undefined;
 	const v = value.trim().toLowerCase();
-	if (
-		v === "single" ||
-		v === "sequential" ||
-		v === "concurrent" ||
-		v === "complex-dag" ||
-		v === "dynamic"
-	) {
+	if (v === "single" || v === "sequential" || v === "concurrent" || v === "complex-dag" || v === "dynamic") {
 		return v;
 	}
 	return undefined;
@@ -129,15 +110,9 @@ function hasSectionBoundary(body: string, match: RegExpMatchArray): boolean {
 }
 
 function isStepHeading(body: string, match: RegExpMatchArray): boolean {
-	const sectionStart =
-		match.index! +
-		match[0].length +
-		(body[match.index! + match[0].length] === "\n" ? 1 : 0);
+	const sectionStart = match.index! + match[0].length + (body[match.index! + match[0].length] === "\n" ? 1 : 0);
 	const nextHeading = body.slice(sectionStart).search(/^##\s+.+[^\S\n]*$/m);
-	const section = body.slice(
-		sectionStart,
-		nextHeading >= 0 ? sectionStart + nextHeading : body.length,
-	);
+	const section = body.slice(sectionStart, nextHeading >= 0 ? sectionStart + nextHeading : body.length);
 	for (const line of section.split("\n")) {
 		const trimmed = line.trim();
 		if (!trimmed) continue;
@@ -148,58 +123,39 @@ function isStepHeading(body: string, match: RegExpMatchArray): boolean {
 	return false;
 }
 
-function parseWorkflowFile(
-	filePath: string,
-	source: ResourceSource,
-): WorkflowConfig | undefined {
+function parseWorkflowFile(filePath: string, source: ResourceSource): WorkflowConfig | undefined {
 	try {
 		const content = fs.readFileSync(filePath, "utf-8");
 		const { frontmatter, body } = parseFrontmatter(content);
-		const name =
-			frontmatter.name?.trim() || path.basename(filePath, ".workflow.md");
+		const name = frontmatter.name?.trim() || path.basename(filePath, ".workflow.md");
 		const matches = [...body.matchAll(/^##\s+(.+)[^\S\n]*$/gm)];
 		const explicitStepIndexes = new Set(
 			matches
-				.map((match, index) =>
-					isStepHeading(body, match) ? index : undefined,
-				)
+				.map((match, index) => (isStepHeading(body, match) ? index : undefined))
 				.filter((index): index is number => index !== undefined),
 		);
 		const effectiveMatches = matches.filter(
 			(match, index) =>
-				explicitStepIndexes.has(index) ||
-				(hasSectionBoundary(body, match) &&
-					/^[a-z][a-z0-9-]*$/.test(match[1]?.trim() ?? "")),
+				explicitStepIndexes.has(index) || (hasSectionBoundary(body, match) && /^[a-z][a-z0-9-]*$/.test(match[1]?.trim() ?? "")),
 		);
-		const parseMatches = explicitStepIndexes.size
-			? effectiveMatches
-			: matches;
+		const parseMatches = explicitStepIndexes.size ? effectiveMatches : matches;
 		const steps: WorkflowStep[] = [];
 		for (let i = 0; i < parseMatches.length; i++) {
 			const match = parseMatches[i]!;
 			const id = match[1]!.trim();
-			const sectionStart =
-				match.index! +
-				match[0].length +
-				(body[match.index! + match[0].length] === "\n" ? 1 : 0);
-			const sectionEnd =
-				i + 1 < parseMatches.length
-					? parseMatches[i + 1]!.index!
-					: body.length;
-			const step = parseStepSection(
-				id,
-				body.slice(sectionStart, sectionEnd),
-			);
+			const sectionStart = match.index! + match[0].length + (body[match.index! + match[0].length] === "\n" ? 1 : 0);
+			const sectionEnd = i + 1 < parseMatches.length ? parseMatches[i + 1]!.index! : body.length;
+			const step = parseStepSection(id, body.slice(sectionStart, sectionEnd));
 			if (step) steps.push(step);
 		}
 		return {
 			name,
-			description:
-				frontmatter.description?.trim() || "No description provided.",
+			description: frontmatter.description?.trim() || "No description provided.",
 			source,
 			filePath,
 			maxConcurrency: parseOptionalInteger(frontmatter.maxConcurrency),
 			topology: parseTopology(frontmatter.topology),
+			coalesceMicroTasks: parseOptionalBoolean(frontmatter.coalesceMicroTasks),
 			steps,
 		};
 	} catch {
@@ -207,37 +163,25 @@ function parseWorkflowFile(
 	}
 }
 
-function readWorkflowDir(
-	dir: string,
-	source: ResourceSource,
-): WorkflowConfig[] {
+function readWorkflowDir(dir: string, source: ResourceSource): WorkflowConfig[] {
 	if (!fs.existsSync(dir)) return [];
 	const staticWorkflows = fs
 		.readdirSync(dir)
 		.filter((entry) => entry.endsWith(".workflow.md"))
 		.map((entry) => parseWorkflowFile(path.join(dir, entry), source))
-		.filter(
-			(workflow): workflow is WorkflowConfig => workflow !== undefined,
-		)
+		.filter((workflow): workflow is WorkflowConfig => workflow !== undefined)
 		.sort((a, b) => a.name.localeCompare(b.name));
 	// P2: also discover dynamic workflows (*.dwf.ts). A .dwf.ts's default export is a JS orchestrator.
 	const dynamicWorkflows = fs
 		.readdirSync(dir)
 		.filter((entry) => entry.endsWith(".dwf.ts"))
 		.map((entry) => parseDynamicWorkflowFile(path.join(dir, entry), source))
-		.filter(
-			(workflow): workflow is WorkflowConfig => workflow !== undefined,
-		);
-	return [...staticWorkflows, ...dynamicWorkflows].sort((a, b) =>
-		a.name.localeCompare(b.name),
-	);
+		.filter((workflow): workflow is WorkflowConfig => workflow !== undefined);
+	return [...staticWorkflows, ...dynamicWorkflows].sort((a, b) => a.name.localeCompare(b.name));
 }
 
 /** P2: a .dwf.ts is a dynamic workflow. Name = filename stem; script = the file itself. */
-function parseDynamicWorkflowFile(
-	filePath: string,
-	source: ResourceSource,
-): WorkflowConfig | undefined {
+function parseDynamicWorkflowFile(filePath: string, source: ResourceSource): WorkflowConfig | undefined {
 	try {
 		const basename = path.basename(filePath, ".dwf.ts");
 		return {
@@ -259,28 +203,16 @@ export function discoverWorkflows(cwd: string): WorkflowDiscoveryResult {
 		return { builtin: [], user: [], project: [] };
 	}
 	return {
-		builtin: readWorkflowDir(
-			path.join(packageRoot(), "workflows"),
-			"builtin",
-		),
+		builtin: readWorkflowDir(path.join(packageRoot(), "workflows"), "builtin"),
 		user: readWorkflowDir(path.join(userPiRoot(), "workflows"), "user"),
-		project: readWorkflowDir(
-			path.join(projectCrewRoot(cwd), "workflows"),
-			"project",
-		),
+		project: readWorkflowDir(path.join(projectCrewRoot(cwd), "workflows"), "project"),
 	};
 }
 
-export function allWorkflows(
-	discovery: WorkflowDiscoveryResult | undefined,
-): WorkflowConfig[] {
+export function allWorkflows(discovery: WorkflowDiscoveryResult | undefined): WorkflowConfig[] {
 	if (!discovery) return [];
 	const byName = new Map<string, WorkflowConfig>();
-	for (const workflow of [
-		...discovery.project,
-		...discovery.builtin,
-		...discovery.user,
-	]) {
+	for (const workflow of [...discovery.project, ...discovery.builtin, ...discovery.user]) {
 		byName.set(workflow.name, workflow);
 	}
 	return [...byName.values()].sort((a, b) => a.name.localeCompare(b.name));

package/src/workflows/intermediate-store.ts

@@ -9,7 +9,7 @@
  * - Debugging (inspect intermediate outputs)
  */
 
-import { mkdirSync, readFileSync, writeFileSync, existsSync, readdirSync, unlinkSync } from "node:fs";
+import { existsSync, mkdirSync, readdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
 import path from "node:path";
 import { logInternalError } from "../utils/internal-error.ts";
 import { isSafePathId } from "../utils/safe-paths.ts";
@@ -55,12 +55,7 @@ export function ensureIntermediateDir(config: Partial<IntermediateStoreConfig> =
  * @param data - Phase output data
  * @returns Path to the written file
  */
-export function writeIntermediate(
-	config: Partial<IntermediateStoreConfig>,
-	phase: string,
-	stepId: string,
-	data: unknown,
-): string {
+export function writeIntermediate(config: Partial<IntermediateStoreConfig>, phase: string, stepId: string, data: unknown): string {
 	const dir = ensureIntermediateDir(config);
 	const filename = `${phase}-${stepId}.json`;
 	const filePath = path.join(dir, filename);
@@ -84,11 +79,7 @@ export function writeIntermediate(
  * @param stepId - Step ID
  * @returns Parsed intermediate output, or undefined if not found
  */
-export function readIntermediate(
-	config: Partial<IntermediateStoreConfig>,
-	phase: string,
-	stepId: string,
-): IntermediateOutput | undefined {
+export function readIntermediate(config: Partial<IntermediateStoreConfig>, phase: string, stepId: string): IntermediateOutput | undefined {
 	// M-2 fix (code-review 2026-06-23): validate phase/stepId before building the
 	// filename to prevent path traversal via a poisoned stepId.
 	if (!isSafePathId(phase) || !isSafePathId(stepId)) return undefined;
@@ -113,10 +104,7 @@ export function readIntermediate(
  * Useful when you don't know the exact stepId but want the latest
  * output from a phase (e.g., for incremental re-runs).
  */
-export function readLatestIntermediate(
-	config: Partial<IntermediateStoreConfig>,
-	phase: string,
-): IntermediateOutput | undefined {
+export function readLatestIntermediate(config: Partial<IntermediateStoreConfig>, phase: string): IntermediateOutput | undefined {
 	const dir = config.intermediateDir ?? DEFAULT_CONFIG.intermediateDir;
 	if (!existsSync(dir)) return undefined;
 
@@ -168,10 +156,6 @@ export function cleanupIntermediates(config: Partial<IntermediateStoreConfig> =
 /**
  * Check if a phase has completed (intermediate exists).
  */
-export function hasPhaseCompleted(
-	config: Partial<IntermediateStoreConfig>,
-	phase: string,
-	stepId: string,
-): boolean {
+export function hasPhaseCompleted(config: Partial<IntermediateStoreConfig>, phase: string, stepId: string): boolean {
 	return readIntermediate(config, phase, stepId) !== undefined;
 }

package/src/workflows/preflight-validator.ts

@@ -11,10 +11,7 @@
 // result they receive. This keeps the validator side-effect free and
 // trivially testable.
 
-import {
-	analyzeWorkflowTopology,
-	type WorkflowTopology,
-} from "./topology-analyzer.ts";
+import { analyzeWorkflowTopology, type WorkflowTopology } from "./topology-analyzer.ts";
 import type { WorkflowConfig } from "./workflow-config.ts";
 
 export interface PreflightOptions {
@@ -43,12 +40,7 @@ export interface PreflightResult {
 	recommendation: string;
 }
 
-function info(
-	topology: WorkflowTopology,
-	stepCount: number,
-	recommendation: string,
-	message: string,
-): PreflightResult {
+function info(topology: WorkflowTopology, stepCount: number, recommendation: string, message: string): PreflightResult {
 	return {
 		level: "info",
 		message,
@@ -59,12 +51,7 @@ function info(
 	};
 }
 
-function note(
-	topology: WorkflowTopology,
-	stepCount: number,
-	recommendation: string,
-	message: string,
-): PreflightResult {
+function note(topology: WorkflowTopology, stepCount: number, recommendation: string, message: string): PreflightResult {
 	return {
 		level: "note",
 		message,
@@ -75,13 +62,7 @@ function note(
 	};
 }
 
-function warn(
-	topology: WorkflowTopology,
-	stepCount: number,
-	recommendation: string,
-	message: string,
-	suggestion: string,
-): PreflightResult {
+function warn(topology: WorkflowTopology, stepCount: number, recommendation: string, message: string, suggestion: string): PreflightResult {
 	return {
 		level: "warn",
 		message,
@@ -108,21 +89,13 @@ function warn(
  * No `block` level exists — there is no scenario where pi-crew hard-rejects a call.
  * The agent always gets to decide.
  */
-export function validateWorkflowUsage(
-	workflow: WorkflowConfig,
-	options: PreflightOptions = {},
-): PreflightResult {
+export function validateWorkflowUsage(workflow: WorkflowConfig, options: PreflightOptions = {}): PreflightResult {
 	const analysis = analyzeWorkflowTopology(workflow);
 	const { topology, stepCount, recommendation } = analysis;
 
 	// Rule 0: dynamic workflows are runtime-decided.
 	if (topology === "dynamic") {
-		return info(
-			topology,
-			stepCount,
-			recommendation,
-			"Dynamic workflow — runtime decides topology.",
-		);
+		return info(topology, stepCount, recommendation, "Dynamic workflow — runtime decides topology.");
 	}
 
 	// Rule 0b: explicit force-bypass acknowledged (still log it).

package/src/workflows/topology-analyzer.ts

@@ -20,19 +20,9 @@
 
 import type { WorkflowConfig, WorkflowStep } from "./workflow-config.ts";
 
-export type WorkflowTopology =
-	| "single"
-	| "sequential"
-	| "concurrent"
-	| "complex-dag"
-	| "dynamic";
+export type WorkflowTopology = "single" | "sequential" | "concurrent" | "complex-dag" | "dynamic";
 
-export type TopologyRecommendation =
-	| "raw_agent"
-	| "fast_fix"
-	| "parallel_research"
-	| "implementation_adaptive"
-	| "any";
+export type TopologyRecommendation = "raw_agent" | "fast_fix" | "parallel_research" | "implementation_adaptive" | "any";
 
 export interface TopologyAnalysis {
 	topology: WorkflowTopology;
@@ -58,10 +48,7 @@ export function fanOutDegreeFromSteps(steps: WorkflowStep[]): number {
 	const counts = new Map<string, number>();
 	for (const step of steps) {
 		if (step.parallelGroup === undefined) continue;
-		counts.set(
-			step.parallelGroup,
-			(counts.get(step.parallelGroup) ?? 0) + 1,
-		);
+		counts.set(step.parallelGroup, (counts.get(step.parallelGroup) ?? 0) + 1);
 	}
 	let max = 0;
 	for (const count of counts.values()) if (count > max) max = count;
@@ -129,9 +116,7 @@ function hasMultiDeps(steps: WorkflowStep[]): boolean {
  * Classify a workflow's topology. See file header for the rule order.
  * Pure function — no I/O, no side effects, no exceptions.
  */
-export function analyzeWorkflowTopology(
-	workflow: WorkflowConfig,
-): TopologyAnalysis {
+export function analyzeWorkflowTopology(workflow: WorkflowConfig): TopologyAnalysis {
 	// Chain/dynamic workflows: runtime decides the topology at execution time.
 	// Don't try to classify the empty `steps: []` (DynamicWorkflowConfig forces
 	// it to be empty).
@@ -186,8 +171,7 @@ export function analyzeWorkflowTopology(
 	if (stepCount === 1) {
 		topology = "single";
 		recommendation = "raw_agent";
-		reason =
-			"Single-task workflow: no concurrency or DAG structure to justify pi-crew overhead.";
+		reason = "Single-task workflow: no concurrency or DAG structure to justify pi-crew overhead.";
 	} else if (parallelGroupCount >= 1 && fanOutDegree >= 3) {
 		topology = "concurrent";
 		recommendation = "parallel_research";

package/src/workflows/workflow-config.ts

@@ -53,12 +53,13 @@ export interface WorkflowConfig {
 	 *  Used by preflight-validator to enforce "don't use pi-crew for sequential chains".
 	 *  Valid values: 'single' | 'sequential' | 'concurrent' | 'complex-dag' | 'dynamic'.
 	 *  Absent = auto-classify from step structure (default). */
-	topology?:
-		| "single"
-		| "sequential"
-		| "concurrent"
-		| "complex-dag"
-		| "dynamic";
+	topology?: "single" | "sequential" | "concurrent" | "complex-dag" | "dynamic";
+	/** Round 25 (M6): coalesce micro-tasks. Default false (off). When true,
+	 *  multiple ready tasks sharing role + cwd are grouped into a single
+	 *  multi-task worker prompt to reduce per-task cold-start cost. Disabled
+	 *  in v0.9.17 unless the workflow author opts in. See
+	 *  src/runtime/coalesce-tasks.ts. */
+	coalesceMicroTasks?: boolean;
 }
 
 /** A dynamic workflow (runtime === "dynamic"). steps is empty — the script is the source of truth. */