Release v0.1.26

Author: janvitos

README.md

@@ -19,7 +19,7 @@ A global [Pi coding agent](https://github.com/badlogic/pi-mono) extension that a
   - **Switch to Build and implement here**
   - **Start fresh and implement**
   - **Stay in Plan mode**
-- Staying in Plan mode produces a durable acknowledgement and stops the run until the user responds.
+- Staying in Plan mode—or pressing Escape in the approval dialog—produces a durable acknowledgement and stops the run until the user responds.
 - Mode state survives reloads, resumes, and forks.
 - When Pi recreates the custom editor, the latest 100 user prompts from the active session branch are restored for Up/Down history navigation.
 
@@ -76,11 +76,11 @@ When planning is complete, `plan_exit` displays the entire persisted plan and as
 
 Selecting **Start fresh and implement** stops the current run and pre-fills `/build-fresh`. Press Enter to confirm. Pi only exposes session creation to user-invoked command contexts, so this confirmation is required. The command creates a linked child session, copies the approved plan to its canonical plan file, switches it to Build, and starts implementation without transferring the planning conversation.
 
-Selecting **Stay in Plan mode** displays:
+Selecting **Stay in Plan mode**, or pressing Escape while the approval dialog is open, displays:
 
 > Staying in Plan mode. Let me know when you’re ready to revise or implement the plan.
 
-The agent then stops and waits for the next user message.
+Both actions leave Plan mode active, stop the agent, and wait for the next user message.
 
 ## Plan-mode permissions
 

index.ts

@@ -29,6 +29,7 @@ import {
 	makePlanPath,
 	nextMode,
 	nextThinkingLevel,
+	normalizePlanExitChoice,
 	PLAN_EXIT_APPROVE_CHOICE,
 	PLAN_EXIT_FRESH_CHOICE,
 	PLAN_EXIT_STAY_ACKNOWLEDGEMENT,
@@ -289,15 +290,15 @@ export default function planBuildModes(pi: ExtensionAPI): void {
 			if (!plan.trim()) throw new Error("Cannot request plan approval because the plan file is empty");
 			pi.appendEntry(PLAN_REVIEW_ENTRY_TYPE, { plan, planPath });
 			const displayPath = shorten(planPath, ctx.cwd);
-			const choice = await ctx.ui.select(
+			const selection = normalizePlanExitChoice(await ctx.ui.select(
 				`Build Agent: Plan at ${displayPath} is complete. What would you like to do?`,
 				[PLAN_EXIT_APPROVE_CHOICE, PLAN_EXIT_FRESH_CHOICE, PLAN_EXIT_STAY_CHOICE],
-			);
-			const decision = classifyPlanExitChoice(choice);
+			));
+			const decision = classifyPlanExitChoice(selection.choice);
 			if (decision === "stay") {
 				freshImplementationPlan = undefined;
 				pi.appendEntry(MODE_NOTICE_ENTRY_TYPE, { message: PLAN_EXIT_STAY_ACKNOWLEDGEMENT });
-				return buildPlanExitStayResult(planPath, choice === undefined);
+				return buildPlanExitStayResult(planPath, selection.cancelled);
 			}
 			if (decision === "implement-fresh") {
 				freshImplementationPlan = plan;

package.json

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

utils.test.ts

@@ -22,6 +22,7 @@ import {
 	makePlanPath,
 	nextMode,
 	nextThinkingLevel,
+	normalizePlanExitChoice,
 	PLAN_EXIT_APPROVE_CHOICE,
 	PLAN_EXIT_FRESH_CHOICE,
 	PLAN_EXIT_STAY_ACKNOWLEDGEMENT,
@@ -202,11 +203,21 @@ test("declining plan exit stays in Plan mode and terminates the run", () => {
 	assert.equal(cancelled.details.cancelled, true);
 });
 
+test("plan exit normalizes Escape to the explicit Stay choice", () => {
+	assert.deepEqual(normalizePlanExitChoice(undefined), {
+		choice: PLAN_EXIT_STAY_CHOICE,
+		cancelled: true,
+	});
+	assert.deepEqual(normalizePlanExitChoice(PLAN_EXIT_STAY_CHOICE), {
+		choice: PLAN_EXIT_STAY_CHOICE,
+		cancelled: false,
+	});
+});
+
 test("plan exit classifies all three choices and fails safe", () => {
 	assert.equal(classifyPlanExitChoice(PLAN_EXIT_APPROVE_CHOICE), "implement-here");
 	assert.equal(classifyPlanExitChoice(PLAN_EXIT_FRESH_CHOICE), "implement-fresh");
 	assert.equal(classifyPlanExitChoice(PLAN_EXIT_STAY_CHOICE), "stay");
-	assert.equal(classifyPlanExitChoice(undefined), "stay");
 	assert.equal(classifyPlanExitChoice("unexpected value"), "stay");
 });
 

utils.ts

@@ -139,7 +139,19 @@ export const PLAN_EXIT_STAY_ACKNOWLEDGEMENT =
 
 export type PlanExitDecision = "implement-here" | "implement-fresh" | "stay";
 
-export function classifyPlanExitChoice(choice: string | undefined): PlanExitDecision {
+export interface NormalizedPlanExitChoice {
+	choice: string;
+	cancelled: boolean;
+}
+
+export function normalizePlanExitChoice(choice: string | undefined): NormalizedPlanExitChoice {
+	return {
+		choice: choice ?? PLAN_EXIT_STAY_CHOICE,
+		cancelled: choice === undefined,
+	};
+}
+
+export function classifyPlanExitChoice(choice: string): PlanExitDecision {
 	if (choice === PLAN_EXIT_APPROVE_CHOICE) return "implement-here";
 	if (choice === PLAN_EXIT_FRESH_CHOICE) return "implement-fresh";
 	return "stay";