@hienlh/ppm 0.5.3 → 0.5.5

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.5.5] - 2026-03-18
+
+### Fixed
+- **Windows: PowerShell Start-Process fails with empty config arg** — filter empty strings from daemon spawn arguments
+
+## [0.5.4] - 2026-03-18
+
+### Fixed
+- **Windows: daemon process dies when parent exits** — use PowerShell `Start-Process` for truly detached daemon and tunnel processes on Windows
+- **Windows: `ppm status -a` / `stop -a` crash** — replaced `pgrep` (Unix-only) with `wmic` on Windows
+- Daemon startup now verifies child process is alive after 500ms — shows clear error + suggests `-f` if daemon fails
+
 ## [0.5.3] - 2026-03-18
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hienlh/ppm",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "description": "Personal Project Manager — mobile-first web IDE with AI assistance",
   "module": "src/index.ts",
   "type": "module",

package/src/cli/commands/status.ts

@@ -61,6 +61,25 @@ interface ProcessInfo {
 
 function findSystemProcesses(name: string): ProcessInfo[] {
   try {
+    if (process.platform === "win32") {
+      // Windows: use wmic to list processes matching name
+      const result = Bun.spawnSync(
+        ["wmic", "process", "where", `CommandLine like '%${name}%'`, "get", "ProcessId,CommandLine", "/format:csv"],
+        { stdout: "pipe", stderr: "ignore" },
+      );
+      const output = result.stdout.toString().trim();
+      if (!output) return [];
+      return output.split("\n").slice(1) // skip header
+        .map((line) => {
+          const parts = line.trim().split(",");
+          if (parts.length < 3) return null;
+          const pid = parseInt(parts[parts.length - 1]!, 10);
+          const cmdLine = parts.slice(1, -1).join(",");
+          return { pid, command: name, args: cmdLine };
+        })
+        .filter((p): p is ProcessInfo => p !== null && !isNaN(p.pid) && p.args.includes(name));
+    }
+    // macOS/Linux: use pgrep
     const result = Bun.spawnSync(["pgrep", "-afl", name], { stdout: "pipe", stderr: "ignore" });
     const output = result.stdout.toString().trim();
     if (!output) return [];

package/src/cli/commands/stop.ts

@@ -17,20 +17,37 @@ function killPid(pid: number, label: string): boolean {
   }
 }
 
-function killAllByName(name: string): number {
+function findPidsByName(name: string): number[] {
   try {
+    if (process.platform === "win32") {
+      // Windows: use wmic to find processes
+      const result = Bun.spawnSync(
+        ["wmic", "process", "where", `CommandLine like '%${name}%'`, "get", "ProcessId", "/format:csv"],
+        { stdout: "pipe", stderr: "ignore" },
+      );
+      const output = result.stdout.toString().trim();
+      if (!output) return [];
+      return output.split("\n").slice(1)
+        .map((line) => parseInt(line.trim().split(",").pop() ?? "", 10))
+        .filter((pid) => !isNaN(pid) && pid !== process.pid);
+    }
+    // macOS/Linux: use pgrep
     const result = Bun.spawnSync(["pgrep", "-fl", name], { stdout: "pipe", stderr: "ignore" });
     const output = result.stdout.toString().trim();
-    if (!output) return 0;
-    const pids = output.split("\n")
+    if (!output) return [];
+    return output.split("\n")
       .map((line) => parseInt(line.trim(), 10))
       .filter((pid) => !isNaN(pid) && pid !== process.pid);
-    let killed = 0;
-    for (const pid of pids) {
-      if (killPid(pid, `${name}`)) killed++;
-    }
-    return killed;
-  } catch { return 0; }
+  } catch { return []; }
+}
+
+function killAllByName(name: string): number {
+  const pids = findPidsByName(name);
+  let killed = 0;
+  for (const pid of pids) {
+    if (killPid(pid, name)) killed++;
+  }
+  return killed;
 }
 
 export async function stopServer(options?: { all?: boolean }) {

package/src/server/index.ts

@@ -254,14 +254,32 @@ export async function startServer(options: {
         const tunnelLog = resolve(ppmDir, "tunnel.log");
         // Truncate old log so we only match the new tunnel URL
         writeFs(tunnelLog, "");
-        const tfd = openFd(tunnelLog, "a");
-        const tunnelProc = Bun.spawn({
-          cmd: [bin, "tunnel", "--url", `http://localhost:${port}`],
-          stdio: ["ignore", "ignore", tfd],
-          env: process.env,
-        });
-        tunnelProc.unref();
-        tunnelPid = tunnelProc.pid;
+
+        if (process.platform === "win32") {
+          // Windows: use PowerShell for detached tunnel process
+          const psCmd = [
+            `$p = Start-Process -PassThru -WindowStyle Hidden`,
+            `-FilePath '${bin.replace(/\\/g, "\\\\")}'`,
+            `-ArgumentList 'tunnel','--url','http://localhost:${port}'`,
+            `-RedirectStandardError '${tunnelLog.replace(/\\/g, "\\\\")}'`,
+            `; Write-Output $p.Id`,
+          ].join(" ");
+          const result = Bun.spawnSync({
+            cmd: ["powershell", "-NoProfile", "-Command", psCmd],
+            stdout: "pipe", stderr: "pipe",
+          });
+          tunnelPid = parseInt(result.stdout.toString().trim(), 10);
+          if (isNaN(tunnelPid)) tunnelPid = null;
+        } else {
+          const tfd = openFd(tunnelLog, "a");
+          const tunnelProc = Bun.spawn({
+            cmd: [bin, "tunnel", "--url", `http://localhost:${port}`],
+            stdio: ["ignore", "ignore", tfd],
+            env: process.env,
+          });
+          tunnelProc.unref();
+          tunnelPid = tunnelProc.pid;
+        }
 
         // Parse URL from tunnel.log (poll stderr output)
         const urlRegex = /https:\/\/[a-z0-9-]+\.trycloudflare\.com/;
@@ -283,22 +301,64 @@ export async function startServer(options: {
     const logFile = resolve(ppmDir, "ppm.log");
     const logFd = openSync(logFile, "a");
     const { resolve: resolvePath } = await import("node:path");
-    const child = Bun.spawn({
-      cmd: [
-        process.execPath, "run", resolvePath(import.meta.dir, "index.ts"), "__serve__",
-        String(port), host, options.config ?? "",
-      ],
-      stdio: ["ignore", logFd, logFd],
-      env: process.env,
-    });
-    child.unref();
+    const script = resolvePath(import.meta.dir, "index.ts");
+    const args = ["__serve__", String(port), host, options.config ?? ""].filter(Boolean);
+
+    let childPid: number;
+
+    if (process.platform === "win32") {
+      // Windows: Bun.spawn child may die when parent exits (same job object).
+      // Use PowerShell Start-Process to create a truly detached process.
+      const bunExe = process.execPath.replace(/\\/g, "\\\\");
+      const argStr = ["run", script, ...args].map((a) => `'${a}'`).join(",");
+      const psCmd = [
+        `$p = Start-Process -PassThru -WindowStyle Hidden`,
+        `-FilePath '${bunExe}'`,
+        `-ArgumentList ${argStr}`,
+        `-RedirectStandardOutput '${logFile.replace(/\\/g, "\\\\")}'`,
+        `-RedirectStandardError '${logFile.replace(/\\/g, "\\\\")}'`,
+        `; Write-Output $p.Id`,
+      ].join(" ");
+      const result = Bun.spawnSync({
+        cmd: ["powershell", "-NoProfile", "-Command", psCmd],
+        stdout: "pipe",
+        stderr: "pipe",
+      });
+      childPid = parseInt(result.stdout.toString().trim(), 10);
+      if (isNaN(childPid)) {
+        console.error("  ✗  Failed to start daemon on Windows.");
+        console.error(`     ${result.stderr.toString().trim()}`);
+        console.error("     Try: ppm start -f (foreground mode)");
+        process.exit(1);
+      }
+    } else {
+      // macOS/Linux: Bun.spawn + unref works fine
+      const child = Bun.spawn({
+        cmd: [process.execPath, "run", script, ...args],
+        stdio: ["ignore", logFd, logFd],
+        env: process.env,
+      });
+      child.unref();
+      childPid = child.pid;
+    }
+
+    // Verify daemon is alive after brief startup
+    await Bun.sleep(500);
+    let alive = false;
+    try { process.kill(childPid, 0); alive = true; } catch {}
+    if (!alive) {
+      console.error("  ✗  Daemon exited immediately after start.");
+      console.error("     Check logs: ppm logs");
+      console.error("     Or try: ppm start -f (foreground mode)");
+      process.exit(1);
+    }
 
     // Write status file with both PIDs
-    const status = { pid: child.pid, port, host, shareUrl, tunnelPid };
+    const status = { pid: childPid, port, host, shareUrl, tunnelPid };
     writeFileSync(statusFile, JSON.stringify(status));
-    writeFileSync(pidFile, String(child.pid));
+    writeFileSync(pidFile, String(childPid));
 
-    console.log(`\n  PPM v${VERSION} daemon started (PID: ${child.pid})\n`);
+    console.log(`\n  PPM v${VERSION} daemon started (PID: ${childPid})\n`);
     console.log(`  ➜  Local:   http://localhost:${port}/`);
     if (shareUrl) {
       console.log(`  ➜  Share:   ${shareUrl}`);