@dreki-gg/pi-plan-mode 0.4.0 → 0.5.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @dreki-gg/pi-plan-mode
 
+## 0.5.0
+
+### Minor Changes
+
+- [`32797ff`](https://github.com/dreki-gg/pi-extensions/commit/32797ff18d968e22c6c44e95c46e3393d8928cef) Thanks [@jalbarrang](https://github.com/jalbarrang)! - feat(plan-mode): add Windows compatibility — replace Unix shell commands with cross-platform Bun/Node APIs
+
+  Plan-mode no longer shells out to `cat`, `bash`, or `mkdir` via `pi.exec()`. File I/O now uses `Bun.file()` / `Bun.write()` and `node:fs/promises` `mkdir`, making the extension fully cross-platform. Destructive and safe command pattern lists now include Windows equivalents (`del`, `rd`, `copy`, `move`, `powershell`, `dir`, `where`, `tasklist`, etc.).
+
+  Also fixes Windows compatibility in three other packages:
+
+  - **browser-tools**: `spawn` now uses `shell: true` on Windows so `.cmd` wrappers resolve correctly; `shellEscape` uses double-quote style on Windows; install guidance is platform-aware (Homebrew shown only on macOS).
+  - **subagent**: `spawn` uses `shell: true` on Windows when the command is bare `pi`, allowing `pi.cmd` resolution.
+  - **lsp**: `globalConfigPath()` now uses `os.homedir()` on Windows instead of the unreliable `process.env.HOME`.
+
 ## 0.4.0
 
 ### Minor Changes

package/extensions/plan-mode/index.ts

@@ -22,6 +22,7 @@ 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,
@@ -111,7 +112,7 @@ export default function planMode(pi: ExtensionAPI): void {
     status: 'in-progress' | 'done',
     title?: string,
   ): Promise<void> {
-    const manifest = await readPlansJson((cmd, args) => pi.exec(cmd, args));
+    const manifest = await readPlansJson();
     const existing = manifest[planName];
     const now = new Date().toISOString();
 
@@ -122,10 +123,9 @@ export default function planMode(pi: ExtensionAPI): void {
       completed: status === 'done' ? now : null,
     };
 
-    await pi.exec('mkdir', ['-p', '.plans']);
+    await mkdir('.plans', { recursive: true });
     const content = serializePlansJson(manifest);
-    // Write via a temp approach — use bash echo to avoid needing the write tool
-    await pi.exec('bash', ['-c', `cat > .plans/plans.json << 'PLANS_EOF'\n${content}PLANS_EOF`]);
+    await Bun.write('.plans/plans.json', content);
   }
 
   // ── UI updates ────────────────────────────────────────────────────────────
@@ -419,10 +419,8 @@ 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 result = await pi.exec('cat', [path]);
-          if (result.code === 0) {
-            title = extractPlanTitle(result.stdout);
-          }
+          const content = await Bun.file(path).text();
+          title = extractPlanTitle(content);
         } catch {
           // Fall through
         }
@@ -432,11 +430,9 @@ 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 result = await pi.exec('cat', [path]);
-        if (result.code === 0) {
-          const title = extractPlanTitle(result.stdout);
-          await updatePlansManifest(match[1], 'in-progress', title);
-        }
+        const content = await Bun.file(path).text();
+        const title = extractPlanTitle(content);
+        await updatePlansManifest(match[1], 'in-progress', title);
       } catch {
         // Fall through
       }
@@ -500,10 +496,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 {
-        const result = await pi.exec('cat', [planMdPath]);
-        if (result.code === 0) {
-          planContent = result.stdout;
-        }
+        planContent = await Bun.file(planMdPath).text();
       } catch {
         // Fall through — will use empty plan content
       }
@@ -516,10 +509,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 {
-        const result = await pi.exec('cat', [startPromptPath]);
-        if (result.code === 0) {
-          startPrompt = result.stdout.trim();
-        }
+        startPrompt = (await Bun.file(startPromptPath).text()).trim();
       } catch {
         // Fall through
       }

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

@@ -25,12 +25,15 @@ export type PlansManifest = Record<string, PlanEntry>;
 
 const PLANS_JSON = '.plans/plans.json';
 
-/** Read plans.json via pi.exec, returning current manifest (empty object if missing). */
-export async function readPlansJson(exec: (cmd: string, args: string[]) => Promise<{ code: number; stdout: string }>): Promise<PlansManifest> {
+/** Read plans.json, returning current manifest (empty object if missing). */
+export async function readPlansJson(): Promise<PlansManifest> {
   try {
-    const result = await exec('cat', [PLANS_JSON]);
-    if (result.code === 0 && result.stdout.trim()) {
-      return JSON.parse(result.stdout) as PlansManifest;
+    const file = Bun.file(PLANS_JSON);
+    if (await file.exists()) {
+      const text = await file.text();
+      if (text.trim()) {
+        return JSON.parse(text) as PlansManifest;
+      }
     }
   } catch {
     // File doesn't exist or isn't valid JSON

package/extensions/plan-mode/utils.ts

@@ -43,6 +43,17 @@ const DESTRUCTIVE_PATTERNS = [
   /\bsystemctl\s+(start|stop|restart|enable|disable)/i,
   /\bservice\s+\S+\s+(start|stop|restart)/i,
   /\b(vim?|nano|emacs|code|subl)\b/i,
+  // Windows equivalents
+  /\bdel\b/i,
+  /\brd\b/i,
+  /\bcopy\b/i,
+  /\bmove\b/i,
+  /\bren\b/i,
+  /\brename\b/i,
+  /\bicacls\b/i,
+  /\battrib\b/i,
+  /\bpowershell\b/i,
+  /\bpwsh\b/i,
 ];
 
 // ── Safe read-only bash patterns (allowed in plan mode) ─────────────────────
@@ -98,6 +109,12 @@ const SAFE_PATTERNS = [
   /^\s*fd\b/,
   /^\s*bat\b/,
   /^\s*eza\b/,
+  // Windows equivalents
+  /^\s*dir\b/,
+  /^\s*where\b/,
+  /^\s*set\b/,
+  /^\s*systeminfo\b/,
+  /^\s*tasklist\b/,
 ];
 
 export function isSafeCommand(command: string): boolean {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dreki-gg/pi-plan-mode",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "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"
@@ -36,6 +36,7 @@
   },
   "devDependencies": {
     "@types/node": "24",
+    "bun-types": "latest",
     "oxfmt": "^0.43.0",
     "oxlint": "^1.58.0",
     "typescript": "^6.0.0"