preview: de-namespace win32 entry (loadableEntry) + entry instrumentation

Author: modacker

.github/workflows/preview.yml

@@ -0,0 +1,27 @@
+name: preview
+
+# Throwaway windows-latest verification loop for the community/win-launcher
+# work: pushed only to preview/** branches on the fork, never landed on a PR
+# branch. Mirrors the windows-process-lifecycle job of dynamic-workflow.yml.
+on:
+  push:
+    branches: ['preview/**']
+
+permissions:
+  contents: read
+
+jobs:
+  windows-launcher:
+    runs-on: windows-latest
+    timeout-minutes: 15
+    defaults:
+      run:
+        working-directory: plugins/hetaoBackend/mcode-dynamic-workflows
+    steps:
+      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b18 # v7.0.1
+      - uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
+        with:
+          node-version: 22
+      - run: npm ci --ignore-scripts --registry=https://registry.npmjs.org
+      - run: node --test checks/process-tree.check.mjs
+      - run: node --test checks/win-launcher.check.mjs

plugins/hetaoBackend/mcode-dynamic-workflows/checks/win-launcher.check.mjs

@@ -2,7 +2,7 @@ import test from 'node:test';import assert from 'node:assert/strict';
 import {mkdir,writeFile,rm,readFile,chmod} from 'node:fs/promises';
 import {spawn} from 'node:child_process';
 import {tmpdir} from 'node:os';import {join} from 'node:path';
-import {resolveMcode} from '../src/mcode-location.mjs';
+import {resolveMcode,loadableEntry} from '../src/mcode-location.mjs';
 // Windows launcher resolution is exercised from POSIX by faking the on-disk
 // layout and passing platform:'win32' + a controlled PATH/PATHEXT: resolution
 // is pure filesystem probing, so the real bug (mixed-install layouts) is
@@ -181,6 +181,7 @@ test('end-to-end argv fidelity: resolved node entry spawns with spaced cwd and f
   await release(root,'0.4.12','0.4.12','process.stdout.write(JSON.stringify(process.argv.slice(2)))\n');
   const r=await resolveMcode('mcode',{env:winEnv(root),home:f.official,platform:'win32'});
   assert.ok(r);assert.equal(r.command,process.execPath,'direct node entry');
+  console.error('ENTRY=',JSON.stringify({entry:r.args[0],command:r.command,tmpdir:tmpdir(),cwd:process.cwd()}));
   const cwd=join(f.root,'work dir with spaces');
   await mkdir(cwd,{recursive:true});
   const sent=['exec','--input','-','--cwd',cwd,'--output-format','stream-json','--flag-shape-toy','-'];
@@ -199,3 +200,8 @@ test('last-resort PS hop prefers pwsh (PS7) over powershell (PS5.1 -File is brok
   assert.match(r.command,/pwsh\.exe$/i);
  }finally{await f.cleanup();}
 });
+test('namespaced win32 entries are de-namespaced: node >=22 realpathSync dies on a \\\\?\\ main entry with EISDIR lstat C: (real-Windows fork runs 35491670398/35492809510)',()=>{
+ assert.equal(loadableEntry('\\\\?\\C:\\Users\\a\\.minimax-code\\releases\\0.4.12\\node_modules\\@minimax-ai\\code\\cli.js'),'C:\\Users\\a\\.minimax-code\\releases\\0.4.12\\node_modules\\@minimax-ai\\code\\cli.js','drive form loses the namespace marker');
+ assert.equal(loadableEntry('\\\\?\\UNC\\srv\\share\\code\\cli.js'),'\\\\srv\\share\\code\\cli.js','UNC form maps back to \\\\server\\share');
+ assert.equal(loadableEntry('/tmp/wf-winloc/cli.js'),'/tmp/wf-winloc/cli.js','posix passes through untouched');
+});

plugins/hetaoBackend/mcode-dynamic-workflows/dist/main.mjs

@@ -13964,6 +13964,9 @@ async function fileExists(file, executable = false, platform = process.platform)
     return false;
   }
 }
+function loadableEntry(entry) {
+  return String(entry).replace(/^\\\\\?\\UNC\\/, "\\\\").replace(/^\\\\\?\\([a-zA-Z]:)/, "$1");
+}
 async function executablePath(command, env = process.env, platform = process.platform) {
   if (typeof command !== "string" || !command) return null;
   const direct = /[\\/]/.test(command);
@@ -13996,7 +13999,7 @@ async function resolveMcode(command = "mcode", { env = process.env, home = homed
   if (path) {
     if (platform === "win32" && /\.(cmd|bat)$/i.test(path)) {
       const active = await launcherEntry(join2(dirname(path), ".mcode-launcher.cmd"), dirname(path));
-      if (active) return { command: process.execPath, args: [active], source };
+      if (active) return { command: process.execPath, args: [loadableEntry(active)], source };
       const root = officialRoot(home, env);
       const candidates = [
         { entry: join2(dirname(path), "node_modules", "@minimax-ai", "code", "cli.js"), tie: "" },
@@ -14045,7 +14048,7 @@ async function resolveMcode(command = "mcode", { env = process.env, home = homed
         const version3 = await versionOf(candidate.entry);
         if (!best || cmpSemVer(version3, best.version) > 0 || cmpSemVer(version3, best.version) === 0 && candidate.tie > best.tie) best = { ...candidate, version: version3 };
       }
-      if (best) return { command: process.execPath, args: [best.entry], source };
+      if (best) return { command: process.execPath, args: [loadableEntry(best.entry)], source };
       const launcher = join2(dirname(path), "mcode.ps1");
       if (await fileExists(launcher)) {
         const powershell = await executablePath("pwsh.exe", env, platform) ?? await executablePath("powershell.exe", env, platform);
@@ -14058,7 +14061,7 @@ async function resolveMcode(command = "mcode", { env = process.env, home = homed
   }
   if (command === "mcode") {
     const entry = managedEntry(home, platform);
-    if (await fileExists(entry)) return { command: process.execPath, args: [entry], source: "workflow-managed" };
+    if (await fileExists(entry)) return { command: process.execPath, args: [loadableEntry(entry)], source: "workflow-managed" };
   }
   return null;
 }

plugins/hetaoBackend/mcode-dynamic-workflows/src/mcode-location.mjs

@@ -9,6 +9,21 @@ export const managedEntry = (home = homedir(), platform = process.platform) => j
 async function fileExists(file, executable = false, platform = process.platform) {
   try { await access(file, executable && platform !== 'win32' ? constants.X_OK : constants.F_OK); return (await stat(file)).isFile(); } catch { return false; }
 }
+// A \\?\-namespaced win32 path is fine for fs probing but fatal as node's
+// main entry on every Node line this plugin supports (>=22): node runs
+// fs.realpathSync on argv[1], whose JS implementation probes the namespaced
+// drive root `\\?\C:\`; the fs binding drops the trailing separator, lstat
+// receives the bare drive `C:` and dies with EISDIR (nodejs/node#62446,
+// fixed by nodejs/node#65378 — first shipped in v24.21.0/v26.8.0). Real-
+// Windows evidence: fork preview runs 35491670398 / 35492809510
+// (windows-latest, Node 22) — fs.promises.mkdir(recursive) hands back a
+// namespaced created path, every join below it keeps the prefix, and the
+// spawned entry crashed exactly there. Strip the namespace marker (drive
+// and UNC forms) so a returned entry is always a plain path node can load
+// verbatim; POSIX strings never match and pass through unchanged.
+export function loadableEntry(entry) {
+  return String(entry).replace(/^\\\\\?\\UNC\\/, '\\\\').replace(/^\\\\\?\\([a-zA-Z]:)/, '$1');
+}
 export async function executablePath(command, env = process.env, platform = process.platform) {
   if (typeof command !== 'string' || !command) return null;
   const direct = /[\\/]/.test(command);
@@ -66,7 +81,7 @@ export async function resolveMcode(command = 'mcode', { env = process.env, home
       // any version arithmetic — the launcher runs what it runs, so we run what
       // it runs, and rollbacks and channel switches are followed for free.
       const active = await launcherEntry(join(dirname(path), '.mcode-launcher.cmd'), dirname(path));
-      if (active) return { command: process.execPath, args: [active], source };
+      if (active) return { command: process.execPath, args: [loadableEntry(active)], source };
       // No launcher pointer: fall back to ranking every coexisting layout by
       // version. When several installs coexist (PATH shim with an old sibling,
       // newer official root) a stale 0.2.x entry lacks current exec flags.
@@ -126,7 +141,7 @@ export async function resolveMcode(command = 'mcode', { env = process.env, home
         const version = await versionOf(candidate.entry);
         if (!best || cmpSemVer(version, best.version) > 0 || (cmpSemVer(version, best.version) === 0 && candidate.tie > best.tie)) best = { ...candidate, version };
       }
-      if (best) return { command: process.execPath, args: [best.entry], source };
+      if (best) return { command: process.execPath, args: [loadableEntry(best.entry)], source };
       const launcher = join(dirname(path), 'mcode.ps1');
       if (await fileExists(launcher)) {
         // pwsh (PS7) first: PS 5.1 binds flag-shaped argv as its own named parameters
@@ -141,7 +156,7 @@ export async function resolveMcode(command = 'mcode', { env = process.env, home
   }
   if (command === 'mcode') {
     const entry = managedEntry(home, platform);
-    if (await fileExists(entry)) return { command: process.execPath, args: [entry], source: 'workflow-managed' };
+    if (await fileExists(entry)) return { command: process.execPath, args: [loadableEntry(entry)], source: 'workflow-managed' };
   }
   return null;
 }