ultimate-pi 0.10.0 → 0.10.1

package/.pi/extensions/harness-live-widget.ts

@@ -329,6 +329,52 @@ export default function harnessLiveWidget(pi: ExtensionAPI) {
 	let component: HarnessWidgetComponent | null = null;
 	let refreshQueued = false;
 	let lastRenderHash = "";
+	let mountCtx: ExtensionContext | null = null;
+
+	function mountHarnessWidget(ctx: ExtensionContext): void {
+		if (!ctx.hasUI) return;
+		const state = stateStore.refresh(ctx);
+		const inFlight: InFlightState = { toolCount: 0, lastToolName: null };
+		lastRenderHash = computeRenderHash(state, inFlight);
+
+		ctx.ui.setWidget(
+			"harness-live",
+			(tui, theme) => {
+				widgetMounted = true;
+				tuiHandle = tui;
+				component = new HarnessWidgetComponent(
+					stateStore.snapshot(),
+					inFlight,
+					theme,
+				);
+				return {
+					render(width: number): string[] {
+						component?.setTheme(theme);
+						return component?.render(width) ?? [];
+					},
+					invalidate(): void {
+						component?.invalidate();
+					},
+				};
+			},
+			{ placement: "aboveEditor" },
+		);
+		updateStatusFallback(ctx, state);
+	}
+
+	function remountHarnessLiveWidget(ctx: ExtensionContext): void {
+		if (!ctx.hasUI || !widgetMounted) return;
+		ctx.ui.setWidget("harness-live", undefined);
+		mountHarnessWidget(ctx);
+	}
+
+	pi.events.on("subagents:agents-widget-mounted", () => {
+		if (mountCtx) remountHarnessLiveWidget(mountCtx);
+	});
+
+	pi.events.on("plan-approval:mounted", () => {
+		if (mountCtx) remountHarnessLiveWidget(mountCtx);
+	});
 
 	function updateStatusFallback(
 		ctx: ExtensionContext,
@@ -385,34 +431,8 @@ export default function harnessLiveWidget(pi: ExtensionAPI) {
 	}
 
 	pi.on("session_start", (_event, ctx) => {
-		if (!ctx.hasUI) return;
-		const state = stateStore.refresh(ctx);
-		const inFlight: InFlightState = { toolCount: 0, lastToolName: null };
-		lastRenderHash = computeRenderHash(state, inFlight);
-
-		ctx.ui.setWidget(
-			"harness-live",
-			(tui, theme) => {
-				widgetMounted = true;
-				tuiHandle = tui;
-				component = new HarnessWidgetComponent(
-					stateStore.snapshot(),
-					inFlight,
-					theme,
-				);
-				return {
-					render(width: number): string[] {
-						component?.setTheme(theme);
-						return component?.render(width) ?? [];
-					},
-					invalidate(): void {
-						component?.invalidate();
-					},
-				};
-			},
-			{ placement: "aboveEditor" },
-		);
-		updateStatusFallback(ctx, state);
+		mountCtx = ctx;
+		mountHarnessWidget(ctx);
 	});
 
 	pi.on("context", (_event, ctx) => {

package/.pi/extensions/harness-plan-approval.ts

@@ -7,6 +7,7 @@ import { Text } from "@earendil-works/pi-tui";
 import {
 	appendPlanApprovalIfNew,
 	getLatestRunContext,
+	hasPlanUserApproval,
 	parsePlanApprovalFromMessage,
 } from "../lib/harness-run-context.js";
 import { runPlanApprovalDialog } from "./lib/plan-approval/dialog.js";
@@ -32,26 +33,29 @@ import {
 } from "./lib/plan-approval/validate.js";
 
 export default function harnessPlanApproval(pi: ExtensionAPI) {
-	pi.registerMessageRenderer("harness-plan-draft", (message, _options, theme) => {
-		const data = message.details as
-			| {
-					plan_packet?: unknown;
-					human_summary?: string | null;
-			  }
-			| undefined;
-		if (!data?.plan_packet) return undefined;
-		const lines = renderHarnessPlanDraft(
-			{
-				plan_packet: data.plan_packet as Parameters<
-					typeof renderHarnessPlanDraft
-				>[0]["plan_packet"],
-				human_summary: data.human_summary,
-			},
-			80,
-			theme,
-		);
-		return new Text(lines.join("\n"), 0, 0);
-	});
+	pi.registerMessageRenderer(
+		"harness-plan-draft",
+		(message, _options, theme) => {
+			const data = message.details as
+				| {
+						plan_packet?: unknown;
+						human_summary?: string | null;
+				  }
+				| undefined;
+			if (!data?.plan_packet) return undefined;
+			const lines = renderHarnessPlanDraft(
+				{
+					plan_packet: data.plan_packet as Parameters<
+						typeof renderHarnessPlanDraft
+					>[0]["plan_packet"],
+					human_summary: data.human_summary,
+				},
+				80,
+				theme,
+			);
+			return new Text(lines.join("\n"), 0, 0);
+		},
+	);
 
 	pi.registerTool({
 		name: "approve_plan",
@@ -76,6 +80,33 @@ export default function harnessPlanApproval(pi: ExtensionAPI) {
 				};
 			}
 
+			const entries = ctx.sessionManager.getEntries();
+			if (
+				hasPlanUserApproval(entries, {
+					sincePlanCommand: true,
+					planId: validated.plan_packet.plan_id ?? null,
+				})
+			) {
+				const planId = String(validated.plan_packet.plan_id ?? "plan");
+				return {
+					content: [
+						{
+							type: "text",
+							text: `Plan ${planId} already approved in this harness run (planner subagent). Proceed with /harness-run.`,
+						},
+					],
+					details: {
+						plan_packet: validated.plan_packet,
+						options: validated.options,
+						response: {
+							kind: "selection",
+							selections: ["Approve"],
+						},
+						cancelled: false,
+					},
+				};
+			}
+
 			const planId = String(validated.plan_packet.plan_id ?? "plan");
 			const summary =
 				validated.human_summary?.trim() ||
@@ -94,7 +125,11 @@ export default function harnessPlanApproval(pi: ExtensionAPI) {
 
 			let outcome: PlanApprovalDialogResult;
 			if (ctx.hasUI) {
-				outcome = await runPlanApprovalDialog(ctx.ui, validated);
+				outcome = await runPlanApprovalDialog(ctx.ui, validated, {
+					onMounted: () => {
+						pi.events.emit("plan-approval:mounted", {});
+					},
+				});
 			} else {
 				outcome = await runPlanApprovalFallback(ctx.ui, validated);
 			}
@@ -109,7 +144,6 @@ export default function harnessPlanApproval(pi: ExtensionAPI) {
 				details,
 			});
 			if (approval) {
-				const entries = ctx.sessionManager.getEntries();
 				const runCtx = getLatestRunContext(entries);
 				appendPlanApprovalIfNew(
 					(type, data) => pi.appendEntry(type, data),

package/.pi/extensions/harness-run-context.ts

@@ -671,20 +671,29 @@ export default function harnessRunContext(pi: ExtensionAPI) {
 	});
 
 	pi.on("tool_call", async (event, ctx) => {
-		if (event.toolName === "ask_user" && activeCtx?.plan_packet_path) {
-			const input = event.input as {
-				question?: string;
-				options?: unknown[];
-			};
-			if (
-				isPlanApprovalAskUser(input) &&
-				hasPlanUserApproval(getEntries(ctx), { sincePlanCommand: true })
-			) {
-				return {
-					block: true,
-					reason:
-						"harness-run-context: plan already approved via planner subagent; do not call ask_user for plan approval in the parent session.",
-				};
+		if (activeCtx?.plan_packet_path) {
+			const entries = getEntries(ctx);
+			if (hasPlanUserApproval(entries, { sincePlanCommand: true })) {
+				if (event.toolName === "approve_plan") {
+					return {
+						block: true,
+						reason:
+							"harness-run-context: plan already approved via planner subagent; do not call approve_plan again in the parent session.",
+					};
+				}
+				if (event.toolName === "ask_user") {
+					const input = event.input as {
+						question?: string;
+						options?: unknown[];
+					};
+					if (isPlanApprovalAskUser(input)) {
+						return {
+							block: true,
+							reason:
+								"harness-run-context: plan already approved via planner subagent; do not call ask_user for plan approval in the parent session.",
+						};
+					}
+				}
 			}
 		}
 		if (!activeCtx?.plan_packet_path) return undefined;

package/.pi/extensions/lib/harness-subagents/parent-harness-ui-bridge.ts

@@ -198,7 +198,11 @@ export function createParentHarnessUiBridgeFactory(
 
 				let outcome: DialogResult;
 				if (parentCtx.hasUI) {
-					outcome = await runPlanApprovalDialog(parentCtx.ui, validated);
+					outcome = await runPlanApprovalDialog(parentCtx.ui, validated, {
+						onMounted: () => {
+							pi.events.emit("plan-approval:mounted", {});
+						},
+					});
 				} else {
 					outcome = await runPlanApprovalFallback(parentCtx.ui, validated);
 				}

package/.pi/extensions/lib/harness-subagents/vendored/index.ts

@@ -759,7 +759,9 @@ export function createHarnessSubagentsExtension(packageRoot: string) {
 		});
 
 		// Live widget: show running agents above editor
-		const widget = new AgentWidget(manager, agentActivity);
+		const widget = new AgentWidget(manager, agentActivity, () => {
+			pi.events.emit("subagents:agents-widget-mounted", {});
+		});
 
 		// ---- Join mode configuration ----
 		let defaultJoinMode: JoinMode = "smart";

package/.pi/extensions/lib/harness-subagents/vendored/ui/agent-widget.ts

@@ -264,6 +264,7 @@ export class AgentWidget {
 	constructor(
 		private manager: AgentManager,
 		private agentActivity: Map<string, AgentActivity>,
+		private onWidgetRegistered?: () => void,
 	) {}
 
 	/** Set the UI context (grabbed from first tool execution). */
@@ -615,6 +616,7 @@ export class AgentWidget {
 				{ placement: "aboveEditor" },
 			);
 			this.widgetRegistered = true;
+			this.onWidgetRegistered?.();
 		} else {
 			// Widget already registered — just request a re-render of existing components.
 			this.tui?.requestRender();

package/.pi/extensions/lib/plan-approval/dialog.ts

@@ -12,6 +12,51 @@ interface CustomAnswer {
 	response: { kind: "selection"; selections: string[] };
 }
 
+/** Lines reserved below overlay: harness-live widget + editor + footer. */
+export const PLAN_APPROVAL_BOTTOM_RESERVE_LINES = 11;
+/** Estimate agents widget height when stacking above harness live. */
+export const PLAN_APPROVAL_AGENTS_TOP_RESERVE_LINES = 12;
+export const PLAN_APPROVAL_MIN_VIEWPORT = 6;
+
+export function computePlanViewport(
+	availableHeight: number,
+	chromeLines: number,
+): number {
+	return Math.max(PLAN_APPROVAL_MIN_VIEWPORT, availableHeight - chromeLines);
+}
+
+export function computePlanOverlayMaxHeight(termHeight: number): number {
+	return Math.max(
+		PLAN_APPROVAL_MIN_VIEWPORT + 8,
+		termHeight -
+			PLAN_APPROVAL_BOTTOM_RESERVE_LINES -
+			PLAN_APPROVAL_AGENTS_TOP_RESERVE_LINES,
+	);
+}
+
+function countPlanChromeLines(
+	validated: ValidatedApprovePlanParams,
+	displayOptions: ValidatedApprovePlanParams["options"],
+	useOverlay: boolean,
+): number {
+	let chrome = useOverlay ? 2 : 0; // borders
+	chrome += 1; // title
+	if (validated.human_summary) {
+		chrome += validated.human_summary.split("\n").length;
+	}
+	chrome += 1; // blank before plan
+	chrome += 1; // plan label
+	chrome += 1; // blank before options
+	chrome += 1; // options label
+	for (const opt of displayOptions) {
+		chrome += 1;
+		if (opt.description) chrome += 1;
+	}
+	chrome += 1; // blank before hints
+	chrome += 1; // hints
+	return chrome;
+}
+
 function withTimeout<T>(
 	promise: Promise<T | null>,
 	ms: number | undefined,
@@ -25,177 +70,216 @@ function withTimeout<T>(
 	]);
 }
 
+export type RunPlanApprovalDialogOptions = {
+	onMounted?: () => void;
+};
+
 export async function runPlanApprovalDialog(
 	ui: ExtensionUIContext,
 	validated: ValidatedApprovePlanParams,
+	options?: RunPlanApprovalDialogOptions,
 ): Promise<PlanApprovalDialogResult> {
 	const planLines = formatPlanPacketLines(validated.plan_packet, 100);
 	const displayOptions = validated.options;
+	const useOverlay = validated.displayMode !== "inline";
+	let overlayTermHeight = 24;
 
 	const result = await withTimeout(
-		ui.custom<CustomAnswer | null>((tui, theme, _kb, done) => {
-			let scrollOffset = 0;
-			let optionIndex = 0;
-			let focus: FocusRegion = "plan";
-			let cachedLines: string[] | undefined;
-
-			function refresh() {
-				cachedLines = undefined;
-				tui.requestRender();
-			}
-
-			function submitSelection() {
-				const opt = displayOptions[optionIndex];
-				done({
-					response: { kind: "selection", selections: [opt.title] },
-				});
-			}
-
-			function handleInput(data: string) {
-				if (focus === "plan") {
-					if (matchesKey(data, Key.up) || data === "k") {
-						scrollOffset = Math.max(0, scrollOffset - 1);
-						refresh();
-						return;
-					}
-					if (matchesKey(data, Key.down) || data === "j") {
-						scrollOffset += 1;
-						refresh();
-						return;
-					}
-					if (matchesKey(data, Key.pageUp)) {
-						scrollOffset = Math.max(0, scrollOffset - 8);
-						refresh();
-						return;
-					}
-					if (matchesKey(data, Key.pageDown)) {
-						scrollOffset += 8;
-						refresh();
-						return;
-					}
-					if (matchesKey(data, Key.tab)) {
-						focus = "options";
-						refresh();
-						return;
-					}
+		ui.custom<CustomAnswer | null>(
+			(tui, theme, _kb, done) => {
+				const tuiHeight = (tui as unknown as { height?: number }).height;
+				overlayTermHeight =
+					typeof tuiHeight === "number" && tuiHeight > 10 ? tuiHeight : 24;
+				options?.onMounted?.();
+
+				let scrollOffset = 0;
+				let optionIndex = 0;
+				let focus: FocusRegion = "plan";
+				let cachedLines: string[] | undefined;
+
+				function refresh() {
+					cachedLines = undefined;
+					tui.requestRender();
 				}
 
-				if (focus === "options") {
-					if (matchesKey(data, Key.up)) {
-						optionIndex = Math.max(0, optionIndex - 1);
-						refresh();
-						return;
-					}
-					if (matchesKey(data, Key.down)) {
-						optionIndex = Math.min(displayOptions.length - 1, optionIndex + 1);
-						refresh();
-						return;
+				function submitSelection() {
+					const opt = displayOptions[optionIndex];
+					done({
+						response: { kind: "selection", selections: [opt.title] },
+					});
+				}
+
+				function handleInput(data: string) {
+					if (focus === "plan") {
+						if (matchesKey(data, Key.up) || data === "k") {
+							scrollOffset = Math.max(0, scrollOffset - 1);
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.down) || data === "j") {
+							scrollOffset += 1;
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.pageUp)) {
+							scrollOffset = Math.max(0, scrollOffset - 8);
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.pageDown)) {
+							scrollOffset += 8;
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.tab)) {
+							focus = "options";
+							refresh();
+							return;
+						}
 					}
-					if (matchesKey(data, Key.tab)) {
-						focus = "plan";
-						refresh();
-						return;
+
+					if (focus === "options") {
+						if (matchesKey(data, Key.up)) {
+							optionIndex = Math.max(0, optionIndex - 1);
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.down)) {
+							optionIndex = Math.min(
+								displayOptions.length - 1,
+								optionIndex + 1,
+							);
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.tab)) {
+							focus = "plan";
+							refresh();
+							return;
+						}
+						if (matchesKey(data, Key.enter)) {
+							submitSelection();
+							return;
+						}
 					}
-					if (matchesKey(data, Key.enter)) {
-						submitSelection();
-						return;
+
+					if (matchesKey(data, Key.escape)) {
+						done(null);
 					}
 				}
 
-				if (matchesKey(data, Key.escape)) {
-					done(null);
-				}
-			}
-
-			function render(width: number): string[] {
-				if (cachedLines) return cachedLines;
-
-				const lines: string[] = [];
-				const add = (s: string) => lines.push(truncateToWidth(s, width));
-				const useOverlay = validated.displayMode !== "inline";
-				const dims = (tui as { height?: number }).height;
-				const termHeight = typeof dims === "number" && dims > 10 ? dims : 24;
-				const footerLines = displayOptions.length * 2 + 8;
-				const planViewport = Math.max(
-					6,
-					Math.floor(termHeight * 0.55) - footerLines,
-				);
-
-				if (useOverlay) {
-					add(theme.fg("accent", "─".repeat(width)));
-				}
+				function render(width: number): string[] {
+					if (cachedLines) return cachedLines;
 
-				add(theme.fg("accent", " Plan approval"));
-				if (validated.human_summary) {
-					for (const line of validated.human_summary.split("\n")) {
-						add(theme.fg("muted", ` ${line}`));
-					}
-				}
-				lines.push("");
-
-				const maxScroll = Math.max(0, planLines.length - planViewport);
-				scrollOffset = Math.min(scrollOffset, maxScroll);
-				const visible = planLines.slice(
-					scrollOffset,
-					scrollOffset + planViewport,
-				);
-				const planLabel =
-					focus === "plan"
-						? theme.fg("accent", " [plan — ↑↓/Pg scroll, Tab → options]")
-						: theme.fg("dim", " [plan]");
-				add(planLabel);
-				for (const line of visible) {
-					add(theme.fg("text", ` ${line}`));
-				}
-				if (planLines.length > planViewport) {
-					add(
-						theme.fg(
-							"dim",
-							` … ${scrollOffset + 1}-${scrollOffset + visible.length} of ${planLines.length}`,
-						),
+					const lines: string[] = [];
+					const add = (s: string) => lines.push(truncateToWidth(s, width));
+					const dims = (tui as { height?: number }).height;
+					const termHeight = typeof dims === "number" && dims > 10 ? dims : 24;
+					const chromeLines = countPlanChromeLines(
+						validated,
+						displayOptions,
+						useOverlay,
 					);
-				}
-				lines.push("");
-
-				const optLabel =
-					focus === "options"
-						? theme.fg("accent", " Options (↑↓, Enter, Tab → plan):")
-						: theme.fg("dim", " Options (Tab to focus):");
-				add(optLabel);
-				for (let i = 0; i < displayOptions.length; i++) {
-					const opt = displayOptions[i];
-					const focused = focus === "options" && i === optionIndex;
-					const prefix = focused ? theme.fg("accent", "> ") : "  ";
-					const num = `${i + 1}. `;
-					if (focused) {
-						add(prefix + theme.fg("accent", `${num}${opt.title}`));
+
+					let availableHeight: number;
+					if (useOverlay) {
+						const overlayMax = computePlanOverlayMaxHeight(termHeight);
+						availableHeight = overlayMax;
 					} else {
-						add(`${prefix}${theme.fg("text", `${num}${opt.title}`)}`);
+						availableHeight = termHeight - PLAN_APPROVAL_BOTTOM_RESERVE_LINES;
 					}
-					if (opt.description) {
-						add(`     ${theme.fg("muted", opt.description)}`);
+					const planViewport = computePlanViewport(
+						availableHeight,
+						chromeLines,
+					);
+
+					if (useOverlay) {
+						add(theme.fg("accent", "─".repeat(width)));
 					}
-				}
 
-				lines.push("");
-				add(theme.fg("dim", " Tab: plan ↔ options • Esc: cancel"));
+					add(theme.fg("accent", " Plan approval"));
+					if (validated.human_summary) {
+						for (const line of validated.human_summary.split("\n")) {
+							add(theme.fg("muted", ` ${line}`));
+						}
+					}
+					lines.push("");
 
-				if (useOverlay) {
-					add(theme.fg("accent", "─".repeat(width)));
-				}
+					const maxScroll = Math.max(0, planLines.length - planViewport);
+					scrollOffset = Math.min(scrollOffset, maxScroll);
+					const visible = planLines.slice(
+						scrollOffset,
+						scrollOffset + planViewport,
+					);
+					const planLabel =
+						focus === "plan"
+							? theme.fg("accent", " [plan — ↑↓/Pg scroll, Tab → options]")
+							: theme.fg("dim", " [plan]");
+					add(planLabel);
+					for (const line of visible) {
+						add(theme.fg("text", ` ${line}`));
+					}
+					if (planLines.length > planViewport) {
+						add(
+							theme.fg(
+								"dim",
+								` … ${scrollOffset + 1}-${scrollOffset + visible.length} of ${planLines.length}`,
+							),
+						);
+					}
+					lines.push("");
 
-				cachedLines = lines;
-				return lines;
-			}
+					const optLabel =
+						focus === "options"
+							? theme.fg("accent", " Options (↑↓, Enter, Tab → plan):")
+							: theme.fg("dim", " Options (Tab to focus):");
+					add(optLabel);
+					for (let i = 0; i < displayOptions.length; i++) {
+						const opt = displayOptions[i];
+						const focused = focus === "options" && i === optionIndex;
+						const prefix = focused ? theme.fg("accent", "> ") : "  ";
+						const num = `${i + 1}. `;
+						if (focused) {
+							add(prefix + theme.fg("accent", `${num}${opt.title}`));
+						} else {
+							add(`${prefix}${theme.fg("text", `${num}${opt.title}`)}`);
+						}
+						if (opt.description) {
+							add(`     ${theme.fg("muted", opt.description)}`);
+						}
+					}
 
-			return {
-				render,
-				invalidate: () => {
-					cachedLines = undefined;
-				},
-				handleInput,
-			};
-		}),
+					lines.push("");
+					add(theme.fg("dim", " Tab: plan ↔ options • Esc: cancel"));
+
+					if (useOverlay) {
+						add(theme.fg("accent", "─".repeat(width)));
+					}
+
+					cachedLines = lines;
+					return lines;
+				}
+
+				return {
+					render,
+					invalidate: () => {
+						cachedLines = undefined;
+					},
+					handleInput,
+				};
+			},
+			useOverlay
+				? {
+						overlay: true,
+						overlayOptions: () => ({
+							anchor: "bottom-center",
+							width: "100%",
+							margin: { bottom: PLAN_APPROVAL_BOTTOM_RESERVE_LINES },
+							maxHeight: computePlanOverlayMaxHeight(overlayTermHeight),
+						}),
+					}
+				: undefined,
+		),
 		undefined,
 	);
 

package/.pi/prompts/harness-plan.md

@@ -40,11 +40,24 @@ Otherwise use `HarnessSpawnContext` from `[HarnessRunContext]` for greenfield `m
 Agent({ subagent_type: "harness/planner", prompt: "<task + HarnessSpawnContext JSON + output schema>" })
 ```
 
-3. `get_subagent_result` — parse final JSON (`status`, `plan_packet`, `human_summary`, `clarification`) via fenced `json` block.
-4. If `status === "ready"` and parent `harness-run-context` shows `plan_ready: true` (planner called `create_plan`), confirm `plan_packet_path` exists — do **not** write the file yourself.
+3. `get_subagent_result` — parse final JSON (`status`, `plan_packet`, `human_summary`, `clarification`) via fenced `json` block. Treat `plan_packet` in that JSON as **read-only summary context** — not input for another approval tool call.
+4. If `status === "ready"` and `[HarnessRunContext]` shows `plan_ready: true` (planner called `create_plan`), confirm `plan_packet_path` exists — do **not** write the file yourself.
 5. If `needs_clarification`, tell the user the planner is waiting — do **not** re-spawn; user should answer in the subagent or re-run `/harness-plan`.
 6. Do **not** call `ask_user`, `approve_plan`, or `create_plan` in this parent session.
 
+## After subagent returns (no second approval)
+
+User approval happens **once**, inside the planner subagent: `approve_plan` uses the parent TUI bridge. You are the orchestrator, **not** an approver.
+
+After `get_subagent_result`:
+
+- If `[HarnessRunContext]` shows `plan_ready: true`, or the transcript already has `harness-plan-approval` / bridged `approve_plan` with **Approve** → planning is complete. **Stop.** Summarize the plan and set `next_command: /harness-run`.
+- Do **not** call `approve_plan` to “confirm” using `plan_packet` from subagent JSON.
+- Do **not** call `ask_user` with Approve / Request changes / Cancel for the same plan.
+- Do **not** re-spawn the planner to “get approval again”.
+
+If `status === "ready"` but `plan_ready` is false → planner approved but `create_plan` may have failed; tell the user to run `/harness-plan-commit` — **not** a second `approve_plan`.
+
 ## Parent rules
 
 - Do not mutate project source files in the plan phase.

package/CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [v0.10.1] — 2026-05-17
+
+### 🐛 Fixes
+
+- **Harness plan TUI:** agents widget stacks above harness-live; full-height bottom `approve_plan` overlay above the harness band; block duplicate parent `approve_plan` / plan `ask_user` after planner subagent approval.
+
 ## [v0.10.0] — 2026-05-17
 
 ### ✨ Features

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ultimate-pi",
-	"version": "0.10.0",
+	"version": "0.10.1",
 	"description": "Ultimate AI coding harness for pi.dev — extensible skills, Obsidian wiki knowledge layer, compressed context, deterministic output",
 	"keywords": [
 		"pi-package",
@@ -82,7 +82,7 @@
 		"format": "biome format --write",
 		"format:check": "biome format",
 		"prepare": "lefthook install",
-		"test": "node --test test/harness-verify.test.mjs test/harness-ask-user.test.mjs test/harness-subagents-loader.test.mjs test/harness-subagents-import-path.test.mjs test/sentrux-rules-sync.test.mjs test/harness-budget-guard.test.mjs && npx -y tsx --test test/harness-vcc-settings.test.ts test/harness-plan-phase-policy.test.mjs test/harness-subagent-policy.test.mjs test/harness-turn-routing.test.mjs test/plan-approval-format.test.mjs test/plan-approval-sync.test.mjs test/plan-create-plan.test.mjs",
+		"test": "node --test test/harness-verify.test.mjs test/harness-ask-user.test.mjs test/harness-subagents-loader.test.mjs test/harness-subagents-import-path.test.mjs test/sentrux-rules-sync.test.mjs test/harness-budget-guard.test.mjs && npx -y tsx --test test/harness-vcc-settings.test.ts test/harness-plan-phase-policy.test.mjs test/harness-subagent-policy.test.mjs test/harness-turn-routing.test.mjs test/plan-approval-format.test.mjs test/plan-approval-dialog.test.mjs test/plan-approval-sync.test.mjs test/plan-create-plan.test.mjs",
 		"test:vcc": "npx -y tsx --test vendor/pi-vcc/tests/*.test.ts",
 		"harness:sentrux-bootstrap": "node .pi/scripts/harness-sentrux-bootstrap.mjs",
 		"harness:sentrux-sync": "node .pi/scripts/sentrux-rules-sync.mjs --force",