@bobfrankston/npmglobalize 1.0.114 → 1.0.115

package/README.md

@@ -165,6 +165,10 @@ npmglobalize --rebase     # Auto-rebase if behind remote
 
 For single-developer projects, this safely pulls remote changes before publishing.
 
+**Failed publish recovery**: If a previous run bumped the version locally but the publish or push failed (e.g., network error, push protection), `npmglobalize` detects this automatically on the next run. It checks whether the current version actually exists on npm — if not, it republishes without re-bumping the version. Unpushed commits from a failed push are also pushed automatically.
+
+**GitHub Push Protection (GH013)**: When GitHub's secret scanning blocks a push, `npmglobalize` detects the error and checks whether the flagged secrets are actually safe. For Google OAuth desktop/installed app credentials (where the "client_secret" is just a public app identifier, not a real secret), it automatically bypasses push protection via the GitHub API (`gh` CLI required) and retries the push. For other secret types, it displays the unblock URLs with guidance.
+
 **Note:** This tool is designed for single-developer, single-branch workflows where automatic rebase and tag cleanup are safe operations.
 
 ### 📂 Git Repository Setup
@@ -390,17 +394,18 @@ npmglobalize --dry-run     # See what would happen
 ## How It Works
 
 1. **Validates** package.json and git status
-2. **Updates dependencies** (if `--update-deps`)
-3. **Publishes file: dependencies** (if needed)
-4. **Backs up** original file: references to `.dependencies`
-5. **Converts** `file:` → npm version references
-6. **Commits** changes
-7. **Bumps** version (using npm version)
-8. **Publishes** to npm
-9. **Pushes** to git
-10. **Installs** globally (if `--install`)
-11. **Restores** file: references (if `--files`, default)
-12. **Runs audit** (shows security status)
+2. **Checks** if current version is on npm (recovers from failed publishes)
+3. **Updates dependencies** (if `--update-deps`)
+4. **Publishes file: dependencies** (if needed)
+5. **Backs up** original file: references to `.dependencies`
+6. **Converts** `file:` → npm version references
+7. **Commits** changes
+8. **Bumps** version (using npm version) — skipped if recovering a failed publish
+9. **Publishes** to npm
+10. **Pushes** to git (with push-protection detection and auto-bypass)
+11. **Installs** globally (if `--install`)
+12. **Restores** file: references (if `--files`, default)
+13. **Runs audit** (shows security status)
 
 ## Operational Details
 

package/lib.js

@@ -3012,103 +3012,127 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     // Check if there are changes to commit or a custom message
     // Skip this check for first publish (currentAccess null) or just-initialized repos
     const isFirstPublish = !currentAccess;
+    let skipVersionBump = false;
     if (!currentGitStatus.hasUncommitted && !message && !justInitialized && !isFirstPublish) {
-        console.log('');
-        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.`);
+        // Check if the current version is actually on npm (it might have failed to publish previously)
+        const versionCheck = spawnSafe('npm', ['view', `${pkg.name}@${pkg.version}`, 'version'], {
+            shell: process.platform === 'win32',
+            stdio: ['pipe', 'pipe', 'pipe'],
+            encoding: 'utf-8'
+        });
+        const versionIsOnNpm = versionCheck.status === 0 && versionCheck.stdout?.trim() === pkg.version;
+        if (!versionIsOnNpm && currentAccess) {
+            // Version not on npm — previous publish/push likely failed
+            // Skip version bump but fall through to publish+push
+            console.log('');
+            console.log(colors.yellow(`${pkg.name}@${pkg.version} is NOT on npm (previous publish may have failed).`));
+            console.log(colors.yellow('Republishing current version...'));
+            skipVersionBump = true;
         }
         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) {
-                console.log('');
-                console.log(link ? 'Installing from local directory (link)...' : 'Installing from registry...');
-            }
-            const pkgName = pkg.name;
-            const pkgVersion = pkg.version;
-            if (!pkg.bin && (install || link || wsl)) {
-                const proceed = await offerAddBin(cwd, pkg);
-                if (!proceed) {
-                    console.log(colors.yellow('Skipping global install — library packages should not be installed globally.'));
-                    console.log(colors.yellow(`To use in projects: npm install ${pkgName}`));
-                }
-            }
-            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, showCommand: true });
-                    if (localInstallResult.success) {
-                        console.log(colors.green(`✓ Linked globally: ${pkgName}@${pkgVersion}`));
-                    }
-                    else {
-                        console.error(colors.red(`✗ Global link install failed`));
-                        console.error(colors.yellow('  Try running manually: npm install -g .'));
+            // Version is on npm — truly nothing to publish
+            console.log('');
+            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.'));
+            // Push any unpushed commits (e.g., from a previous failed push)
+            if (currentGitStatus.hasRemote && currentGitStatus.hasUnpushed && !dryRun) {
+                console.log(colors.yellow('Pushing unpushed commits...'));
+                pushWithProtection(cwd, verbose);
+            }
+            // If install/link flag is set, install globally
+            if (install || link || wsl) {
+                if (verbose) {
+                    console.log('');
+                    console.log(link ? 'Installing from local directory (link)...' : 'Installing from registry...');
+                }
+                const pkgName = pkg.name;
+                const pkgVersion = pkg.version;
+                if (!pkg.bin && (install || link || wsl)) {
+                    const proceed = await offerAddBin(cwd, pkg);
+                    if (!proceed) {
+                        console.log(colors.yellow('Skipping global install — library packages should not be installed globally.'));
+                        console.log(colors.yellow(`To use in projects: npm install ${pkgName}`));
                     }
                 }
-                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 versionOnNpm = vCheck.status === 0 && vCheck.stdout?.trim() === pkgVersion;
-                    if (versionOnNpm) {
-                        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.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}`));
+                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, showCommand: true });
+                        if (localInstallResult.success) {
+                            console.log(colors.green(`✓ Linked globally: ${pkgName}@${pkgVersion}`));
                         }
                         else {
-                            console.error(colors.red(`✗ Local install failed`));
+                            console.error(colors.red(`✗ Global link 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 || (() => {
-                        const vc = spawnSafe('npm', ['view', `${pkgName}@${pkgVersion}`, 'version'], {
+                    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'
                         });
-                        return !(vc.status === 0 && vc.stdout?.trim() === pkgVersion);
-                    })();
-                    const wslArgs = useLocal ? ['npm', 'install', '-g', '.'] : ['npm', 'install', '-g', `${pkgName}@${pkgVersion}`];
-                    if (!useLocal)
-                        waitForNpmVersion(pkgName, pkgVersion);
-                    console.log(`Installing ${pkgName} in WSL${useLocal ? ' (local)' : ' from registry'}...`);
-                    const wslInstallResult = runCommand('wsl', wslArgs, { cwd, silent: false, showCommand: true });
-                    if (wslInstallResult.success) {
-                        console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
+                        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.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 .'));
+                            }
+                        }
                     }
-                    else {
-                        console.error(colors.red(`✗ WSL install failed`));
-                        console.error(colors.yellow('  Try running manually in WSL: npm install -g ' + (useLocal ? '.' : pkgName)));
+                    if (wsl) {
+                        // Check if version is on npm for registry-based WSL install
+                        const useLocal = link || (() => {
+                            const vc = spawnSafe('npm', ['view', `${pkgName}@${pkgVersion}`, 'version'], {
+                                shell: process.platform === 'win32',
+                                stdio: ['pipe', 'pipe', 'pipe'],
+                                encoding: 'utf-8'
+                            });
+                            return !(vc.status === 0 && vc.stdout?.trim() === pkgVersion);
+                        })();
+                        const wslArgs = useLocal ? ['npm', 'install', '-g', '.'] : ['npm', 'install', '-g', `${pkgName}@${pkgVersion}`];
+                        if (!useLocal)
+                            waitForNpmVersion(pkgName, pkgVersion);
+                        console.log(`Installing ${pkgName} in WSL${useLocal ? ' (local)' : ' from registry'}...`);
+                        const wslInstallResult = runCommand('wsl', wslArgs, { cwd, silent: false, showCommand: true });
+                        if (wslInstallResult.success) {
+                            console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
+                        }
+                        else {
+                            console.error(colors.red(`✗ WSL install failed`));
+                            console.error(colors.yellow('  Try running manually in WSL: npm install -g ' + (useLocal ? '.' : pkgName)));
+                        }
                     }
                 }
             }
+            return true;
         }
-        return true;
     }
     // Ensure node_modules is in .gitignore before any git operations
     if (currentGitStatus.isRepo && !dryRun) {
@@ -3199,9 +3223,14 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             console.log('  Pulled latest from remote');
         }
     }
-    // Version bump
-    console.log(`Bumping version (${bump})...`);
-    if (!dryRun) {
+    // Version bump (skip if republishing a version that's already bumped locally)
+    if (skipVersionBump) {
+        console.log(`Skipping version bump — ${pkg.version} already set locally.`);
+    }
+    else {
+        console.log(`Bumping version (${bump})...`);
+    }
+    if (!dryRun && !skipVersionBump) {
         // Temporarily disable postversion hook to prevent double-publishing or push
         // (push is handled separately with push-protection detection)
         const pkg = readPackageJson(cwd);
@@ -3411,7 +3440,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             }
         }
     }
-    else {
+    else if (!skipVersionBump) {
         console.log(`  [dry-run] Would run: npm version ${bump}`);
     }
     // Publish

package/package.json

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