@bobfrankston/npmglobalize 1.0.142 → 1.0.143

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, getBuildIssues, clearBuildIssues } from './lib.js';
+import { globalize, globalizeWorkspace, readConfig, readPackageJson, readUserNpmConfig, writeConfig, writePackageJson, confirm, getBuildIssues, clearBuildIssues, recordBuildIssue, extractFirstTscError } from './lib.js';
 import fs from 'fs';
 import path from 'path';
 import { styleText } from 'util';
@@ -402,15 +402,23 @@ export async function main() {
                 }
             }
             if (pkg.scripts?.build) {
-                const { execSync } = await import('child_process');
-                try {
-                    console.log(`Building ${cwd}...`);
-                    execSync('npm run build', { cwd, stdio: 'inherit' });
+                const { spawnSync } = await import('child_process');
+                console.log(`Building ${cwd}...`);
+                const buildResult = spawnSync('npm', ['run', 'build'], {
+                    cwd, encoding: 'utf-8', stdio: 'pipe', shell: true
+                });
+                if (buildResult.status === 0) {
                     console.log(styleText('green', '✓ Build succeeded'));
                 }
-                catch (error) {
+                else {
+                    const buildOutput = (buildResult.stderr || '') + (buildResult.stdout || '');
+                    if (buildOutput)
+                        console.error(buildOutput);
                     console.error(styleText('red', `Build failed in ${cwd}`));
+                    const firstErr = extractFirstTscError(buildOutput);
+                    recordBuildIssue(pkg.name || path.basename(cwd), 'error', firstErr || 'Build failed');
                     if (!cliOptions.force) {
+                        printBuildSummary();
                         process.exit(1);
                     }
                     console.log(styleText('yellow', 'Continuing with --force...'));

package/lib.d.ts

@@ -22,6 +22,9 @@ export declare function recordBuildIssue(module: string, severity: 'error' | 'wa
 export declare function getBuildIssues(): readonly BuildIssue[];
 /** Clear accumulated issues (call at start of run) */
 export declare function clearBuildIssues(): void;
+/** Extract the first TypeScript error line from build output for the summary.
+ *  Returns a short string like "file.ts(42,5): error TS2339: Property 'foo' ..." */
+export declare function extractFirstTscError(output: string): string | null;
 /** Options for the globalize operation */
 export interface GlobalizeOptions {
     /** Bump type: patch (default), minor, major */

package/lib.js

@@ -46,6 +46,25 @@ export function getBuildIssues() {
 export function clearBuildIssues() {
     _buildIssues.length = 0;
 }
+/** Extract the first TypeScript error line from build output for the summary.
+ *  Returns a short string like "file.ts(42,5): error TS2339: Property 'foo' ..." */
+export function extractFirstTscError(output) {
+    if (!output)
+        return null;
+    // tsc errors: src/file.ts(line,col): error TS1234: message
+    const tscMatch = output.match(/^(.+?\(\d+,\d+\): error TS\d+: .+)$/m);
+    if (tscMatch) {
+        const line = tscMatch[1];
+        return line.length > 120 ? line.slice(0, 117) + '...' : line;
+    }
+    // Generic "error" line
+    const errMatch = output.match(/^(error .+)$/m);
+    if (errMatch) {
+        const line = errMatch[1];
+        return line.length > 120 ? line.slice(0, 117) + '...' : line;
+    }
+    return null;
+}
 /**
  * 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.
@@ -2746,17 +2765,24 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     if (pkg.scripts?.build && !options._fromCli) {
         console.log(`${timestamp()} Running build...`);
         if (!dryRun) {
-            const buildResult = runCommand('npm', ['run', 'build'], { cwd, silent: !verbose });
+            // Always capture output so we can extract tsc errors for the summary
+            const buildResult = runCommand('npm', ['run', 'build'], { cwd, silent: true });
             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');
+                const buildOutput = buildResult.stderr || buildResult.output;
+                if (buildOutput)
+                    console.error(buildOutput);
+                console.error(colors.red('ERROR: Build failed'));
+                diagnoseBuildFailure(buildOutput || '', cwd);
+                const firstErr = extractFirstTscError(buildOutput || '');
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', firstErr || 'Build failed');
                 if (!force) {
                     return false;
                 }
                 console.log(colors.yellow('Continuing with --force despite build failure...'));
             }
             else {
+                if (verbose && buildResult.output)
+                    process.stdout.write(buildResult.output);
                 console.log(`${timestamp()} ${colors.green('✓ Build succeeded')}`);
             }
         }
@@ -4003,22 +4029,17 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             // Ignore cleanup errors
         }
         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
+            // Check for specific error types before recording
             const output = (publishResult.output + '\n' + publishResult.stderr).toLowerCase();
-            if (output.includes('err_string_too_long') || output.includes('string longer than')) {
-                console.error(colors.red('Tarball too large — check .npmignore. Run: npm pack --dry-run'));
-            }
-            else if (output.includes('e409') || output.includes('409 conflict')) {
-                console.error(colors.yellow('npm still processing previous version — wait and retry'));
-            }
-            else if (output.includes('cannot publish over') || output.includes('previously published')) {
+            // "Already published" is benign — don't record as error
+            if (output.includes('cannot publish over') || output.includes('previously published')) {
                 const currentPkgVersion = readPackageJson(cwd).version;
                 if (output.includes(currentPkgVersion)) {
                     console.log(colors.green('✓ Already published — continuing'));
                 }
                 else {
+                    console.error(colors.red('\nERROR: npm publish failed\n'));
+                    recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
                     console.error(colors.yellow('Version conflict — run npmglobalize again'));
                     if (transformResult.transformed) {
                         const failPkg = readPackageJson(cwd);
@@ -4030,7 +4051,19 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                     return false;
                 }
             }
+            else if (output.includes('err_string_too_long') || output.includes('string longer than')) {
+                console.error(colors.red('\nERROR: npm publish failed\n'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
+                console.error(colors.red('Tarball too large — check .npmignore. Run: npm pack --dry-run'));
+            }
+            else if (output.includes('e409') || output.includes('409 conflict')) {
+                console.error(colors.red('\nERROR: npm publish failed\n'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
+                console.error(colors.yellow('npm still processing previous version — wait and retry'));
+            }
             else if (output.includes('403') || output.includes('forbidden')) {
+                console.error(colors.red('\nERROR: npm publish failed\n'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
                 console.error(colors.yellow('Publish forbidden — check npm login and package access'));
                 if (transformResult.transformed) {
                     const failPkg = readPackageJson(cwd);
@@ -4042,9 +4075,13 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 return false;
             }
             else if (output.includes('402') || output.includes('payment required')) {
+                console.error(colors.red('\nERROR: npm publish failed\n'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
                 console.error(colors.yellow('Private packages need paid npm account — use --npm public'));
             }
             else {
+                console.error(colors.red('\nERROR: npm publish failed\n'));
+                recordBuildIssue(pkg.name || path.basename(cwd), 'error', 'npm publish failed');
                 console.error(colors.yellow('Publish failed — run npm login or check npm whoami'));
             }
             if (transformResult.transformed) {

package/package.json

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