@agegr/pi-web 0.5.12 → 0.5.13

package/bin/pi-web.js

@@ -11,7 +11,6 @@ const fs = require("fs");
 const { parseArgs } = require("util");
 
 const pkgDir = path.join(__dirname, "..");
-const isWindows = process.platform === "win32";
 const nextDir = path.join(pkgDir, ".next");
 
 // Resolve next's CLI entry directly to avoid relying on .bin symlinks (which
@@ -20,8 +19,13 @@ let nextBin;
 try {
   nextBin = require.resolve("next/dist/bin/next", { paths: [pkgDir] });
 } catch {
-  // Fallback to the .bin symlink for older setups / Windows
-  nextBin = path.join(pkgDir, "node_modules", ".bin", isWindows ? "next.cmd" : "next");
+  // Fallback: locate next package root and derive the bin path manually.
+  try {
+    const nextPkg = require.resolve("next/package.json", { paths: [pkgDir] });
+    nextBin = path.join(path.dirname(nextPkg), "dist", "bin", "next");
+  } catch {
+    nextBin = path.join(pkgDir, "node_modules", "next", "dist", "bin", "next");
+  }
 }
 
 const { values: cliArgs } = parseArgs({
@@ -43,16 +47,12 @@ if (!fs.existsSync(nextDir)) {
 const nextArgs = ["start", "-p", port];
 if (hostname) nextArgs.push("-H", hostname);
 
-// If we resolved a .js file, run it with node directly; otherwise exec the binary.
-const useNode = nextBin.endsWith(".js");
-const spawnCmd  = useNode ? process.execPath : nextBin;
-const spawnArgs = useNode ? [nextBin, ...nextArgs] : nextArgs;
-
-const child = spawn(spawnCmd, spawnArgs, {
+// Always run next's JS entry with node directly — avoids .bin symlink issues
+// and path-with-spaces problems on Windows when shell: true is used.
+const child = spawn(process.execPath, [nextBin, ...nextArgs], {
   cwd: pkgDir,
   stdio: "inherit",
   env: { ...process.env },
-  shell: isWindows && !useNode,
 });
 
 child.on("exit", (code) => process.exit(code ?? 0));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agegr/pi-web",
-  "version": "0.5.12",
+  "version": "0.5.13",
   "description": "Web UI for the pi coding agent",
   "bin": {
     "pi-web": "bin/pi-web.js"