forge-jsxy 1.0.82 → 1.0.83

package/dist/assets/files-explorer-template.html

@@ -10,7 +10,7 @@
 <link rel="apple-touch-icon" href="/forge-explorer-favicon.svg"/>
 <link rel="stylesheet" href="/forge-explorer-codicons/codicon.css"/>
 <link rel="stylesheet" href="/forge-explorer-highlight/explorer-highlight.css"/>
-<!-- forge-jsxy@1.0.82 reconnect-ui npm-isolated-cache hub-20gib-delete-watch -->
+<!-- forge-jsxy@1.0.83 reconnect-ui npm-isolated-cache hub-20gib-delete-watch -->
 <script>
 (function () {
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "forge-jsxy",
-  "version": "1.0.82",
+  "version": "1.0.83",
   "description": "Node.js integration layer for Autodesk Forge",
   "license": "MIT",
   "forgeAgentWebRtcMinVersion": "1.0.71",

package/scripts/forge-isolated-runtime.mjs

@@ -79,6 +79,40 @@ function npmEnv() {
   };
 }
 
+/**
+ * Run npm subcommands with argv-style args.
+ * Windows: `shell: true` so `npm.cmd` resolves from PATH like npm's own lifecycle scripts (Git Bash /
+ * non-interactive spawn often returns `status: null` without shell when PATH differs).
+ */
+function npmSpawnSync(npmArgv, extraOptions) {
+  const cmd = npmCmd();
+  const merged = {
+    encoding: "utf8",
+    windowsHide: true,
+    env: npmEnv(),
+    ...extraOptions,
+  };
+  if (process.platform === "win32") {
+    return spawnSync(cmd, npmArgv, { ...merged, shell: true });
+  }
+  return spawnSync(cmd, npmArgv, merged);
+}
+
+/** When stderr/stdout are empty but status is null or non-zero — explain spawn/PATH/signal. */
+function formatSpawnFailure(pr, label) {
+  const io = (pr.stderr || pr.stdout || "").trim();
+  if (io) return io.slice(0, 2000);
+  const parts = [`${label}: subprocess failed`];
+  if (pr.status !== null && pr.status !== undefined) parts.push(`exit=${pr.status}`);
+  else parts.push("exit=null (npm not started — PATH/npm.cmd, antivirus, or shell issue)");
+  if (pr.signal) parts.push(`signal=${pr.signal}`);
+  if (pr.error) parts.push(pr.error.message || String(pr.error));
+  parts.push(
+    "Windows: prefer cmd.exe or PowerShell for npm install; ensure Node/npm are on PATH; folder paths with spaces sometimes confuse tooling."
+  );
+  return parts.join(" — ");
+}
+
 /**
  * @returns {{ ok: boolean, tgzPath: string, error?: string }}
  */
@@ -88,18 +122,15 @@ function npmPackToDir(packDir) {
   } catch (e) {
     return { ok: false, tgzPath: "", error: e instanceof Error ? e.message : String(e) };
   }
-  const pr = spawnSync(npmCmd(), ["pack", pkgRoot, "--pack-destination", packDir], {
+  const pr = npmSpawnSync(["pack", pkgRoot, "--pack-destination", packDir], {
     cwd: packDir,
-    encoding: "utf8",
-    windowsHide: true,
-    env: npmEnv(),
     maxBuffer: 10 * 1024 * 1024,
   });
   if (pr.status !== 0) {
     return {
       ok: false,
       tgzPath: "",
-      error: (pr.stderr || pr.stdout || "").trim() || `npm pack exit ${pr.status}`,
+      error: formatSpawnFailure(pr, "npm pack"),
     };
   }
   const lines = (pr.stdout || "")
@@ -129,11 +160,8 @@ function npmInstallTgzLocalPrefix(tgzPath, prefixRoot) {
     "--no-fund",
   ];
   const cwd = existsSync(prefixRoot) ? prefixRoot : os.tmpdir();
-  return spawnSync(npmCmd(), args, {
+  return npmSpawnSync(args, {
     cwd,
-    encoding: "utf8",
-    windowsHide: true,
-    env: npmEnv(),
     maxBuffer: 40 * 1024 * 1024,
   });
 }
@@ -187,8 +215,13 @@ export function bootstrapDurableForgeJsxy() {
 
   const r = npmInstallTgzLocalPrefix(tgzPath, versionDir);
   if (r.status !== 0) {
-    const err = (r.stderr || r.stdout || "").trim() || `npm install exit ${r.status}`;
-    return { ok: false, distDir: "", versionDir, version, error: err };
+    return {
+      ok: false,
+      distDir: "",
+      versionDir,
+      version,
+      error: formatSpawnFailure(r, "npm install (durable prefix)"),
+    };
   }
 
   const distDir = resolveDistDirUnderLocalPrefix(versionDir);