@bobfrankston/npmglobalize 1.0.104 → 1.0.106

package/README.md

@@ -169,6 +169,12 @@ For single-developer projects, this safely pulls remote changes before publishin
 
 ### 📂 Git Repository Setup
 
+**Change visibility of an existing repo:**
+```bash
+npmglobalize -git public     # Makes the GitHub repo public (with confirmation)
+npmglobalize -git private    # Makes the GitHub repo private
+```
+
 When initializing a new repository with `--init`, npmglobalize automatically sets up:
 
 **File structure** (per programming.md standards):
@@ -266,8 +272,9 @@ Publishing requires being on a branch so commits and tags can be properly tracke
 
 ### Git/npm Visibility
 ```
--git private    Make git repo private (default)
--git public     Make git repo public (requires confirmation)
+-git private    Set GitHub repo to private (default for new repos)
+-git public     Set GitHub repo to public (requires confirmation)
+                Works on both new and existing repos
 -npm private    Mark npm package private (skip publish) (default)
 -npm public     Publish to npm
 ```

package/cli.js

@@ -47,8 +47,9 @@ Mode Options:
   -nofiles        Keep npm versions permanently
 
 Git/npm Visibility:
-  -git private    Make git repo private (default)
-  -git public     Make git repo public (requires confirmation)
+  -git private    Set GitHub repo to private (default for new repos)
+  -git public     Set GitHub repo to public (requires confirmation)
+                  Works on both new and existing repos
   -npm private    Mark npm package private (skip publish) (default)
   -npm public     Publish to npm
 

package/lib.d.ts

@@ -78,6 +78,8 @@ export interface GlobalizeOptions {
     _fromWorkspace?: boolean;
     /** Internal: signals this call is from CLI (version already printed) */
     _fromCli?: boolean;
+    /** Internal: tracks which options were explicitly set on the CLI */
+    explicitKeys?: Set<string>;
 }
 /** Result from a single package in workspace mode */
 interface WorkspacePackageResult {
@@ -186,6 +188,8 @@ export declare function fixVersionTagMismatch(cwd: string, pkg: any, verbose?: b
 /** Run a command and return success status */
 export declare function runCommand(cmd: string, args: string[], options?: {
     silent?: boolean;
+    verbose?: boolean;
+    showCommand?: boolean;
     cwd?: string;
 }): {
     success: boolean;

package/lib.js

@@ -36,6 +36,7 @@ const colors = {
     green: (text) => styleText('green', text),
     italic: (text) => styleText('italic', text),
     blue: (text) => styleText('blue', text),
+    cyan: (text) => styleText('cyan', text),
     dim: (text) => styleText('dim', text),
 };
 /**
@@ -962,23 +963,23 @@ function waitForNpmVersion(pkgName, version, maxWaitMs = 90000) {
 }
 /** Run npm install -g with retries for registry propagation delay */
 function installGlobalWithRetry(pkgSpec, cwd, maxRetries = 3) {
-    let result = runCommand('npm', ['install', '-g', pkgSpec], { cwd, silent: false });
+    let result = runCommand('npm', ['install', '-g', pkgSpec], { cwd, silent: false, showCommand: true });
     for (let attempt = 1; attempt < maxRetries && !result.success; attempt++) {
         console.log(colors.yellow(`  Retrying install (attempt ${attempt + 1}/${maxRetries}) in 10 seconds...`));
         spawnSafe(process.platform === 'win32' ? 'timeout' : 'sleep', process.platform === 'win32' ? ['/t', '10', '/nobreak'] : ['10'], { stdio: 'pipe', shell: process.platform === 'win32' });
-        result = runCommand('npm', ['install', '-g', pkgSpec], { cwd, silent: false });
+        result = runCommand('npm', ['install', '-g', pkgSpec], { cwd, silent: false, showCommand: true });
     }
     return result;
 }
 /** Run a command and return success status */
 export function runCommand(cmd, args, options = {}) {
-    const { silent = false, cwd } = options;
+    const { silent = false, verbose = false, showCommand = false, cwd } = options;
     try {
         // Use shell:true for npm/gh commands which need it on Windows
         const needsShell = cmd === 'npm' || cmd === 'npm.cmd' || cmd === 'gh';
-        // Debug: log the command being executed
-        if (!silent) {
-            console.log(colors.dim(`[DEBUG] Running: ${cmd} ${args.join(' ')}`));
+        // Show command for important actions (install, etc.) or in verbose mode
+        if (!silent && (showCommand || verbose)) {
+            console.log(colors.cyan(`> ${cmd} ${args.join(' ')}`));
         }
         const result = spawnSafe(cmd, args, {
             encoding: 'utf-8',
@@ -1865,7 +1866,7 @@ async function doLocalInstall(cwd, options) {
             console.log('  [dry-run] Would run: wsl npm install -g .');
         return true;
     }
-    const result = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false });
+    const result = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
     if (result.success) {
         console.log(colors.green(`✓ Installed locally: ${pkgName}@${pkgVersion}`));
     }
@@ -1876,7 +1877,7 @@ async function doLocalInstall(cwd, options) {
     }
     if (wsl) {
         console.log(`Installing ${pkgName} in WSL (local)...`);
-        const wslResult = runCommand('wsl', ['npm', 'install', '-g', '.'], { cwd, silent: false });
+        const wslResult = runCommand('wsl', ['npm', 'install', '-g', '.'], { cwd, silent: false, showCommand: true });
         if (wslResult.success) {
             console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
         }
@@ -1971,7 +1972,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 console.log('  [dry-run] Would run: wsl npm install -g .');
             return true;
         }
-        const result = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false });
+        const result = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
         if (result.success) {
             console.log(colors.green(`✓ Installed locally: ${pkgName}@${pkgVersion}`));
         }
@@ -1982,7 +1983,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         }
         if (wsl) {
             console.log(`Installing ${pkgName} in WSL (local)...`);
-            const wslResult = runCommand('wsl', ['npm', 'install', '-g', '.'], { cwd, silent: false });
+            const wslResult = runCommand('wsl', ['npm', 'install', '-g', '.'], { cwd, silent: false, showCommand: true });
             if (wslResult.success) {
                 console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
             }
@@ -2115,6 +2116,33 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             return false;
         justInitialized = true;
     }
+    // Change GitHub repo visibility if explicitly requested on an existing repo
+    const explicitKeys = options.explicitKeys;
+    if (!justInitialized && gitStatus.isRepo && gitStatus.hasRemote && explicitKeys?.has('gitVisibility')) {
+        const targetVis = gitVisibility;
+        if (targetVis === 'public') {
+            const ok = await confirm(`Change GitHub repo visibility to PUBLIC? Anyone will be able to see your code.`, false);
+            if (!ok) {
+                console.log('Aborted visibility change.');
+                return false;
+            }
+        }
+        if (dryRun) {
+            console.log(`  [dry-run] Would change GitHub repo visibility to ${targetVis}`);
+        }
+        else {
+            console.log(`Changing GitHub repo visibility to ${targetVis}...`);
+            const visResult = runCommand('gh', ['repo', 'edit', '--visibility', targetVis], { cwd, silent: true });
+            if (visResult.success) {
+                console.log(colors.green(`✓ GitHub repo is now ${targetVis.toUpperCase()}`));
+            }
+            else {
+                console.error(colors.red(`✗ Failed to change repo visibility: ${visResult.stderr}`));
+                if (!force)
+                    return false;
+            }
+        }
+    }
     // Re-check git status after potential init
     let currentGitStatus = getGitStatus(cwd);
     // Validate git state
@@ -2349,12 +2377,22 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             }
         }
         if (currentAccess === 'public') {
-            console.error(colors.red(`ERROR: Package '${pkg.name}' is currently PUBLIC on npm.`));
-            console.error(colors.red(`Cannot change to private. Options:`));
-            console.error(colors.yellow(`  1. Deprecate the public package: npm deprecate ${pkg.name} "moved to private"`));
-            console.error(colors.yellow(`  2. Continue as public: npmglobalize --npm public`));
-            console.error(colors.yellow(`  3. Use a different package name`));
-            return false;
+            console.log(colors.yellow(`Package '${pkg.name}' is currently PUBLIC on npm. Converting to private...`));
+            if (!dryRun) {
+                const accessResult = runCommand('npm', ['access', 'set', 'status=restricted', pkg.name], { cwd, silent: false });
+                if (accessResult.success) {
+                    console.log(colors.green(`✓ Changed ${pkg.name} to PRIVATE on npm`));
+                    currentAccess = 'restricted';
+                }
+                else {
+                    console.error(colors.red(`Failed to change access. Try manually: npm access set status=restricted ${pkg.name}`));
+                    if (!force)
+                        return false;
+                }
+            }
+            else {
+                console.log(colors.dim(`  [dry-run] Would run: npm access set status=restricted ${pkg.name}`));
+            }
         }
         // Don't set "private": true in package.json - that blocks all publishing
         console.log(`Package '${pkg.name}' will publish as PRIVATE (restricted access).`);
@@ -2365,12 +2403,22 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     else if (effectiveNpmVisibility === 'public') {
         // User explicitly wants public (or confirmed via prompt)
         if (currentAccess === 'restricted') {
-            console.error(colors.red(`ERROR: Package '${pkg.name}' is currently PRIVATE on npm.`));
-            console.error(colors.red(`Cannot change to public. Options:`));
-            console.error(colors.yellow(`  1. Unpublish and republish: npm unpublish ${pkg.name} --force`));
-            console.error(colors.yellow(`  2. Continue as private: npmglobalize --npm private`));
-            console.error(colors.yellow(`  3. Use a different package name`));
-            return false;
+            console.log(colors.yellow(`Package '${pkg.name}' is currently PRIVATE on npm. Converting to public...`));
+            if (!dryRun) {
+                const accessResult = runCommand('npm', ['access', 'set', 'status=public', pkg.name], { cwd, silent: false });
+                if (accessResult.success) {
+                    console.log(colors.green(`✓ Changed ${pkg.name} to PUBLIC on npm`));
+                    currentAccess = 'public';
+                }
+                else {
+                    console.error(colors.red(`Failed to change access. Try manually: npm access set status=public ${pkg.name}`));
+                    if (!force)
+                        return false;
+                }
+            }
+            else {
+                console.log(colors.dim(`  [dry-run] Would run: npm access set status=public ${pkg.name}`));
+            }
         }
         if (!publicConfirmed) {
             console.log(`Will publish '${pkg.name}' to PUBLIC npm registry.`);
@@ -2677,7 +2725,16 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     const isFirstPublish = !currentAccess;
     if (!currentGitStatus.hasUncommitted && !message && !justInitialized && !isFirstPublish) {
         console.log('');
-        console.log('No changes to commit and no custom message specified.');
+        if (currentAccess === 'public') {
+            console.log(`${pkg.name}@${pkg.version} is already PUBLIC on npm. No changes to publish.`);
+        }
+        else if (currentAccess === 'restricted') {
+            console.log(`${pkg.name}@${pkg.version} is PRIVATE on npm. No changes to publish.`);
+        }
+        else {
+            console.log('No changes to commit and no custom message specified.');
+        }
+        console.log(colors.dim('  Use -m "message" to force a version bump and republish.'));
         // If install/link flag is set, install globally
         if (install || link || wsl) {
             if (verbose) {
@@ -2696,7 +2753,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             if (pkg.bin && (install || link || wsl)) {
                 if (link) {
                     console.log(`Installing ${pkgName} globally from local directory (link)...`);
-                    const localInstallResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false });
+                    const localInstallResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
                     if (localInstallResult.success) {
                         console.log(colors.green(`✓ Linked globally: ${pkgName}@${pkgVersion}`));
                     }
@@ -2722,7 +2779,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                     if (!link && !install)
                         waitForNpmVersion(pkgName, pkgVersion);
                     console.log(`Installing ${pkgName} in WSL${link ? ' (link)' : ' from registry'}...`);
-                    const wslInstallResult = runCommand('wsl', wslArgs, { cwd, silent: false });
+                    const wslInstallResult = runCommand('wsl', wslArgs, { cwd, silent: false, showCommand: true });
                     if (wslInstallResult.success) {
                         console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
                     }
@@ -3194,7 +3251,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         if (link) {
             console.log(`Linking globally: ${pkgName}@${pkgVersion}...`);
             if (!dryRun) {
-                const installResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false });
+                const installResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
                 if (installResult.success) {
                     console.log(colors.green(`✓ Linked globally: ${pkgName}@${pkgVersion}`));
                 }
@@ -3230,7 +3287,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 waitForNpmVersion(pkgName, pkgVersion);
             console.log(`Installing in WSL${link ? ' (link)' : ' from registry'}: ${pkgName}@${pkgVersion}...`);
             if (!dryRun) {
-                const wslResult = runCommand('wsl', wslArgs, { cwd, silent: false });
+                const wslResult = runCommand('wsl', wslArgs, { cwd, silent: false, showCommand: true });
                 if (wslResult.success) {
                     console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
                 }
@@ -3279,6 +3336,9 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     console.log(`  Version: ${colors.green('v' + finalPkg.version)}`);
     console.log(`  Published: ${colors.green('✓')} (${accessLabel})`);
     console.log(`  Git pushed: ${colors.green('✓')}`);
+    if (explicitKeys?.has('gitVisibility')) {
+        console.log(`  Git visibility: ${colors.green(gitVisibility.toUpperCase())}`);
+    }
     if (link) {
         console.log(`  Linked globally: ${colors.green('✓')}`);
     }
@@ -3357,6 +3417,52 @@ export async function globalizeWorkspace(rootDir, options = {}, configOptions =
         console.log(`Filtered to: ${filteredOrder.join(', ')}`);
         console.log('');
     }
+    // Check for visibility mismatches: public packages depending on would-be-private siblings
+    const visibilityMap = new Map();
+    for (const pkgInfo of packages) {
+        const pkgConfig = readConfig(pkgInfo.dir);
+        const vis = pkgConfig.npmVisibility
+            || pkgInfo.pkg.publishConfig?.access
+            || options.npmVisibility; // CLI-level default
+        if (vis === 'public') {
+            visibilityMap.set(pkgInfo.name, 'public');
+        }
+        else if (pkgInfo.pkg.private) {
+            visibilityMap.set(pkgInfo.name, 'private');
+        }
+        else {
+            visibilityMap.set(pkgInfo.name, 'unknown');
+        }
+    }
+    const wsNameSet = new Set(packages.map(p => p.name));
+    for (const pkgInfo of packages) {
+        if (visibilityMap.get(pkgInfo.name) !== 'public')
+            continue;
+        const deps = { ...pkgInfo.pkg.dependencies, ...pkgInfo.pkg.devDependencies };
+        for (const depName of Object.keys(deps || {})) {
+            if (!wsNameSet.has(depName))
+                continue;
+            if (visibilityMap.get(depName) === 'public')
+                continue;
+            const depInfo = packages.find(p => p.name === depName);
+            if (!depInfo)
+                continue;
+            console.log(colors.yellow(`⚠ Public package ${pkgInfo.name} depends on ${depName} which has no public visibility set.`));
+            if (!options.dryRun) {
+                const makePublic = await confirm(`Make ${depName} public too?`, true);
+                if (makePublic) {
+                    const depConfig = readConfig(depInfo.dir);
+                    depConfig.npmVisibility = 'public';
+                    writeConfig(depInfo.dir, depConfig, new Set(['npmVisibility']));
+                    visibilityMap.set(depName, 'public');
+                    console.log(colors.green(`✓ Set ${depName} to public in .globalize.json5`));
+                }
+            }
+            else {
+                console.log(colors.dim(`  [dry-run] Would ask to make ${depName} public`));
+            }
+        }
+    }
     // Process each package in dependency order
     const results = [];
     for (const pkgName of filteredOrder) {

package/package.json

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