@codemoreira/esad 2.0.1-1 → 2.0.1-2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codemoreira/esad",
-  "version": "2.0.1-1",
+  "version": "2.0.1-2",
   "description": "Easy Super App Development - Zero-Config CLI and DevTools for React Native Module Federation",
   "main": "src/plugin/index.js",
   "types": "./src/plugin/index.d.ts",

package/src/cli/utils/process.js

@@ -4,6 +4,7 @@ const path = require('path');
 const fs = require('fs-extra');
 
 const runProcess = (cmd, args, cwd = process.cwd()) => {
+    let childRef;
     const promise = new Promise((resolve, reject) => {
       let finished = false;
       let started = false;
@@ -15,47 +16,45 @@ const runProcess = (cmd, args, cwd = process.cwd()) => {
       };
 
       const isWin = process.platform === 'win32';
-      const localBinPath = path.join(cwd, 'node_modules', '.bin', isWin ? `${cmd}.cmd` : cmd);
+      const localBin = isWin ? `${cmd}.cmd` : cmd;
+      const localBinPath = path.join(cwd, 'node_modules', '.bin', localBin);
       
       let command = cmd;
       let finalArgs = args;
       let useNativeSpawn = false;
 
       if (fs.existsSync(localBinPath)) {
-        command = isWin ? `node_modules\\.bin\\${cmd}.cmd` : `./node_modules/.bin/${cmd}`;
-        useNativeSpawn = isWin; // Use native spawn on Windows for local binaries to avoid cross-spawn issues
+        // 1. Priority: Local node_modules/.bin (fastest & consistent)
+        command = isWin ? `node_modules\\.bin\\${localBin}` : `./node_modules/.bin/${localBin}`;
+        useNativeSpawn = isWin; 
       } else {
-        command = isWin ? 'npx.cmd' : 'npx';
-        finalArgs = [cmd, ...args];
+        // 2. Fallback: System Path (git, npm, npx, etc)
+        // cross-spawn handles .cmd/.exe resolution automatically
+        command = cmd;
+        finalArgs = args;
+        useNativeSpawn = false;
       }
       
       console.log(`[ESAD] Resolved Command: ${command} (CWD: ${cwd})`);
 
-      // Mark as started after a short delay
       setTimeout(() => { started = true; }, 2000);
 
       const spawnFn = useNativeSpawn ? nativeSpawn : spawn;
-      const child = spawnFn(command, finalArgs, { 
+      childRef = spawnFn(command, finalArgs, { 
         stdio: 'inherit', 
         cwd, 
         shell: true
       });
 
-      // Attach child to promise for control
-      promise.child = child;
-      promise.kill = (signal) => child.kill(signal);
-
-      child.on('error', err => {
+      childRef.on('error', err => {
         if (started && err.code === 'ENOENT') {
           console.warn(`[ESAD] Warning: Late ENOENT ignored for ${cmd}.`);
           return;
         }
-
-        console.error(`[ESAD] Process Error: ${err.code} - ${err.message}`);
-        finalize(reject, new Error(`Failed to start ${cmd}: ${err.message}`));
+        finalize(reject, err);
       });
 
-      child.on('close', code => {
+      childRef.on('close', code => {
         if (code !== 0) {
           finalize(reject, new Error(`Process ${cmd} exited with code ${code}`));
         } else {
@@ -64,6 +63,9 @@ const runProcess = (cmd, args, cwd = process.cwd()) => {
       });
     });
 
+    promise.child = childRef;
+    promise.kill = (signal) => childRef && childRef.kill(signal);
+
     return promise;
   };