@bobfrankston/npmglobalize 1.0.122 → 1.0.124

package/lib.d.ts

@@ -78,6 +78,8 @@ export interface GlobalizeOptions {
     local?: boolean;
     /** Freeze node_modules: replace symlinks/junctions with real copies for network share use */
     freeze?: boolean;
+    /** Internal: auto-initialize git repos without prompting (user chose "all") */
+    autoInit?: boolean;
     /** Internal: signals this call is from workspace orchestrator */
     _fromWorkspace?: boolean;
     /** Internal: signals this call is from CLI (version already printed) */

package/lib.js

@@ -814,18 +814,6 @@ export function restoreDeps(pkg, verbose = false) {
             }
         }
     }
-    // Remove bundledDependencies added for workspace hoisting
-    if (pkg['.bundledDependencies']) {
-        if (pkg['.bundledDependencies'].length === 0) {
-            delete pkg.bundledDependencies;
-            delete pkg.bundleDependencies;
-        }
-        else {
-            pkg.bundledDependencies = pkg['.bundledDependencies'];
-        }
-        delete pkg['.bundledDependencies'];
-        restored = true;
-    }
     return restored;
 }
 /** Check if .dependencies exist (already transformed) */
@@ -2391,8 +2379,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         if (dryRun) {
             console.log('  [dry-run] Would initialize git repository');
         }
-        else if (!init) {
-            const choice = await promptChoice('No git repository found. What would you like to do?\n  1) Initialize git repository (default)\n  2) Use local install only (skip git/publish)\n  3) Abort\nChoice:', ['1', '2', '3', '']);
+        else if (!init && !options.autoInit) {
+            const choice = await promptChoice('No git repository found. What would you like to do?\n  1) Initialize git repository (default)\n  a) Initialize ALL (don\'t ask again for remaining deps)\n  2) Use local install only (skip git/publish)\n  3) Abort\nChoice:', ['1', 'a', '2', '3', '']);
             if (choice === '2') {
                 console.log(colors.dim('Switching to local-only mode...'));
                 writeConfig(cwd, { ...configOptions, local: true }, new Set(['local']));
@@ -2402,7 +2390,10 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 console.log('Aborted. Run with --init to initialize.');
                 return false;
             }
-            // choice is '1' or '' (default)
+            if (choice === 'a') {
+                options.autoInit = true;
+            }
+            // choice is '1', 'a', or '' (default)
             const success = await initGit(cwd, gitVisibility, dryRun);
             if (!success)
                 return false;
@@ -2417,8 +2408,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     }
     else if (!gitStatus.hasRemote) {
         // Git repo exists but no remote - need to create GitHub repo
-        if (!init) {
-            const choice = await promptChoice('No git remote configured. What would you like to do?\n  1) Create GitHub repository (default)\n  2) Use local install only (skip git/publish)\n  3) Abort\nChoice:', ['1', '2', '3', '']);
+        if (!init && !options.autoInit) {
+            const choice = await promptChoice('No git remote configured. What would you like to do?\n  1) Create GitHub repository (default)\n  a) Initialize ALL (don\'t ask again for remaining deps)\n  2) Use local install only (skip git/publish)\n  3) Abort\nChoice:', ['1', 'a', '2', '3', '']);
             if (choice === '2') {
                 console.log(colors.dim('Switching to local-only mode...'));
                 writeConfig(cwd, { ...configOptions, local: true }, new Set(['local']));
@@ -2428,6 +2419,9 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 console.log('Aborted. Run with --init to set up GitHub repository.');
                 return false;
             }
+            if (choice === 'a') {
+                options.autoInit = true;
+            }
         }
         const success = await initGit(cwd, gitVisibility, dryRun);
         if (!success)
@@ -2971,7 +2965,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                             conform, // Propagate conform to dependencies
                             publishDeps, // Propagate so transitive deps also get published
                             forcePublish, // Propagate so transitive deps get force-published too
-                            rebase // Propagate so behind-remote deps get rebased automatically
+                            rebase, // Propagate so behind-remote deps get rebased automatically
+                            autoInit: options.autoInit // Propagate "init all" choice
                         });
                         if (!depSuccess) {
                             console.error(colors.red(`Failed to publish ${name}`));
@@ -3009,13 +3004,12 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     else if (!alreadyTransformed && verbose) {
         console.log('  No file: dependencies found.');
     }
-    // Hoist workspace sub-package deps into root for publishing/global install.
-    // npm install -g only installs root-level dependencies and doesn't process
-    // the workspaces field, so:
-    //   1. Third-party deps (express, ws, etc.) must be in root dependencies
-    //   2. Sibling workspace packages must be in bundledDependencies so npm
-    //      includes them in the tarball's node_modules/
-    // All changes are temporary — restoreDeps removes them via .dependencies backup.
+    // Hoist workspace sub-package third-party deps into root for global install.
+    // npm install -g from local handles workspace linking, but third-party deps
+    // (express, ws, etc.) in sub-packages must also be in root dependencies
+    // so they get installed. Workspace sibling refs are skipped — npm install -g .
+    // handles those via workspace linking.
+    // Changes are temporary — restoreDeps removes them via .dependencies backup.
     if (pkg.workspaces) {
         const wsPatterns = Array.isArray(pkg.workspaces) ? pkg.workspaces : pkg.workspaces.packages || [];
         // Collect all workspace sub-package dirs
@@ -3036,8 +3030,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                     wsDirs.push(dir);
             }
         }
-        // Collect workspace package names and their relative paths
-        const wsPackages = new Map(); // name -> relative dir
+        // Collect workspace package names (to skip as siblings)
+        const wsNames = new Set();
         for (const dir of wsDirs) {
             const pkgJsonPath = path.join(dir, 'package.json');
             if (!fs.existsSync(pkgJsonPath))
@@ -3045,7 +3039,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             try {
                 const subPkg = JSON.parse(fs.readFileSync(pkgJsonPath, 'utf-8'));
                 if (subPkg.name)
-                    wsPackages.set(subPkg.name, path.relative(cwd, dir).replace(/\\/g, '/'));
+                    wsNames.add(subPkg.name);
             }
             catch { /* skip */ }
         }
@@ -3065,8 +3059,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             }
             const deps = subPkg.dependencies || {};
             for (const [dep, ver] of Object.entries(deps)) {
-                if (wsPackages.has(dep))
-                    continue; // sibling workspace — handled via bundledDependencies
+                if (wsNames.has(dep))
+                    continue; // sibling workspace — handled by npm install -g .
                 if (dep in rootDeps)
                     continue; // already in root
                 if (hoisted.some(m => m.dep === dep))
@@ -3074,8 +3068,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 hoisted.push({ dep, version: ver, from: subPkg.name || path.basename(dir) });
             }
         }
-        const needsUpdate = hoisted.length > 0 || wsPackages.size > 0;
-        if (needsUpdate) {
+        if (hoisted.length > 0) {
             // Ensure .dependencies backup exists so restoreDeps will remove hoisted deps
             if (!pkg['.dependencies']) {
                 pkg['.dependencies'] = { ...(pkg.dependencies || {}) };
@@ -3083,7 +3076,6 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             }
             if (!pkg.dependencies)
                 pkg.dependencies = {};
-            // Hoist third-party deps
             for (const m of hoisted) {
                 let version = m.version;
                 if (typeof version === 'string' && version.startsWith('file:')) {
@@ -3099,22 +3091,6 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 pkg.dependencies[m.dep] = version;
                 console.log(colors.green(`  + hoisted ${m.dep}: ${version} (from ${m.from})`));
             }
-            // Add workspace sibling packages to dependencies + bundledDependencies
-            // so npm includes them in the tarball's node_modules/
-            // Save original bundledDependencies for restore
-            const origBundled = pkg.bundledDependencies || pkg.bundleDependencies || [];
-            pkg['.bundledDependencies'] = [...origBundled];
-            const bundled = [...origBundled];
-            for (const [name, relDir] of wsPackages) {
-                if (!(name in pkg.dependencies)) {
-                    pkg.dependencies[name] = 'file:./' + relDir;
-                }
-                if (!bundled.includes(name)) {
-                    bundled.push(name);
-                }
-                console.log(colors.green(`  + bundled workspace ${name} (./${relDir})`));
-            }
-            pkg.bundledDependencies = bundled;
             if (!dryRun) {
                 writePackageJson(cwd, pkg);
             }
@@ -3256,40 +3232,56 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                         }
                     }
                     else if (install) {
-                        // Quick check: does this version actually exist on npm?
-                        const vCheck = spawnSafe('npm', ['view', `${pkgName}@${pkgVersion}`, 'version'], {
-                            shell: process.platform === 'win32',
-                            stdio: ['pipe', 'pipe', 'pipe'],
-                            encoding: 'utf-8'
-                        });
-                        const vOnNpm = vCheck.status === 0 && vCheck.stdout?.trim() === pkgVersion;
-                        if (vOnNpm) {
-                            console.log(`Installing ${pkgName}@${pkgVersion} globally from registry...`);
-                            const registryInstallResult = installGlobalWithRetry(`${pkgName}@${pkgVersion}`, cwd);
-                            if (registryInstallResult.success) {
+                        // Workspace packages must install from local — npm install -g from
+                        // registry doesn't process workspaces, so sub-packages can't resolve
+                        // sibling imports. Local install handles workspace linking correctly.
+                        if (pkg.workspaces) {
+                            console.log(`Installing ${pkgName}@${pkgVersion} globally from local (workspace)...`);
+                            const localResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
+                            if (localResult.success) {
                                 console.log(colors.green(`✓ Installed globally: ${pkgName}@${pkgVersion}`));
                             }
                             else {
                                 console.error(colors.red(`✗ Global install failed`));
-                                console.error(colors.yellow(`  Try running manually: npm install -g ${pkgName}@${pkgVersion}`));
+                                console.error(colors.yellow('  Try running manually: npm install -g .'));
                             }
                         }
                         else {
-                            console.log(colors.yellow(`${pkgName}@${pkgVersion} not found on npm — installing from local directory.`));
-                            console.log(colors.dim('  Use -m "message" to publish this version to npm.'));
-                            const localResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
-                            if (localResult.success) {
-                                console.log(colors.green(`✓ Installed globally from local: ${pkgName}@${pkgVersion}`));
+                            // Quick check: does this version actually exist on npm?
+                            const vCheck = spawnSafe('npm', ['view', `${pkgName}@${pkgVersion}`, 'version'], {
+                                shell: process.platform === 'win32',
+                                stdio: ['pipe', 'pipe', 'pipe'],
+                                encoding: 'utf-8'
+                            });
+                            const vOnNpm = vCheck.status === 0 && vCheck.stdout?.trim() === pkgVersion;
+                            if (vOnNpm) {
+                                console.log(`Installing ${pkgName}@${pkgVersion} globally from registry...`);
+                                const registryInstallResult = installGlobalWithRetry(`${pkgName}@${pkgVersion}`, cwd);
+                                if (registryInstallResult.success) {
+                                    console.log(colors.green(`✓ Installed globally: ${pkgName}@${pkgVersion}`));
+                                }
+                                else {
+                                    console.error(colors.red(`✗ Global install failed`));
+                                    console.error(colors.yellow(`  Try running manually: npm install -g ${pkgName}@${pkgVersion}`));
+                                }
                             }
                             else {
-                                console.error(colors.red(`✗ Local install failed`));
-                                console.error(colors.yellow('  Try running manually: npm install -g .'));
+                                console.log(colors.yellow(`${pkgName}@${pkgVersion} not found on npm — installing from local directory.`));
+                                console.log(colors.dim('  Use -m "message" to publish this version to npm.'));
+                                const localResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
+                                if (localResult.success) {
+                                    console.log(colors.green(`✓ Installed globally from local: ${pkgName}@${pkgVersion}`));
+                                }
+                                else {
+                                    console.error(colors.red(`✗ Local install failed`));
+                                    console.error(colors.yellow('  Try running manually: npm install -g .'));
+                                }
                             }
                         }
                     }
                     if (wsl) {
-                        // Check if version is on npm for registry-based WSL install
-                        const useLocal = link || (() => {
+                        // Workspace packages must install from local (see above)
+                        const useLocal = link || !!pkg.workspaces || (() => {
                             const vc = spawnSafe('npm', ['view', `${pkgName}@${pkgVersion}`, 'version'], {
                                 shell: process.platform === 'win32',
                                 stdio: ['pipe', 'pipe', 'pipe'],
@@ -3868,28 +3860,49 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             }
         }
         else if (install) {
-            console.log(`Installing globally from registry: ${pkgName}@${pkgVersion}...`);
-            if (!dryRun) {
-                waitForNpmVersion(pkgName, pkgVersion);
-                const installResult = installGlobalWithRetry(`${pkgName}@${pkgVersion}`, cwd);
-                if (installResult.success) {
-                    globalInstallOk = true;
-                    console.log(colors.green(`✓ Installed globally: ${pkgName}@${pkgVersion}`));
+            if (updatedPkg.workspaces) {
+                // Workspace packages: install from local for proper workspace linking
+                console.log(`Installing ${pkgName}@${pkgVersion} globally from local (workspace)...`);
+                if (!dryRun) {
+                    const installResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
+                    if (installResult.success) {
+                        globalInstallOk = true;
+                        console.log(colors.green(`✓ Installed globally: ${pkgName}@${pkgVersion}`));
+                    }
+                    else {
+                        console.error(colors.red(`✗ Global install failed`));
+                        console.error(colors.yellow('  Try running manually: npm install -g .'));
+                    }
                 }
                 else {
-                    console.error(colors.red(`✗ Global install failed`));
-                    console.error(colors.yellow(`  Try running manually: npm install -g ${pkgName}@${pkgVersion}`));
+                    console.log(`  [dry-run] Would run: npm install -g .`);
                 }
             }
             else {
-                console.log(`  [dry-run] Would run: npm install -g ${pkgName}@${pkgVersion}`);
+                console.log(`Installing globally from registry: ${pkgName}@${pkgVersion}...`);
+                if (!dryRun) {
+                    waitForNpmVersion(pkgName, pkgVersion);
+                    const installResult = installGlobalWithRetry(`${pkgName}@${pkgVersion}`, cwd);
+                    if (installResult.success) {
+                        globalInstallOk = true;
+                        console.log(colors.green(`✓ Installed globally: ${pkgName}@${pkgVersion}`));
+                    }
+                    else {
+                        console.error(colors.red(`✗ Global install failed`));
+                        console.error(colors.yellow(`  Try running manually: npm install -g ${pkgName}@${pkgVersion}`));
+                    }
+                }
+                else {
+                    console.log(`  [dry-run] Would run: npm install -g ${pkgName}@${pkgVersion}`);
+                }
             }
         }
         if (wsl) {
-            const wslArgs = link ? ['npm', 'install', '-g', '.'] : ['npm', 'install', '-g', `${pkgName}@${pkgVersion}`];
-            if (!link && !install)
+            const useLocalWsl = link || !!updatedPkg.workspaces;
+            const wslArgs = useLocalWsl ? ['npm', 'install', '-g', '.'] : ['npm', 'install', '-g', `${pkgName}@${pkgVersion}`];
+            if (!useLocalWsl)
                 waitForNpmVersion(pkgName, pkgVersion);
-            console.log(`Installing in WSL${link ? ' (link)' : ' from registry'}: ${pkgName}@${pkgVersion}...`);
+            console.log(`Installing in WSL${useLocalWsl ? ' (local)' : ' from registry'}: ${pkgName}@${pkgVersion}...`);
             if (!dryRun) {
                 const wslResult = runCommand('wsl', wslArgs, { cwd, silent: false, showCommand: true });
                 if (wslResult.success) {

package/package.json

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