@bobfrankston/npmglobalize 1.0.82 → 1.0.84

package/cli.js

@@ -2,7 +2,7 @@
 /**
  * npmglobalize CLI - Transform file: dependencies to npm versions for publishing
  */
-import { globalize, globalizeWorkspace, readConfig, readPackageJson, writeConfig, writePackageJson } from './lib.js';
+import { globalize, globalizeWorkspace, readConfig, readPackageJson, writeConfig, writePackageJson, confirm } from './lib.js';
 import fs from 'fs';
 import path from 'path';
 import { styleText } from 'util';
@@ -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,47 +327,46 @@ 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 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) {
+                // TypeScript project without a build script — offer to add one
+                console.log(styleText('yellow', `TypeScript project has no "build" script in ${cwd}`));
+                const addIt = await confirm('Add "build": "tsc" to package.json?', true);
+                if (addIt) {
+                    if (!pkg.scripts)
+                        pkg.scripts = {};
+                    pkg.scripts.build = 'tsc';
+                    writePackageJson(cwd, pkg);
+                    console.log(styleText('green', '✓ Added "build": "tsc" to package.json'));
                 }
             }
-            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);
+            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...'));
+                }
             }
         }
     }
@@ -414,6 +415,7 @@ export async function main() {
                 }
             }
         }
+        options._fromCli = true;
         const success = await globalize(cwd, options, configOptions);
         process.exit(success ? 0 : 1);
     }

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;
@@ -1418,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.82",
+  "version": "1.0.84",
   "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": [