@bobfrankston/npmglobalize 1.0.181 → 1.0.182

package/lib.js

@@ -5463,6 +5463,27 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         // before any subsequent install -g symlinks to these dep targets.
         if (stashedDepModules.length > 0)
             restoreNestedDepModules(stashedDepModules, verbose);
+        // Auto-retry on arborist crash if -clean-nested-modules wasn't already
+        // tried. The crash is the canonical sibling-junctions-in-junctions
+        // failure mode; the fix is mechanical (wipe nested node_modules in
+        // file: targets, pack, restore), so there's no reason to make the user
+        // re-run with the flag they'd pick 100% of the time anyway.
+        if (!packResult.success && !options.cleanNestedModules) {
+            const probe = diagnoseNpmPackFailure(cwd, packResult.output, packResult.stderr, pkg);
+            if (probe.summary.includes('arborist')) {
+                console.error(colors.yellow(`  Detected arborist crash — auto-retrying with --clean-nested-modules...`));
+                const autoStash = cleanNestedDepModules(pkg, cwd, verbose);
+                if (autoStash.length > 0) {
+                    console.log(colors.yellow(`  Cleaned node_modules in ${autoStash.length} file: dep target(s): ${autoStash.map(s => s.name).join(', ')}`));
+                }
+                packResult = await runCommandAsync('npm', ['pack'], { cwd, silent: true });
+                if (autoStash.length > 0)
+                    restoreNestedDepModules(autoStash, verbose);
+                if (packResult.success) {
+                    console.log(colors.green('  ✓ Auto-retry succeeded after cleaning nested node_modules.'));
+                }
+            }
+        }
         if (!packResult.success) {
             const d = diagnoseNpmPackFailure(cwd, packResult.output, packResult.stderr, pkg);
             console.error(colors.red(`ERROR: ${d.summary}`));
@@ -5472,8 +5493,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 console.error(colors.yellow(`  Caused by referenced module: ${d.referencedModule}`));
             if (d.hint)
                 console.error(colors.yellow(`  Hint: ${d.hint}`));
-            if (d.summary.includes('arborist') && !options.cleanNestedModules) {
-                console.error(colors.yellow(`  Retry with --clean-nested-modules to wipe node_modules/ inside each file: dep target`));
+            if (d.summary.includes('arborist') && options.cleanNestedModules) {
+                console.error(colors.yellow(`  Already used --clean-nested-modules; the crash is from a deeper sibling. Try running in WSL or wiping nested node_modules manually.`));
             }
             recordBuildIssue(pkg.name || path.basename(cwd), 'error', d.referencedModule ? `${d.summary} (via ${d.referencedModule})` : d.summary);
             return false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/npmglobalize",
-  "version": "1.0.181",
+  "version": "1.0.182",
   "description": "Transform file: dependencies to npm versions for publishing",
   "main": "index.js",
   "type": "module",