taskplane 0.22.16 → 0.22.18

package/bin/rpc-wrapper.mjs

@@ -770,10 +770,36 @@ piArgs.push(...args.passthrough);
 
 // ── Spawn pi process ─────────────────────────────────────────────────
 
+// ── System prompt: file-based passthrough to avoid command line limits ────
+// Windows CreateProcess has a ~32K command line limit. Orchestrated worker
+// system prompts routinely exceed this (PROMPT.md + context docs + steps).
+// When the system prompt is large, write it to a temp file and use shell
+// expansion `$(cat file)` to pass it. This works in MSYS2/Git Bash tmux
+// (where the lane sessions run) without hitting the Win32 limit.
+//
+// For small system prompts (< 8K), pass inline for simplicity.
+const SYSTEM_PROMPT_FILE_THRESHOLD = 8192;
+let systemPromptTempFile = null;
+
+if (systemPromptContent && systemPromptContent.length >= SYSTEM_PROMPT_FILE_THRESHOLD) {
+	// Remove --system-prompt from piArgs (was added above) and use file instead
+	const sysIdx = piArgs.indexOf("--system-prompt");
+	if (sysIdx >= 0) piArgs.splice(sysIdx, 2);
+	// Write to temp file and use --append-system-prompt with @file syntax.
+	// Pi's --append-system-prompt accepts @filepath to read from a file.
+	// We use --system-prompt "" (empty base) + --append-system-prompt @file
+	// to effectively set the system prompt from a file.
+	systemPromptTempFile = join(tmpdir(), `pi-rpc-sysprompt-${Date.now()}-${Math.random().toString(36).slice(2, 8)}.txt`);
+	writeFileSync(systemPromptTempFile, systemPromptContent, "utf-8");
+	piArgs.push("--system-prompt", "");
+	piArgs.push("--append-system-prompt", `@${systemPromptTempFile}`);
+	process.stderr.write(`[rpc-wrapper] system prompt written to file (${systemPromptContent.length} chars): ${systemPromptTempFile}\n`);
+}
+
 const proc = spawn("pi", piArgs, {
 	stdio: ["pipe", "pipe", "pipe"],
 	env: { ...process.env },
-	shell: true, // Required for Windows: resolves pi.cmd shim. Matches task-runner.ts pattern.
+	shell: true,
 });
 
 // ── TP-097: Write PID file for orphan cleanup ──────────────────
@@ -796,6 +822,9 @@ try {
 // Clean up PID file on process exit (best-effort)
 function cleanupPidFile() {
 	try { unlinkSync(pidFilePath); } catch { /* ignore */ }
+	if (systemPromptTempFile) {
+		try { unlinkSync(systemPromptTempFile); } catch { /* ignore */ }
+	}
 }
 process.on("exit", cleanupPidFile);
 

package/extensions/task-runner.ts

@@ -3519,80 +3519,41 @@ export default function (pi: ExtensionAPI) {
 				|| workerDef?.model
 				|| (ctx.model ? `${ctx.model.provider}/${ctx.model.id}` : "anthropic/claude-sonnet-4-20250514"));
 
-		const contextDocsList = task.contextDocs.length > 0
-			? "\n\nContext docs to read if needed:\n" + task.contextDocs.map(d => `- ${d}`).join("\n")
-			: "";
+		// ── Lean worker prompt: pass file paths, not content ──────────
+		// The worker reads PROMPT.md and STATUS.md itself using the read tool.
+		// This keeps the initial prompt small (~500 chars) instead of embedding
+		// 50K+ of compiled content that exceeds Windows command line limits
+		// and wastes initial context window capacity.
+		const promptLines = [
+			`Read your task instructions at: ${task.promptPath}`,
+			`Read your execution state at: ${statusPath}`,
+			``,
+			`Task: ${task.taskId}`,
+			`Task folder: ${task.taskFolder}/`,
+			`Iteration: ${state.totalIterations}`,
+			`Wrap-up signal file: ${wrapUpFile}`,
+		];
 
-		// When running under the parallel orchestrator, workers must NOT
-		// archive or move the task folder — the orchestrator polls for .DONE
-		// at the original path and handles post-merge archival itself.
-		const archiveSuppression = isOrchestratedMode()
-			? "\n\n⚠️ ORCHESTRATED RUN: Do NOT archive or move the task folder. " +
-			  "Do NOT rename, relocate, or reorganize the task folder path. " +
-			  "The orchestrator handles post-merge archival. " +
-			  "Just create the .DONE file in the task folder when complete."
-			: "";
+		if (isOrchestratedMode()) {
+			promptLines.push(``, `⚠️ ORCHESTRATED RUN: Do NOT archive or move the task folder. The orchestrator handles post-merge archival.`);
+		}
 
-		// Build step listing for the worker prompt — show ALL steps with status
-		const remainingSet = new Set(remainingSteps.map(s => s.number));
-		const stepListing = task.steps.map(s =>
-			remainingSet.has(s.number)
-				? `  - Step ${s.number}: ${s.name}`
-				: `  - Step ${s.number}: ${s.name}  [already complete — skip]`
-		).join("\n");
-
-		// TP-073: Build nudge for subsequent iterations (iter > 0)
-		// When the worker exited without completing all steps, the next iteration
-		// gets an explicit nudge listing completed/remaining steps and a warning
-		// not to exit prematurely again.
-		let iterationNudge = "";
 		if (state.totalIterations > 1 && remainingSteps.length > 0) {
+			const remainingSet = new Set(remainingSteps.map(s => s.number));
 			const completedSteps = task.steps.filter(s => !remainingSet.has(s.number));
 			const completedList = completedSteps.length > 0
 				? completedSteps.map(s => `Step ${s.number}: ${s.name}`).join(", ")
 				: "(none)";
 			const remainingList = remainingSteps.map(s => `Step ${s.number}: ${s.name}`).join(", ");
-			iterationNudge = [
-				``,
-				`IMPORTANT: You exited on your previous iteration without completing all steps.`,
-				`Do NOT repeat this — you must complete all remaining steps before stopping.`,
-				``,
-				`Completed steps (do not redo): ${completedList}`,
-				`Remaining steps (focus here): ${remainingList}`,
+			promptLines.push(
 				``,
-				`Your final action MUST be a tool call (update STATUS.md). Do NOT produce a`,
-				`text-only response — that will terminate your session prematurely.`,
-				``,
-			].join("\n");
+				`IMPORTANT: You exited previously without completing all steps.`,
+				`Completed (do not redo): ${completedList}`,
+				`Remaining (focus here): ${remainingList}`,
+			);
 		}
 
-		const prompt = [
-			`Execute all remaining steps for task ${task.taskId}.`,
-			``,
-			`Task: ${task.taskId} — ${task.taskName}`,
-			`Task folder: ${task.taskFolder}/`,
-			`PROMPT: ${task.promptPath}`,
-			`STATUS: ${statusPath}`,
-			``,
-			`This is iteration ${state.totalIterations}.`,
-			`Read STATUS.md FIRST to find where you left off.`,
-			iterationNudge,
-			`Steps:`,
-			stepListing,
-			``,
-			`Work through these steps in order. For each step:`,
-			`1. Read STATUS.md to find unchecked items for that step`,
-			`2. Complete all items for the step`,
-			`3. Update STATUS.md step status to "complete"`,
-			`4. Commit your changes: feat(${task.taskId}): complete Step N — description`,
-			`5. Check for wrap-up signal files before starting the next step`,
-			`6. Proceed to the next incomplete step`,
-			``,
-			`Wrap-up signal file: ${wrapUpFile}`,
-			`Check for this file after each checkpoint. If it exists, stop.`,
-			archiveSuppression,
-			contextDocsList,
-		].join("\n");
+		const prompt = promptLines.join("\n");
 
 		state.workerStatus = "running";
 		state.workerElapsed = 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "taskplane",
-  "version": "0.22.16",
+  "version": "0.22.18",
   "description": "AI agent orchestration for pi — parallel task execution with checkpoint discipline",
   "keywords": [
     "pi-package",