cli-tunnel 1.2.0-beta.2 → 1.2.0-beta.4

package/dist/index.js

@@ -448,6 +448,16 @@ async function main() {
                         installProc.on('close', (code) => code === 0 ? resolve() : reject(new Error(`Install exited with code ${code}`)));
                         installProc.on('error', reject);
                     });
+                    // Refresh PATH — winget updates the registry but current process has stale PATH
+                    if (process.platform === 'win32') {
+                        try {
+                            const userPath = execFileSync('reg', ['query', 'HKCU\\Environment', '/v', 'Path'], { encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] });
+                            const sysPath = execFileSync('reg', ['query', 'HKLM\\SYSTEM\\CurrentControlSet\\Control\\Session Manager\\Environment', '/v', 'Path'], { encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] });
+                            const extractPath = (out) => out.split('\n').find(l => l.includes('REG_'))?.split('REG_EXPAND_SZ')[1]?.trim() || out.split('\n').find(l => l.includes('REG_'))?.split('REG_SZ')[1]?.trim() || '';
+                            process.env.PATH = `${extractPath(userPath)};${extractPath(sysPath)}`;
+                        }
+                        catch { /* keep existing PATH */ }
+                    }
                     // Verify installation
                     execFileSync('devtunnel', ['--version'], { stdio: 'pipe' });
                     console.log(`\n  ${GREEN}✓${RESET} devtunnel installed successfully!\n`);
@@ -463,16 +473,18 @@ async function main() {
                 console.log(`\n  ${DIM}More info: https://aka.ms/devtunnels/doc${RESET}`);
                 console.log(`  ${DIM}Continuing without tunnel (local only)...${RESET}\n`);
             }
-            // If just installed, check login
-            if (devtunnelInstalled) {
+        }
+        if (devtunnelInstalled) {
+            try {
+                // Check if logged in before attempting tunnel creation
                 try {
                     const userInfo = execFileSync('devtunnel', ['user', 'show'], { encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] });
-                    if (userInfo.includes('not logged in') || userInfo.includes('No user')) {
+                    if (userInfo.includes('not logged in') || userInfo.includes('No user') || userInfo.includes('Anonymous')) {
                         throw new Error('not logged in');
                     }
                 }
                 catch {
-                    console.log(`  ${YELLOW}⚠ devtunnel not authenticated.${RESET}\n`);
+                    console.log(`\n  ${YELLOW}⚠ devtunnel not authenticated.${RESET}\n`);
                     const loginAnswer = await askUser(`  Would you like to log in now? [Y/n] `);
                     if (loginAnswer === '' || loginAnswer === 'y' || loginAnswer === 'yes') {
                         try {
@@ -496,6 +508,10 @@ async function main() {
                     }
                 }
             }
+            catch (err) {
+                console.log(`  ${YELLOW}⚠${RESET} Tunnel failed: ${err.message}\n`);
+                devtunnelInstalled = false;
+            }
         }
         if (devtunnelInstalled) {
             try {
@@ -601,7 +617,7 @@ async function main() {
         cols, rows, cwd,
         env: safeEnv,
     });
-    // Detect CSPRNG crash (Node.js + node-pty + ConPTY issue) and retry with useConpty: false
+    // Detect CSPRNG crash (Node.js 23 + node-pty issue) and retry via cmd.exe wrapper
     let ptyExitedEarly = false;
     const earlyExitCheck = new Promise((resolve) => {
         ptyProcess.onExit(({ exitCode }) => {
@@ -614,13 +630,32 @@ async function main() {
     });
     await earlyExitCheck;
     if (ptyExitedEarly && process.platform === 'win32') {
-        console.log(`  ${YELLOW}⚠${RESET} ConPTY crash detected, retrying with legacy PTY backend...\n`);
-        ptyProcess = nodePty.spawn(resolvedCmd, commandArgs, {
+        console.log(`  ${YELLOW}⚠${RESET} CSPRNG crash detected (Node.js + PTY issue), retrying via cmd.exe wrapper...\n`);
+        // Spawn through cmd.exe /c — this adds a shell layer that avoids the crash
+        const cmdLine = [resolvedCmd, ...commandArgs].map(a => a.includes(' ') ? `"${a}"` : a).join(' ');
+        ptyProcess = nodePty.spawn('cmd.exe', ['/c', cmdLine], {
             name: 'xterm-256color',
             cols, rows, cwd,
             env: safeEnv,
-            useConpty: false,
         });
+        // Check if cmd.exe wrapper also fails
+        let retryFailed = false;
+        const retryCheck = new Promise((resolve) => {
+            ptyProcess.onExit(({ exitCode }) => {
+                if (exitCode === 134 || exitCode === 3221226505) {
+                    retryFailed = true;
+                    resolve();
+                }
+            });
+            setTimeout(resolve, 2000);
+        });
+        await retryCheck;
+        if (retryFailed) {
+            const nodeVer = process.version;
+            console.log(`  ${YELLOW}⚠${RESET} The command crashed due to a known Node.js ${nodeVer} + PTY compatibility issue.`);
+            console.log(`  ${BOLD}Fix:${RESET} Install Node.js 22 LTS: ${GREEN}nvm install 22${RESET} or ${GREEN}winget install OpenJS.NodeJS.LTS${RESET}\n`);
+            process.exit(1);
+        }
     }
     ptyProcess.onData((data) => {
         process.stdout.write(data);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cli-tunnel",
-  "version": "1.2.0-beta.2",
+  "version": "1.2.0-beta.4",
   "description": "Tunnel any CLI app to your phone — PTY + devtunnel + xterm.js",
   "type": "module",
   "main": "dist/index.js",