@bobfrankston/npmglobalize 1.0.38 → 1.0.39

package/README.md

@@ -397,6 +397,40 @@ Check authentication:
 npm whoami
 ```
 
+## Development
+
+### Build Check
+
+`npmglobalize` includes automatic build verification to ensure TypeScript files are compiled before execution. This prevents runtime errors from outdated JavaScript files.
+
+**How it works:**
+- When you run `npmglobalize`, it automatically checks if `.js` files are newer than their `.ts` sources
+- If `.js` files are missing or outdated, execution stops with an error
+- The check is skipped if `noEmit: true` is set in `tsconfig.json`
+
+**Building the project:**
+```bash
+npm run build     # Compile TypeScript files
+npm run watch     # Watch mode for development
+npm run check     # Manually verify build status
+```
+
+**Bypassing the check:**
+```bash
+npmglobalize --force    # Skip build check (not recommended)
+```
+
+**Error example:**
+```
+❌ Error: TypeScript files not compiled
+cli.js is older than cli.ts
+
+Please run: npm run build
+Or use --force to skip this check
+```
+
+This ensures you never accidentally run outdated code when the TypeScript source has changed.
+
 ## License
 
 MIT

package/check-build.d.ts

@@ -0,0 +1,3 @@
+#!/usr/bin/env node
+export {};
+//# sourceMappingURL=check-build.d.ts.map

package/check-build.js

@@ -0,0 +1,139 @@
+#!/usr/bin/env node
+/**
+ * Check if TypeScript files are compiled before running
+ * Ensures .js files exist and are newer than their .ts source files
+ * Unless --force flag is specified or noEmit is set in tsconfig.json
+ */
+import fs from 'fs';
+import path from 'path';
+import { fileURLToPath } from 'url';
+import JSON5 from 'json5';
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+// Check for --force flag
+const hasForceFlag = process.argv.includes('--force');
+// Read and parse tsconfig.json
+function readTsConfig() {
+    try {
+        const tsconfigPath = path.join(__dirname, 'tsconfig.json');
+        const content = fs.readFileSync(tsconfigPath, 'utf8');
+        return JSON5.parse(content);
+    }
+    catch (error) {
+        console.error('Warning: Could not read tsconfig.json');
+        return null;
+    }
+}
+// Check if a path matches any of the exclude patterns
+function isExcluded(relativePath, excludePatterns) {
+    if (!excludePatterns || excludePatterns.length === 0) {
+        return false;
+    }
+    // Normalize path separators for comparison
+    const normalizedPath = relativePath.replace(/\\/g, '/');
+    for (const pattern of excludePatterns) {
+        const normalizedPattern = pattern.replace(/\\/g, '/');
+        // Simple pattern matching (supports exact matches and wildcards)
+        if (normalizedPath === normalizedPattern) {
+            return true;
+        }
+        // Check if path starts with pattern (for directory exclusions)
+        if (normalizedPath.startsWith(normalizedPattern + '/') ||
+            normalizedPath === normalizedPattern) {
+            return true;
+        }
+        // Check if any path segment matches the pattern
+        const pathSegments = normalizedPath.split('/');
+        if (pathSegments.includes(normalizedPattern)) {
+            return true;
+        }
+    }
+    return false;
+}
+// Get all .ts files in the project (respecting tsconfig exclude)
+function getTypeScriptFiles(tsconfig) {
+    const files = [];
+    const excludePatterns = tsconfig?.exclude || [];
+    // Always exclude node_modules and .git
+    const alwaysExclude = ['node_modules', '.git'];
+    const allExcludes = [...alwaysExclude, ...excludePatterns];
+    function scanDir(dir, relativePath = '') {
+        const entries = fs.readdirSync(dir, { withFileTypes: true });
+        for (const entry of entries) {
+            const fullPath = path.join(dir, entry.name);
+            const relPath = relativePath ? path.join(relativePath, entry.name) : entry.name;
+            // Check if this path is excluded
+            if (isExcluded(relPath, allExcludes)) {
+                continue;
+            }
+            if (entry.isDirectory()) {
+                scanDir(fullPath, relPath);
+            }
+            else if (entry.isFile() && entry.name.endsWith('.ts')) {
+                files.push(fullPath);
+            }
+        }
+    }
+    scanDir(__dirname);
+    return files;
+}
+// Check if .js file exists and is newer than .ts file
+function checkBuildStatus() {
+    const tsconfig = readTsConfig();
+    // Check for noEmit
+    if (tsconfig?.compilerOptions?.noEmit === true) {
+        console.log('✓ noEmit is set in tsconfig.json, skipping build check');
+        return true;
+    }
+    if (hasForceFlag) {
+        console.log('✓ --force flag specified, skipping build check');
+        return true;
+    }
+    const tsFiles = getTypeScriptFiles(tsconfig);
+    const errors = [];
+    for (const tsFile of tsFiles) {
+        const jsFile = tsFile.replace(/\.ts$/, '.js');
+        // Check if .js file exists
+        if (!fs.existsSync(jsFile)) {
+            const relPath = path.relative(__dirname, tsFile);
+            errors.push(`Missing: ${relPath.replace(/\.ts$/, '.js')}`);
+            continue;
+        }
+        // Check if .js is older than .ts
+        const tsStat = fs.statSync(tsFile);
+        const jsStat = fs.statSync(jsFile);
+        if (jsStat.mtime < tsStat.mtime) {
+            const relPath = path.relative(__dirname, tsFile);
+            errors.push(`Outdated: ${relPath.replace(/\.ts$/, '.js')} is older than ${relPath}`);
+        }
+    }
+    if (errors.length > 0) {
+        console.error('\n❌ Build check failed:\n');
+        errors.forEach(err => console.error(`  • ${err}`));
+        console.error('\nPlease run: npm run build');
+        console.error('Or use --force to skip this check\n');
+        return false;
+    }
+    console.log('✓ All TypeScript files are compiled');
+    return true;
+}
+// Run the check
+if (!checkBuildStatus()) {
+    process.exit(1);
+}
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map
+//# sourceMappingURL=check-build.js.map

package/cli.js

@@ -6,6 +6,7 @@ import { globalize, readConfig } from './lib.js';
 import fs from 'fs';
 import path from 'path';
 import { fileURLToPath } from 'url';
+import JSON5 from 'json5';
 function printHelp() {
     console.log(`
 npmglobalize - Transform file: dependencies to npm versions for publishing
@@ -250,6 +251,45 @@ export async function main() {
         console.log(pkg.version);
         process.exit(0);
     }
+    // Check if TypeScript files are compiled (unless --force or --help/--version)
+    if (!cliOptions.force && !cliOptions.help && !cliOptions.version) {
+        const __filename = fileURLToPath(import.meta.url);
+        const __dirname = path.dirname(__filename);
+        // Check for noEmit in tsconfig.json
+        let noEmit = false;
+        try {
+            const tsconfigPath = path.join(__dirname, 'tsconfig.json');
+            const content = fs.readFileSync(tsconfigPath, 'utf-8');
+            const tsconfig = JSON5.parse(content);
+            noEmit = tsconfig.compilerOptions?.noEmit === true;
+        }
+        catch (error) {
+            // If we can't read tsconfig, assume we need to check
+        }
+        if (!noEmit) {
+            // Check if this .js file is older than its .ts source
+            const jsFile = __filename;
+            const tsFile = jsFile.replace(/\.js$/, '.ts');
+            if (fs.existsSync(tsFile)) {
+                if (!fs.existsSync(jsFile)) {
+                    console.error('\n❌ Error: TypeScript files not compiled');
+                    console.error(`Missing: ${path.basename(jsFile)}`);
+                    console.error('\nPlease run: npm run build');
+                    console.error('Or use --force to skip this check\n');
+                    process.exit(1);
+                }
+                const tsStat = fs.statSync(tsFile);
+                const jsStat = fs.statSync(jsFile);
+                if (jsStat.mtime < tsStat.mtime) {
+                    console.error('\n❌ Error: TypeScript files not compiled');
+                    console.error(`${path.basename(jsFile)} is older than ${path.basename(tsFile)}`);
+                    console.error('\nPlease run: npm run build');
+                    console.error('Or use --force to skip this check\n');
+                    process.exit(1);
+                }
+            }
+        }
+    }
     if (cliOptions.error) {
         console.error(`Error: ${cliOptions.error}`);
         console.error('Run with --help for usage.');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/npmglobalize",
-  "version": "1.0.38",
+  "version": "1.0.39",
   "description": "Transform file: dependencies to npm versions for publishing",
   "main": "index.js",
   "type": "module",
@@ -9,7 +9,9 @@
   },
   "scripts": {
     "build": "tsc",
-    "watch": "tsc -w"
+    "watch": "tsc -w",
+    "prestart": "node check-build.js",
+    "check": "node check-build.js"
   },
   "keywords": [
     "npm",