@janvitos/pi-plan-build 0.1.38 → 0.1.39

package/README.md

@@ -99,7 +99,7 @@ The passive 64-column right panel reserves terminal columns, so the transcript a
 - “Pause the plan,” “hide the plan,” or “show the plan.”
 - “Cancel this plan” at any point to end step-by-step execution immediately.
 
-The extension exposes these actions to the agent through `plan_step_control`; project mutations remain blocked until the user clearly approves a ready step or explicitly indicates that it is already complete. A ready step may be marked complete without implementation when the user says they already handled or verified it. An implemented step is marked completed immediately after verification, and the next step becomes ready without a review or acceptance phase. The agent interprets intent contextually rather than requiring exact phrases, while the extension validates every resulting state transition. Step tool results identify the affected step, for example `Step 2: complete`, rather than presenting a plan-wide completion label. When the final step is completed or skipped, the panel and execution guards are removed immediately and the main window receives a completion confirmation with a concise plan summary. Cancelling removes the panel and execution guards immediately, restores the full-width layout, and preserves the saved plan file for reference. The agent implements only that step, calls `plan_step_complete`, and waits for the user's next prompt. Progress, revisions, summaries, and panel visibility survive reload/resume. If such a session is opened in regular mode, progress is retained but cannot advance until fullscreen mode is restored; no overlay fallback is used.
+The extension exposes these actions to the agent through `plan_step_control`; project mutations remain blocked until the user clearly approves a ready step or explicitly indicates that it is already complete. A ready step may be marked complete without implementation when the user says they already handled or verified it. An implemented step is marked completed immediately after verification, and the next step becomes ready without a review or acceptance phase. The agent interprets intent contextually rather than requiring exact phrases, while the extension validates every resulting state transition. Step tool results identify the affected step, for example `Step 2: complete`, rather than presenting a plan-wide completion label. When the final step is completed or skipped, the panel and execution guards are removed immediately and the main window receives a Markdown-formatted completion summary with plan-style headings, spacing, and colors. Cancelling removes the panel and execution guards immediately, restores the full-width layout, and preserves the saved plan file for reference. The agent implements only that step, calls `plan_step_complete`, and waits for the user's next prompt. Progress, revisions, summaries, and panel visibility survive reload/resume. If such a session is opened in regular mode, progress is retained but cannot advance until fullscreen mode is restored; no overlay fallback is used.
 
 Enable fullscreen in `~/.pi/agent/settings.json` and restart Pi:
 

package/index.ts

@@ -417,9 +417,9 @@ export default function planBuildModes(pi: ExtensionAPI): void {
 			const target = params.step === undefined
 				? execution.steps.find((step) => step.status === "ready")
 				: execution.steps[Math.floor(params.step) - 1];
-			const finish = (message: string) => ({
+			const finish = (message: string, extraDetails?: { planCompleted?: boolean }) => ({
 				content: [{ type: "text" as const, text: message }],
-				details: { action: params.action, stepId: target?.id },
+				details: { action: params.action, stepId: target?.id, ...extraDetails },
 				terminate: true,
 			});
 
@@ -448,11 +448,17 @@ export default function planBuildModes(pi: ExtensionAPI): void {
 			}
 			if (params.action === "complete") {
 				const completion = applyExecutionTransition(completePlanStep(execution, target.id));
-				return finish(completion ?? "The step was marked complete. The next step is ready and awaits user instruction.");
+				return finish(
+					completion ?? "The step was marked complete. The next step is ready and awaits user instruction.",
+					{ planCompleted: completion !== undefined },
+				);
 			}
 			if (params.action === "skip") {
 				const completion = applyExecutionTransition(skipPlanStep(execution, target.id));
-				return finish(completion ?? "The step was skipped. The next step awaits user instruction.");
+				return finish(
+					completion ?? "The step was skipped. The next step awaits user instruction.",
+					{ planCompleted: completion !== undefined },
+				);
 			}
 			if (!params.instruction?.trim()) throw new Error("Revising a step requires a replacement instruction");
 			const plan = await fs.promises.readFile(planPath, "utf8");
@@ -471,6 +477,8 @@ export default function planBuildModes(pi: ExtensionAPI): void {
 		},
 		renderResult(result, _options, theme, context) {
 			const text = result.content.find((item) => item.type === "text")?.text ?? "Plan state updated";
+			const details = result.details as { planCompleted?: boolean } | undefined;
+			if (details?.planCompleted && !context.isError) return new Markdown(text, 0, 0, getMarkdownTheme());
 			return new Text(theme.fg(context.isError ? "error" : "success", text), 0, 0);
 		},
 	});
@@ -489,7 +497,7 @@ export default function planBuildModes(pi: ExtensionAPI): void {
 			const completion = applyExecutionTransition(completePlanStep(execution, step.id, params.summary));
 			return {
 				content: [{ type: "text", text: completion ?? "The step was completed. The next step is ready and awaits user instruction." }],
-				details: { stepId: step.id, completed: true },
+				details: { stepId: step.id, completed: true, planCompleted: completion !== undefined },
 				terminate: true,
 			};
 		},
@@ -498,6 +506,8 @@ export default function planBuildModes(pi: ExtensionAPI): void {
 		},
 		renderResult(result, _options, theme, context) {
 			const text = result.content.find((item) => item.type === "text")?.text ?? "Plan step completed";
+			const details = result.details as { planCompleted?: boolean } | undefined;
+			if (details?.planCompleted && !context.isError) return new Markdown(text, 0, 0, getMarkdownTheme());
 			return new Text(theme.fg(context.isError ? "error" : "success", text), 0, 0);
 		},
 	});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@janvitos/pi-plan-build",
-  "version": "0.1.38",
+  "version": "0.1.39",
   "description": "Plan safely, approve explicitly, then implement here or in a clean session.",
   "type": "module",
   "license": "MIT",

package/plan-execution.ts

@@ -114,11 +114,12 @@ function makeNextReady(state: PlanExecutionState, afterId: string): void {
 }
 
 export function formatPlanCompletionSummary(state: PlanExecutionState): string {
-	const lines = ["Plan complete.", "", "Summary:"];
+	const lines = ["# Plan complete", "", "## Summary", ""];
 	for (const [index, step] of state.steps.entries()) {
 		const outcome = step.status === "skipped" ? "Skipped" : "Completed";
-		lines.push(`${index + 1}. ${outcome}: ${step.text}`);
-		if (step.summary?.trim()) lines.push(`   ${step.summary.trim()}`);
+		lines.push(`${index + 1}. **${outcome}:** ${step.text}`);
+		if (step.summary?.trim()) lines.push("", `   ${step.summary.trim()}`);
+		if (index < state.steps.length - 1) lines.push("");
 	}
 	return lines.join("\n");
 }