@bobfrankston/npmglobalize 1.0.110 → 1.0.112

package/README.md

@@ -179,7 +179,8 @@ When initializing a new repository with `--init`, npmglobalize automatically set
 
 **File structure** (per programming.md standards):
 - `.gitignore` - Node.js best practices (node_modules, secrets, certificates, etc.)
-- `.npmignore` - Publishing filters (excludes .git, tests, source files, etc.)  
+- `.npmignore` - Publishing filters (excludes .git, tests, source files, etc.)
+  - **noEmit projects:** `*.ts`, `*.map`, and `tsconfig.json` are kept (not ignored) since TS files are the runtime files
 - `.gitattributes` - Forces LF line endings for all text files
 
 **Git configuration** (ensures cross-platform compatibility):
@@ -298,6 +299,7 @@ Workspace mode is auto-detected when run from a root with `"private": true` and
 -verbose        Show detailed output
 -conform        Update .gitignore/.npmignore/.gitattributes to best practices
                 and configure git for LF line endings (fixes existing repos)
+                For noEmit projects: removes *.ts/*.map/tsconfig.json from .npmignore
 -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

package/cli.js

@@ -67,6 +67,7 @@ Other Options:
   -quiet          Suppress npm warnings (default)
   -verbose        Show detailed output
   -conform        Update .gitignore/.npmignore to best practices
+                  (noEmit projects: removes TS ignores from .npmignore)
   -asis           Skip ignore file checks (or set "asis": true in .globalize.json5)
   -rebase         Automatically rebase if local is behind remote
   -show           Show package.json dependency changes

package/lib.js

@@ -1338,6 +1338,29 @@ const IGNORE_PATTERNS = loadIgnorePatterns();
 const ALL_GITIGNORE = [...IGNORE_PATTERNS.gitignore.security, ...IGNORE_PATTERNS.gitignore.recommended, ...(IGNORE_PATTERNS.gitignore.presenceOnly ?? [])];
 /** All .npmignore patterns (security + recommended) */
 const ALL_NPMIGNORE = [...IGNORE_PATTERNS.npmignore.security, ...IGNORE_PATTERNS.npmignore.recommended];
+/** Patterns that should NOT be in .npmignore for noEmit projects (TS files are the runtime files) */
+const TS_NPMIGNORE_PATTERNS = new Set(['*.ts', '!*.d.ts', '*.map', 'tsconfig.json']);
+/** Check if target project uses noEmit (TS files run directly, no compilation) */
+function isNoEmitProject(cwd) {
+    const tsconfigPath = path.join(cwd, 'tsconfig.json');
+    if (!fs.existsSync(tsconfigPath))
+        return false;
+    try {
+        const content = fs.readFileSync(tsconfigPath, 'utf-8');
+        const tsconfig = JSON5.parse(content);
+        return tsconfig.compilerOptions?.noEmit === true;
+    }
+    catch {
+        return false;
+    }
+}
+/** Get applicable npmignore patterns, excluding TS patterns for noEmit projects */
+function getApplicableNpmignorePatterns(cwd) {
+    if (isNoEmitProject(cwd)) {
+        return ALL_NPMIGNORE.filter(p => !TS_NPMIGNORE_PATTERNS.has(p));
+    }
+    return ALL_NPMIGNORE;
+}
 /** Presence-only extensions derived from data file */
 const PRESENCE_ONLY_EXTENSIONS = new Set((IGNORE_PATTERNS.gitignore.presenceOnly ?? [])
     .map(p => p.match(/^\*(\.\w+)$/)?.[1])
@@ -1417,7 +1440,8 @@ function checkIgnoreFiles(cwd, options) {
     if (fs.existsSync(npmignorePath)) {
         const content = fs.readFileSync(npmignorePath, 'utf-8');
         const lines = content.split('\n').map(l => l.trim());
-        for (const pattern of ALL_NPMIGNORE) {
+        const applicableNpm = getApplicableNpmignorePatterns(cwd);
+        for (const pattern of applicableNpm) {
             if (!lineHasPattern(lines, pattern)) {
                 if (securityNpm.has(pattern)) {
                     securityChanges.push(`  .npmignore missing: ${pattern}`);
@@ -1452,7 +1476,8 @@ function conformIgnoreFiles(cwd, securityOnly = false) {
         }
     }
     const patternsGit = securityOnly ? getSecurityGitignorePatterns() : getApplicableGitignorePatterns(cwd);
-    const patternsNpm = securityOnly ? getSecurityNpmignorePatterns() : ALL_NPMIGNORE;
+    const patternsNpm = securityOnly ? getSecurityNpmignorePatterns() : getApplicableNpmignorePatterns(cwd);
+    const noEmit = isNoEmitProject(cwd);
     // Update .gitignore
     const gitignorePath = path.join(cwd, '.gitignore');
     if (fs.existsSync(gitignorePath)) {
@@ -1479,8 +1504,8 @@ function conformIgnoreFiles(cwd, securityOnly = false) {
         const lines = content.split('\n').map(l => l.trim());
         const newLines = new Set(lines.filter(l => l));
         let updated = false;
-        if (!securityOnly) {
-            // Add TypeScript exclusions (only in full conform)
+        if (!securityOnly && !noEmit) {
+            // Add TypeScript exclusions (only in full conform, skip for noEmit projects)
             for (const ts of ['*.ts', '!*.d.ts', '*.map']) {
                 if (!newLines.has(ts)) {
                     newLines.add(ts);
@@ -1488,6 +1513,15 @@ function conformIgnoreFiles(cwd, securityOnly = false) {
                 }
             }
         }
+        // For noEmit projects, remove TS patterns that shouldn't be ignored
+        if (noEmit) {
+            for (const ts of TS_NPMIGNORE_PATTERNS) {
+                if (newLines.has(ts)) {
+                    newLines.delete(ts);
+                    updated = true;
+                }
+            }
+        }
         for (const pattern of patternsNpm) {
             if (!newLines.has(pattern)) {
                 newLines.add(pattern);
@@ -1497,7 +1531,12 @@ function conformIgnoreFiles(cwd, securityOnly = false) {
         if (updated) {
             const newContent = Array.from(newLines).join('\n') + '\n';
             fs.writeFileSync(npmignorePath, newContent);
-            console.log(colors.cyan('  ✓ Auto-added security patterns to .npmignore'));
+            if (noEmit) {
+                console.log(colors.cyan('  ✓ .npmignore updated (noEmit: kept *.ts files)'));
+            }
+            else {
+                console.log(colors.cyan('  ✓ Auto-added security patterns to .npmignore'));
+            }
         }
     }
 }
@@ -1547,7 +1586,8 @@ function ensureNpmignore(cwd) {
     const npmignorePath = path.join(cwd, '.npmignore');
     if (!fs.existsSync(npmignorePath)) {
         console.log('  Creating .npmignore...');
-        const content = ALL_NPMIGNORE.join('\n') + '\n';
+        const patterns = getApplicableNpmignorePatterns(cwd);
+        const content = patterns.join('\n') + '\n';
         fs.writeFileSync(npmignorePath, content);
         console.log(colors.green('  ✓ .npmignore created'));
     }
@@ -3310,6 +3350,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     const updatedPkg = readPackageJson(cwd); // Re-read to get updated version
     const pkgName = updatedPkg.name;
     const pkgVersion = updatedPkg.version;
+    let globalInstallOk = false;
+    let wslInstallOk = false;
     if (!updatedPkg.bin && (install || link || wsl)) {
         const proceed = await offerAddBin(cwd, updatedPkg);
         if (!proceed) {
@@ -3323,6 +3365,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             if (!dryRun) {
                 const installResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false, showCommand: true });
                 if (installResult.success) {
+                    globalInstallOk = true;
                     console.log(colors.green(`✓ Linked globally: ${pkgName}@${pkgVersion}`));
                 }
                 else {
@@ -3340,6 +3383,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 waitForNpmVersion(pkgName, pkgVersion);
                 const installResult = installGlobalWithRetry(`${pkgName}@${pkgVersion}`, cwd);
                 if (installResult.success) {
+                    globalInstallOk = true;
                     console.log(colors.green(`✓ Installed globally: ${pkgName}@${pkgVersion}`));
                 }
                 else {
@@ -3359,6 +3403,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             if (!dryRun) {
                 const wslResult = runCommand('wsl', wslArgs, { cwd, silent: false, showCommand: true });
                 if (wslResult.success) {
+                    wslInstallOk = true;
                     console.log(colors.green(`✓ Installed in WSL: ${pkgName}@${pkgVersion}`));
                 }
                 else {
@@ -3424,13 +3469,13 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
         console.log(`  Git visibility: ${colors.green(gitVisibility.toUpperCase())}`);
     }
     if (link) {
-        console.log(`  Linked globally: ${colors.green('✓')}`);
+        console.log(`  Linked globally: ${globalInstallOk ? colors.green('✓') : colors.red('✗')}`);
     }
     else if (install) {
-        console.log(`  Installed globally: ${colors.green('✓')}`);
+        console.log(`  Installed globally: ${globalInstallOk ? colors.green('✓') : colors.red('✗')}`);
     }
     if (wsl) {
-        console.log(`  Installed in WSL: ${colors.green('✓')}`);
+        console.log(`  Installed in WSL: ${wslInstallOk ? colors.green('✓') : colors.red('✗')}`);
     }
     if (freeze) {
         console.log(`  Frozen node_modules: ${colors.green('✓')}`);

package/package.json

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