pi-crew 0.1.30 → 0.1.32

package/src/utils/sleep.ts

@@ -1,32 +1,32 @@
-/**
- * Sleep helper that respects abort signal.
- */
-export function sleep(ms: number, signal?: AbortSignal): Promise<void> {
-	return new Promise((resolve, reject) => {
-		if (signal?.aborted) {
-			reject(new Error("Aborted"));
-			return;
-		}
-
-		let settled = false;
-		const cleanup = (): void => {
-			if (signal) signal.removeEventListener("abort", onAbort);
-		};
-		const timeout = setTimeout(() => {
-			if (settled) return;
-			settled = true;
-			cleanup();
-			resolve();
-		}, ms);
-
-		const onAbort = (): void => {
-			if (settled) return;
-			settled = true;
-			clearTimeout(timeout);
-			cleanup();
-			reject(new Error("Aborted"));
-		};
-
-		signal?.addEventListener("abort", onAbort);
-	});
-}
+/**
+ * Sleep helper that respects abort signal.
+ */
+export function sleep(ms: number, signal?: AbortSignal): Promise<void> {
+	return new Promise((resolve, reject) => {
+		if (signal?.aborted) {
+			reject(new Error("Aborted"));
+			return;
+		}
+
+		let settled = false;
+		const cleanup = (): void => {
+			if (signal) signal.removeEventListener("abort", onAbort);
+		};
+		const timeout = setTimeout(() => {
+			if (settled) return;
+			settled = true;
+			cleanup();
+			resolve();
+		}, ms);
+
+		const onAbort = (): void => {
+			if (settled) return;
+			settled = true;
+			clearTimeout(timeout);
+			cleanup();
+			reject(new Error("Aborted"));
+		};
+
+		signal?.addEventListener("abort", onAbort);
+	});
+}

package/src/utils/timings.ts

@@ -1,31 +1,31 @@
-/**
- * Central timing instrumentation for startup profiling.
- * Enable with PI_TIMING=1 environment variable.
- */
-
-const ENABLED = process.env.PI_TIMING === "1";
-const timings: Array<{ label: string; ms: number }> = [];
-let lastTime = Date.now();
-
-export function resetTimings(): void {
-	if (!ENABLED) return;
-	timings.length = 0;
-	lastTime = Date.now();
-}
-
-export function time(label: string): void {
-	if (!ENABLED) return;
-	const now = Date.now();
-	timings.push({ label, ms: now - lastTime });
-	lastTime = now;
-}
-
-export function printTimings(): void {
-	if (!ENABLED || timings.length === 0) return;
-	console.error("\n--- Startup Timings ---");
-	for (const t of timings) {
-		console.error(`  ${t.label}: ${t.ms}ms`);
-	}
-	console.error(`  TOTAL: ${timings.reduce((a, b) => a + b.ms, 0)}ms`);
-	console.error("------------------------\n");
-}
+/**
+ * Central timing instrumentation for startup profiling.
+ * Enable with PI_TIMING=1 environment variable.
+ */
+
+const ENABLED = process.env.PI_TIMING === "1";
+const timings: Array<{ label: string; ms: number }> = [];
+let lastTime = Date.now();
+
+export function resetTimings(): void {
+	if (!ENABLED) return;
+	timings.length = 0;
+	lastTime = Date.now();
+}
+
+export function time(label: string): void {
+	if (!ENABLED) return;
+	const now = Date.now();
+	timings.push({ label, ms: now - lastTime });
+	lastTime = now;
+}
+
+export function printTimings(): void {
+	if (!ENABLED || timings.length === 0) return;
+	console.error("\n--- Startup Timings ---");
+	for (const t of timings) {
+		console.error(`  ${t.label}: ${t.ms}ms`);
+	}
+	console.error(`  TOTAL: ${timings.reduce((a, b) => a + b.ms, 0)}ms`);
+	console.error("------------------------\n");
+}

package/src/utils/visual.ts

@@ -1,159 +1,159 @@
-export const ANSI_PATTERN = /\u001b\[[0-?]*[ -/]*[@-~]/g;
-
-const WIDTH_CACHE_LIMIT = 256;
-const widthCache = new Map<string, number>();
-
-export function visibleWidth(value: string): number {
-	const cached = widthCache.get(value);
-	if (cached !== undefined) return cached;
-	let length = 0;
-	for (const char of value.replace(ANSI_PATTERN, "")) {
-		if (char !== "\n") length += 1;
-	}
-	if (widthCache.size >= WIDTH_CACHE_LIMIT) {
-		const firstKey = widthCache.keys().next().value;
-		if (firstKey !== undefined) widthCache.delete(firstKey);
-	}
-	widthCache.set(value, length);
-	return length;
-}
-
-export function __test__clearVisibleWidthCache(): void {
-	widthCache.clear();
-}
-
-export function __test__visibleWidthCacheSize(): number {
-	return widthCache.size;
-}
-
-function consumeAnsi(input: string, index: number): number {
-	const char = input[index];
-	if (!char || char !== "\u001b") return 0;
-	if (input[index + 1] !== "[") return 0;
-	let i = index + 2;
-	while (i < input.length) {
-		const code = input.charCodeAt(i);
-		if (code >= 0x40 && code <= 0x7e) return i - index + 1;
-		i++;
-	}
-	return 0;
-}
-
-function splitGraphemes(value: string): string[] {
-	return Array.from(value.replace(ANSI_PATTERN, ""));
-}
-
-export function truncateToWidth(value: string, width: number, ellipsis = "…"): string {
-	if (width <= 0) return "";
-	if (visibleWidth(value) <= width) return value;
-	if (width <= ellipsis.length) return ellipsis.slice(0, width);
-	let output = "";
-	let renderedWidth = 0;
-	for (let i = 0; i < value.length; i++) {
-		const ansiLen = consumeAnsi(value, i);
-		if (ansiLen) {
-			output += value.slice(i, i + ansiLen);
-			i += ansiLen - 1;
-			continue;
-		}
-		const char = value[i] as string;
-		const nextIndex = i + (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) {
-			return `${output}${ellipsis}`;
-		}
-		output += segment;
-		renderedWidth += charWidth;
-		i = nextIndex - 1;
-	}
-	return output;
-}
-
-export const truncate = truncateToWidth;
-
-export function pad(value: string, width: number): string {
-	const current = visibleWidth(value);
-	if (current >= width) return value;
-	return `${value}${" ".repeat(width - current)}`;
-}
-
-export function boxLine(text: string, innerWidth: number): string {
-	return `│ ${truncate(text, innerWidth - 4)} │`;
-}
-
-function readAnsiCode(input: string, index: number): string | undefined {
-	const ansiLength = consumeAnsi(input, index);
-	if (ansiLength > 0) return input.slice(index, index + ansiLength);
-	return 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) {
-		return { chunk: input.slice(index, index + 2), nextIndex: index + 2 };
-	}
-	return { chunk: input[index] ?? "", nextIndex: index + 1 };
-}
-
-export function wrapHard(value: string, width: number): string[] {
-	if (width <= 0 || !value) return [];
-	const lines: string[] = [];
-	let current = "";
-	let currentWidth = 0;
-	let i = 0;
-	while (i < value.length) {
-		const ansi = readAnsiCode(value, i);
-		if (ansi) {
-			current += ansi;
-			i += ansi.length;
-			continue;
-		}
-		const { chunk, nextIndex } = takeCodePoint(value, i);
-		const chunkWidth = visibleWidth(chunk);
-		if (chunkWidth > width) {
-			lines.push(current ? current + chunk : chunk);
-			current = "";
-			currentWidth = 0;
-			i = nextIndex;
-			continue;
-		}
-		if (currentWidth + chunkWidth > width) {
-			if (current) lines.push(current);
-			current = chunk;
-			currentWidth = chunkWidth;
-			i = nextIndex;
-			continue;
-		}
-		current += chunk;
-		currentWidth += chunkWidth;
-		i = nextIndex;
-	}
-	if (current) lines.push(current);
-	return lines.length > 0 ? lines : [""];
-}
-
-export interface VisualTruncateResult {
-	visualLines: string[];
-	skippedCount: number;
-}
-
-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));
-	if (visualLines.length <= limit) return { visualLines, skippedCount: 0 };
-	const truncated = visualLines.slice(-limit);
-	return { visualLines: truncated, skippedCount: visualLines.length - limit };
-}
+export const ANSI_PATTERN = /\u001b\[[0-?]*[ -/]*[@-~]/g;
+
+const WIDTH_CACHE_LIMIT = 256;
+const widthCache = new Map<string, number>();
+
+export function visibleWidth(value: string): number {
+	const cached = widthCache.get(value);
+	if (cached !== undefined) return cached;
+	let length = 0;
+	for (const char of value.replace(ANSI_PATTERN, "")) {
+		if (char !== "\n") length += 1;
+	}
+	if (widthCache.size >= WIDTH_CACHE_LIMIT) {
+		const firstKey = widthCache.keys().next().value;
+		if (firstKey !== undefined) widthCache.delete(firstKey);
+	}
+	widthCache.set(value, length);
+	return length;
+}
+
+export function __test__clearVisibleWidthCache(): void {
+	widthCache.clear();
+}
+
+export function __test__visibleWidthCacheSize(): number {
+	return widthCache.size;
+}
+
+function consumeAnsi(input: string, index: number): number {
+	const char = input[index];
+	if (!char || char !== "\u001b") return 0;
+	if (input[index + 1] !== "[") return 0;
+	let i = index + 2;
+	while (i < input.length) {
+		const code = input.charCodeAt(i);
+		if (code >= 0x40 && code <= 0x7e) return i - index + 1;
+		i++;
+	}
+	return 0;
+}
+
+function splitGraphemes(value: string): string[] {
+	return Array.from(value.replace(ANSI_PATTERN, ""));
+}
+
+export function truncateToWidth(value: string, width: number, ellipsis = "…"): string {
+	if (width <= 0) return "";
+	if (visibleWidth(value) <= width) return value;
+	if (width <= ellipsis.length) return ellipsis.slice(0, width);
+	let output = "";
+	let renderedWidth = 0;
+	for (let i = 0; i < value.length; i++) {
+		const ansiLen = consumeAnsi(value, i);
+		if (ansiLen) {
+			output += value.slice(i, i + ansiLen);
+			i += ansiLen - 1;
+			continue;
+		}
+		const char = value[i] as string;
+		const nextIndex = i + (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) {
+			return `${output}${ellipsis}`;
+		}
+		output += segment;
+		renderedWidth += charWidth;
+		i = nextIndex - 1;
+	}
+	return output;
+}
+
+export const truncate = truncateToWidth;
+
+export function pad(value: string, width: number): string {
+	const current = visibleWidth(value);
+	if (current >= width) return value;
+	return `${value}${" ".repeat(width - current)}`;
+}
+
+export function boxLine(text: string, innerWidth: number): string {
+	return `│ ${truncate(text, innerWidth - 4)} │`;
+}
+
+function readAnsiCode(input: string, index: number): string | undefined {
+	const ansiLength = consumeAnsi(input, index);
+	if (ansiLength > 0) return input.slice(index, index + ansiLength);
+	return 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) {
+		return { chunk: input.slice(index, index + 2), nextIndex: index + 2 };
+	}
+	return { chunk: input[index] ?? "", nextIndex: index + 1 };
+}
+
+export function wrapHard(value: string, width: number): string[] {
+	if (width <= 0 || !value) return [];
+	const lines: string[] = [];
+	let current = "";
+	let currentWidth = 0;
+	let i = 0;
+	while (i < value.length) {
+		const ansi = readAnsiCode(value, i);
+		if (ansi) {
+			current += ansi;
+			i += ansi.length;
+			continue;
+		}
+		const { chunk, nextIndex } = takeCodePoint(value, i);
+		const chunkWidth = visibleWidth(chunk);
+		if (chunkWidth > width) {
+			lines.push(current ? current + chunk : chunk);
+			current = "";
+			currentWidth = 0;
+			i = nextIndex;
+			continue;
+		}
+		if (currentWidth + chunkWidth > width) {
+			if (current) lines.push(current);
+			current = chunk;
+			currentWidth = chunkWidth;
+			i = nextIndex;
+			continue;
+		}
+		current += chunk;
+		currentWidth += chunkWidth;
+		i = nextIndex;
+	}
+	if (current) lines.push(current);
+	return lines.length > 0 ? lines : [""];
+}
+
+export interface VisualTruncateResult {
+	visualLines: string[];
+	skippedCount: number;
+}
+
+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));
+	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

@@ -2,7 +2,7 @@ import * as fs from "node:fs";
 import * as path from "node:path";
 import type { ResourceSource } from "../agents/agent-config.ts";
 import { parseCsv, parseFrontmatter } from "../utils/frontmatter.ts";
-import { packageRoot, projectPiRoot, userPiRoot } from "../utils/paths.ts";
+import { packageRoot, projectCrewRoot, userPiRoot } from "../utils/paths.ts";
 import type { WorkflowConfig, WorkflowStep } from "./workflow-config.ts";
 
 export interface WorkflowDiscoveryResult {
@@ -96,7 +96,7 @@ export function discoverWorkflows(cwd: string): WorkflowDiscoveryResult {
 	return {
 		builtin: readWorkflowDir(path.join(packageRoot(), "workflows"), "builtin"),
 		user: readWorkflowDir(path.join(userPiRoot(), "workflows"), "user"),
-		project: readWorkflowDir(path.join(projectPiRoot(cwd), "workflows"), "project"),
+		project: readWorkflowDir(path.join(projectCrewRoot(cwd), "workflows"), "project"),
 	};
 }
 

package/src/workflows/validate-workflow.ts

@@ -1,40 +1,40 @@
-import type { TeamConfig } from "../teams/team-config.ts";
-import type { WorkflowConfig } from "./workflow-config.ts";
-
-export function validateWorkflowForTeam(workflow: WorkflowConfig, team: TeamConfig): string[] {
-	const errors: string[] = [];
-	const roles = new Set(team.roles.map((role) => role.name));
-	const stepIds = new Set<string>();
-
-	for (const step of workflow.steps) {
-		if (stepIds.has(step.id)) errors.push(`Duplicate workflow step id '${step.id}'.`);
-		stepIds.add(step.id);
-		if (!roles.has(step.role)) errors.push(`Step '${step.id}' references unknown team role '${step.role}'.`);
-	}
-
-	for (const step of workflow.steps) {
-		for (const dep of step.dependsOn ?? []) {
-			if (!stepIds.has(dep)) errors.push(`Step '${step.id}' depends on unknown step '${dep}'.`);
-		}
-	}
-
-	const visiting = new Set<string>();
-	const visited = new Set<string>();
-	const byId = new Map(workflow.steps.map((step) => [step.id, step]));
-
-	function visit(id: string, trail: string[]): void {
-		if (visited.has(id)) return;
-		if (visiting.has(id)) {
-			errors.push(`Workflow dependency cycle detected: ${[...trail, id].join(" -> ")}.`);
-			return;
-		}
-		visiting.add(id);
-		const step = byId.get(id);
-		for (const dep of step?.dependsOn ?? []) visit(dep, [...trail, id]);
-		visiting.delete(id);
-		visited.add(id);
-	}
-
-	for (const step of workflow.steps) visit(step.id, []);
-	return [...new Set(errors)];
-}
+import type { TeamConfig } from "../teams/team-config.ts";
+import type { WorkflowConfig } from "./workflow-config.ts";
+
+export function validateWorkflowForTeam(workflow: WorkflowConfig, team: TeamConfig): string[] {
+	const errors: string[] = [];
+	const roles = new Set(team.roles.map((role) => role.name));
+	const stepIds = new Set<string>();
+
+	for (const step of workflow.steps) {
+		if (stepIds.has(step.id)) errors.push(`Duplicate workflow step id '${step.id}'.`);
+		stepIds.add(step.id);
+		if (!roles.has(step.role)) errors.push(`Step '${step.id}' references unknown team role '${step.role}'.`);
+	}
+
+	for (const step of workflow.steps) {
+		for (const dep of step.dependsOn ?? []) {
+			if (!stepIds.has(dep)) errors.push(`Step '${step.id}' depends on unknown step '${dep}'.`);
+		}
+	}
+
+	const visiting = new Set<string>();
+	const visited = new Set<string>();
+	const byId = new Map(workflow.steps.map((step) => [step.id, step]));
+
+	function visit(id: string, trail: string[]): void {
+		if (visited.has(id)) return;
+		if (visiting.has(id)) {
+			errors.push(`Workflow dependency cycle detected: ${[...trail, id].join(" -> ")}.`);
+			return;
+		}
+		visiting.add(id);
+		const step = byId.get(id);
+		for (const dep of step?.dependsOn ?? []) visit(dep, [...trail, id]);
+		visiting.delete(id);
+		visited.add(id);
+	}
+
+	for (const step of workflow.steps) visit(step.id, []);
+	return [...new Set(errors)];
+}

package/src/worktree/branch-freshness.ts

@@ -1,45 +1,45 @@
-import { execFileSync } from "node:child_process";
-
-export type BranchFreshnessStatus = "fresh" | "stale" | "diverged" | "unknown";
-export type StaleBranchPolicy = "warn" | "block" | "auto_rebase" | "auto_merge_forward";
-
-export interface BranchFreshness {
-	status: BranchFreshnessStatus;
-	branch?: string;
-	mainRef: string;
-	ahead: number;
-	behind: number;
-	missingFixes: string[];
-	message: string;
-	error?: string;
-}
-
-function git(cwd: string, args: string[]): string {
-	return execFileSync("git", args, { cwd, encoding: "utf-8", stdio: ["ignore", "pipe", "pipe"] }).trim();
-}
-
-function count(cwd: string, range: string): number {
-	const raw = git(cwd, ["rev-list", "--count", range]);
-	const parsed = Number.parseInt(raw, 10);
-	return Number.isFinite(parsed) ? parsed : 0;
-}
-
-export function checkBranchFreshness(cwd: string, mainRef = "main"): BranchFreshness {
-	try {
-		git(cwd, ["rev-parse", "--is-inside-work-tree"]);
-		const branch = git(cwd, ["rev-parse", "--abbrev-ref", "HEAD"]);
-		const behind = count(cwd, `${branch}..${mainRef}`);
-		const ahead = count(cwd, `${mainRef}..${branch}`);
-		const missingFixes = behind > 0 ? git(cwd, ["log", "--format=%s", `${branch}..${mainRef}`]).split("\n").map((line) => line.trim()).filter(Boolean) : [];
-		if (behind === 0) return { status: "fresh", branch, mainRef, ahead, behind, missingFixes, message: `Branch '${branch}' is fresh against ${mainRef}.` };
-		if (ahead > 0) return { status: "diverged", branch, mainRef, ahead, behind, missingFixes, message: `Branch '${branch}' diverged from ${mainRef}: ahead=${ahead}, behind=${behind}.` };
-		return { status: "stale", branch, mainRef, ahead, behind, missingFixes, message: `Branch '${branch}' is ${behind} commit(s) behind ${mainRef}.` };
-	} catch (error) {
-		const message = error instanceof Error ? error.message : String(error);
-		return { status: "unknown", mainRef, ahead: 0, behind: 0, missingFixes: [], message: "Branch freshness could not be determined.", error: message };
-	}
-}
-
-export function shouldBlockForBranchFreshness(freshness: BranchFreshness, policy: StaleBranchPolicy = "warn"): boolean {
-	return policy === "block" && (freshness.status === "stale" || freshness.status === "diverged");
-}
+import { execFileSync } from "node:child_process";
+
+export type BranchFreshnessStatus = "fresh" | "stale" | "diverged" | "unknown";
+export type StaleBranchPolicy = "warn" | "block" | "auto_rebase" | "auto_merge_forward";
+
+export interface BranchFreshness {
+	status: BranchFreshnessStatus;
+	branch?: string;
+	mainRef: string;
+	ahead: number;
+	behind: number;
+	missingFixes: string[];
+	message: string;
+	error?: string;
+}
+
+function git(cwd: string, args: string[]): string {
+	return execFileSync("git", args, { cwd, encoding: "utf-8", stdio: ["ignore", "pipe", "pipe"] }).trim();
+}
+
+function count(cwd: string, range: string): number {
+	const raw = git(cwd, ["rev-list", "--count", range]);
+	const parsed = Number.parseInt(raw, 10);
+	return Number.isFinite(parsed) ? parsed : 0;
+}
+
+export function checkBranchFreshness(cwd: string, mainRef = "main"): BranchFreshness {
+	try {
+		git(cwd, ["rev-parse", "--is-inside-work-tree"]);
+		const branch = git(cwd, ["rev-parse", "--abbrev-ref", "HEAD"]);
+		const behind = count(cwd, `${branch}..${mainRef}`);
+		const ahead = count(cwd, `${mainRef}..${branch}`);
+		const missingFixes = behind > 0 ? git(cwd, ["log", "--format=%s", `${branch}..${mainRef}`]).split("\n").map((line) => line.trim()).filter(Boolean) : [];
+		if (behind === 0) return { status: "fresh", branch, mainRef, ahead, behind, missingFixes, message: `Branch '${branch}' is fresh against ${mainRef}.` };
+		if (ahead > 0) return { status: "diverged", branch, mainRef, ahead, behind, missingFixes, message: `Branch '${branch}' diverged from ${mainRef}: ahead=${ahead}, behind=${behind}.` };
+		return { status: "stale", branch, mainRef, ahead, behind, missingFixes, message: `Branch '${branch}' is ${behind} commit(s) behind ${mainRef}.` };
+	} catch (error) {
+		const message = error instanceof Error ? error.message : String(error);
+		return { status: "unknown", mainRef, ahead: 0, behind: 0, missingFixes: [], message: "Branch freshness could not be determined.", error: message };
+	}
+}
+
+export function shouldBlockForBranchFreshness(freshness: BranchFreshness, policy: StaleBranchPolicy = "warn"): boolean {
+	return policy === "block" && (freshness.status === "stale" || freshness.status === "diverged");
+}