@bobfrankston/npmglobalize 1.0.123 → 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

@@ -2379,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']));
@@ -2390,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;
@@ -2405,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']));
@@ -2416,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)
@@ -2959,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}`));
@@ -2997,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
@@ -3024,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))
@@ -3033,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 */ }
         }
@@ -3053,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))
@@ -3062,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 || {}) };
@@ -3071,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:')) {

package/package.json

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