@bobfrankston/npmglobalize 1.0.140 → 1.0.142

package/cli.js

@@ -2,7 +2,7 @@
 /**
  * npmglobalize CLI - Transform file: dependencies to npm versions for publishing
  */
-import { globalize, globalizeWorkspace, readConfig, readPackageJson, readUserNpmConfig, writeConfig, writePackageJson, confirm } from './lib.js';
+import { globalize, globalizeWorkspace, readConfig, readPackageJson, readUserNpmConfig, writeConfig, writePackageJson, confirm, getBuildIssues, clearBuildIssues } from './lib.js';
 import fs from 'fs';
 import path from 'path';
 import { styleText } from 'util';
@@ -328,6 +328,20 @@ function parseArgs(args) {
     }
     return options;
 }
+/** Print accumulated build issues in warning color */
+function printBuildSummary() {
+    const issues = getBuildIssues();
+    if (issues.length === 0)
+        return;
+    console.log('');
+    console.log(styleText('yellow', '━━━ Issues Summary ━━━━━━━━━━━━━━━━━━━━'));
+    for (const issue of issues) {
+        const icon = issue.severity === 'error' ? '✗' : '⚠';
+        console.log(styleText('yellow', `  ${icon} ${issue.module}: ${issue.message}`));
+    }
+    console.log(styleText('yellow', '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'));
+    console.log('');
+}
 export async function main() {
     // Show version at the very start
     const ownPkgPath = path.join(__dirname, 'package.json');
@@ -444,6 +458,7 @@ export async function main() {
         }
         writeConfig(cwd, persistable, cliOptions.explicitKeys);
     }
+    clearBuildIssues();
     try {
         // Auto-detect workspace root: private package with workspaces[] field
         if (!options.noWorkspace) {
@@ -452,12 +467,14 @@ export async function main() {
                 const rootPkg = readPackageJson(cwd);
                 if (rootPkg.private && Array.isArray(rootPkg.workspaces)) {
                     const result = await globalizeWorkspace(cwd, options, configOptions);
+                    printBuildSummary();
                     process.exit(result.success ? 0 : 1);
                 }
             }
         }
         options._fromCli = true;
         const success = await globalize(cwd, options, configOptions);
+        printBuildSummary();
         process.exit(success ? 0 : 1);
     }
     catch (error) {
@@ -465,6 +482,7 @@ export async function main() {
         if (options.verbose) {
             console.error(error.stack);
         }
+        printBuildSummary();
         process.exit(1);
     }
 }

package/lib.d.ts

@@ -10,6 +10,18 @@
  * Consider library-based approach if async operations or cross-platform issues arise.
  */
 import { type NpmCommonConfig } from '@bobfrankston/userconfig';
+/** Issue recorded during a build/publish run for end-of-run summary */
+export interface BuildIssue {
+    module: string;
+    severity: 'error' | 'warning';
+    message: string;
+}
+/** Record an issue for the end-of-run summary */
+export declare function recordBuildIssue(module: string, severity: 'error' | 'warning', message: string): void;
+/** Get all accumulated build issues */
+export declare function getBuildIssues(): readonly BuildIssue[];
+/** Clear accumulated issues (call at start of run) */
+export declare function clearBuildIssues(): void;
 /** Options for the globalize operation */
 export interface GlobalizeOptions {
     /** Bump type: patch (default), minor, major */

package/lib.js

@@ -33,6 +33,19 @@ import { fileURLToPath } from 'url';
 import { themeColors } from '@bobfrankston/themecolors';
 /** Semantic color functions — adapts to terminal light/dark theme */
 const colors = themeColors();
+const _buildIssues = [];
+/** Record an issue for the end-of-run summary */
+export function recordBuildIssue(module, severity, message) {
+    _buildIssues.push({ module, severity, message });
+}
+/** Get all accumulated build issues */
+export function getBuildIssues() {
+    return _buildIssues;
+}
+/** Clear accumulated issues (call at start of run) */
+export function clearBuildIssues() {
+    _buildIssues.length = 0;
+}
 /**
  * Remove 'nul' files from a directory tree (Windows reserved name issue).
  * These files break git and npm on Windows. Uses \\?\ prefix to bypass name validation.
@@ -1072,6 +1085,96 @@ export function runCommand(cmd, args, options = {}) {
         return { success: false, output: '', stderr: error.message };
     }
 }
+/** Diagnose common build/version failure patterns and print actionable hints.
+ *  Returns true if a diagnosis was printed. */
+function diagnoseBuildFailure(errorText, cwd) {
+    let diagnosed = false;
+    // Pattern: missing module / package not found
+    const missingPkg = errorText.match(/Cannot find (?:package|module) ['"](@?[^'"\/]+(?:\/[^'"\/]+)?)/);
+    if (missingPkg) {
+        const pkgName = missingPkg[1];
+        console.error(colors.yellow(`\n  Hint: missing package "${pkgName}"`));
+        // Check for broken symlinks in node_modules tree
+        const searchDirs = [cwd];
+        try {
+            for (const entry of fs.readdirSync(cwd, { withFileTypes: true })) {
+                if (entry.isDirectory() && entry.name !== 'node_modules' && entry.name !== '.git' && entry.name !== 'prev') {
+                    searchDirs.push(path.join(cwd, entry.name));
+                }
+            }
+        }
+        catch { /* ignore */ }
+        for (const dir of searchDirs) {
+            const nmPath = path.join(dir, 'node_modules', ...pkgName.split('/'));
+            try {
+                const stat = fs.lstatSync(nmPath);
+                if (stat.isSymbolicLink()) {
+                    const target = fs.readlinkSync(nmPath);
+                    const resolved = path.resolve(path.dirname(nmPath), target);
+                    if (!fs.existsSync(resolved)) {
+                        console.error(colors.yellow(`  Broken symlink: ${nmPath}`));
+                        console.error(colors.yellow(`    → ${resolved} (target missing)`));
+                        console.error(colors.yellow(`  Fix: cd "${dir}" && npm install`));
+                        diagnosed = true;
+                    }
+                    else {
+                        console.error(colors.yellow(`  Symlink exists at ${nmPath} but module still failed to load.`));
+                        console.error(colors.yellow(`  Check that the target package has been built (has .js files).`));
+                        diagnosed = true;
+                    }
+                }
+            }
+            catch {
+                // No symlink found at this location — check if it should exist
+                const subPkgPath = path.join(dir, 'package.json');
+                if (fs.existsSync(subPkgPath)) {
+                    try {
+                        const subPkg = JSON.parse(fs.readFileSync(subPkgPath, 'utf-8'));
+                        const allDeps = { ...subPkg.dependencies, ...subPkg.devDependencies };
+                        if (allDeps[pkgName]) {
+                            const depValue = allDeps[pkgName];
+                            if (typeof depValue === 'string' && depValue.startsWith('file:')) {
+                                const target = path.resolve(dir, depValue.replace('file:', ''));
+                                console.error(colors.yellow(`  "${pkgName}" is a file: dep in ${path.relative(cwd, subPkgPath) || 'package.json'}`));
+                                console.error(colors.yellow(`    → ${target}`));
+                                if (!fs.existsSync(target)) {
+                                    console.error(colors.yellow(`    Target directory does not exist!`));
+                                }
+                                else {
+                                    console.error(colors.yellow(`  Fix: cd "${dir}" && npm install`));
+                                }
+                                diagnosed = true;
+                            }
+                            else {
+                                console.error(colors.yellow(`  "${pkgName}" is listed in ${path.relative(cwd, subPkgPath) || 'package.json'} but not installed.`));
+                                console.error(colors.yellow(`  Fix: cd "${dir}" && npm install`));
+                                diagnosed = true;
+                            }
+                        }
+                    }
+                    catch { /* ignore parse errors */ }
+                }
+            }
+        }
+        if (!diagnosed) {
+            console.error(colors.yellow(`  "${pkgName}" is not installed and not listed as a dependency.`));
+            console.error(colors.yellow(`  If needed, add it: npm install ${pkgName}`));
+            diagnosed = true;
+        }
+    }
+    // Pattern: permission denied
+    if (!diagnosed && /permission denied|EPERM|EACCES/.test(errorText)) {
+        console.error(colors.yellow('\n  Hint: permission error during build.'));
+        console.error(colors.yellow('  Check file ownership and that no other process has files locked.'));
+        diagnosed = true;
+    }
+    // Pattern: TypeScript compilation error
+    if (!diagnosed && /error TS\d+/.test(errorText)) {
+        console.error(colors.yellow('\n  Hint: TypeScript compilation errors. Fix the type errors above before publishing.'));
+        diagnosed = true;
+    }
+    return diagnosed;
+}
 /** Run git commit silently and print a compact summary (no per-file create/delete mode lines). */
 function gitCommit(msg, cwd) {
     const result = runCommand('git', ['commit', '-m', msg], { cwd, silent: true });
@@ -2646,6 +2749,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             const buildResult = runCommand('npm', ['run', 'build'], { cwd, silent: !verbose });
             if (!buildResult.success) {
                 console.error(colors.red('ERROR: Build failed:'), buildResult.stderr || buildResult.output);
+                diagnoseBuildFailure(buildResult.stderr || buildResult.output || '', cwd);
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'Build failed');
                 if (!force) {
                     return false;
                 }
@@ -3052,6 +3157,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                         });
                         if (!depSuccess) {
                             console.error(colors.red(`Failed to publish ${name}`));
+                            recordBuildIssue(pkg.name || cwd, 'error', `Dependency failed: ${name}`);
                             if (!force) {
                                 return false;
                             }
@@ -3551,6 +3657,9 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 if (error.code)
                     console.error('  exit code:', error.code);
             }
+            // Diagnose common failures (missing modules, permissions, TS errors)
+            const versionErrorText = [error.message, error.stderr, error.stdout].filter(Boolean).join('\n');
+            diagnoseBuildFailure(versionErrorText, cwd);
             // Show the full error stack in verbose mode
             if (verbose && error.stack) {
                 console.error('  Stack trace:', error.stack);
@@ -3777,6 +3886,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 }
                 console.error(colors.yellow('Consumers will get E404 when installing this package.'));
                 console.error(colors.yellow('Fix: publish those deps as public, or make this package private.'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'warning', `Public pkg depends on private: ${privateDeps.join(', ')}`);
                 console.log('');
             }
         }
@@ -3788,6 +3898,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         const authStatus = checkNpmAuth();
         if (!authStatus.authenticated) {
             console.error(colors.red(`Not authenticated to npm (${authStatus.error}) — run: npm login`));
+            recordBuildIssue(pkg.name || path.basename(cwd), 'error', `npm auth: ${authStatus.error}`);
             return false;
         }
         if (verbose) {
@@ -3799,6 +3910,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             console.error(colors.red('ERROR: Failed to create package tarball'));
             console.error(colors.yellow('Output:'), packResult.output);
             console.error(colors.yellow('Error:'), packResult.stderr);
+            recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm pack failed');
             return false;
         }
         // Get the tarball filename from npm pack output
@@ -3831,6 +3943,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                     }
                 }
                 console.error(colors.yellow('  Check .npmignore — you may need to exclude large files or directories.'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', `Tarball too large (${sizeMB.toFixed(0)}MB)`);
                 // Clean up and abort
                 try {
                     fs.unlinkSync(tarballPath);
@@ -3891,6 +4004,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         }
         if (!publishResult.success) {
             console.error(colors.red('\nERROR: npm publish failed\n'));
+            recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
             // Check for specific error types
             const output = (publishResult.output + '\n' + publishResult.stderr).toLowerCase();
             if (output.includes('err_string_too_long') || output.includes('string longer than')) {
@@ -3992,6 +4106,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 else {
                     console.error(colors.red(`✗ Global link install failed`));
                     console.error(colors.yellow('  Try running manually: npm install -g .'));
+                    recordBuildIssue(pkgName, 'warning', 'Global link install failed');
                 }
             }
             else {
@@ -4011,6 +4126,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                     else {
                         console.error(colors.red(`✗ Global install failed`));
                         console.error(colors.yellow('  Try running manually: npm install -g .'));
+                        recordBuildIssue(pkgName, 'warning', 'Global install failed');
                     }
                 }
                 else {
@@ -4029,6 +4145,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                     else {
                         console.error(colors.red(`✗ Global install failed`));
                         console.error(colors.yellow(`  Try running manually: npm install -g ${pkgName}@${pkgVersion}`));
+                        recordBuildIssue(pkgName, 'warning', 'Global install failed');
                     }
                 }
                 else {
@@ -4050,6 +4167,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 }
                 else {
                     console.error(colors.yellow('✗ WSL install failed (is npm installed in WSL?)'));
+                    recordBuildIssue(pkgName, 'warning', 'WSL install failed');
                 }
             }
             else {
@@ -4277,6 +4395,7 @@ export async function globalizeWorkspace(rootDir, options = {}, configOptions =
                 success: false,
                 error: error.message,
             });
+            recordBuildIssue(pkgName, 'error', error.message);
             console.error(colors.red(`\nError processing ${pkgName}: ${error.message}`));
             if (!options.continueOnError) {
                 console.error(colors.red('Use --continue-on-error to continue with remaining packages.'));
@@ -4359,6 +4478,7 @@ export async function globalizeWorkspace(rootDir, options = {}, configOptions =
                 else {
                     console.error(colors.red(`✗ Global install failed`));
                     console.error(colors.yellow('  Try running manually: npm install -g .'));
+                    recordBuildIssue(pkgName, 'warning', 'Global install failed');
                 }
             }
             else {
@@ -4374,6 +4494,7 @@ export async function globalizeWorkspace(rootDir, options = {}, configOptions =
                 }
                 else {
                     console.error(colors.yellow('✗ WSL install failed (is npm installed in WSL?)'));
+                    recordBuildIssue(pkgName, 'warning', 'WSL install failed');
                 }
             }
             else {

package/package.json

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