pi-crew 0.2.3 → 0.2.4

package/src/extension/cross-extension-rpc.ts

@@ -1,104 +1,104 @@
-import type { ExtensionContext } from "@mariozechner/pi-coding-agent";
-import type { TeamToolParamsValue } from "../schema/team-tool-schema.ts";
-// Lazy-loaded to avoid pulling team-tool.ts (and its entire runtime chain) into module load.
-import type { handleTeamTool as HandleTeamToolFn } from "./team-tool.ts";
-let _cachedHandleTeamTool: typeof HandleTeamToolFn | undefined;
-async function handleTeamTool(params: Parameters<typeof HandleTeamToolFn>[0], ctx: Parameters<typeof HandleTeamToolFn>[1]): Promise<Awaited<ReturnType<typeof HandleTeamToolFn>>> {
-	if (!_cachedHandleTeamTool) {
-		// LAZY: avoid pulling team-tool.ts (and its entire runtime chain) into module load.
-		const mod = await import("./team-tool.ts");
-		_cachedHandleTeamTool = mod.handleTeamTool;
-	}
-	return _cachedHandleTeamTool(params, ctx);
-}
-import { parseLiveControlRealtimeMessage, publishLiveControlRealtime } from "../runtime/live-control-realtime.ts";
-
-export interface EventBusLike {
-	on(event: string, handler: (data: unknown) => void): (() => void) | void;
-	emit(event: string, data: unknown): void;
-}
-
-export type RpcReply<T = unknown> = { success: true; data?: T } | { success: false; error: string };
-export const PI_CREW_RPC_VERSION = 1;
-
-export interface PiCrewRpcHandle {
-	unsubscribe(): void;
-}
-
-function requestId(raw: unknown): string | undefined {
-	return raw && typeof raw === "object" && !Array.isArray(raw) && typeof (raw as { requestId?: unknown }).requestId === "string" ? (raw as { requestId: string }).requestId : undefined;
-}
-
-function reply(events: EventBusLike, channel: string, id: string | undefined, payload: RpcReply): void {
-	if (!id) return;
-	events.emit(`${channel}:reply:${id}`, payload);
-}
-
-function textOf(result: Awaited<ReturnType<typeof handleTeamTool>>): string {
-	return result.content?.map((item) => item.type === "text" ? item.text : "").join("\n") ?? "";
-}
-
-function on(events: EventBusLike, channel: string, handler: (raw: unknown) => void): () => void {
-	const unsub = events.on(channel, handler);
-	return typeof unsub === "function" ? unsub : () => {};
-}
-
-export function registerPiCrewRpc(events: EventBusLike | undefined, getCtx: () => ExtensionContext | undefined): PiCrewRpcHandle | undefined {
-	if (!events) return undefined;
-	const unsubs = [
-		on(events, "pi-crew:rpc:ping", (raw) => reply(events, "pi-crew:rpc:ping", requestId(raw), { success: true, data: { version: PI_CREW_RPC_VERSION } })),
-		on(events, "pi-crew:rpc:run", async (raw) => {
-			const id = requestId(raw);
-			try {
-				const ctx = getCtx();
-				if (!ctx) throw new Error("No active pi-crew session context.");
-				// Validate payload: only allow known fields from TeamToolParamsValue
-				const ALLOWED_RPC_RUN_KEYS = new Set(["goal", "team", "workflow", "async", "cwd", "config", "skill", "model"]);
-				let params: TeamToolParamsValue;
-				if (raw && typeof raw === "object" && !Array.isArray(raw)) {
-					const filtered: Record<string, unknown> = { ...(raw as object) };
-					// Strip any keys not in the allowlist to prevent injection of unexpected fields
-					for (const key of Object.keys(filtered)) {
-						if (!ALLOWED_RPC_RUN_KEYS.has(key)) delete filtered[key];
-					}
-					params = { ...filtered, action: "run" } as TeamToolParamsValue;
-				} else {
-					params = { action: "run" };
-				}
-				const result = await handleTeamTool(params, ctx);
-				reply(events, "pi-crew:rpc:run", id, result.isError ? { success: false, error: textOf(result) } : { success: true, data: result.details });
-			} catch (error) {
-				reply(events, "pi-crew:rpc:run", id, { success: false, error: error instanceof Error ? error.message : String(error) });
-			}
-		}),
-		on(events, "pi-crew:rpc:status", async (raw) => {
-			const id = requestId(raw);
-			try {
-				const ctx = getCtx();
-				if (!ctx) throw new Error("No active pi-crew session context.");
-				const runId = raw && typeof raw === "object" && !Array.isArray(raw) ? (raw as { runId?: string }).runId : undefined;
-				const result = await handleTeamTool({ action: "status", runId }, ctx);
-				reply(events, "pi-crew:rpc:status", id, result.isError ? { success: false, error: textOf(result) } : { success: true, data: { text: textOf(result), details: result.details } });
-			} catch (error) {
-				reply(events, "pi-crew:rpc:status", id, { success: false, error: error instanceof Error ? error.message : String(error) });
-			}
-		}),
-		on(events, "pi-crew:live-control", (raw) => {
-			const request = parseLiveControlRealtimeMessage(raw);
-			if (request) publishLiveControlRealtime(request);
-		}),
-		on(events, "pi-crew:rpc:live-control", async (raw) => {
-			const id = requestId(raw);
-			try {
-				const ctx = getCtx();
-				if (!ctx) throw new Error("No active pi-crew session context.");
-				const obj = raw && typeof raw === "object" && !Array.isArray(raw) ? raw as Record<string, unknown> : {};
-				const result = await handleTeamTool({ action: "api", runId: typeof obj.runId === "string" ? obj.runId : undefined, config: { operation: typeof obj.operation === "string" ? obj.operation : "steer-agent", agentId: obj.agentId, message: obj.message, prompt: obj.prompt } }, ctx);
-				reply(events, "pi-crew:rpc:live-control", id, result.isError ? { success: false, error: textOf(result) } : { success: true, data: { text: textOf(result), details: result.details } });
-			} catch (error) {
-				reply(events, "pi-crew:rpc:live-control", id, { success: false, error: error instanceof Error ? error.message : String(error) });
-			}
-		}),
-	];
-	return { unsubscribe: () => unsubs.forEach((unsub) => unsub()) };
-}
+import type { ExtensionContext } from "@mariozechner/pi-coding-agent";
+import type { TeamToolParamsValue } from "../schema/team-tool-schema.ts";
+// Lazy-loaded to avoid pulling team-tool.ts (and its entire runtime chain) into module load.
+import type { handleTeamTool as HandleTeamToolFn } from "./team-tool.ts";
+let _cachedHandleTeamTool: typeof HandleTeamToolFn | undefined;
+async function handleTeamTool(params: Parameters<typeof HandleTeamToolFn>[0], ctx: Parameters<typeof HandleTeamToolFn>[1]): Promise<Awaited<ReturnType<typeof HandleTeamToolFn>>> {
+	if (!_cachedHandleTeamTool) {
+		// LAZY: avoid pulling team-tool.ts (and its entire runtime chain) into module load.
+		const mod = await import("./team-tool.ts");
+		_cachedHandleTeamTool = mod.handleTeamTool;
+	}
+	return _cachedHandleTeamTool(params, ctx);
+}
+import { parseLiveControlRealtimeMessage, publishLiveControlRealtime } from "../runtime/live-control-realtime.ts";
+
+export interface EventBusLike {
+	on(event: string, handler: (data: unknown) => void): (() => void) | void;
+	emit(event: string, data: unknown): void;
+}
+
+export type RpcReply<T = unknown> = { success: true; data?: T } | { success: false; error: string };
+export const PI_CREW_RPC_VERSION = 1;
+
+export interface PiCrewRpcHandle {
+	unsubscribe(): void;
+}
+
+function requestId(raw: unknown): string | undefined {
+	return raw && typeof raw === "object" && !Array.isArray(raw) && typeof (raw as { requestId?: unknown }).requestId === "string" ? (raw as { requestId: string }).requestId : undefined;
+}
+
+function reply(events: EventBusLike, channel: string, id: string | undefined, payload: RpcReply): void {
+	if (!id) return;
+	events.emit(`${channel}:reply:${id}`, payload);
+}
+
+function textOf(result: Awaited<ReturnType<typeof handleTeamTool>>): string {
+	return result.content?.map((item) => item.type === "text" ? item.text : "").join("\n") ?? "";
+}
+
+function on(events: EventBusLike, channel: string, handler: (raw: unknown) => void): () => void {
+	const unsub = events.on(channel, handler);
+	return typeof unsub === "function" ? unsub : () => {};
+}
+
+export function registerPiCrewRpc(events: EventBusLike | undefined, getCtx: () => ExtensionContext | undefined): PiCrewRpcHandle | undefined {
+	if (!events) return undefined;
+	const unsubs = [
+		on(events, "pi-crew:rpc:ping", (raw) => reply(events, "pi-crew:rpc:ping", requestId(raw), { success: true, data: { version: PI_CREW_RPC_VERSION } })),
+		on(events, "pi-crew:rpc:run", async (raw) => {
+			const id = requestId(raw);
+			try {
+				const ctx = getCtx();
+				if (!ctx) throw new Error("No active pi-crew session context.");
+				// Validate payload: only allow known fields from TeamToolParamsValue
+				const ALLOWED_RPC_RUN_KEYS = new Set(["goal", "team", "workflow", "async", "cwd", "config", "skill", "model"]);
+				let params: TeamToolParamsValue;
+				if (raw && typeof raw === "object" && !Array.isArray(raw)) {
+					const filtered: Record<string, unknown> = { ...(raw as object) };
+					// Strip any keys not in the allowlist to prevent injection of unexpected fields
+					for (const key of Object.keys(filtered)) {
+						if (!ALLOWED_RPC_RUN_KEYS.has(key)) delete filtered[key];
+					}
+					params = { ...filtered, action: "run" } as TeamToolParamsValue;
+				} else {
+					params = { action: "run" };
+				}
+				const result = await handleTeamTool(params, ctx);
+				reply(events, "pi-crew:rpc:run", id, result.isError ? { success: false, error: textOf(result) } : { success: true, data: result.details });
+			} catch (error) {
+				reply(events, "pi-crew:rpc:run", id, { success: false, error: error instanceof Error ? error.message : String(error) });
+			}
+		}),
+		on(events, "pi-crew:rpc:status", async (raw) => {
+			const id = requestId(raw);
+			try {
+				const ctx = getCtx();
+				if (!ctx) throw new Error("No active pi-crew session context.");
+				const runId = raw && typeof raw === "object" && !Array.isArray(raw) ? (raw as { runId?: string }).runId : undefined;
+				const result = await handleTeamTool({ action: "status", runId }, ctx);
+				reply(events, "pi-crew:rpc:status", id, result.isError ? { success: false, error: textOf(result) } : { success: true, data: { text: textOf(result), details: result.details } });
+			} catch (error) {
+				reply(events, "pi-crew:rpc:status", id, { success: false, error: error instanceof Error ? error.message : String(error) });
+			}
+		}),
+		on(events, "pi-crew:live-control", (raw) => {
+			const request = parseLiveControlRealtimeMessage(raw);
+			if (request) publishLiveControlRealtime(request);
+		}),
+		on(events, "pi-crew:rpc:live-control", async (raw) => {
+			const id = requestId(raw);
+			try {
+				const ctx = getCtx();
+				if (!ctx) throw new Error("No active pi-crew session context.");
+				const obj = raw && typeof raw === "object" && !Array.isArray(raw) ? raw as Record<string, unknown> : {};
+				const result = await handleTeamTool({ action: "api", runId: typeof obj.runId === "string" ? obj.runId : undefined, config: { operation: typeof obj.operation === "string" ? obj.operation : "steer-agent", agentId: obj.agentId, message: obj.message, prompt: obj.prompt } }, ctx);
+				reply(events, "pi-crew:rpc:live-control", id, result.isError ? { success: false, error: textOf(result) } : { success: true, data: { text: textOf(result), details: result.details } });
+			} catch (error) {
+				reply(events, "pi-crew:rpc:live-control", id, { success: false, error: error instanceof Error ? error.message : String(error) });
+			}
+		}),
+	];
+	return { unsubscribe: () => unsubs.forEach((unsub) => unsub()) };
+}

package/src/extension/help.ts

@@ -1,47 +1,47 @@
-export function piTeamsHelp(): string {
-	return [
-		"pi-crew commands:",
-		"",
-		"Core:",
-		"- Agent can use the `team` tool autonomously; slash commands are manual controls.",
-		"- Tool action `recommend` suggests the best team/workflow for a goal.",
-		"- /teams — list teams, workflows, agents, recent runs",
-		"- /team-run [--team=name] [--workflow=name] [--async] [--worktree] <goal>",
-		"- /team-status <runId>",
-		"- /team-summary <runId>",
-		"- /team-resume <runId>",
-		"- /team-cancel <runId>",
-	"- /team-retry <runId> [taskId]",
-		"",
-		"Inspection:",
-		"- /team-events <runId>",
-		"- /team-artifacts <runId>",
-		"- /team-worktrees <runId>",
-		"- /team-api <runId> <operation> [taskId=<taskId>] [body=<message>]",
-		"- /team-dashboard",
-		"- /team-mascot",
-		"- /team-transcript <runId> [taskId]",
-		"- /team-result <runId> [taskId]",
-		"- /team-manager",
-		"",
-		"Maintenance:",
-		"- /team-cleanup <runId> [--force]",
-		"- /team-forget <runId> --confirm [--force]",
-		"- /team-prune --keep=20 --confirm",
-		"",
-		"Portability:",
-		"- /team-export <runId>",
-		"- /team-import <path-to-run-export.json> [--user]",
-		"- /team-imports",
-		"",
-		"Diagnostics:",
-		"- /team-doctor",
-		"- /team-init [--copy-builtins] [--overwrite]",
-		"- /team-config [key=value] [--unset=key.path] [--project]",
-		"- /team-autonomy [status|on|off|manual|suggested|assisted|aggressive] [--prefer-async] [--no-worktree-suggest]",
-		"- /team-validate",
-		"- /team-help",
-		"",
-		"Real child workers are enabled by default. Use runtime.mode=scaffold or executeWorkers=false only for dry runs.",
-	].join("\n");
-}
+export function piTeamsHelp(): string {
+	return [
+		"pi-crew commands:",
+		"",
+		"Core:",
+		"- Agent can use the `team` tool autonomously; slash commands are manual controls.",
+		"- Tool action `recommend` suggests the best team/workflow for a goal.",
+		"- /teams — list teams, workflows, agents, recent runs",
+		"- /team-run [--team=name] [--workflow=name] [--async] [--worktree] <goal>",
+		"- /team-status <runId>",
+		"- /team-summary <runId>",
+		"- /team-resume <runId>",
+		"- /team-cancel <runId>",
+	"- /team-retry <runId> [taskId]",
+		"",
+		"Inspection:",
+		"- /team-events <runId>",
+		"- /team-artifacts <runId>",
+		"- /team-worktrees <runId>",
+		"- /team-api <runId> <operation> [taskId=<taskId>] [body=<message>]",
+		"- /team-dashboard",
+		"- /team-mascot",
+		"- /team-transcript <runId> [taskId]",
+		"- /team-result <runId> [taskId]",
+		"- /team-manager",
+		"",
+		"Maintenance:",
+		"- /team-cleanup <runId> [--force]",
+		"- /team-forget <runId> --confirm [--force]",
+		"- /team-prune --keep=20 --confirm",
+		"",
+		"Portability:",
+		"- /team-export <runId>",
+		"- /team-import <path-to-run-export.json> [--user]",
+		"- /team-imports",
+		"",
+		"Diagnostics:",
+		"- /team-doctor",
+		"- /team-init [--copy-builtins] [--overwrite]",
+		"- /team-config [key=value] [--unset=key.path] [--project]",
+		"- /team-autonomy [status|on|off|manual|suggested|assisted|aggressive] [--prefer-async] [--no-worktree-suggest]",
+		"- /team-validate",
+		"- /team-help",
+		"",
+		"Real child workers are enabled by default. Use runtime.mode=scaffold or executeWorkers=false only for dry runs.",
+	].join("\n");
+}

package/src/extension/import-index.ts

@@ -1,69 +1,69 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { projectCrewRoot, userCrewRoot } from "../utils/paths.ts";
-import { DEFAULT_PATHS } from "../config/defaults.ts";
-import { isSafePathId, resolveRealContainedPath } from "../utils/safe-paths.ts";
-
-export interface ImportedRunIndexEntry {
-	runId: string;
-	scope: "project" | "user";
-	bundlePath: string;
-	summaryPath: string;
-	importedAt?: string;
-	status?: string;
-	team?: string;
-	workflow?: string;
-	goal?: string;
-}
-
-function readEntry(root: string, scope: "project" | "user", runId: string): ImportedRunIndexEntry | undefined {
-	if (!isSafePathId(runId)) return undefined;
-	let bundlePath: string;
-	let summaryPath: string;
-	try {
-		const entryRoot = resolveRealContainedPath(root, runId);
-		bundlePath = resolveRealContainedPath(root, path.join(runId, "run-export.json"));
-		summaryPath = path.join(entryRoot, "README.md");
-	} catch {
-		return undefined;
-	}
-	if (!fs.existsSync(bundlePath)) return undefined;
-	try {
-		const raw = JSON.parse(fs.readFileSync(bundlePath, "utf-8")) as Record<string, unknown>;
-		const manifest = raw.manifest && typeof raw.manifest === "object" && !Array.isArray(raw.manifest) ? raw.manifest as Record<string, unknown> : {};
-		return {
-			runId,
-			scope,
-			bundlePath,
-			summaryPath,
-			importedAt: typeof raw.importedAt === "string" ? raw.importedAt : undefined,
-			status: typeof manifest.status === "string" ? manifest.status : undefined,
-			team: typeof manifest.team === "string" ? manifest.team : undefined,
-			workflow: typeof manifest.workflow === "string" ? manifest.workflow : undefined,
-			goal: typeof manifest.goal === "string" ? manifest.goal : undefined,
-		};
-	} catch {
-		return { runId, scope, bundlePath, summaryPath };
-	}
-}
-
-function collect(root: string, scope: "project" | "user"): ImportedRunIndexEntry[] {
-	if (!fs.existsSync(root)) return [];
-	try {
-		if (fs.lstatSync(root).isSymbolicLink()) return [];
-		resolveRealContainedPath(path.dirname(root), path.basename(root));
-	} catch {
-		return [];
-	}
-	return fs.readdirSync(root)
-		.filter((entry) => isSafePathId(entry))
-		.map((entry) => readEntry(root, scope, entry))
-		.filter((entry): entry is ImportedRunIndexEntry => entry !== undefined);
-}
-
-export function listImportedRuns(cwd: string): ImportedRunIndexEntry[] {
-	const projectRoot = path.join(projectCrewRoot(cwd), DEFAULT_PATHS.state.importsSubdir);
-	const userRoot = path.join(userCrewRoot(), DEFAULT_PATHS.state.importsSubdir);
-	return [...collect(userRoot, "user"), ...collect(projectRoot, "project")]
-		.sort((a, b) => (b.importedAt ?? "").localeCompare(a.importedAt ?? ""));
-}
+import * as fs from "node:fs";
+import * as path from "node:path";
+import { projectCrewRoot, userCrewRoot } from "../utils/paths.ts";
+import { DEFAULT_PATHS } from "../config/defaults.ts";
+import { isSafePathId, resolveRealContainedPath } from "../utils/safe-paths.ts";
+
+export interface ImportedRunIndexEntry {
+	runId: string;
+	scope: "project" | "user";
+	bundlePath: string;
+	summaryPath: string;
+	importedAt?: string;
+	status?: string;
+	team?: string;
+	workflow?: string;
+	goal?: string;
+}
+
+function readEntry(root: string, scope: "project" | "user", runId: string): ImportedRunIndexEntry | undefined {
+	if (!isSafePathId(runId)) return undefined;
+	let bundlePath: string;
+	let summaryPath: string;
+	try {
+		const entryRoot = resolveRealContainedPath(root, runId);
+		bundlePath = resolveRealContainedPath(root, path.join(runId, "run-export.json"));
+		summaryPath = path.join(entryRoot, "README.md");
+	} catch {
+		return undefined;
+	}
+	if (!fs.existsSync(bundlePath)) return undefined;
+	try {
+		const raw = JSON.parse(fs.readFileSync(bundlePath, "utf-8")) as Record<string, unknown>;
+		const manifest = raw.manifest && typeof raw.manifest === "object" && !Array.isArray(raw.manifest) ? raw.manifest as Record<string, unknown> : {};
+		return {
+			runId,
+			scope,
+			bundlePath,
+			summaryPath,
+			importedAt: typeof raw.importedAt === "string" ? raw.importedAt : undefined,
+			status: typeof manifest.status === "string" ? manifest.status : undefined,
+			team: typeof manifest.team === "string" ? manifest.team : undefined,
+			workflow: typeof manifest.workflow === "string" ? manifest.workflow : undefined,
+			goal: typeof manifest.goal === "string" ? manifest.goal : undefined,
+		};
+	} catch {
+		return { runId, scope, bundlePath, summaryPath };
+	}
+}
+
+function collect(root: string, scope: "project" | "user"): ImportedRunIndexEntry[] {
+	if (!fs.existsSync(root)) return [];
+	try {
+		if (fs.lstatSync(root).isSymbolicLink()) return [];
+		resolveRealContainedPath(path.dirname(root), path.basename(root));
+	} catch {
+		return [];
+	}
+	return fs.readdirSync(root)
+		.filter((entry) => isSafePathId(entry))
+		.map((entry) => readEntry(root, scope, entry))
+		.filter((entry): entry is ImportedRunIndexEntry => entry !== undefined);
+}
+
+export function listImportedRuns(cwd: string): ImportedRunIndexEntry[] {
+	const projectRoot = path.join(projectCrewRoot(cwd), DEFAULT_PATHS.state.importsSubdir);
+	const userRoot = path.join(userCrewRoot(), DEFAULT_PATHS.state.importsSubdir);
+	return [...collect(userRoot, "user"), ...collect(projectRoot, "project")]
+		.sort((a, b) => (b.importedAt ?? "").localeCompare(a.importedAt ?? ""));
+}