pi-crew 0.9.16 → 0.9.18

package/src/extension/team-tool/plan.ts

@@ -1,10 +1,10 @@
+import { composeSingleAgentPrompt } from "../../runtime/single-agent-compose.ts";
+import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import { allTeams, discoverTeams } from "../../teams/discover-teams.ts";
 import { allWorkflows, discoverWorkflows } from "../../workflows/discover-workflows.ts";
 import { validateWorkflowForTeam } from "../../workflows/validate-workflow.ts";
-import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import type { PiTeamsToolResult } from "../tool-result.ts";
 import { result, type TeamContext } from "./context.ts";
-import { composeSingleAgentPrompt } from "../../runtime/single-agent-compose.ts";
 
 export function handlePlan(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
 	const teamName = params.team ?? "default";
@@ -14,13 +14,37 @@ export function handlePlan(params: TeamToolParamsValue, ctx: TeamContext): PiTea
 	const workflow = allWorkflows(discoverWorkflows(ctx.cwd)).find((item) => item.name === workflowName);
 	if (!workflow) return result(`Workflow '${workflowName}' not found.`, { action: "plan", status: "error" }, true);
 	const errors = validateWorkflowForTeam(workflow, team);
-	if (errors.length > 0) return result([`Workflow '${workflow.name}' is not valid for team '${team.name}':`, ...errors.map((error) => `- ${error}`)].join("\n"), { action: "plan", status: "error" }, true);
+	if (errors.length > 0)
+		return result(
+			[`Workflow '${workflow.name}' is not valid for team '${team.name}':`, ...errors.map((error) => `- ${error}`)].join("\n"),
+			{ action: "plan", status: "error" },
+			true,
+		);
 	const goal = params.goal ?? params.task ?? "(not provided)";
 	// ROADMAP T2.2: single-agent composition mode (cliff hedge).
 	if (params.singleAgent) {
 		const composed = composeSingleAgentPrompt(workflow, goal);
-		return result([`Single-agent plan for ${team.name} / ${workflow.name} (${composed.stepCount} phases composed into one sequential prompt):`, "", composed.prompt, "", "This prompt can be handed to a single agent to execute the entire workflow sequentially — pi-crew's cliff-resilient mode (survives single-agent domination)."].join("\n"), { action: "plan", status: "ok" });
+		return result(
+			[
+				`Single-agent plan for ${team.name} / ${workflow.name} (${composed.stepCount} phases composed into one sequential prompt):`,
+				"",
+				composed.prompt,
+				"",
+				"This prompt can be handed to a single agent to execute the entire workflow sequentially — pi-crew's cliff-resilient mode (survives single-agent domination).",
+			].join("\n"),
+			{ action: "plan", status: "ok" },
+		);
 	}
-	const lines = [`Team plan: ${team.name}`, `Workflow: ${workflow.name}`, `Goal: ${goal}`, "", "Steps:", ...workflow.steps.map((step, index) => `${index + 1}. ${step.id} [${step.role}]${step.dependsOn?.length ? ` after ${step.dependsOn.join(", ")}` : ""}`)];
+	const lines = [
+		`Team plan: ${team.name}`,
+		`Workflow: ${workflow.name}`,
+		`Goal: ${goal}`,
+		"",
+		"Steps:",
+		...workflow.steps.map(
+			(step, index) =>
+				`${index + 1}. ${step.id} [${step.role}]${step.dependsOn?.length ? ` after ${step.dependsOn.join(", ")}` : ""}`,
+		),
+	];
 	return result(lines.join("\n"), { action: "plan", status: "ok" });
 }

package/src/extension/team-tool/respond.ts

@@ -1,12 +1,12 @@
+import { readCrewAgents, recordFromTask, saveCrewAgents } from "../../runtime/crew-agent-records.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
-import { withRunLockSync } from "../../state/locks.ts";
-import { loadRunManifestById, saveRunTasks, updateRunStatus } from "../../state/state-store.ts";
 import { appendEvent } from "../../state/event-log.ts";
+import { withRunLockSync } from "../../state/locks.ts";
 import { appendMailboxMessage, updateMailboxMessageReply } from "../../state/mailbox.ts";
-import { readCrewAgents, saveCrewAgents, recordFromTask } from "../../runtime/crew-agent-records.ts";
+import { loadRunManifestById, saveRunTasks, updateRunStatus } from "../../state/state-store.ts";
 import { logInternalError } from "../../utils/internal-error.ts";
-import type { PiTeamsToolResult } from "../tool-result.ts";
 import { locateRunCwd } from "../team-tool.ts";
+import type { PiTeamsToolResult } from "../tool-result.ts";
 import { result, type TeamContext } from "./context.ts";
 import { RUN_NOT_FOUND_HINT } from "./run-not-found.ts";
 
@@ -17,7 +17,8 @@ import { RUN_NOT_FOUND_HINT } from "./run-not-found.ts";
  */
 export function handleRespond(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
 	if (!params.runId) return result("Respond requires runId.", { action: "respond", status: "error" }, true);
-	if (!params.message && !params.taskId) return result("Respond requires taskId and/or message.", { action: "respond", status: "error" }, true);
+	if (!params.message && !params.taskId)
+		return result("Respond requires taskId and/or message.", { action: "respond", status: "error" }, true);
 
 	const runCwd = locateRunCwd(params.runId, ctx.cwd);
 	if (!runCwd) return result(`Run '${params.runId}' not found.${RUN_NOT_FOUND_HINT}`, { action: "respond", status: "error" }, true);
@@ -28,7 +29,16 @@ export function handleRespond(params: TeamToolParamsValue, ctx: TeamContext): Pi
 		const fresh = loadRunManifestById(loaded.manifest.cwd, params.runId!); // NOTE: inside withRunLockSync - consistent read
 		if (!fresh) return result(`Run '${params.runId}' not found.${RUN_NOT_FOUND_HINT}`, { action: "respond", status: "error" }, true);
 		const foreignRun = typeof fresh.manifest.ownerSessionId === "string" && fresh.manifest.ownerSessionId !== ctx.sessionId;
-		if (foreignRun && !params.force) return result(`Run ${fresh.manifest.runId} belongs to another session. Use force: true to override.`, { action: "respond", status: "error", runId: fresh.manifest.runId }, true);
+		if (foreignRun && !params.force)
+			return result(
+				`Run ${fresh.manifest.runId} belongs to another session. Use force: true to override.`,
+				{
+					action: "respond",
+					status: "error",
+					runId: fresh.manifest.runId,
+				},
+				true,
+			);
 
 		const taskId = params.taskId;
 		const message = params.message ?? "";
@@ -39,14 +49,19 @@ export function handleRespond(params: TeamToolParamsValue, ctx: TeamContext): Pi
 
 		if (targetTasks.length === 0) {
 			const existing = taskId ? fresh.tasks.find((t) => t.id === taskId) : undefined;
-			const hint = " Use api operation=follow-up-agent for continuation prompts or api operation=steer-agent to interrupt active work.";
+			const hint =
+				" Use api operation=follow-up-agent for continuation prompts or api operation=steer-agent to interrupt active work.";
 			return result(
 				(taskId
 					? existing
 						? `Task '${taskId}' is ${existing.status}, not waiting.`
 						: `Task '${taskId}' not found.`
 					: `No waiting tasks in run ${fresh.manifest.runId}.`) + hint,
-				{ action: "respond", status: "error", runId: fresh.manifest.runId },
+				{
+					action: "respond",
+					status: "error",
+					runId: fresh.manifest.runId,
+				},
 				true,
 			);
 		}
@@ -95,23 +110,41 @@ export function handleRespond(params: TeamToolParamsValue, ctx: TeamContext): Pi
 
 		saveRunTasks(fresh.manifest, updatedTasks);
 		let manifest = fresh.manifest;
-		if (manifest.status === "blocked" || manifest.status === "completed" || manifest.status === "failed" || manifest.status === "cancelled") {
+		if (
+			manifest.status === "blocked" ||
+			manifest.status === "completed" ||
+			manifest.status === "failed" ||
+			manifest.status === "cancelled"
+		) {
 			manifest = updateRunStatus(manifest, "running", `Resumed ${resumed.size} waiting task(s).`);
 		}
 		for (const taskId of resumed) {
-			appendEvent(manifest.eventsPath, { type: "task.resumed", runId: manifest.runId, taskId, message: message || "Task re-queued after respond.", data: { mailboxIds } });
+			appendEvent(manifest.eventsPath, {
+				type: "task.resumed",
+				runId: manifest.runId,
+				taskId,
+				message: message || "Task re-queued after respond.",
+				data: { mailboxIds },
+			});
 		}
 		try {
 			const existingRuntimes = new Map(readCrewAgents(fresh.manifest).map((a) => [a.taskId, a.runtime]));
-			saveCrewAgents(fresh.manifest, updatedTasks.map((task) => recordFromTask(fresh.manifest, task, existingRuntimes.get(task.id) ?? "child-process")));
+			saveCrewAgents(
+				fresh.manifest,
+				updatedTasks.map((task) => recordFromTask(fresh.manifest, task, existingRuntimes.get(task.id) ?? "child-process")),
+			);
 		} catch (error) {
 			logInternalError("team-tool.handleRespond.crewAgents", error, `runId=${fresh.manifest.runId}`);
 		}
 
 		const resumedIds = targetTasks.map((t) => t.id);
-		return result(
-			`Resumed ${resumedIds.length} waiting task(s): ${resumedIds.join(", ")}. Message: ${message || "(no message)"}`,
-			{ action: "respond", status: "ok", runId: fresh.manifest.runId, resumedIds, mailboxIds, intent: `responding to ${resumedIds.join(", ")} in ${fresh.manifest.runId}` },
-		);
+		return result(`Resumed ${resumedIds.length} waiting task(s): ${resumedIds.join(", ")}. Message: ${message || "(no message)"}`, {
+			action: "respond",
+			status: "ok",
+			runId: fresh.manifest.runId,
+			resumedIds,
+			mailboxIds,
+			intent: `responding to ${resumedIds.join(", ")} in ${fresh.manifest.runId}`,
+		});
 	});
-}
+}

package/src/extension/team-tool/run-not-found.ts

@@ -12,12 +12,11 @@
  * consistent and the hint never drifts.
  */
 
-import { result, type TeamContext } from "./context.ts";
 import type { TeamToolDetails } from "../team-tool-types.ts";
+import { result, type TeamContext } from "./context.ts";
 
 /** Recovery hint appended to every "Run not found" message. */
-export const RUN_NOT_FOUND_HINT =
-	"\n\nTip: run action='list' to see recent runs and their IDs.";
+export const RUN_NOT_FOUND_HINT = "\n\nTip: run action='list' to see recent runs and their IDs.";
 
 /**
  * Build the standard "Run not found" error result with a recovery hint.
@@ -26,11 +25,7 @@ export const RUN_NOT_FOUND_HINT =
  * @param action the action that was attempted (for the details.action field)
  */
 export function runNotFound(runId: string, action: string): ReturnType<typeof result> {
-	return result(
-		`Run '${runId}' not found.${RUN_NOT_FOUND_HINT}`,
-		{ action, status: "error" } satisfies TeamToolDetails,
-		true,
-	);
+	return result(`Run '${runId}' not found.${RUN_NOT_FOUND_HINT}`, { action, status: "error" } satisfies TeamToolDetails, true);
 }
 
 /**