@bobfrankston/npmglobalize 1.0.29 → 1.0.31

package/lib.js

@@ -1520,11 +1520,35 @@ export async function globalize(cwd, options = {}) {
     // Version bump
     console.log(`Bumping version (${bump})...`);
     if (!dryRun) {
+        // Temporarily disable postversion hook to prevent double-publishing
+        const pkg = readPackageJson(cwd);
+        const originalPostversion = pkg.scripts?.postversion;
+        let postversDisabled = false;
+        if (originalPostversion && originalPostversion.includes('npm publish')) {
+            if (verbose) {
+                console.log('Temporarily disabling postversion hook to prevent double-publish...');
+            }
+            delete pkg.scripts.postversion;
+            writePackageJson(cwd, pkg);
+            postversDisabled = true;
+        }
         try {
             // Use libnpmversion - it will create commit and tag automatically
             const newVersion = await libversion(bump, {
-                path: cwd
+                path: cwd,
+                'git-tag-version': true
             });
+            // Restore postversion hook
+            if (postversDisabled && originalPostversion) {
+                const updatedPkg = readPackageJson(cwd);
+                if (!updatedPkg.scripts)
+                    updatedPkg.scripts = {};
+                updatedPkg.scripts.postversion = originalPostversion;
+                writePackageJson(cwd, updatedPkg);
+                if (verbose) {
+                    console.log('Restored postversion hook');
+                }
+            }
             console.log(colors.green(`Version bumped to ${newVersion}`));
         }
         catch (error) {
@@ -1726,11 +1750,11 @@ export async function globalize(cwd, options = {}) {
                 console.error(colors.green('Solution: Wait a few more minutes and try again'));
                 console.error('');
             }
-            else if (output.includes('403') || output.includes('forbidden')) {
+            else if (output.includes('403') || output.includes('forbidden') || output.includes('cannot publish over')) {
                 // Check if it's the "version already published" case
                 if (output.includes('cannot publish over') || output.includes('previously published')) {
                     // Extract the version number from the error if possible
-                    const versionMatch = output.match(/versions?:\s*(\d+\.\d+\.\d+)/);
+                    const versionMatch = output.match(/versions?[:\s]+(\d+\.\d+\.\d+)/);
                     const failedVersion = versionMatch ? versionMatch[1] : null;
                     // Check if this is the version we just bumped to
                     const currentPkgVersion = readPackageJson(cwd).version;
@@ -1745,6 +1769,9 @@ export async function globalize(cwd, options = {}) {
                         console.error('');
                         console.error('Verify with: ' + colors.yellow(`npm view ${pkg.name}@${currentPkgVersion}`));
                         console.error('');
+                        console.error(colors.yellow('Note: You may have a postversion script that auto-publishes.'));
+                        console.error(colors.yellow('npmglobalize will now skip that hook to prevent this issue.'));
+                        console.error('');
                         // Treat as success and continue
                         console.log(colors.green('✓ Treating as successful publish'));
                         // Don't return false - let it continue to push git tags

package/package.json

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