pi-teams 0.9.8 → 0.9.10

package/extensions/index.ts

@@ -15,6 +15,29 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import { spawnSync } from "node:child_process";
 
+/**
+ * Build the command used to relaunch pi for teammate processes.
+ *
+ * In Bun-compiled pi binaries, process.argv[1] points at a virtual $bunfs path
+ * like /$bunfs/root/pi, which is not a real file and breaks when prefixed with
+ * `node`. process.execPath points at the actual executable in both compiled and
+ * regular environments, so prefer that when available.
+ */
+function getPiLaunchCommand(): string {
+  // If we have an execPath, use it directly (works for both compiled binaries and node scripts)
+  if (process.execPath) {
+    return JSON.stringify(process.execPath);
+  }
+
+  // Fallback: try argv[1] with node prefix for regular node environments
+  if (process.argv[1]) {
+    return `node ${JSON.stringify(process.argv[1])}`;
+  }
+
+  // Last resort: just use "pi" and hope it's on PATH
+  return "pi";
+}
+
 // Cache for available models
 let availableModelsCache: Array<{ provider: string; model: string }> | null = null;
 let modelsCacheTime = 0;
@@ -553,7 +576,7 @@ export default function (pi: ExtensionAPI) {
       await teams.addMember(safeTeamName, member);
       await messaging.sendPlainMessage(safeTeamName, "team-lead", safeName, params.prompt, "Initial prompt");
 
-      const piBinary = process.argv[1] ? `node ${process.argv[1]}` : "pi";
+      const piBinary = getPiLaunchCommand();
       let piCmd = piBinary;
 
       if (chosenModel) {
@@ -638,7 +661,7 @@ export default function (pi: ExtensionAPI) {
 
       const teamConfig = await teams.readConfig(safeTeamName);
       const cwd = params.cwd || process.cwd();
-      const piBinary = process.argv[1] ? `node ${process.argv[1]}` : "pi";
+      const piBinary = getPiLaunchCommand();
       let piCmd = piBinary;
       if (teamConfig.defaultModel) {
         // Use the combined --model provider/model format
@@ -1105,7 +1128,7 @@ export default function (pi: ExtensionAPI) {
           await teams.addMember(safeTeamName, member);
           await messaging.sendPlainMessage(safeTeamName, "team-lead", safeName, agentDef.prompt, "Initial prompt from predefined team");
 
-          const piBinary = process.argv[1] ? `node ${process.argv[1]}` : "pi";
+          const piBinary = getPiLaunchCommand();
           let piCmd = piBinary;
 
           if (chosenModel) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-teams",
-  "version": "0.9.8",
+  "version": "0.9.10",
   "description": "Agent teams for pi, ported from claude-code-teams-mcp",
   "repository": {
     "type": "git",

package/src/adapters/windows-adapter.test.ts

@@ -94,6 +94,8 @@ describe("WindowsAdapter", () => {
       Object.defineProperty(process, "platform", { value: "win32" });
       // Mock findWtBinary check
       mockExecCommand.mockReturnValueOnce({ stdout: "wt", status: 0 });
+      // Mock findPsBinary check (pwsh found)
+      mockExecCommand.mockReturnValueOnce({ stdout: "found", status: 0 });
       // Mock getPanes - no existing panes
       mockExecCommand.mockReturnValueOnce({ stdout: "[]", status: 0 });
       // Mock actual spawn
@@ -114,6 +116,8 @@ describe("WindowsAdapter", () => {
       Object.defineProperty(process, "platform", { value: "win32" });
       // Mock findWtBinary check
       mockExecCommand.mockReturnValueOnce({ stdout: "wt", status: 0 });
+      // Mock findPsBinary check (pwsh found)
+      mockExecCommand.mockReturnValueOnce({ stdout: "found", status: 0 });
       // Mock getPanes - existing panes
       mockExecCommand.mockReturnValueOnce({ stdout: '[{"window":1}]', status: 0 });
       // Mock actual spawn
@@ -169,6 +173,8 @@ describe("WindowsAdapter", () => {
       Object.defineProperty(process, "platform", { value: "win32" });
       // Mock findWtBinary check
       mockExecCommand.mockReturnValueOnce({ stdout: "wt", status: 0 });
+      // Mock findPsBinary check (pwsh found)
+      mockExecCommand.mockReturnValueOnce({ stdout: "found", status: 0 });
       // Mock actual spawn
       mockExecCommand.mockReturnValueOnce({ stdout: "", status: 0 });
 
@@ -183,6 +189,31 @@ describe("WindowsAdapter", () => {
       // Returns synthetic ID: windows_win_<timestamp>_<name>
       expect(windowId).toMatch(/^windows_win_\d+_team-lead$/);
     });
+
+    it("should fallback to powershell when pwsh is not available", () => {
+      Object.defineProperty(process, "platform", { value: "win32" });
+      // Mock findWtBinary check
+      mockExecCommand.mockReturnValueOnce({ stdout: "wt", status: 0 });
+      // Mock findPsBinary check - pwsh fails, powershell succeeds
+      mockExecCommand.mockReturnValueOnce({ stdout: "", status: 1 }); // pwsh not found
+      mockExecCommand.mockReturnValueOnce({ stdout: "found", status: 0 }); // powershell found
+      // Mock actual spawn
+      mockExecCommand.mockReturnValueOnce({ stdout: "", status: 0 });
+
+      const windowId = adapter.spawnWindow({
+        name: "team-lead",
+        cwd: "/test/path",
+        command: "pi",
+        env: {},
+      });
+
+      expect(windowId).toMatch(/^windows_win_\d+_team-lead$/);
+      // Verify that powershell was used (check the call args)
+      const lastCall = mockExecCommand.mock.calls[mockExecCommand.mock.calls.length - 1];
+      expect(lastCall[0]).toBe("wt");
+      // The command should use 'powershell' not 'pwsh'
+      expect(lastCall[1]).toContain("powershell");
+    });
   });
 
   describe("kill()", () => {

package/src/adapters/windows-adapter.ts

@@ -18,6 +18,7 @@ export class WindowsAdapter implements TerminalAdapter {
   ];
 
   private wtPath: string | null = null;
+  private psPath: string | null = null;
 
   private findWtBinary(): string | null {
     if (this.wtPath !== null) {
@@ -62,6 +63,42 @@ export class WindowsAdapter implements TerminalAdapter {
     return null;
   }
 
+  /**
+   * Find the PowerShell binary to use.
+   * Prefers PowerShell Core (pwsh) if available, falls back to Windows PowerShell (powershell).
+   */
+  private findPsBinary(): string {
+    if (this.psPath !== null) {
+      return this.psPath;
+    }
+
+    // Try pwsh (PowerShell Core / PowerShell 7+) first
+    try {
+      const result = execCommand("pwsh", ["-NoProfile", "-Command", "echo 'found'"]);
+      if (result.status === 0 && result.stdout.trim() === "found") {
+        this.psPath = "pwsh";
+        return "pwsh";
+      }
+    } catch {
+      // pwsh not found, try powershell
+    }
+
+    // Fall back to powershell (Windows PowerShell 5.1)
+    try {
+      const result = execCommand("powershell", ["-NoProfile", "-Command", "echo 'found'"]);
+      if (result.status === 0 && result.stdout.trim() === "found") {
+        this.psPath = "powershell";
+        return "powershell";
+      }
+    } catch {
+      // powershell not found either
+    }
+
+    // Default to powershell as it's built into Windows
+    this.psPath = "powershell";
+    return "powershell";
+  }
+
   detect(): boolean {
     // Windows only - check platform
     if (process.platform !== "win32") {
@@ -111,6 +148,7 @@ export class WindowsAdapter implements TerminalAdapter {
       throw new Error("Windows Terminal (wt) CLI binary not found.");
     }
 
+    const psBin = this.findPsBinary();
     const panes = this.getPanes();
 
     // Build environment variables for PowerShell
@@ -134,7 +172,7 @@ export class WindowsAdapter implements TerminalAdapter {
         "split-pane",
         "--vertical",
         "--size", "50%",
-        "--", "pwsh", "-NoExit", "-Command", psCommand
+        "--", psBin, "-NoExit", "-Command", psCommand
       ];
     } else {
       // Create a new tab for subsequent panes (Windows Terminal limitation)
@@ -143,7 +181,7 @@ export class WindowsAdapter implements TerminalAdapter {
         "split-pane",
         "--horizontal",
         "--size", "50%",
-        "--", "pwsh", "-NoExit", "-Command", psCommand
+        "--", psBin, "-NoExit", "-Command", psCommand
       ];
     }
 
@@ -205,6 +243,8 @@ export class WindowsAdapter implements TerminalAdapter {
       throw new Error("Windows Terminal (wt) CLI binary not found.");
     }
 
+    const psBin = this.findPsBinary();
+
     // Build environment variables for PowerShell
     const envVars = Object.entries(options.env)
       .filter(([k]) => k.startsWith("PI_"))
@@ -223,7 +263,7 @@ export class WindowsAdapter implements TerminalAdapter {
     const spawnArgs = [
       "new-window",
       "--title", windowTitle,
-      "--", "pwsh", "-NoExit", "-Command", psCommand
+      "--", psBin, "-NoExit", "-Command", psCommand
     ];
 
     const result = execCommand(wtBin, spawnArgs);