@bobfrankston/npmglobalize 1.0.165 → 1.0.167

package/cli.js

@@ -24,6 +24,7 @@ Release Options:
   -minor               Bump minor version
   -major               Bump major version
   -nopublish, -np      Just transform, don't publish (persisted to config)
+  -publish             Clear a persisted -nopublish in .globalize.json5
   -cleanup             Restore from .dependencies
   -m, -message <msg>   Custom commit message (forces release even without changes)
                        If -m not given and a .commitmsg file exists in cwd, its
@@ -158,6 +159,10 @@ function parseArgs(args) {
                 options.noPublish = true;
                 options.explicitKeys.add('noPublish');
                 break;
+            case '-publish':
+                options.noPublish = false;
+                options.explicitKeys.add('noPublish');
+                break;
             case '-cleanup':
                 options.cleanup = true;
                 break;

package/lib.js

@@ -3143,8 +3143,6 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             settings.push('--dry-run');
         if (configOptions.bump)
             settings.push(`--bump ${configOptions.bump}`);
-        if (configOptions.noPublish)
-            settings.push('--no-publish');
         if (configOptions.cleanup)
             settings.push('--cleanup');
         if (configOptions.wsl)
@@ -3172,6 +3170,9 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         if (settings.length > 0) {
             console.log(colors.dim(`Settings from .globalize.json5: ${settings.join(', ')}`));
         }
+        if (configOptions.noPublish) {
+            console.log(colors.yellow(`!  noPublish=true in .globalize.json5 — transform only, no npm publish (clear with -publish)`));
+        }
     }
     console.log('');
     // -local: skip all transform/publish, just install from local directory with file: deps intact
@@ -3526,10 +3527,14 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     }
     // Read package.json
     const pkg = readPackageJson(cwd);
+    // Ensure node_modules are present in this dir and every transitive file: dep
+    // before doing anything else. Independent of build script presence — a dep
+    // with new declared deps still needs its own `npm install` even if it has
+    // no build step. CLI entrypoint already ran this, so skip when _fromCli.
+    if (!options._fromCli && !dryRun)
+        ensureFileDepModules(cwd, verbose);
     // Run build step if package.json has a build script (skip if CLI already built)
     if (pkg.scripts?.build && !options._fromCli) {
-        if (!dryRun)
-            ensureFileDepModules(cwd, verbose);
         console.log(`${timestamp()} Running build...`);
         if (!dryRun) {
             // For workspace roots whose build invokes `--workspaces`, npm uses
@@ -4147,6 +4152,36 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 }
             }
         }
+        // Highlighted summary so noPublish (and other surprises) don't slip past unnoticed
+        const finalPkg2 = readPackageJson(cwd);
+        const surprises = [];
+        const fromConfig = (k) => k in configOptions ? ' (from .globalize.json5)' : '';
+        surprises.push(`noPublish=true${fromConfig('noPublish')} — nothing was published; clear with -publish`);
+        if (configOptions.forcePublish || forcePublish)
+            surprises.push(`forcePublish=true${fromConfig('forcePublish')}`);
+        if (configOptions.publishDeps === false || publishDeps === false)
+            surprises.push(`publishDeps=false${fromConfig('publishDeps')} — file: deps NOT republished`);
+        if (configOptions.usePaths === false || usePaths === false)
+            surprises.push(`usePaths=false${fromConfig('usePaths')} — file: deps resolved from npm, not siblings`);
+        if (configOptions.freeze || freeze)
+            surprises.push(`freeze=true${fromConfig('freeze')}`);
+        if (configOptions.local || local)
+            surprises.push(`local=true${fromConfig('local')}`);
+        if (configOptions.files === false || files === false)
+            surprises.push(`files=false${fromConfig('files')} — npm versions left in package.json`);
+        if (configOptions.allowTs === true || allowTs === true)
+            surprises.push(`allowTs=true${fromConfig('allowTs')} — .ts source kept in tarball`);
+        console.log('');
+        console.log(colors.yellow('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'));
+        console.log(colors.yellow('!  Transform Summary  (NO PUBLISH)'));
+        console.log(colors.yellow('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'));
+        console.log(`  Package: ${colors.green(finalPkg2.name)}`);
+        console.log(`  Version: ${colors.green('v' + finalPkg2.version)}`);
+        console.log(`  Published: ${colors.red('✗')}`);
+        for (const s of surprises)
+            console.log(`  ${colors.yellow('!')} ${s}`);
+        console.log(colors.yellow('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'));
+        console.log('');
         return true;
     }
     // Skip if private
@@ -5208,6 +5243,23 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     if (files && transformResult.transformed) {
         console.log(`  Restored file: deps: ${colors.green('✓')}`);
     }
+    // Surface non-default settings the user may have forgotten about
+    {
+        const surprises = [];
+        const fromConfig = (k) => k in configOptions ? ' (from .globalize.json5)' : '';
+        if (configOptions.forcePublish || forcePublish)
+            surprises.push(`forcePublish=true${fromConfig('forcePublish')}`);
+        if (configOptions.publishDeps === false || publishDeps === false)
+            surprises.push(`publishDeps=false${fromConfig('publishDeps')} — file: deps NOT republished`);
+        if (configOptions.usePaths === false || usePaths === false)
+            surprises.push(`usePaths=false${fromConfig('usePaths')} — file: deps resolved from npm, not siblings`);
+        if (configOptions.files === false || files === false)
+            surprises.push(`files=false${fromConfig('files')} — npm versions left in package.json`);
+        if (configOptions.allowTs === true || allowTs === true)
+            surprises.push(`allowTs=true${fromConfig('allowTs')} — .ts source kept in tarball`);
+        for (const s of surprises)
+            console.log(`  ${colors.yellow('!')} ${s}`);
+    }
     console.log(colors.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'));
     console.log('');
     // Only show "To use run" message if package provides commands (has bin field)

package/package.json

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