@blackbelt-technology/pi-agent-dashboard 0.2.4 → 0.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blackbelt-technology/pi-agent-dashboard",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Web dashboard for monitoring and interacting with pi agent sessions",
   "repository": {
     "type": "git",

package/packages/extension/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blackbelt-technology/pi-dashboard-extension",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Pi bridge extension for pi-dashboard",
   "type": "module",
   "pi": {

package/packages/server/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blackbelt-technology/pi-dashboard-server",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Dashboard server for monitoring and interacting with pi agent sessions",
   "type": "module",
   "main": "src/cli.ts",

package/packages/server/src/process-manager.ts

@@ -104,7 +104,91 @@ function resolvePiCommand(): string[] | null {
   return resolver.resolvePi();
 }
 
-function spawnHeadless(cwd: string, options?: SessionOptions): SpawnResult {
+/** Windows-specific headless spawn with error detection and stderr capture.
+ *  Waits briefly to detect immediate process death (e.g., missing deps, config errors).
+ */
+async function spawnHeadlessWindows(
+  cwd: string,
+  piCmd: string[],
+  args: string[],
+  env: NodeJS.ProcessEnv,
+): Promise<SpawnResult> {
+  const [bin, ...prefixArgs] = piCmd;
+  const needsShell = bin.endsWith(".cmd");
+  const spawnBin = needsShell ? `"${bin}"` : bin;
+  const spawnArgs = needsShell
+    ? [...prefixArgs, ...args].map(a => `"${a}"`)
+    : [...prefixArgs, ...args];
+
+  const cmdForLog = `${bin} ${[...prefixArgs, ...args].join(" ")}`;
+  console.error(`[spawn] Windows headless: ${cmdForLog} (cwd=${cwd})`);
+
+  // Capture stderr for diagnostics (pi might log errors there)
+  const child = spawn(spawnBin, spawnArgs, {
+    cwd,
+    detached: false,
+    stdio: ["pipe", "ignore", "pipe"],
+    env,
+    shell: needsShell,
+    windowsHide: true,
+  });
+
+  // Collect stderr for early crash diagnostics
+  let stderrBuf = "";
+  child.stderr?.on("data", (chunk: Buffer) => {
+    stderrBuf += chunk.toString();
+    // Limit memory: keep only last 4 KB
+    if (stderrBuf.length > 4096) stderrBuf = stderrBuf.slice(-4096);
+  });
+
+  // Handle async spawn errors (e.g., ENOENT if binary disappears between check and exec)
+  let spawnError: string | null = null;
+  child.on("error", (err: Error) => {
+    spawnError = err.message;
+    console.error(`[spawn] Windows spawn error: ${err.message}`);
+  });
+
+  child.unref();
+  (child.stdin as any)?.unref();
+  child.stderr?.unref();
+
+  // Guard: if pid is undefined, spawn failed synchronously
+  if (!child.pid) {
+    // Wait briefly for the async error event
+    await new Promise(r => setTimeout(r, 200));
+    return {
+      success: false,
+      message: `Failed to spawn pi: ${spawnError || "unknown error (no PID)"}. Command: ${cmdForLog}`,
+    };
+  }
+
+  // Wait briefly to detect immediate crash (e.g., missing module, config error)
+  const exitCode = await Promise.race([
+    new Promise<number | null>(resolve => child.on("exit", resolve)),
+    new Promise<undefined>(resolve => setTimeout(() => resolve(undefined), 1500)),
+  ]);
+
+  if (exitCode !== undefined) {
+    const detail = stderrBuf.trim()
+      ? `\nstderr: ${stderrBuf.trim().split("\n").slice(-5).join("\n")}`
+      : "";
+    console.error(`[spawn] Pi exited immediately with code ${exitCode}${detail}`);
+    return {
+      success: false,
+      message: `Pi process exited immediately (code ${exitCode}).${detail}\nCommand: ${cmdForLog}`,
+    };
+  }
+
+  return {
+    success: true,
+    dashboardSpawned: true,
+    message: `Pi session spawned headless (pid ${child.pid})`,
+    pid: child.pid,
+    process: child,
+  };
+}
+
+async function spawnHeadless(cwd: string, options?: SessionOptions): Promise<SpawnResult> {
   try {
     const args = buildHeadlessArgs(options);
     const env = buildSpawnEnv();
@@ -119,31 +203,7 @@ function spawnHeadless(cwd: string, options?: SessionOptions): SpawnResult {
     }
 
     if (process.platform === "win32") {
-      // Windows: spawn node.exe directly with pi's CLI entry point (no .cmd, no cmd window)
-      const [bin, ...prefixArgs] = piCmd_;
-      const needsShell = bin.endsWith(".cmd");
-      const spawnBin = needsShell ? `"${bin}"` : bin;
-      const spawnArgs = needsShell
-        ? [...prefixArgs, ...args].map(a => `"${a}"`)
-        : [...prefixArgs, ...args];
-      const child = spawn(spawnBin, spawnArgs, {
-        cwd,
-        detached: false,
-        stdio: ["pipe", "ignore", "ignore"],
-        env,
-        shell: needsShell,
-        windowsHide: true,
-      });
-      child.unref();
-      (child.stdin as any)?.unref();
-
-      return {
-        success: true,
-        dashboardSpawned: true,
-        message: `Pi session spawned headless (pid ${child.pid})`,
-        pid: child.pid,
-        process: child,
-      };
+      return await spawnHeadlessWindows(cwd, piCmd_, args, env);
     }
 
     // Unix (macOS / Linux / WSL): wrap with "tail -f /dev/null | pi" so stdin

package/packages/shared/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blackbelt-technology/pi-dashboard-shared",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Shared types and utilities for pi-dashboard",
   "type": "module",
   "exports": {

package/packages/shared/src/__tests__/bridge-register.test.ts

@@ -114,6 +114,32 @@ describe("shared bridge-register", () => {
       expect(packages).toContain(newPath);
     });
 
+    it("removes stale app bundle paths with spaces or mixed case (macOS PI Dashboard.app)", () => {
+      const stalePath = "/Applications/PI Dashboard.app/Contents/Resources/server/packages/extension";
+      writeSettings({ packages: [stalePath] });
+
+      const newPath = "/Applications/PI-Dashboard.app/Contents/Resources/server/packages/extension";
+      registerBridgeExtension(newPath);
+
+      const settings = readSettings();
+      const packages = settings.packages as string[];
+      expect(packages).not.toContain(stalePath);
+      expect(packages).toContain(newPath);
+    });
+
+    it("removes stale Windows-style dashboard paths with mixed case", () => {
+      const stalePath = "C:\\Program Files\\PI Dashboard\\resources\\server\\packages\\extension";
+      writeSettings({ packages: [stalePath] });
+
+      const newPath = "C:\\Program Files\\PI-Dashboard\\resources\\server\\packages\\extension";
+      registerBridgeExtension(newPath);
+
+      const settings = readSettings();
+      const packages = settings.packages as string[];
+      expect(packages).not.toContain(stalePath);
+      expect(packages).toContain(newPath);
+    });
+
     it("preserves non-dashboard paths", () => {
       writeSettings({ packages: ["/some/other/extension"] });
 

package/packages/shared/src/bridge-register.ts

@@ -72,7 +72,9 @@ export function registerBridgeExtension(extensionPath: string): void {
     const isLocalPath = p.startsWith("/") || /^[a-zA-Z]:[/\\]/.test(p);
     if (!isLocalPath) return true;
     // Only consider dashboard-related paths for cleanup
-    if (!p.includes("pi-dashboard") && !p.includes("pi-agent-dashboard")) return true;
+    // Normalize: lowercase + collapse spaces/hyphens so "PI Dashboard" matches "pi-dashboard"
+    const normalized = p.toLowerCase().replace(/[\s_-]/g, "");
+    if (!normalized.includes("pidashboard") && !normalized.includes("piagentdashboard")) return true;
     // Keep paths that point to existing directories with a package.json
     try {
       return fs.existsSync(p) && fs.existsSync(path.join(p, "package.json"));