pi-crew 0.9.16 → 0.9.18

package/src/runtime/workspace-tree.ts

@@ -82,15 +82,15 @@ function compareByRecency(a: TreeNode, b: TreeNode): number {
 	return a.name.localeCompare(b.name);
 }
 
-function applyDirLimit(
-	children: TreeNode[],
-	limit: number,
-): { visible: TreeNode[]; dropped: number } {
+function applyDirLimit(children: TreeNode[], limit: number): { visible: TreeNode[]; dropped: number } {
 	if (children.length <= limit) {
 		return { visible: children, dropped: 0 };
 	}
 	if (limit <= 1) {
-		return { visible: children.slice(0, limit), dropped: children.length - limit };
+		return {
+			visible: children.slice(0, limit),
+			dropped: children.length - limit,
+		};
 	}
 	// Keep first (limit-1) by recency + always keep the oldest (last after sort)
 	const recent = children.slice(0, limit - 1);
@@ -99,14 +99,8 @@ function applyDirLimit(
 	return { visible, dropped: children.length - limit };
 }
 
-async function readChildren(
-	rootPath: string,
-	parent: TreeNode,
-	excludedDirs: ReadonlySet<string>,
-): Promise<TreeNode[]> {
-	const dirPath = parent.relativePath
-		? path.join(rootPath, parent.relativePath)
-		: rootPath;
+async function readChildren(rootPath: string, parent: TreeNode, excludedDirs: ReadonlySet<string>): Promise<TreeNode[]> {
+	const dirPath = parent.relativePath ? path.join(rootPath, parent.relativePath) : rootPath;
 
 	let names: string[];
 	try {
@@ -119,9 +113,7 @@ async function readChildren(
 		names.map(async (name): Promise<TreeNode | null> => {
 			// Skip hidden entries
 			if (name.startsWith(".")) return null;
-			const relativePath = parent.relativePath
-				? `${parent.relativePath}/${name}`
-				: name;
+			const relativePath = parent.relativePath ? `${parent.relativePath}/${name}` : name;
 			const absolutePath = path.join(rootPath, relativePath);
 			try {
 				const stat = await fs.stat(absolutePath);
@@ -201,7 +193,11 @@ function collectLines(node: TreeNode, nowMs: number, lines: RenderLine[]): void
 			const ageSeconds = Math.max(0, Math.floor((nowMs - node.mtimeMs) / 1000));
 			const size = formatBytes(node.size);
 			const age = formatAge(ageSeconds);
-			lines.push({ text: `${label}  ${size}  ${age}`, depth: node.depth, isRoot: false });
+			lines.push({
+				text: `${label}  ${size}  ${age}`,
+				depth: node.depth,
+				isRoot: false,
+			});
 		}
 	}
 
@@ -225,10 +221,7 @@ function collectLines(node: TreeNode, nowMs: number, lines: RenderLine[]): void
 	}
 }
 
-function applyLineCap(
-	lines: RenderLine[],
-	cap: number,
-): { lines: RenderLine[]; elided: number } {
+function applyLineCap(lines: RenderLine[], cap: number): { lines: RenderLine[]; elided: number } {
 	if (lines.length <= cap) return { lines, elided: 0 };
 
 	const target = Math.max(1, cap - 1);
@@ -280,10 +273,7 @@ function treeCacheKey(cwd: string, options?: WorkspaceTreeOptions): string {
 	return `${path.resolve(cwd)}|${options?.maxDepth ?? ""}|${options?.dirLimit ?? ""}|${options?.lineCap ?? ""}`;
 }
 
-export async function buildWorkspaceTree(
-	cwd: string,
-	options?: WorkspaceTreeOptions,
-): Promise<WorkspaceTree> {
+export async function buildWorkspaceTree(cwd: string, options?: WorkspaceTreeOptions): Promise<WorkspaceTree> {
 	const rootPath = path.resolve(cwd);
 	const cacheKey = treeCacheKey(cwd, options);
 	const cached = treeCache.get(cacheKey);
@@ -296,12 +286,7 @@ export async function buildWorkspaceTree(
 		const lineCap = options?.lineCap ?? DEFAULT_LINE_CAP;
 		const excludedDirs = options?.excludedDirs ?? DEFAULT_EXCLUDED_DIRS;
 
-		const { root, truncated: dirTruncated } = await collectTree(
-			rootPath,
-			maxDepth,
-			dirLimit,
-			excludedDirs,
-		);
+		const { root, truncated: dirTruncated } = await collectTree(rootPath, maxDepth, dirLimit, excludedDirs);
 
 		const nowMs = Date.now();
 		const lines: RenderLine[] = [];
@@ -316,7 +301,10 @@ export async function buildWorkspaceTree(
 			truncated: dirTruncated || elided > 0,
 			totalLines: capped.length,
 		};
-		treeCache.set(cacheKey, { tree: result, expiresAt: Date.now() + TREE_CACHE_TTL_MS });
+		treeCache.set(cacheKey, {
+			tree: result,
+			expiresAt: Date.now() + TREE_CACHE_TTL_MS,
+		});
 		return result;
 	} catch {
 		return emptyResult(rootPath);

package/src/runtime/yield-handler.ts

@@ -1,4 +1,4 @@
-import { subprocessToolRegistry, type SubprocessToolEvent } from "./subprocess-tool-registry.ts";
+import { type SubprocessToolEvent, subprocessToolRegistry } from "./subprocess-tool-registry.ts";
 
 // G3: Full AJV-based schema validation for yield data.
 // Falls back to lightweight validation if AJV is unavailable.
@@ -8,7 +8,15 @@ let _ajv: Awaited<ReturnType<typeof getAjvInternal>> | null | undefined;
 async function getAjvInternal() {
 	// LAZY: AJV is an optional heavy validator — only load on first use.
 	const mod = await import("ajv");
-	const AjvCtor = ("default" in mod ? (mod as Record<string, unknown>).default : mod) as unknown as new (opts: Record<string, unknown>) => { compile: (schema: unknown) => { (data: unknown): boolean; errors?: unknown[] }; errorsText: (errors?: unknown[]) => string };
+	const AjvCtor = ("default" in mod ? (mod as Record<string, unknown>).default : mod) as unknown as new (
+		opts: Record<string, unknown>,
+	) => {
+		compile: (schema: unknown) => {
+			(data: unknown): boolean;
+			errors?: unknown[];
+		};
+		errorsText: (errors?: unknown[]) => string;
+	};
 	return new AjvCtor({ allErrors: true, strict: false, logger: false });
 }
 
@@ -44,7 +52,10 @@ export async function validateYieldData(data: unknown, schema: unknown): Promise
 			const validate = ajv.compile(schema as Record<string, unknown>);
 			const valid = validate(data);
 			if (!valid) {
-				return { valid: false, error: ajv.errorsText(validate.errors ?? undefined) };
+				return {
+					valid: false,
+					error: ajv.errorsText(validate.errors ?? undefined),
+				};
 			}
 			return { valid: true };
 		} catch {
@@ -65,11 +76,31 @@ function lightweightValidate(data: unknown, schema: unknown): SchemaValidationRe
 	if (typeof schemaObj.type === "string") {
 		const expected = schemaObj.type;
 		const actual = Array.isArray(data) ? "array" : typeof data;
-		if (expected === "object" && (typeof data !== "object" || Array.isArray(data))) return { valid: false, error: `Expected type '${expected}' but got '${actual}'.` };
-		if (expected === "array" && !Array.isArray(data)) return { valid: false, error: `Expected type 'array' but got '${actual}'.` };
-		if (expected === "string" && typeof data !== "string") return { valid: false, error: `Expected type 'string' but got '${actual}'.` };
-		if (expected === "number" && typeof data !== "number") return { valid: false, error: `Expected type 'number' but got '${actual}'.` };
-		if (expected === "boolean" && typeof data !== "boolean") return { valid: false, error: `Expected type 'boolean' but got '${actual}'.` };
+		if (expected === "object" && (typeof data !== "object" || Array.isArray(data)))
+			return {
+				valid: false,
+				error: `Expected type '${expected}' but got '${actual}'.`,
+			};
+		if (expected === "array" && !Array.isArray(data))
+			return {
+				valid: false,
+				error: `Expected type 'array' but got '${actual}'.`,
+			};
+		if (expected === "string" && typeof data !== "string")
+			return {
+				valid: false,
+				error: `Expected type 'string' but got '${actual}'.`,
+			};
+		if (expected === "number" && typeof data !== "number")
+			return {
+				valid: false,
+				error: `Expected type 'number' but got '${actual}'.`,
+			};
+		if (expected === "boolean" && typeof data !== "boolean")
+			return {
+				valid: false,
+				error: `Expected type 'boolean' but got '${actual}'.`,
+			};
 	}
 
 	// Check required fields
@@ -77,7 +108,10 @@ function lightweightValidate(data: unknown, schema: unknown): SchemaValidationRe
 		const dataObj = data as Record<string, unknown>;
 		for (const field of schemaObj.required) {
 			if (typeof field === "string" && !(field in dataObj)) {
-				return { valid: false, error: `Missing required field '${field}'.` };
+				return {
+					valid: false,
+					error: `Missing required field '${field}'.`,
+				};
 			}
 		}
 	}

package/src/runtime/zombie-scanner.ts

@@ -69,7 +69,10 @@ function readProcStat(pid: number): { ppid: number; elapsedSec: number | undefin
 		// comm may contain spaces/parens, so parse from the LAST ')' backwards.
 		const closeParen = stat.lastIndexOf(")");
 		if (closeParen < 0) return undefined;
-		const rest = stat.slice(closeParen + 2).trim().split(/\s+/);
+		const rest = stat
+			.slice(closeParen + 2)
+			.trim()
+			.split(/\s+/);
 		// rest[0] = state, rest[1] = ppid
 		const ppid = Number.parseInt(rest[1] ?? "", 10);
 		// starttime (clock ticks since boot) is field 22 in the full stat → index 19 in `rest`
@@ -236,9 +239,7 @@ export function formatZombieReport(scan: ZombieScanResult): string {
 	const lines: string[] = [];
 	lines.push("## Zombie sub-agent scan (read-only — nothing killed)");
 	lines.push("");
-	lines.push(
-		`Sub-agents identified by PI_CREW_KIND=subagent marker. Main sessions (no marker) are never listed.`,
-	);
+	lines.push(`Sub-agents identified by PI_CREW_KIND=subagent marker. Main sessions (no marker) are never listed.`);
 	lines.push("");
 
 	if (scan.zombies.length === 0 && scan.live.length === 0) {