@bobfrankston/npmglobalize 1.0.93 → 1.0.95

package/lib.js

@@ -1595,26 +1595,24 @@ export function getToolVersion() {
         return 'unknown';
     }
 }
-/** Offer to add a bin field if missing — returns true if bin was added or already existed */
+/** Offer to add a bin field if missing — returns true if bin was added, false to skip/abort */
 async function offerAddBin(cwd, pkg) {
     if (pkg.bin)
         return true;
     // Determine likely entry point
     const mainFile = pkg.main || 'index.js';
     const cmdName = (pkg.name || path.basename(cwd)).replace(/^@[^/]+\//, '');
-    console.log(colors.yellow('No bin field — this package won\'t install as a CLI command.'));
-    const choice = await promptChoice(`Add bin field to make it a CLI tool?\n  1) Yes, use "${cmdName}" → "${mainFile}" (default)\n  2) No, install as library link\n  3) Abort\nChoice:`, ['1', '2', '3', '']);
-    if (choice === '3')
-        return false;
-    if (choice === '2') {
-        console.log(colors.dim('Installing as library link (no bin)...'));
+    console.log(colors.yellow('No bin field — this is a library, not a CLI tool.'));
+    console.log(colors.yellow('Libraries should not be installed globally. Use npm install <name> in projects instead.'));
+    const choice = await promptChoice(`Add bin field to make it a CLI tool?\n  1) Yes, use "${cmdName}" → "${mainFile}"\n  2) Skip global install (default)\nChoice:`, ['1', '2', '']);
+    if (choice === '1') {
+        pkg.bin = { [cmdName]: mainFile };
+        writePackageJson(cwd, pkg);
+        console.log(colors.green(`✓ Added bin: { "${cmdName}": "${mainFile}" }`));
         return true;
     }
-    // choice is '1' or '' (default)
-    pkg.bin = { [cmdName]: mainFile };
-    writePackageJson(cwd, pkg);
-    console.log(colors.green(`✓ Added bin: { "${cmdName}": "${mainFile}" }`));
-    return true;
+    // choice is '2' or '' (default) — skip
+    return false;
 }
 /** Perform local-only install (npm install -g .) — extracted for reuse from git-init prompts */
 async function doLocalInstall(cwd, options) {
@@ -1626,8 +1624,11 @@ async function doLocalInstall(cwd, options) {
     console.log(colors.dim('Skipping transform/publish — installing with file: deps as-is'));
     if (!pkg.bin) {
         const proceed = await offerAddBin(cwd, pkg);
-        if (!proceed)
-            return false;
+        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}`));
+            return true;
+        }
     }
     if (dryRun) {
         console.log('  [dry-run] Would run: npm install -g .');
@@ -1729,8 +1730,11 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         console.log(colors.dim('Skipping transform/publish — installing with file: deps as-is'));
         if (!pkg.bin) {
             const proceed = await offerAddBin(cwd, pkg);
-            if (!proceed)
-                return false;
+            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}`));
+                return true;
+            }
         }
         if (dryRun) {
             console.log('  [dry-run] Would run: npm install -g .');
@@ -2044,8 +2048,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         console.log(colors.yellow('Continuing with --force despite errors...'));
     }
     // Check npm visibility and current publication status
-    const currentAccess = checkNpmAccess(pkg.name);
-    const isScoped = pkg.name.startsWith('@');
+    let currentAccess = checkNpmAccess(pkg.name);
+    let isScoped = pkg.name.startsWith('@');
     // Check if public intent was explicitly declared anywhere:
     // CLI (--npm public), config file (.globalize.json5), or package.json publishConfig
     const npmVisibilityExplicitlySet = 'npmVisibility' in options
@@ -2102,6 +2106,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             const normalizedScope = scope.startsWith('@') ? scope : `@${scope}`;
             const newName = `${normalizedScope}/${pkg.name}`;
             pkg.name = newName;
+            isScoped = true;
+            currentAccess = checkNpmAccess(newName);
             writePackageJson(cwd, pkg);
             console.log(colors.green(`✓ Renamed package to ${newName}`));
             // Save scope to .userconfig if not already there
@@ -2185,6 +2191,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                             const normalizedScope = scope.startsWith('@') ? scope : `@${scope}`;
                             const newName = `${normalizedScope}/${pkg.name}`;
                             pkg.name = newName;
+                            isScoped = true;
+                            currentAccess = checkNpmAccess(newName);
                             writePackageJson(cwd, pkg);
                             console.log(colors.green(`✓ Renamed package to ${newName}`));
                             if (!ucfg.scope) {

package/package.json

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