@bobfrankston/npmglobalize 1.0.101 → 1.0.103

package/lib.js

@@ -752,6 +752,47 @@ export function transformDeps(pkg, baseDir, verbose = false, forcePublish = fals
     }
     return { transformed, unpublished };
 }
+/** Build and print a dependency tree of file: references.
+ *  Recursively walks file: deps showing the full hierarchy with indentation. */
+function printDepTree(baseDir, indent = 0, visited = new Set()) {
+    const realDir = fs.realpathSync(baseDir);
+    if (visited.has(realDir))
+        return;
+    visited.add(realDir);
+    let pkg;
+    try {
+        pkg = readPackageJson(baseDir);
+    }
+    catch {
+        return;
+    }
+    for (const key of DEP_KEYS) {
+        if (!pkg[key])
+            continue;
+        // Use the backup (.dependencies) if present — it has the original file: refs
+        const deps = pkg['.' + key] || pkg[key];
+        for (const [name, value] of Object.entries(deps)) {
+            if (!isFileRef(value))
+                continue;
+            const prefix = ' '.repeat(indent * 2);
+            let targetPath;
+            let version = '?';
+            try {
+                targetPath = resolveFilePath(value, baseDir);
+                const targetPkg = readPackageJson(targetPath);
+                version = targetPkg.version || '?';
+            }
+            catch {
+                console.log(`${prefix}  ${name} → ${value} ${colors.red('(unresolvable)')}`);
+                continue;
+            }
+            const exists = checkVersionExists(name, version);
+            const marker = exists ? colors.green('✓') : colors.red('✗');
+            console.log(`${prefix}  ${marker} ${name}@${version} → ${value}`);
+            printDepTree(targetPath, indent + 1, visited);
+        }
+    }
+}
 /** Restore file: dependencies from .dependencies */
 export function restoreDeps(pkg, verbose = false) {
     let restored = false;
@@ -1301,6 +1342,37 @@ const RECOMMENDED_GITIGNORE = [
     '*.ldf',
     '*.ndf'
 ];
+/** Extensions only recommended for .gitignore when matching files exist in the project */
+const PRESENCE_ONLY_EXTENSIONS = new Set(['.mdf', '.ldf', '.ndf']);
+/** Check if any files with the given extension exist in dir (skips node_modules, .git, prev) */
+function hasFilesWithExtension(dir, ext) {
+    const skip = new Set(['node_modules', '.git', 'prev']);
+    function check(d) {
+        try {
+            for (const entry of fs.readdirSync(d, { withFileTypes: true })) {
+                if (skip.has(entry.name))
+                    continue;
+                if (entry.isFile() && entry.name.endsWith(ext))
+                    return true;
+                if (entry.isDirectory() && check(path.join(d, entry.name)))
+                    return true;
+            }
+        }
+        catch { }
+        return false;
+    }
+    return check(dir);
+}
+/** Filter RECOMMENDED_GITIGNORE to skip presence-only patterns when no matching files exist */
+function getApplicableGitignorePatterns(cwd) {
+    return RECOMMENDED_GITIGNORE.filter(pattern => {
+        const extMatch = pattern.match(/^\*(\.\w+)$/);
+        if (extMatch && PRESENCE_ONLY_EXTENSIONS.has(extMatch[1])) {
+            return hasFilesWithExtension(cwd, extMatch[1]);
+        }
+        return true;
+    });
+}
 /** Recommended .npmignore patterns */
 const RECOMMENDED_NPMIGNORE = [
     '.git/',
@@ -1341,7 +1413,7 @@ function checkIgnoreFiles(cwd, options) {
     if (fs.existsSync(gitignorePath)) {
         const content = fs.readFileSync(gitignorePath, 'utf-8');
         const lines = content.split('\n').map(l => l.trim());
-        for (const pattern of RECOMMENDED_GITIGNORE) {
+        for (const pattern of getApplicableGitignorePatterns(cwd)) {
             if (!lines.some(line => line === pattern || line === pattern.replace('/', ''))) {
                 changes.push(`  .gitignore missing: ${pattern}`);
             }
@@ -1395,7 +1467,7 @@ function conformIgnoreFiles(cwd) {
         const content = fs.readFileSync(gitignorePath, 'utf-8');
         const lines = new Set(content.split('\n').map(l => l.trim()).filter(l => l && !l.startsWith('#')));
         let updated = false;
-        for (const pattern of RECOMMENDED_GITIGNORE) {
+        for (const pattern of getApplicableGitignorePatterns(cwd)) {
             const normalized = pattern.replace('/', '');
             if (!lines.has(pattern) && !lines.has(normalized)) {
                 lines.add(pattern);
@@ -1468,9 +1540,9 @@ function ensureGitignore(cwd) {
     // Update .gitignore if needed
     if (needsUpdate) {
         if (!content || content.trim() === '') {
-            // Create new .gitignore from RECOMMENDED_GITIGNORE plus extras
+            // Create new .gitignore from applicable patterns plus extras
             const extras = ['*certs*/', 'configuration.json', 'cruft/', 'prev/', 'tests/'];
-            content = [...RECOMMENDED_GITIGNORE, ...extras].join('\n') + '\n';
+            content = [...getApplicableGitignorePatterns(cwd), ...extras].join('\n') + '\n';
         }
         else {
             // Add node_modules to existing .gitignore
@@ -2477,13 +2549,16 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             console.log('');
         }
         // Check if target packages need to be published
-        if (transformResult.unpublished.length > 0) {
+        if (transformResult.unpublished.length > 0 && !noPublish) {
             console.log('');
             console.log(colors.red('⚠ WARNING: Some file: dependencies are not published on npm:'));
             for (const { name, version, path } of transformResult.unpublished) {
                 console.log(colors.yellow(`  ${name}@${version} (${path})`));
             }
             console.log('');
+            console.log('Dependency tree (✓ = on npm, ✗ = needs publishing):');
+            printDepTree(cwd);
+            console.log('');
             if (publishDeps) {
                 const action = forcePublish ? 'Publishing/updating' : 'Publishing';
                 console.log(`${action} file: dependencies first (--publish-deps)...`);
@@ -2508,7 +2583,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                             fix,
                             conform, // Propagate conform to dependencies
                             publishDeps, // Propagate so transitive deps also get published
-                            forcePublish // Propagate so transitive deps get force-published too
+                            forcePublish, // Propagate so transitive deps get force-published too
+                            rebase // Propagate so behind-remote deps get rebased automatically
                         });
                         if (!depSuccess) {
                             console.error(colors.red(`Failed to publish ${name}`));
@@ -2529,6 +2605,7 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
                 console.log(colors.yellow('  1. Publish them manually first'));
                 console.log(colors.yellow('  2. Use --publish-deps to publish them automatically'));
                 console.log(colors.yellow('  3. Use --force to continue anyway (NOT RECOMMENDED)'));
+                console.log(colors.yellow('  4. Use -npd (--no-publish-deps) to skip dependency publishing'));
                 console.log('');
                 if (!force) {
                     const shouldContinue = await confirm('Continue with unpublished dependencies?', false);
@@ -2610,10 +2687,13 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             const pkgName = pkg.name;
             const pkgVersion = pkg.version;
             if (!pkg.bin && (install || link || wsl)) {
-                console.log(colors.yellow('Note: This is a library (no bin field), skipping global installation.'));
-                console.log(colors.yellow('To use in projects: npm install ' + pkgName));
+                const proceed = await offerAddBin(cwd, pkg);
+                if (!proceed) {
+                    console.log(colors.yellow('Skipping global install — library packages should not be installed globally.'));
+                    console.log(colors.yellow(`To use in projects: npm install ${pkgName}`));
+                }
             }
-            else {
+            if (pkg.bin && (install || link || wsl)) {
                 if (link) {
                     console.log(`Installing ${pkgName} globally from local directory (link)...`);
                     const localInstallResult = runCommand('npm', ['install', '-g', '.'], { cwd, silent: false });
@@ -3104,10 +3184,13 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     const pkgName = updatedPkg.name;
     const pkgVersion = updatedPkg.version;
     if (!updatedPkg.bin && (install || link || wsl)) {
-        console.log(colors.yellow('Note: This is a library (no bin field), skipping global installation.'));
-        console.log(colors.yellow('To use in projects: npm install ' + pkgName));
+        const proceed = await offerAddBin(cwd, updatedPkg);
+        if (!proceed) {
+            console.log(colors.yellow('Skipping global install — library packages should not be installed globally.'));
+            console.log(colors.yellow(`To use in projects: npm install ${pkgName}`));
+        }
     }
-    else {
+    if (updatedPkg.bin && (install || link || wsl)) {
         if (link) {
             console.log(`Linking globally: ${pkgName}@${pkgVersion}...`);
             if (!dryRun) {

package/package.json

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