@bobfrankston/npmglobalize 1.0.81 → 1.0.83

package/README.md

@@ -224,29 +224,36 @@ Publishing requires being on a branch so commits and tags can be properly tracke
 
 ## Command Reference
 
+<!-- NOTE: Keep this in sync with the --help output in cli.ts printHelp().
+     The README expands on options with examples and context;
+     cli.ts is the concise quick-reference. Update both when adding/changing flags. -->
+
 ### Release Options
 ```
---patch         Bump patch version (default: 1.0.0 → 1.0.1)
---minor         Bump minor version (1.0.0 → 1.1.0)
---major         Bump major version (1.0.0 → 2.0.0)
---nopublish, -np  Just transform, don't publish
---cleanup       Restore file: dependencies from backup
+--patch              Bump patch version (default)
+--minor              Bump minor version
+--major              Bump major version
+--nopublish, -np     Just transform, don't publish
+--cleanup            Restore file: dependencies from backup
+-m, --message <msg>  Custom commit message (forces release even without changes)
 ```
 
 ### Dependency Options
 ```
---update-deps      Update package.json to latest versions (safe: minor/patch)
---update-major     Allow major version updates (breaking changes)
---no-publish-deps, -npd  Skip auto-publishing file: dependencies
---force-publish    Republish dependencies even if version exists
---fix              Run npm audit fix after transformation
---no-fix           Don't run npm audit
+--update-deps, -ud        Update package.json to latest versions (safe: minor/patch)
+--update-major            Allow major version updates (breaking changes)
+--no-publish-deps, -npd   Skip auto-publishing file: dependencies
+--force-publish           Republish dependencies even if version exists
+--fix                     Run npm audit fix after transformation
+--no-fix                  Don't run npm audit
 ```
 
 ### Install Options
 ```
---install, -i   Install globally after publish (Windows)
+--install, -i   Install globally after publish (from registry)
+--link          Install globally via symlink (npm install -g .)
 --wsl           Also install in WSL
+--once          Don't persist flags to .globalize.json5
 ```
 
 ### Mode Options
@@ -258,14 +265,23 @@ Publishing requires being on a branch so commits and tags can be properly tracke
 ### Git/npm Visibility
 ```
 --git private   Make git repo private (default)
---git public    Make git repo public
---npm private   Mark package private (skip publish) (default)
+--git public    Make git repo public (requires confirmation)
+--npm private   Mark npm package private (skip publish) (default)
 --npm public    Publish to npm
 ```
 
+### Workspace Options
+```
+-w, --workspace <pkg>  Filter to specific package (repeatable)
+--no-workspace         Disable workspace mode at a workspace root
+--continue-on-error    Continue if a package fails in workspace mode
+```
+
+Workspace mode is auto-detected when run from a root with `"private": true` and a `workspaces` field.
+
 ### Other Options
 ```
---init          Initialize git/npm if needed (creates .gitignore, .npmignore, 
+--init          Initialize git/npm if needed (creates .gitignore, .npmignore,
                 .gitattributes, and configures git for LF line endings)
 --force         Continue despite git errors
 --dry-run       Preview what would happen
@@ -273,13 +289,41 @@ Publishing requires being on a branch so commits and tags can be properly tracke
 --verbose       Show detailed output
 --conform       Update .gitignore/.npmignore/.gitattributes to best practices
                 and configure git for LF line endings (fixes existing repos)
---asis          Skip ignore file checks
+--asis          Skip ignore file checks (or set "asis": true in .globalize.json5)
 --fix-tags      Automatically fix version/tag mismatches
 --rebase        Automatically rebase if local is behind remote
---help, -h      Show help
---version, -v   Show version
+--show          Show package.json dependency changes
+--package, -pkg Update package.json scripts to use npmglobalize (see below)
+-h, --help      Show help
+-v, --version   Show version
+```
+
+## Using in package.json
+
+You can wire npmglobalize into your package.json `scripts` so that `npm run release` handles publishing:
+
+```json
+{
+  "scripts": {
+    "release": "npmglobalize"
+  }
+}
 ```
 
+The `--package` (`-pkg`) flag does this automatically — it adds a `release` script (renaming any existing `release`/`installer` scripts to `old-release`/`old-installer`):
+
+```bash
+npmglobalize --package       # Sets up "release": "npmglobalize" in package.json
+```
+
+After that, publishing is just:
+```bash
+npm run release              # Same as running npmglobalize directly
+npm run release -- --minor   # Pass flags through
+```
+
+You can also combine it with `.globalize.json5` for persistent options so `npm run release` always uses your preferred settings (install, visibility, etc.).
+
 ## Configuration File
 
 Settings can be saved in `.globalize.json5`:
@@ -375,12 +419,21 @@ npmglobalize -np --update-deps
 # Force republish all dependencies
 npmglobalize --force-publish --update-major
 
-# Release + install on Windows and WSL
+# Release + install on Windows and WSL (from registry)
 npmglobalize --install --wsl
 
+# Release + link on Windows and WSL (symlink)
+npmglobalize --link --wsl
+
 # Restore original file: references
 npmglobalize --cleanup
 
+# Initialize new git repo + release
+npmglobalize --init
+
+# Migrate package.json scripts to use npmglobalize
+npmglobalize --package
+
 # Preview what would happen
 npmglobalize --dry-run --verbose
 ```

package/cli.js

@@ -295,6 +295,10 @@ function parseArgs(args) {
     return options;
 }
 export async function main() {
+    // Show version at the very start
+    const ownPkgPath = path.join(__dirname, 'package.json');
+    const ownPkg = JSON.parse(fs.readFileSync(ownPkgPath, 'utf-8'));
+    console.log(styleText('cyan', `npmglobalize v${ownPkg.version}`));
     const args = process.argv.slice(2);
     const cliOptions = parseArgs(args);
     if (cliOptions.help) {
@@ -302,9 +306,7 @@ export async function main() {
         process.exit(0);
     }
     if (cliOptions.version) {
-        const pkgPath = path.join(__dirname, 'package.json');
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
-        console.log(pkg.version);
+        // Already printed above, just exit
         process.exit(0);
     }
     if (cliOptions.error) {
@@ -325,48 +327,35 @@ export async function main() {
             process.exit(1);
         }
     }
-    // Check if TypeScript files in target project are compiled
-    if (!cliOptions.force) {
-        let noEmit = false;
+    // Build the target project if it has a build script and tsconfig (not noEmit)
+    if (!cliOptions.cleanup) {
+        let shouldBuild = false;
         try {
             const tsconfigPath = path.join(cwd, 'tsconfig.json');
             const content = fs.readFileSync(tsconfigPath, 'utf-8');
             const tsconfig = JSON5.parse(content);
-            noEmit = tsconfig.compilerOptions?.noEmit === true;
+            shouldBuild = tsconfig.compilerOptions?.noEmit !== true;
         }
         catch (error) {
-            // No tsconfig or can't parse — skip check
-            noEmit = true;
+            // No tsconfig — skip build
         }
-        if (!noEmit) {
-            const outOfSync = [];
-            const jsFiles = fs.readdirSync(cwd).filter(f => f.endsWith('.js') && !f.endsWith('.min.js'));
-            for (const jsFile of jsFiles) {
-                const tsFile = jsFile.replace(/\.js$/, '.ts');
-                const tsPath = path.join(cwd, tsFile);
-                const jsPath = path.join(cwd, jsFile);
-                if (fs.existsSync(tsPath)) {
-                    const tsStat = fs.statSync(tsPath);
-                    const jsStat = fs.statSync(jsPath);
-                    if (jsStat.mtime < tsStat.mtime) {
-                        outOfSync.push(tsFile);
+        if (shouldBuild) {
+            const pkg = JSON.parse(fs.readFileSync(path.join(cwd, 'package.json'), 'utf-8'));
+            if (pkg.scripts?.build) {
+                const { execSync } = await import('child_process');
+                try {
+                    console.log(`Building ${cwd}...`);
+                    execSync('npm run build', { cwd, stdio: 'inherit' });
+                    console.log(styleText('green', '✓ Build succeeded'));
+                }
+                catch (error) {
+                    console.error(styleText('red', `Build failed in ${cwd}`));
+                    if (!cliOptions.force) {
+                        process.exit(1);
                     }
+                    console.log(styleText('yellow', 'Continuing with --force...'));
                 }
             }
-            if (outOfSync.length > 0) {
-                console.error('\n' + styleText('red', '❌ Build check failed:'));
-                console.error(styleText('red', `   ${outOfSync.length} file(s) need compilation in ${cwd}:`));
-                outOfSync.forEach(f => {
-                    const tsPath = path.join(cwd, f);
-                    const jsPath = path.join(cwd, f.replace('.ts', '.js'));
-                    const fmt = (d) => d.toLocaleString();
-                    console.error(styleText('red', `     - ${f}`));
-                    console.error(styleText('red', `       .ts: ${fmt(fs.statSync(tsPath).mtime)}  .js: ${fmt(fs.statSync(jsPath).mtime)}`));
-                });
-                console.error('\n' + styleText('yellow', '   Please run: tsc'));
-                console.error(styleText('yellow', '   Or use --force to skip this check') + '\n');
-                process.exit(1);
-            }
         }
     }
     // Handle --package: update scripts in target package.json

package/lib.d.ts

@@ -73,6 +73,8 @@ export interface GlobalizeOptions {
     once?: boolean;
     /** Internal: signals this call is from workspace orchestrator */
     _fromWorkspace?: boolean;
+    /** Internal: signals this call is from CLI (version already printed) */
+    _fromCli?: boolean;
 }
 /** Result from a single package in workspace mode */
 interface WorkspacePackageResult {

package/lib.js

@@ -23,6 +23,7 @@ const colors = {
     yellow: (text) => styleText('yellow', text),
     green: (text) => styleText('green', text),
     italic: (text) => styleText('italic', text),
+    blue: (text) => styleText('blue', text),
     dim: (text) => styleText('dim', text),
 };
 /**
@@ -541,6 +542,7 @@ export function transformDeps(pkg, baseDir, verbose = false, forcePublish = fals
         for (const [name, value] of Object.entries(pkg[key])) {
             if (isFileRef(value)) {
                 const targetPath = resolveFilePath(value, baseDir);
+                console.log(colors.blue(`  + ${name} → ${value}`));
                 try {
                     const targetPkg = readPackageJson(targetPath);
                     const targetVersion = targetPkg.version;
@@ -744,9 +746,8 @@ function waitForNpmVersion(pkgName, version, maxWaitMs = 60000) {
 export function runCommand(cmd, args, options = {}) {
     const { silent = false, cwd } = options;
     try {
-        // Only use shell:true for npm commands which need it on Windows
-        // Git and other commands work fine without shell
-        const needsShell = cmd === 'npm' || cmd === 'npm.cmd';
+        // Use shell:true for npm/gh commands which need it on Windows
+        const needsShell = cmd === 'npm' || cmd === 'npm.cmd' || cmd === 'gh';
         // Debug: log the command being executed
         if (!silent) {
             console.log(colors.dim(`[DEBUG] Running: ${cmd} ${args.join(' ')}`));
@@ -803,13 +804,13 @@ function getGitHubRepo(pkg) {
 }
 /** Run a command and throw on failure */
 export function runCommandOrThrow(cmd, args, options = {}) {
-    // For commands that should throw, we need to capture output, so use silent=true
-    const captureOptions = { ...options, silent: true };
+    const needsShell = cmd === 'npm' || cmd === 'npm.cmd' || cmd === 'gh';
     const result = spawnSync(cmd, args, {
         encoding: 'utf-8',
         stdio: 'pipe',
-        cwd: captureOptions.cwd,
-        shell: false
+        cwd: options.cwd,
+        env: process.env,
+        shell: needsShell
     });
     const stdout = result.stdout || '';
     const stderr = result.stderr || '';
@@ -1419,9 +1420,9 @@ export function getToolVersion() {
 export async function globalize(cwd, options = {}, configOptions = {}) {
     const { bump = 'patch', noPublish = false, cleanup = false, install = false, link = false, wsl = false, force = false, files = true, dryRun = false, quiet = true, verbose = false, init = false, gitVisibility = 'private', npmVisibility = 'private', message, conform = false, asis = false, updateDeps = false, updateMajor = false, publishDeps = true, // Default to publishing deps for safety
     forcePublish = false, fix = false, fixTags = false, rebase = false, show = false } = options;
-    // Show tool version (skip when called from workspace orchestrator)
+    // Show tool version only for recursive dep calls (CLI already prints it at startup)
     const toolVersion = getToolVersion();
-    if (!options._fromWorkspace) {
+    if (!options._fromWorkspace && !options._fromCli) {
         console.log(colors.italic(`npmglobalize v${toolVersion}`));
     }
     // Show settings from .globalize.json5 if any

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/npmglobalize",
-  "version": "1.0.81",
+  "version": "1.0.83",
   "description": "Transform file: dependencies to npm versions for publishing",
   "main": "index.js",
   "type": "module",
@@ -10,7 +10,7 @@
   "scripts": {
     "build": "tsc",
     "watch": "tsc -w",
-    "prepublishOnly": "npm run build",
+    "prepare": "npm run build",
     "check-links": "node check-links.js"
   },
   "keywords": [