@uxmaltech/collab-cli 0.1.4 → 0.1.5

package/dist/lib/npm-operations.js

@@ -25,36 +25,56 @@ function requireNpm() {
  */
 function npmGlobalInstall(npmPath, version) {
     const spec = `${NPM_PACKAGE}@${version}`;
-    // Try without --force first, then retry with --force on EEXIST
-    for (const force of [false, true]) {
-        try {
-            const args = force
-                ? ['install', '-g', '--force', spec]
-                : ['install', '-g', spec];
-            (0, node_child_process_1.execFileSync)(npmPath, args, {
-                stdio: 'inherit',
-                timeout: 60_000,
-            });
-            return true;
-        }
-        catch (error) {
-            const message = error instanceof Error ? error.message : String(error);
-            // EEXIST: bin symlink collision — retry once with --force
-            if (!force && EEXIST_ERROR.test(message)) {
-                process.stderr.write('Retrying with --force to overwrite existing bin link...\n');
-                continue;
-            }
-            if (PERMISSION_ERROR.test(message)) {
-                process.stderr.write((0, ansi_1.red)(`${ansi_1.CROSS} Permission denied. Try:\n`) +
-                    `  sudo npm install -g ${spec}\n`);
-            }
-            else {
-                process.stderr.write((0, ansi_1.red)(`${ansi_1.CROSS} Install failed: ${message}\n`));
-            }
+    // First attempt: pipe stderr so we can inspect it for EEXIST
+    try {
+        (0, node_child_process_1.execFileSync)(npmPath, ['install', '-g', spec], {
+            stdio: ['inherit', 'inherit', 'pipe'],
+            timeout: 60_000,
+        });
+        return true;
+    }
+    catch (firstError) {
+        // Extract stderr from the failed child process
+        const stderr = extractStderr(firstError);
+        if (!EEXIST_ERROR.test(stderr)) {
+            // Not an EEXIST error — report and bail
+            process.stderr.write(stderr);
+            reportInstallError(spec, firstError);
             return false;
         }
+        // EEXIST: bin symlink collision — retry with --force
+        process.stderr.write('Bin link conflict detected, retrying with --force...\n');
+    }
+    // Second attempt with --force (stdio: inherit for full visibility)
+    try {
+        (0, node_child_process_1.execFileSync)(npmPath, ['install', '-g', '--force', spec], {
+            stdio: 'inherit',
+            timeout: 60_000,
+        });
+        return true;
+    }
+    catch (error) {
+        reportInstallError(spec, error);
+        return false;
+    }
+}
+function extractStderr(error) {
+    if (error && typeof error === 'object' && 'stderr' in error) {
+        const buf = error.stderr;
+        return typeof buf === 'string' ? buf : buf?.toString('utf8') ?? '';
+    }
+    return '';
+}
+function reportInstallError(spec, error) {
+    const message = error instanceof Error ? error.message : String(error);
+    const stderr = extractStderr(error);
+    if (PERMISSION_ERROR.test(message) || PERMISSION_ERROR.test(stderr)) {
+        process.stderr.write((0, ansi_1.red)(`${ansi_1.CROSS} Permission denied. Try:\n`) +
+            `  sudo npm install -g ${spec}\n`);
+    }
+    else {
+        process.stderr.write((0, ansi_1.red)(`${ansi_1.CROSS} Install failed: ${message}\n`));
     }
-    return false;
 }
 /**
  * Runs `npm uninstall -g @uxmaltech/collab-cli`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uxmaltech/collab-cli",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "CLI for collaborative architecture and delivery workflows.",
   "private": false,
   "license": "UNLICENSED",