@bobfrankston/npmglobalize 1.0.212 → 1.0.214

package/README.md

@@ -298,10 +298,10 @@ reachable `repository.url` in `package.json`. If found, it offers two paths:
 ```
 How would you like to set up git?
   1) Adopt history from existing remote (recommended)
-  2) Initialize fresh git repository
-  a) Adopt ALL (don't ask again for remaining deps)
-  3) Use local install only (skip git/publish)
-  4) Abort
+  2) Adopt for ALL remaining deps (don't ask again)
+  3) Initialize fresh git repository
+  4) Use local install only (skip git/publish)
+  5) Abort
 ```
 
 **Adopt** runs:
@@ -421,8 +421,8 @@ not what you meant to publish, so npmglobalize lists the conflicted files and st
 -link           Install globally via symlink (npm install -g .)
 -local          Local install only — skip transform/publish, just npm install -g .
                 (persisted; a .commitmsg is still logged to npmchanges.md as "(local)")
--nolocal        Clear a persisted -local in .globalize.json5 — back to transform/publish
-                (alias: -global)
+-global         Clear a persisted -local in .globalize.json5 — back to the normal
+                transform/publish flow (publishes to npm again)
 -wsl            Also install in WSL
 -once           Don't persist flags to .globalize.json5
 ```
@@ -773,7 +773,7 @@ npmglobalize -local      # same as --local
 Some flags are **persisted** to `.globalize.json5` when set from the CLI:
 - `-install`, `-link`, `-wsl`, `-files`, `-fix` — install/build preferences
 - `-np` (noPublish) — once set, prevents accidental publishes
-- `-local` — remembers "this project is local-only" (clear with `-nolocal` / `-global`)
+- `-local` — remembers "this project is local-only" (clear with `-global`)
 - `-git`/`-npm` visibility
 
 Other flags are **one-shot** (never persisted):
@@ -793,7 +793,7 @@ Skip all transform/publish logic and just run `npm install -g .` with `file:` de
 ```bash
 npmglobalize -local          # Install globally from local directory
 npmglobalize -local -wsl     # Also install in WSL
-npmglobalize -nolocal        # Back to the normal transform/publish flow (alias: -global)
+npmglobalize -global         # Back to the normal transform/publish flow (publishes to npm)
 ```
 
 This is useful for:
@@ -802,7 +802,7 @@ This is useful for:
 - Projects with `file:` deps that should stay as-is
 
 `-local` is persisted to `.globalize.json5` (`"local": true`), so a bare
-`npmglobalize` keeps doing local installs until you run `-nolocal` (or edit the
+`npmglobalize` keeps doing local installs until you run `-global` (or edit the
 file). Nothing is committed, bumped, or published, but a `.commitmsg` is still
 consumed: it is appended to `npmchanges.md` under a
 `## v<version> (local) — <YYYY-MM-DD>` header and deleted, so the notes are not

package/cli.js

@@ -70,8 +70,8 @@ Install Options:
   -link          Global install via symlink (npm install -g .)
   -local         Local install only — skip transform/publish, just npm install -g .
                  (persisted; a .commitmsg is still logged to npmchanges.md as "(local)")
-  -nolocal       Clear a persisted -local in .globalize.json5 — back to transform/publish
-                 (alias: -global)
+  -global        Clear a persisted -local in .globalize.json5 — back to the normal
+                 transform/publish flow (publishes to npm again)
   -wsl           Also install globally in WSL
   -freeze        Freeze node_modules (replace symlinks with real copies for network shares)
   -nofreeze      Disable freeze
@@ -157,7 +157,7 @@ Examples:
   npmglobalize -link -wsl            Release + link on Windows and WSL (symlink)
   npmglobalize -local                Install locally with file: deps as-is (no publish)
   npmglobalize -local -wsl           Local install on Windows and WSL
-  npmglobalize -nolocal              Leave local-only mode; publish normally from now on
+  npmglobalize -global               Leave local-only mode; publish to npm from now on
   npmglobalize -np                   Just transform, no publish (remembered in config)
   npmglobalize -cleanup              Restore original dependencies
   npmglobalize -init                 Initialize git (adopt existing remote if any) + release
@@ -233,7 +233,8 @@ function parseArgs(args) {
             // 2026-09-01 16:30 EDT — Claude Code (Opus 5), at Bob's direction. -local
             // persists to .globalize.json5 but had no counterpart, so leaving local-only
             // mode meant hand-editing the config. Same shape as -publish for -np.
-            case '-nolocal':
+            // 2026-09-01 15:00 EDT — Bob: "-global makes more sense, -nolocal is confusing";
+            // -nolocal dropped, -global is the only spelling.
             case '-global':
                 options.local = false;
                 options.explicitKeys.add('local');

package/lib.d.ts

@@ -589,8 +589,14 @@ export declare function validatePackageJson(pkg: any): string[];
 export declare function confirm(message: string, defaultYes?: boolean): Promise<boolean>;
 /** Prompt user for free text input */
 export declare function promptText(message: string, defaultValue?: string): Promise<string>;
-/** Prompt user for multiple choice */
-export declare function promptChoice(message: string, choices: string[]): Promise<string | null>;
+/**
+ * Prompt user for multiple choice. Re-asks until the answer is one of `choices`.
+ *
+ * 2026-09-01 — Claude Code (Fable 5.1), at Bob's direction. Previously an unrecognized
+ * answer resolved null and every caller fell through to its default action, so a typo
+ * silently did something. Now it says what it didn't understand and asks again.
+ */
+export declare function promptChoice(message: string, choices: string[]): Promise<string>;
 /** Initialize a local git repo by adopting history from an existing remote.
  *  Preserves the working tree — local files appear as uncommitted changes
  *  on top of the remote's HEAD. No force-push is required. */

package/lib.js

@@ -445,7 +445,7 @@ export function writeConfig(dir, config, explicitKeys) {
         ['"gitVisibility": "private"', 'Git repo: private or public'],
         ['"npmVisibility": "private"', 'npm package: private or public'],
         ['"fix": false', 'true = auto-run npm audit fix'],
-        ['"local": false', 'true = local install only (skip transform/publish); clear with -nolocal'],
+        ['"local": false', 'true = local install only (skip transform/publish); clear with -global'],
         ['"noPublish": false', 'true = transform but don\'t publish'],
         ['"freeze": false', 'true = freeze node_modules (replace symlinks with real copies)'],
         ['"usePaths": true', 'true = resolve file: deps from siblings; false = use latest npm version (standalone)'],
@@ -4995,23 +4995,46 @@ export async function promptText(message, defaultValue) {
         });
     });
 }
-/** Prompt user for multiple choice */
-export async function promptChoice(message, choices) {
+/**
+ * Prompt user for multiple choice. Re-asks until the answer is one of `choices`.
+ *
+ * 2026-09-01 — Claude Code (Fable 5.1), at Bob's direction. Previously an unrecognized
+ * answer resolved null and every caller fell through to its default action, so a typo
+ * silently did something. Now it says what it didn't understand and asks again.
+ */
+export function promptChoice(message, choices) {
     const rl = readline.createInterface({
         input: process.stdin,
         output: process.stdout
     });
+    const shown = choices.filter(c => c !== '').join(', ');
+    // A 'line' listener rather than repeated rl.question(): when stdin is piped, several
+    // lines can arrive in one chunk and a line landing between two question() calls is
+    // dropped. The listener sees every line.
     return new Promise((resolve) => {
-        rl.question(`${message} `, (answer) => {
+        let done = false;
+        const finish = (value) => {
+            done = true;
             rl.close();
+            resolve(value);
+        };
+        rl.setPrompt(`${message} `);
+        rl.prompt();
+        rl.on('line', (answer) => {
+            if (done)
+                return;
             const a = answer.trim().toLowerCase();
-            if (choices.includes(a)) {
-                resolve(a);
-            }
-            else {
-                resolve(null);
-            }
+            if (choices.includes(a))
+                return finish(a);
+            console.log(colors.yellow(`Unrecognized choice "${a}" — expected one of: ${shown}`));
+            rl.prompt();
         });
+        // EOF with no valid answer: behave as "just pressed Enter" ('' is the default where
+        // a default exists; callers without one treat '' as abort).
+        rl.on('close', () => { if (!done) {
+            done = true;
+            resolve('');
+        } });
     });
 }
 /** Check npm authentication status */
@@ -6053,7 +6076,7 @@ async function doLocalInstall(cwd, options) {
     const pkgVersion = pkg.version || '0.0.0';
     console.log(colors.blue(`Local install: ${pkgName}@${pkgVersion}`));
     console.log(colors.dim('Skipping transform/publish — installing with file: deps as-is'));
-    console.log(colors.dim('  (switch back to the publish flow with -nolocal)'));
+    console.log(colors.dim('  (switch back to the publish flow with -global)'));
     // Record before the bin check: a library's notes are history too.
     recordLocalChange(cwd, pkgVersion, dryRun);
     if (!pkg.bin) {
@@ -6393,51 +6416,60 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             justInitialized = true;
         }
         else if (!init && !options.autoInit && !publishDepsYes) {
+            // 2026-09-01 — Claude Code (Fable 5.1), at Bob's direction. These menus were
+            // numbered "1 a 2 3": the "ALL" option was lettered and sat between 1 and 2, so
+            // a reader who wanted the second line typed 2 and got something else. Bob:
+            // "I type 2 when I mean a because that's what humans do." Options are now
+            // numbered in display order; the ALL variant is 2, directly under the single.
             let choice;
             if (adoptable) {
                 choice = await promptChoice('How would you like to set up git?\n'
                     + '  1) Adopt history from existing remote (recommended)\n'
-                    + '  a) Adopt ALL (don\'t ask again for remaining deps)\n'
-                    + '  2) Initialize fresh git repository\n'
-                    + '  3) Use local install only (skip git/publish)\n'
-                    + '  4) Abort\nChoice:', ['1', 'a', '2', '3', '4', '']);
-                if (choice === '3') {
+                    + '  2) Adopt for ALL remaining deps (don\'t ask again)\n'
+                    + '  3) Initialize fresh git repository\n'
+                    + '  4) Use local install only (skip git/publish)\n'
+                    + '  5) Abort\nChoice [1]:', ['1', '2', '3', '4', '5', '']);
+                if (choice === '4') {
                     console.log(colors.dim('Switching to local-only mode...'));
                     writeConfig(cwd, { ...configOptions, local: true }, new Set(['local']));
                     return doLocalInstall(cwd, options);
                 }
-                if (choice === '4') {
+                if (choice === '5') {
                     console.log('Aborted. Run with --init to initialize.');
                     return false;
                 }
-                if (choice === 'a')
+                if (choice === '2')
                     options.autoInit = true;
-                if (choice === '2') {
+                if (choice === '3') {
                     const success = await initGit(cwd, gitVisibility, dryRun, allowTs);
                     if (!success)
                         return false;
                 }
                 else {
-                    // '1', 'a', '' default → adopt
+                    // '1', '2', '' default → adopt
                     const success = await adoptExistingRemote(cwd, adoptable.url, adoptable.defaultBranch, dryRun);
                     if (!success)
                         return false;
                 }
             }
             else {
-                choice = await promptChoice('No git repository found. What would you like to do?\n  1) Initialize git repository (default)\n  a) Initialize ALL (don\'t ask again for remaining deps)\n  2) Use local install only (skip git/publish)\n  3) Abort\nChoice:', ['1', 'a', '2', '3', '']);
-                if (choice === '2') {
+                choice = await promptChoice('No git repository found. What would you like to do?\n'
+                    + '  1) Initialize git repository\n'
+                    + '  2) Initialize for ALL remaining deps (don\'t ask again)\n'
+                    + '  3) Use local install only (skip git/publish)\n'
+                    + '  4) Abort\nChoice [1]:', ['1', '2', '3', '4', '']);
+                if (choice === '3') {
                     console.log(colors.dim('Switching to local-only mode...'));
                     writeConfig(cwd, { ...configOptions, local: true }, new Set(['local']));
                     return doLocalInstall(cwd, options);
                 }
-                if (choice === '3') {
+                if (choice === '4') {
                     console.log('Aborted. Run with --init to initialize.');
                     return false;
                 }
-                if (choice === 'a')
+                if (choice === '2')
                     options.autoInit = true;
-                // choice is '1', 'a', or '' (default)
+                // choice is '1', '2', or '' (default) → init
                 const success = await initGit(cwd, gitVisibility, dryRun, allowTs);
                 if (!success)
                     return false;
@@ -6463,17 +6495,22 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     else if (!gitStatus.hasRemote) {
         // Git repo exists but no remote - need to create GitHub repo
         if (!init && !options.autoInit && !publishDepsYes) {
-            const choice = await promptChoice('No git remote configured. What would you like to do?\n  1) Create GitHub repository (default)\n  a) Initialize ALL (don\'t ask again for remaining deps)\n  2) Use local install only (skip git/publish)\n  3) Abort\nChoice:', ['1', 'a', '2', '3', '']);
-            if (choice === '2') {
+            // 2026-09-01 — Claude Code (Fable 5.1): renumbered from "1 a 2 3" (see note above).
+            const choice = await promptChoice('No git remote configured. What would you like to do?\n'
+                + '  1) Create GitHub repository\n'
+                + '  2) Create for ALL remaining deps (don\'t ask again)\n'
+                + '  3) Use local install only (skip git/publish)\n'
+                + '  4) Abort\nChoice [1]:', ['1', '2', '3', '4', '']);
+            if (choice === '3') {
                 console.log(colors.dim('Switching to local-only mode...'));
                 writeConfig(cwd, { ...configOptions, local: true }, new Set(['local']));
                 return doLocalInstall(cwd, options);
             }
-            if (choice === '3') {
+            if (choice === '4') {
                 console.log('Aborted. Run with --init to set up GitHub repository.');
                 return false;
             }
-            if (choice === 'a') {
+            if (choice === '2') {
                 options.autoInit = true;
             }
         }

package/package.json

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