pi-plan-task 1.0.2 → 1.1.0

package/README.md

@@ -29,15 +29,15 @@ Or add the source to `packages` in `~/.pi/agent/settings.json`.
 | Command | What it does |
 | --- | --- |
 | `/plan [file or goal]` | Read-only planning. Writes `.plan_task/plan.md` and `.plan_task/task.md`. |
-| `/build` | Ask where to run, execute the next unfinished task, then ask whether to continue here or in a new session. |
-| `/build here` | Same as `/build`, but skip the first session prompt and stay in this session. |
-| `/build new` | Same as `/build`, but skip the first session prompt and start a new session. |
-| `/goal` | Execute remaining tasks until `task.md` is complete. No session prompts. |
+| `/build` | Execute the next unfinished task in this session, then ask whether to continue here or in a new session. |
+| `/build new` | Same as `/build`, but start that task in a new session. |
+| `/goal` | Execute remaining tasks until `task.md` is complete. No session prompts. All in this session. |
+| `/goal new` | Same as `/goal`, but each remaining task starts in a new session. |
 | `/tasks` | Show the current task list and progress. |
 
-Restarting Pi does not auto-start work. Run `/build` again.
+Restarting Pi does not auto-start work. Run `/build` or `/goal` again.
 
-`/goal` never asks about sessions. After a `/build` task finishes, the continue-here / new-session prompt still appears. Those prompts use this package's `select` dialog, not `ask_user_question`.
+Session prompts only happen after a `/build` task finishes. `/goal` and `/goal new` never ask. Those prompts use this package's `select` dialog, not `ask_user_question`.
 
 ### `/plan` arguments
 
@@ -51,11 +51,27 @@ Restarting Pi does not auto-start work. Run `/build` again.
 
 If the first argument is an existing file — or looks like a path such as `./spec.md`, `notes.txt`, or `@spec.md` — that file is the spec. Anything after it is extra planning guidance. Missing explicit paths are rejected instead of being treated as a prompt.
 
+### `/build` and `/goal`
+
+```
+/build
+/build new
+/goal
+/goal new
+```
+
+`new` and `--new` are equivalent.
+
+- `/build` runs one task here. After it is marked complete, Pi asks whether to continue in this session or a new session.
+- `/build new` opens a new session for that one task, then asks the same question when it finishes.
+- `/goal` runs every remaining task in this session.
+- `/goal new` opens a new session for the next unfinished task, then another new session after each completed task, until the list is done.
+
 ## Planning method
 
 `/plan` uses `extensions/planning-and-task-breakdown.md` as the planning prompt.
 
-That file keeps the same section headings as `planning-and-task-breakdown.md`, so later methodology edits can be copied section-by-section from that source. Plan-task only changes output paths and the checklist form `/build` needs:
+That file keeps the same section headings as `planning-and-task-breakdown.md`, so later methodology edits can be copied section-by-section from that source. Plan-task only changes output paths and the checklist form `/build` and `/goal` need:
 
 - `.plan_task/plan.md` and `.plan_task/task.md`
 - checklist lines in the form `- [ ] N. Title`
@@ -66,7 +82,7 @@ This package does not install or load a skill. It also does not modify Pi's syst
 
 Phase instructions are conversation messages, not system-prompt patches:
 
-- `/plan` and `/build` send a short visible user message to start the turn.
+- `/plan`, `/build`, and `/goal` send a short visible user message to start the turn.
 - The planning method or current-task instructions are injected once, as a hidden message, when that phase or task starts.
 - Later turns in the same phase inject nothing unless the checklist actually changed; then a small build-status message is appended.
 - `context` keeps only the newest framing for the current phase. Stale plan/build messages are dropped from the model context. While idle, every injected message is filtered out.
@@ -75,7 +91,7 @@ Session history still stores the injected messages. Filtering is non-destructive
 
 ## Ask user
 
-`/plan` and `/build` keep juicesharp's `ask_user_question` available and tell the model to call it for consequential choices the repo cannot answer:
+`/plan`, `/build`, and `/goal` keep juicesharp's `ask_user_question` available and tell the model to call it for consequential choices the repo cannot answer:
 
 - 1-4 questions per call, each with a short `header` and 2-4 described options
 - recommended option first, with `(Recommended)` on the label
@@ -99,7 +115,7 @@ Created in the current project:
 - [ ] 2. Add login UI
 ```
 
-`/build` resumes from the first unchecked item.
+`/build` and `/goal` resume from the first unchecked item.
 
 ## Config
 

package/extensions/build-session.test.ts

@@ -1,18 +1,35 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
-import { parseBuildPlacement } from "./build-session.ts";
+import { parseBuildPlacement, parseGoalMode } from "./build-session.ts";
 
 describe("parseBuildPlacement", () => {
-	it("asks by default", () => {
-		assert.equal(parseBuildPlacement(""), "ask");
-		assert.equal(parseBuildPlacement("  "), "ask");
-		assert.equal(parseBuildPlacement("unknown"), "ask");
-	});
-
-	it("accepts here and new aliases", () => {
+	it("stays in this session by default", () => {
+		assert.equal(parseBuildPlacement(""), "here");
+		assert.equal(parseBuildPlacement("  "), "here");
+		assert.equal(parseBuildPlacement("unknown"), "here");
 		assert.equal(parseBuildPlacement("here"), "here");
 		assert.equal(parseBuildPlacement(" --here extra"), "here");
+	});
+
+	it("accepts new aliases", () => {
+		assert.equal(parseBuildPlacement("new"), "new");
 		assert.equal(parseBuildPlacement("NEW"), "new");
 		assert.equal(parseBuildPlacement("--new"), "new");
 	});
 });
+
+describe("parseGoalMode", () => {
+	it("runs remaining tasks in this session by default", () => {
+		assert.equal(parseGoalMode(""), "all-here");
+		assert.equal(parseGoalMode("  "), "all-here");
+		assert.equal(parseGoalMode("unknown"), "all-here");
+	});
+
+	it("accepts new and continue aliases", () => {
+		assert.equal(parseGoalMode("new"), "all-new");
+		assert.equal(parseGoalMode(" --new extra"), "all-new");
+		assert.equal(parseGoalMode("NEW"), "all-new");
+		assert.equal(parseGoalMode("continue"), "chain-here");
+		assert.equal(parseGoalMode("--continue"), "chain-here");
+	});
+});

package/extensions/build-session.ts

@@ -1,8 +1,15 @@
-export type BuildPlacement = "ask" | "here" | "new";
+export type BuildPlacement = "here" | "new";
+export type GoalMode = "all-here" | "all-new" | "chain-here";
 
 export function parseBuildPlacement(args: string): BuildPlacement {
 	const token = args.trim().split(/\s+/)[0]?.toLowerCase() ?? "";
-	if (token === "here" || token === "--here") return "here";
 	if (token === "new" || token === "--new") return "new";
-	return "ask";
+	return "here";
+}
+
+export function parseGoalMode(args: string): GoalMode {
+	const token = args.trim().split(/\s+/)[0]?.toLowerCase() ?? "";
+	if (token === "new" || token === "--new") return "all-new";
+	if (token === "continue" || token === "--continue") return "chain-here";
+	return "all-here";
 }

package/extensions/index.ts

@@ -8,7 +8,7 @@ import {
 import { matchesKey, Text, truncateToWidth } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 import { isSafePlanCommand } from "./bash-guard.ts";
-import { parseBuildPlacement, type BuildPlacement } from "./build-session.ts";
+import { parseBuildPlacement, parseGoalMode, type BuildPlacement } from "./build-session.ts";
 import { ensureDefaultGlobalConfig, loadConfig } from "./config.ts";
 import {
 	ensurePlanDir,
@@ -38,21 +38,14 @@ type Mode = "idle" | "plan" | "build";
 const CONTINUE_THIS = "Continue in this session";
 const CONTINUE_NEW = "Continue in a new session";
 const BUILD_HERE_COMMAND = "/build here";
+const GOAL_CONTINUE_COMMAND = "/goal continue";
 
-async function chooseBuildPlacement(ctx: ExtensionCommandContext): Promise<"here" | "new" | undefined> {
-	if (!ctx.hasUI) return "here";
-	const choice = await ctx.ui.select("Start this task where?", [CONTINUE_THIS, CONTINUE_NEW]);
-	if (choice === CONTINUE_THIS) return "here";
-	if (choice === CONTINUE_NEW) return "new";
-	return undefined;
-}
-
-async function startBuildInNewSession(ctx: ExtensionCommandContext): Promise<void> {
+async function startBuildInNewSession(ctx: ExtensionCommandContext, command = BUILD_HERE_COMMAND): Promise<void> {
 	const parentSession = ctx.sessionManager.getSessionFile();
 	const result = await ctx.newSession({
 		parentSession,
 		withSession: async (nextCtx) => {
-			await nextCtx.sendUserMessage(BUILD_HERE_COMMAND, { expandPromptTemplates: true });
+			await nextCtx.sendUserMessage(command, { expandPromptTemplates: true });
 		},
 	});
 	if (result.cancelled) {
@@ -138,6 +131,7 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 
 	let mode: Mode = "idle";
 	let continueAll = false;
+	let continueNew = false;
 	let currentTaskId: number | undefined;
 	let toolsBeforePlan: string[] | undefined;
 	let awaitingChoice = false;
@@ -193,6 +187,7 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 	async function enterPlanMode(ctx: ExtensionContext): Promise<void> {
 		mode = "plan";
 		continueAll = false;
+		continueNew = false;
 		currentTaskId = undefined;
 		planReadyNotified = false;
 		resetFraming();
@@ -212,6 +207,7 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 	function leaveModes(ctx: ExtensionContext): void {
 		mode = "idle";
 		continueAll = false;
+		continueNew = false;
 		currentTaskId = undefined;
 		resetFraming();
 		restoreTools();
@@ -263,6 +259,15 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 			await startNextTask(ctx);
 			return;
 		}
+		if (continueNew) {
+			mode = "idle";
+			currentTaskId = undefined;
+			resetFraming();
+			updateStatus(ctx);
+			ctx.ui.notify("Task complete. Opening a new session for the next task.", "info");
+			pi.sendUserMessage("/goal-next-session", { expandPromptTemplates: true });
+			return;
+		}
 		if (!ctx.hasUI) {
 			ctx.ui.notify("Task complete. Run /build for the next task.", "info");
 			mode = "idle";
@@ -369,8 +374,11 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 	async function beginBuild(
 		ctx: ExtensionCommandContext,
 		runAll: boolean,
-		placement: BuildPlacement = "ask",
+		placement: BuildPlacement = "here",
+		options: { chainNew?: boolean; forkCommand?: string } = {},
 	): Promise<void> {
+		const chainNew = options.chainNew === true;
+		const forkCommand = options.forkCommand ?? BUILD_HERE_COMMAND;
 		const file = await loadTaskFile(ctx.cwd);
 		if (!file) {
 			ctx.ui.notify("No plan found. Run /plan first.", "error");
@@ -384,30 +392,43 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 			ctx.ui.notify("All tasks are complete.", "info");
 			return;
 		}
-		if (!runAll) {
-			const resolved = placement === "ask" ? await chooseBuildPlacement(ctx) : placement;
-			if (!resolved) return;
-			if (resolved === "new") {
-				await startBuildInNewSession(ctx);
-				return;
-			}
+		if (!runAll && !chainNew && placement === "new") {
+			await startBuildInNewSession(ctx, forkCommand);
+			return;
 		}
 		continueAll = runAll;
+		continueNew = chainNew;
 		await enterBuildMode(ctx);
-		ctx.ui.notify(runAll ? "Building remaining tasks." : "Building the next task.", "info");
+		ctx.ui.notify(
+			runAll
+				? "Building remaining tasks."
+				: chainNew
+					? "Building the next task. Remaining tasks will each start in a new session."
+					: "Building the next task.",
+			"info",
+		);
 		await startNextTask(ctx);
 	}
 
 	pi.registerCommand("build", {
-		description: "Choose this session or a new session, then execute the next planned task",
+		description: "Execute the next planned task in this session. Use /build new to start it in a new session",
 		handler: async (args, ctx) => {
 			await beginBuild(ctx, false, parseBuildPlacement(args));
 		},
 	});
 
 	pi.registerCommand("goal", {
-		description: "Execute remaining planned tasks until the list is complete",
-		handler: async (_args, ctx) => {
+		description: "Execute remaining planned tasks until complete. Use /goal new for one new session per task",
+		handler: async (args, ctx) => {
+			const goalMode = parseGoalMode(args);
+			if (goalMode === "all-new") {
+				await beginBuild(ctx, false, "new", { forkCommand: GOAL_CONTINUE_COMMAND });
+				return;
+			}
+			if (goalMode === "chain-here") {
+				await beginBuild(ctx, false, "here", { chainNew: true });
+				return;
+			}
 			await beginBuild(ctx, true);
 		},
 	});
@@ -432,9 +453,17 @@ export default async function planTaskExtension(pi: ExtensionAPI): Promise<void>
 		},
 	});
 
+	pi.registerCommand("goal-next-session", {
+		description: "Continue the next planned task in a new session, then keep chaining new sessions",
+		handler: async (_args, ctx) => {
+			await startBuildInNewSession(ctx, GOAL_CONTINUE_COMMAND);
+		},
+	});
+
 	pi.on("session_start", async (_event, ctx) => {
 		mode = "idle";
 		continueAll = false;
+		continueNew = false;
 		currentTaskId = undefined;
 		toolsBeforePlan = undefined;
 		awaitingChoice = false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-plan-task",
-  "version": "1.0.2",
+  "version": "1.1.0",
   "license": "MIT",
   "description": "Pi package: /plan, /build, /goal, and /tasks.",
   "keywords": ["pi-package", "pi-extension", "plan", "build"],