pi-teams 0.9.9 → 0.9.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-teams",
-  "version": "0.9.9",
+  "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);