@dreki-gg/pi-plan-mode 0.6.0 → 0.6.2

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @dreki-gg/pi-plan-mode
 
+## 0.6.2
+
+### Patch Changes
+
+- [`376864c`](https://github.com/dreki-gg/pi-extensions/commit/376864c37cefa47530363b47055311269c1724a8) Thanks [@jalbarrang](https://github.com/jalbarrang)! - fix(plan-mode): queue messages with deliverAs to prevent "agent already processing" errors
+
+  All `sendMessage` and `sendUserMessage` calls inside the `agent_end` handler now use `deliverAs: 'followUp'` so they are queued until the agent fully settles. Previously, "Follow up", "Refine Plan", and "Execute Plan" would fire while the agent was still in a processing state, causing silent failures or the error: "Agent is already processing. Specify streamingBehavior ('steer' or 'followUp') to queue the message."
+
+## 0.6.1
+
+### Patch Changes
+
+- [`d95dad2`](https://github.com/dreki-gg/pi-extensions/commit/d95dad2ac85c4e5428252ee691152a0db83a0ced) Thanks [@jalbarrang](https://github.com/jalbarrang)! - fix(plan-mode): replace Bun-specific APIs with Node.js `fs/promises`
+
+  `pi` runs under Node.js (`#!/usr/bin/env node`), so `Bun.file()` and `Bun.write()` are unavailable at runtime. Replaced all usages with `readFile` and `writeFile` from `node:fs/promises`, which work in both runtimes.
+
 ## 0.6.0
 
 ### Minor Changes

package/extensions/plan-mode/index.ts

@@ -22,13 +22,8 @@ import type { AgentMessage } from '@earendil-works/pi-agent-core';
 import type { AssistantMessage, TextContent } from '@earendil-works/pi-ai';
 import type { ExtensionAPI, ExtensionContext } from '@earendil-works/pi-coding-agent';
 import { Key } from '@earendil-works/pi-tui';
-import { mkdir } from 'node:fs/promises';
-import {
-  extractTodoItems,
-  isSafeCommand,
-  markCompletedSteps,
-  type TodoItem,
-} from './utils.js';
+import { mkdir, readFile, writeFile } from 'node:fs/promises';
+import { extractTodoItems, isSafeCommand, markCompletedSteps, type TodoItem } from './utils.js';
 import {
   extractPlanTitle,
   readPlansJson,
@@ -125,7 +120,7 @@ export default function planMode(pi: ExtensionAPI): void {
 
     await mkdir('.plans', { recursive: true });
     const content = serializePlansJson(manifest);
-    await Bun.write('.plans/plans.json', content);
+    await writeFile('.plans/plans.json', content, 'utf-8');
   }
 
   // ── UI updates ────────────────────────────────────────────────────────────
@@ -253,9 +248,7 @@ export default function planMode(pi: ExtensionAPI): void {
         ctx.ui.notify('No plan yet. Use /plan to start planning.', 'info');
         return;
       }
-      const list = todos
-        .map((t, i) => `${i + 1}. ${t.completed ? '✓' : '○'} ${t.text}`)
-        .join('\n');
+      const list = todos.map((t, i) => `${i + 1}. ${t.completed ? '✓' : '○'} ${t.text}`).join('\n');
       ctx.ui.notify(`Plan Progress:\n${list}`, 'info');
     },
   });
@@ -419,7 +412,7 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
       let title = 'Untitled plan';
       if (path.endsWith('PLAN.md')) {
         try {
-          const content = await Bun.file(path).text();
+          const content = await readFile(path, 'utf-8');
           title = extractPlanTitle(content);
         } catch {
           // Fall through
@@ -430,7 +423,7 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
     } else if (match && planDir && path.endsWith('PLAN.md')) {
       // planDir already set but PLAN.md just written — update title
       try {
-        const content = await Bun.file(path).text();
+        const content = await readFile(path, 'utf-8');
         const title = extractPlanTitle(content);
         await updatePlansManifest(match[1], 'in-progress', title);
       } catch {
@@ -496,7 +489,7 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
       // Read the plan to extract todos
       let planContent = '';
       try {
-        planContent = await Bun.file(planMdPath).text();
+        planContent = await readFile(planMdPath, 'utf-8');
       } catch {
         // Fall through — will use empty plan content
       }
@@ -509,7 +502,7 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
       // Read the start prompt for clean handoff
       let startPrompt = '';
       try {
-        startPrompt = (await Bun.file(startPromptPath).text()).trim();
+        startPrompt = (await readFile(startPromptPath, 'utf-8')).trim();
       } catch {
         // Fall through
       }
@@ -524,7 +517,7 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
             content: startPrompt,
             display: true,
           },
-          { triggerTurn: true },
+          { triggerTurn: true, deliverAs: 'followUp' },
         );
       } else {
         // Fallback: ask executor to read the plan
@@ -534,7 +527,7 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
             content: `Execute the plan in ${planMdPath}. Read it first, then execute step by step. Mark each step with [DONE:n] before moving to the next.`,
             display: true,
           },
-          { triggerTurn: true },
+          { triggerTurn: true, deliverAs: 'followUp' },
         );
       }
     } else if (choice === 'Refine Plan') {
@@ -553,12 +546,12 @@ Execute each step in order. You MUST include [DONE:n] in your response after com
 After your review, update PLAN.md and START-PROMPT.md with any improvements.`,
           display: true,
         },
-        { triggerTurn: true },
+        { triggerTurn: true, deliverAs: 'followUp' },
       );
     } else if (choice === 'Follow up') {
       const followUp = await ctx.ui.editor('Follow-up instructions for the planner:', '');
       if (followUp?.trim()) {
-        pi.sendUserMessage(followUp.trim());
+        pi.sendUserMessage(followUp.trim(), { deliverAs: 'followUp' });
       }
     } else if (choice === 'Exit plan mode') {
       await exitPlanMode(ctx);

package/extensions/plan-mode/plans-json.ts

@@ -23,17 +23,16 @@ export interface PlanEntry {
 
 export type PlansManifest = Record<string, PlanEntry>;
 
+import { readFile } from 'node:fs/promises';
+
 const PLANS_JSON = '.plans/plans.json';
 
 /** Read plans.json, returning current manifest (empty object if missing). */
 export async function readPlansJson(): Promise<PlansManifest> {
   try {
-    const file = Bun.file(PLANS_JSON);
-    if (await file.exists()) {
-      const text = await file.text();
-      if (text.trim()) {
-        return JSON.parse(text) as PlansManifest;
-      }
+    const text = await readFile(PLANS_JSON, 'utf-8');
+    if (text.trim()) {
+      return JSON.parse(text) as PlansManifest;
     }
   } catch {
     // File doesn't exist or isn't valid JSON

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dreki-gg/pi-plan-mode",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Two-phase planning workflow for pi — plan with claude-opus-4-6:medium, execute with gpt-5.5:low, with .plans/ file-based handoff",
   "keywords": [
     "pi-package"