pi-crew 0.1.41 → 0.1.44

package/src/state/task-claims.ts

@@ -1,42 +1,44 @@
-import { randomUUID } from "node:crypto";
-import type { TeamTaskState } from "./types.ts";
-
-export interface TaskClaimState {
-	owner: string;
-	token: string;
-	leasedUntil: string;
-}
-
-export function createTaskClaim(owner: string, leaseMs = 5 * 60_000, now = new Date()): TaskClaimState {
-	return { owner, token: randomUUID(), leasedUntil: new Date(now.getTime() + leaseMs).toISOString() };
-}
-
-export function isTaskClaimExpired(claim: TaskClaimState | undefined, now = new Date()): boolean {
-	if (!claim) return false;
-	return Date.parse(claim.leasedUntil) <= now.getTime();
-}
-
-export function canUseTaskClaim(task: Pick<TeamTaskState, "claim">, owner: string, token: string, now = new Date()): boolean {
-	return task.claim?.owner === owner && task.claim.token === token && !isTaskClaimExpired(task.claim, now);
-}
-
-export function claimTask<T extends TeamTaskState>(task: T, owner: string, leaseMs?: number, now = new Date()): T {
-	if (task.claim && !isTaskClaimExpired(task.claim, now)) {
-		throw new Error(`Task '${task.id}' is already claimed by '${task.claim.owner}'.`);
-	}
-	return { ...task, claim: createTaskClaim(owner, leaseMs, now) };
-}
-
-export function releaseTaskClaim<T extends TeamTaskState>(task: T, owner: string, token: string, now = new Date()): T {
-	if (!canUseTaskClaim(task, owner, token, now)) {
-		throw new Error(`Task '${task.id}' claim is not held by '${owner}' or has expired.`);
-	}
-	return { ...task, claim: undefined };
-}
-
-export function transitionClaimedTaskStatus<T extends TeamTaskState>(task: T, owner: string, token: string, status: T["status"], now = new Date()): T {
-	if (!canUseTaskClaim(task, owner, token, now)) {
-		throw new Error(`Task '${task.id}' claim is not held by '${owner}' or has expired.`);
-	}
-	return { ...task, status };
-}
+import { randomUUID } from "node:crypto";
+import type { TeamTaskState } from "./types.ts";
+
+export interface TaskClaimState {
+	owner: string;
+	token: string;
+	leasedUntil: string;
+}
+
+export function createTaskClaim(owner: string, leaseMs = 5 * 60_000, now = new Date()): TaskClaimState {
+	return { owner, token: randomUUID(), leasedUntil: new Date(now.getTime() + leaseMs).toISOString() };
+}
+
+export function isTaskClaimExpired(claim: TaskClaimState | undefined, now = new Date()): boolean {
+	if (!claim) return false;
+	const parsed = Date.parse(claim.leasedUntil);
+	// Corrupt or invalid date strings produce NaN — treat as expired immediately.
+	return Number.isFinite(parsed) ? parsed <= now.getTime() : true;
+}
+
+export function canUseTaskClaim(task: Pick<TeamTaskState, "claim">, owner: string, token: string, now = new Date()): boolean {
+	return task.claim?.owner === owner && task.claim.token === token && !isTaskClaimExpired(task.claim, now);
+}
+
+export function claimTask<T extends TeamTaskState>(task: T, owner: string, leaseMs?: number, now = new Date()): T {
+	if (task.claim && !isTaskClaimExpired(task.claim, now)) {
+		throw new Error(`Task '${task.id}' is already claimed by '${task.claim.owner}'.`);
+	}
+	return { ...task, claim: createTaskClaim(owner, leaseMs, now) };
+}
+
+export function releaseTaskClaim<T extends TeamTaskState>(task: T, owner: string, token: string, now = new Date()): T {
+	if (!canUseTaskClaim(task, owner, token, now)) {
+		throw new Error(`Task '${task.id}' claim is not held by '${owner}' or has expired.`);
+	}
+	return { ...task, claim: undefined };
+}
+
+export function transitionClaimedTaskStatus<T extends TeamTaskState>(task: T, owner: string, token: string, status: T["status"], now = new Date()): T {
+	if (!canUseTaskClaim(task, owner, token, now)) {
+		throw new Error(`Task '${task.id}' claim is not held by '${owner}' or has expired.`);
+	}
+	return { ...task, status };
+}

package/src/state/usage.ts

@@ -1,29 +1,29 @@
-import type { TeamTaskState, UsageState } from "./types.ts";
-
-export function aggregateUsage(tasks: TeamTaskState[]): UsageState | undefined {
-	const total: UsageState = {};
-	let found = false;
-	for (const task of tasks) {
-		if (!task.usage) continue;
-		found = true;
-		total.input = (total.input ?? 0) + (task.usage.input ?? 0);
-		total.output = (total.output ?? 0) + (task.usage.output ?? 0);
-		total.cacheRead = (total.cacheRead ?? 0) + (task.usage.cacheRead ?? 0);
-		total.cacheWrite = (total.cacheWrite ?? 0) + (task.usage.cacheWrite ?? 0);
-		total.cost = (total.cost ?? 0) + (task.usage.cost ?? 0);
-		total.turns = (total.turns ?? 0) + (task.usage.turns ?? 0);
-	}
-	return found ? total : undefined;
-}
-
-export function formatUsage(usage: UsageState | undefined): string {
-	if (!usage) return "(none)";
-	const parts: string[] = [];
-	if (usage.input !== undefined) parts.push(`input=${usage.input}`);
-	if (usage.output !== undefined) parts.push(`output=${usage.output}`);
-	if (usage.cacheRead !== undefined) parts.push(`cacheRead=${usage.cacheRead}`);
-	if (usage.cacheWrite !== undefined) parts.push(`cacheWrite=${usage.cacheWrite}`);
-	if (usage.cost !== undefined) parts.push(`cost=${usage.cost.toFixed(6)}`);
-	if (usage.turns !== undefined) parts.push(`turns=${usage.turns}`);
-	return parts.join(", ") || "(none)";
-}
+import type { TeamTaskState, UsageState } from "./types.ts";
+
+export function aggregateUsage(tasks: TeamTaskState[]): UsageState | undefined {
+	const total: UsageState = {};
+	let found = false;
+	for (const task of tasks) {
+		if (!task.usage) continue;
+		found = true;
+		total.input = (total.input ?? 0) + (task.usage.input ?? 0);
+		total.output = (total.output ?? 0) + (task.usage.output ?? 0);
+		total.cacheRead = (total.cacheRead ?? 0) + (task.usage.cacheRead ?? 0);
+		total.cacheWrite = (total.cacheWrite ?? 0) + (task.usage.cacheWrite ?? 0);
+		total.cost = (total.cost ?? 0) + (task.usage.cost ?? 0);
+		total.turns = (total.turns ?? 0) + (task.usage.turns ?? 0);
+	}
+	return found ? total : undefined;
+}
+
+export function formatUsage(usage: UsageState | undefined): string {
+	if (!usage) return "(none)";
+	const parts: string[] = [];
+	if (usage.input !== undefined) parts.push(`input=${usage.input}`);
+	if (usage.output !== undefined) parts.push(`output=${usage.output}`);
+	if (usage.cacheRead !== undefined) parts.push(`cacheRead=${usage.cacheRead}`);
+	if (usage.cacheWrite !== undefined) parts.push(`cacheWrite=${usage.cacheWrite}`);
+	if (usage.cost !== undefined && Number.isFinite(usage.cost)) parts.push(`cost=${usage.cost.toFixed(6)}`);
+	if (usage.turns !== undefined) parts.push(`turns=${usage.turns}`);
+	return parts.join(", ") || "(none)";
+}

package/src/subagents/async-entry.ts

@@ -1 +1 @@
-export * from "../runtime/async-runner.ts";
+export * from "../runtime/async-runner.ts";

package/src/subagents/index.ts

@@ -1,3 +1,3 @@
-export * from "./spawn.ts";
-export * from "./manager.ts";
-export * from "./async-entry.ts";
+export * from "./spawn.ts";
+export * from "./manager.ts";
+export * from "./async-entry.ts";

package/src/subagents/live/control.ts

@@ -1 +1 @@
-export * from "../../runtime/live-agent-control.ts";
+export * from "../../runtime/live-agent-control.ts";

package/src/subagents/live/manager.ts

@@ -1 +1 @@
-export * from "../../runtime/live-agent-manager.ts";
+export * from "../../runtime/live-agent-manager.ts";

package/src/subagents/live/realtime.ts

@@ -1 +1 @@
-export * from "../../runtime/live-control-realtime.ts";
+export * from "../../runtime/live-control-realtime.ts";

package/src/subagents/live/session-runtime.ts

@@ -1 +1 @@
-export * from "../../runtime/live-session-runtime.ts";
+export * from "../../runtime/live-session-runtime.ts";

package/src/subagents/manager.ts

@@ -1 +1 @@
-export * from "../runtime/subagent-manager.ts";
+export * from "../runtime/subagent-manager.ts";

package/src/subagents/spawn.ts

@@ -1 +1 @@
-export * from "../runtime/child-pi.ts";
+export * from "../runtime/child-pi.ts";

package/src/teams/discover-teams.ts

@@ -39,7 +39,7 @@ function parseRoleLine(line: string): TeamRole | undefined {
 		return "";
 	});
 	const description = descriptionSource.replace(/\s+/g, " ").trim() || undefined;
-	const maxConcurrency = metadata.maxConcurrency ? Number.parseInt(metadata.maxConcurrency, 10) : undefined;
+	const maxConcurrency = metadata.maxConcurrency ? (() => { const p = Number.parseInt(metadata.maxConcurrency, 10); return p > 0 ? p : undefined; })() : undefined;
 	return {
 		name,
 		agent: metadata.agent ?? name,
@@ -81,7 +81,7 @@ function parseTeamFile(filePath: string, source: ResourceSource): TeamConfig | u
 			filePath,
 			roles,
 			defaultWorkflow: frontmatter.defaultWorkflow || frontmatter.workflow || undefined,
-			workspaceMode: frontmatter.workspaceMode === "worktree" ? "worktree" : "single",
+			workspaceMode: frontmatter.workspaceMode?.trim() === "worktree" ? "worktree" : "single",
 			maxConcurrency: frontmatter.maxConcurrency ? Number.parseInt(frontmatter.maxConcurrency, 10) : undefined,
 			routing: triggers || useWhen || avoidWhen || cost || category ? { triggers, useWhen, avoidWhen, cost, category } : undefined,
 		};

package/src/teams/team-serializer.ts

@@ -1,38 +1,38 @@
-import type { TeamConfig, TeamRole } from "./team-config.ts";
-
-function line(key: string, value: string | string[] | undefined): string | undefined {
-	if (value === undefined) return undefined;
-	if (Array.isArray(value)) return `${key}: ${value.join(", ")}`;
-	return `${key}: ${value}`;
-}
-
-function serializeRole(role: TeamRole): string {
-	const parts = [`agent=${role.agent}`];
-	if (role.model) parts.push(`model=${role.model}`);
-	if (role.skills === false) parts.push("skills=false");
-	else if (role.skills?.length) parts.push(`skills=${role.skills.join(",")}`);
-	if (role.maxConcurrency !== undefined) parts.push(`maxConcurrency=${role.maxConcurrency}`);
-	if (role.description) parts.push(role.description);
-	return `- ${role.name}: ${parts.join(" ")}`;
-}
-
-export function serializeTeam(team: TeamConfig): string {
-	const lines = [
-		"---",
-		`name: ${team.name}`,
-		`description: ${team.description}`,
-		team.defaultWorkflow ? `defaultWorkflow: ${team.defaultWorkflow}` : undefined,
-		team.workspaceMode ? `workspaceMode: ${team.workspaceMode}` : undefined,
-		team.maxConcurrency !== undefined ? `maxConcurrency: ${team.maxConcurrency}` : undefined,
-		line("triggers", team.routing?.triggers),
-		line("useWhen", team.routing?.useWhen),
-		line("avoidWhen", team.routing?.avoidWhen),
-		line("cost", team.routing?.cost),
-		line("category", team.routing?.category),
-		"---",
-		"",
-		...team.roles.map(serializeRole),
-		"",
-	].filter((entry): entry is string => entry !== undefined);
-	return lines.join("\n");
-}
+import type { TeamConfig, TeamRole } from "./team-config.ts";
+
+function line(key: string, value: string | string[] | undefined): string | undefined {
+	if (value === undefined) return undefined;
+	if (Array.isArray(value)) return `${key}: ${value.join(", ")}`;
+	return `${key}: ${value}`;
+}
+
+function serializeRole(role: TeamRole): string {
+	const parts = [`agent=${role.agent}`];
+	if (role.model) parts.push(`model=${role.model}`);
+	if (role.skills === false) parts.push("skills=false");
+	else if (role.skills?.length) parts.push(`skills=${role.skills.join(",")}`);
+	if (role.maxConcurrency !== undefined) parts.push(`maxConcurrency=${role.maxConcurrency}`);
+	if (role.description) parts.push(role.description);
+	return `- ${role.name}: ${parts.join(" ")}`;
+}
+
+export function serializeTeam(team: TeamConfig): string {
+	const lines = [
+		"---",
+		`name: ${team.name}`,
+		`description: ${team.description}`,
+		team.defaultWorkflow ? `defaultWorkflow: ${team.defaultWorkflow}` : undefined,
+		team.workspaceMode ? `workspaceMode: ${team.workspaceMode}` : undefined,
+		team.maxConcurrency !== undefined ? `maxConcurrency: ${team.maxConcurrency}` : undefined,
+		line("triggers", team.routing?.triggers),
+		line("useWhen", team.routing?.useWhen),
+		line("avoidWhen", team.routing?.avoidWhen),
+		line("cost", team.routing?.cost),
+		line("category", team.routing?.category),
+		"---",
+		"",
+		...team.roles.map(serializeRole),
+		"",
+	].filter((entry): entry is string => entry !== undefined);
+	return lines.join("\n");
+}

package/src/types/diff.d.ts

@@ -1,18 +1,18 @@
-declare module "diff" {
-	export interface Change {
-		value: string;
-		count?: number;
-		added?: boolean;
-		removed?: boolean;
-	}
-
-	export interface DiffOptions {
-		ignoreCase?: boolean;
-		newlineIsToken?: boolean;
-		ignoreWhitespace?: boolean;
-		stripTrailingCr?: boolean;
-		oneChangePerToken?: boolean;
-	}
-
-	export function diffWords(oldStr: string, newStr: string, options?: DiffOptions): Change[];
-}
+declare module "diff" {
+	export interface Change {
+		value: string;
+		count?: number;
+		added?: boolean;
+		removed?: boolean;
+	}
+
+	export interface DiffOptions {
+		ignoreCase?: boolean;
+		newlineIsToken?: boolean;
+		ignoreWhitespace?: boolean;
+		stripTrailingCr?: boolean;
+		oneChangePerToken?: boolean;
+	}
+
+	export function diffWords(oldStr: string, newStr: string, options?: DiffOptions): Change[];
+}

package/src/ui/crew-footer.ts

@@ -1,101 +1,101 @@
-import type { UsageState } from "../state/types.ts";
-import { pad, truncate } from "../utils/visual.ts";
-import type { RunStatus } from "./status-colors.ts";
-import type { CrewTheme } from "./theme-adapter.ts";
-
-export interface CrewFooterData {
-	pwd: string;
-	branch?: string;
-	runId?: string;
-	status?: RunStatus;
-	usage?: UsageState;
-	contextWindow?: number;
-	contextPercent?: number;
-	badges?: string[];
-}
-
-function formatCount(value: number | undefined): string {
-	if (value === undefined || !Number.isFinite(value)) return "?";
-	if (Math.abs(value) >= 1000) return `${(value / 1000).toFixed(Math.abs(value) >= 10_000 ? 0 : 1)}k`;
-	return `${value}`;
-}
-
-function formatCost(value: number | undefined): string {
-	return value === undefined || !Number.isFinite(value) ? "$0.0000" : `$${value.toFixed(4)}`;
-}
-
-function displayPwd(pwd: string): string {
-	const home = process.env.HOME || process.env.USERPROFILE;
-	if (home && pwd.startsWith(home)) return `~${pwd.slice(home.length) || "/"}`;
-	return pwd || ".";
-}
-
-function contextText(data: CrewFooterData): string {
-	const windowText = data.contextWindow && Number.isFinite(data.contextWindow) ? formatCount(data.contextWindow) : "window";
-	const percent = data.contextPercent;
-	if (percent === undefined || !Number.isFinite(percent)) return `?/${windowText}`;
-	return `${percent.toFixed(1)}%/${windowText}`;
-}
-
-export class CrewFooter {
-	private data: CrewFooterData;
-	private readonly theme: CrewTheme;
-	private cacheKey = "";
-	private cacheWidth = 0;
-	private cacheLines: string[] = [];
-
-	constructor(data: CrewFooterData, theme: CrewTheme) {
-		this.data = data;
-		this.theme = theme;
-	}
-
-	setData(data: CrewFooterData): void {
-		this.data = data;
-		this.invalidate();
-	}
-
-	invalidate(): void {
-		this.cacheKey = "";
-		this.cacheLines = [];
-	}
-
-	render(width: number): string[] {
-		const key = JSON.stringify(this.data);
-		if (this.cacheKey === key && this.cacheWidth === width && this.cacheLines.length) return this.cacheLines;
-		const lineWidth = Math.max(1, width);
-		const firstParts = [
-			displayPwd(this.data.pwd),
-			this.data.branch ? `(${this.data.branch})` : undefined,
-			this.data.runId,
-			this.data.status,
-		].filter((part): part is string => Boolean(part));
-		const usage = this.data.usage;
-		const context = contextText(this.data);
-		const contextPercent = this.data.contextPercent;
-		const contextColor = contextPercent !== undefined && Number.isFinite(contextPercent)
-			? contextPercent > 90
-				? "error"
-				: contextPercent > 70
-					? "warning"
-					: undefined
-			: undefined;
-		const contextRendered = contextColor ? this.theme.fg(contextColor, context) : context;
-		const usageLine = [
-			`↑${formatCount(usage?.input)}`,
-			`↓${formatCount(usage?.output)}`,
-			`R ${formatCount(usage?.cacheRead)} cache`,
-			`W ${formatCount(usage?.cacheWrite)} cache`,
-			formatCost(usage?.cost),
-			contextRendered,
-		].join(" • ");
-		const badges = this.data.badges?.length ? this.data.badges.map((badge) => `[${badge}]`).join(" ") : "";
-		this.cacheLines = [
-			this.theme.fg("dim", pad(truncate(firstParts.join(" • "), lineWidth, "..."), lineWidth)),
-			this.theme.fg("dim", pad(truncate(usageLine, lineWidth, "..."), lineWidth)),
-			this.theme.fg("dim", pad(truncate(badges, lineWidth, "..."), lineWidth)),
-		];
-		this.cacheKey = key;
-		this.cacheWidth = width;
-		return this.cacheLines;
-	}
-}
+import type { UsageState } from "../state/types.ts";
+import { pad, truncate } from "../utils/visual.ts";
+import type { RunStatus } from "./status-colors.ts";
+import type { CrewTheme } from "./theme-adapter.ts";
+
+export interface CrewFooterData {
+	pwd: string;
+	branch?: string;
+	runId?: string;
+	status?: RunStatus;
+	usage?: UsageState;
+	contextWindow?: number;
+	contextPercent?: number;
+	badges?: string[];
+}
+
+function formatCount(value: number | undefined): string {
+	if (value === undefined || !Number.isFinite(value)) return "?";
+	if (Math.abs(value) >= 1000) return `${(value / 1000).toFixed(Math.abs(value) >= 10_000 ? 0 : 1)}k`;
+	return `${value}`;
+}
+
+function formatCost(value: number | undefined): string {
+	return value === undefined || !Number.isFinite(value) ? "$0.0000" : `$${value.toFixed(4)}`;
+}
+
+function displayPwd(pwd: string): string {
+	const home = process.env.HOME || process.env.USERPROFILE;
+	if (home && pwd.startsWith(home)) return `~${pwd.slice(home.length) || "/"}`;
+	return pwd || ".";
+}
+
+function contextText(data: CrewFooterData): string {
+	const windowText = data.contextWindow && Number.isFinite(data.contextWindow) ? formatCount(data.contextWindow) : "window";
+	const percent = data.contextPercent;
+	if (percent === undefined || !Number.isFinite(percent)) return `?/${windowText}`;
+	return `${percent.toFixed(1)}%/${windowText}`;
+}
+
+export class CrewFooter {
+	private data: CrewFooterData;
+	private readonly theme: CrewTheme;
+	private cacheKey = "";
+	private cacheWidth = 0;
+	private cacheLines: string[] = [];
+
+	constructor(data: CrewFooterData, theme: CrewTheme) {
+		this.data = data;
+		this.theme = theme;
+	}
+
+	setData(data: CrewFooterData): void {
+		this.data = data;
+		this.invalidate();
+	}
+
+	invalidate(): void {
+		this.cacheKey = "";
+		this.cacheLines = [];
+	}
+
+	render(width: number): string[] {
+		const key = JSON.stringify(this.data);
+		if (this.cacheKey === key && this.cacheWidth === width && this.cacheLines.length) return this.cacheLines;
+		const lineWidth = Math.max(1, width);
+		const firstParts = [
+			displayPwd(this.data.pwd),
+			this.data.branch ? `(${this.data.branch})` : undefined,
+			this.data.runId,
+			this.data.status,
+		].filter((part): part is string => Boolean(part));
+		const usage = this.data.usage;
+		const context = contextText(this.data);
+		const contextPercent = this.data.contextPercent;
+		const contextColor = contextPercent !== undefined && Number.isFinite(contextPercent)
+			? contextPercent > 90
+				? "error"
+				: contextPercent > 70
+					? "warning"
+					: undefined
+			: undefined;
+		const contextRendered = contextColor ? this.theme.fg(contextColor, context) : context;
+		const usageLine = [
+			`↑${formatCount(usage?.input)}`,
+			`↓${formatCount(usage?.output)}`,
+			`R ${formatCount(usage?.cacheRead)} cache`,
+			`W ${formatCount(usage?.cacheWrite)} cache`,
+			formatCost(usage?.cost),
+			contextRendered,
+		].join(" • ");
+		const badges = this.data.badges?.length ? this.data.badges.map((badge) => `[${badge}]`).join(" ") : "";
+		this.cacheLines = [
+			this.theme.fg("dim", pad(truncate(firstParts.join(" • "), lineWidth, "..."), lineWidth)),
+			this.theme.fg("dim", pad(truncate(usageLine, lineWidth, "..."), lineWidth)),
+			this.theme.fg("dim", pad(truncate(badges, lineWidth, "..."), lineWidth)),
+		];
+		this.cacheKey = key;
+		this.cacheWidth = width;
+		return this.cacheLines;
+	}
+}