dependency-change-report 1.0.5 → 1.0.6

package/lib/generate-html.mjs

@@ -447,7 +447,7 @@ const generateAddedSection = (added) => {
   const rows = added.map(dep => {
     const npmUrl = `https://www.npmjs.com/package/${dep.name}`;
     const versionUrl = `https://www.npmjs.com/package/${dep.name}/v/${dep.version}`;
-    const repoUrl = dep.repository ? 
+    const repoUrl = (dep.repository && typeof dep.repository === 'string') ? 
       dep.repository.replace(/\.git$/, '')
                    .replace(/^git@github\.com:/, 'https://github.com/')
                    .replace(/^git\+https:/, 'https:') : 
@@ -498,15 +498,15 @@ const generateUpgradedSection = (upgraded, changelogs, ciStatus = {}) => {
     const npmUrl = `https://www.npmjs.com/package/${dep.name}`;
     const oldVersionUrl = `https://www.npmjs.com/package/${dep.name}/v/${dep.oldVersion}`;
     const newVersionUrl = `https://www.npmjs.com/package/${dep.name}/v/${dep.newVersion}`;
-    const repoUrl = dep.repository ? 
+    const repoUrl = (dep.repository && typeof dep.repository === 'string') ? 
       dep.repository.replace(/\.git$/, '')
                    .replace(/^git@github\.com:/, 'https://github.com/')
                    .replace(/^git\+https:/, 'https:') : 
-      (changelogs[dep.name] && changelogs[dep.name].repoUrl ? 
+      (changelogs[dep.name] && changelogs[dep.name].repoUrl && typeof changelogs[dep.name].repoUrl === 'string') ? 
         changelogs[dep.name].repoUrl.replace(/\.git$/, '')
                             .replace(/^git@github\.com:/, 'https://github.com/')
                             .replace(/^git\+https:/, 'https:') : 
-        null);
+        null;
       
     const changeTypeBadge = `<span class="badge ${dep.changeType}">${dep.changeType}</span>`;
     
@@ -544,9 +544,11 @@ const generateUpgradedSection = (upgraded, changelogs, ciStatus = {}) => {
     .filter(dep => changelogs[dep.name])
     .map(dep => {
       const changelog = changelogs[dep.name];
-      const repoUrl = changelog.repoUrl.replace(/\.git$/, '')
-                                      .replace(/^git@github\.com:/, 'https://github.com/')
-                                      .replace(/^git\+https:/, 'https:');
+      const repoUrl = (changelog.repoUrl && typeof changelog.repoUrl === 'string') ? 
+        changelog.repoUrl.replace(/\.git$/, '')
+                         .replace(/^git@github\.com:/, 'https://github.com/')
+                         .replace(/^git\+https:/, 'https:') :
+        '#';
       
       const commits = changelog.commits.map(commit => {
         const commitUrl = `${repoUrl}/commit/${commit.hash}`;
@@ -692,9 +694,11 @@ const generateModifiedSection = (modified, changelogs = {}) => {
     .filter(dep => changelogs[dep.newName])
     .map(dep => {
       const changelog = changelogs[dep.newName];
-      const repoUrl = changelog.repoUrl.replace(/\.git$/, '')
-                                      .replace(/^git@github\.com:/, 'https://github.com/')
-                                      .replace(/^git\+https:/, 'https:');
+      const repoUrl = (changelog.repoUrl && typeof changelog.repoUrl === 'string') ? 
+        changelog.repoUrl.replace(/\.git$/, '')
+                         .replace(/^git@github\.com:/, 'https://github.com/')
+                         .replace(/^git\+https:/, 'https:') :
+        '#';
       
       const commits = changelog.commits.map(commit => {
         const commitUrl = `${repoUrl}/commit/${commit.hash}`;
@@ -762,9 +766,11 @@ const generateErrorsSection = (errors) => {
   
   const rows = Object.entries(errors).map(([name, info]) => {
     const npmUrl = `https://www.npmjs.com/package/${name}`;
-    const repoUrl = info.repoUrl.replace(/\.git$/, '')
-                               .replace(/^git@github\.com:/, 'https://github.com/')
-                               .replace(/^git\+https:/, 'https:');
+    const repoUrl = (info.repoUrl && typeof info.repoUrl === 'string') ? 
+      info.repoUrl.replace(/\.git$/, '')
+                  .replace(/^git@github\.com:/, 'https://github.com/')
+                  .replace(/^git\+https:/, 'https:') :
+      '#';
     
     return `
       <tr>
@@ -809,7 +815,7 @@ const generateNestedAddedSection = (added) => {
     const npmUrl = `https://www.npmjs.com/package/${dep.name}`;
     const versionUrl = `https://www.npmjs.com/package/${dep.name}/v/${dep.version}`;
     const parentUrl = `https://www.npmjs.com/package/${dep.parent}`;
-    const repoUrl = dep.repository ? 
+    const repoUrl = (dep.repository && typeof dep.repository === 'string') ? 
       dep.repository.replace(/\.git$/, '')
                    .replace(/^git@github\.com:/, 'https://github.com/')
                    .replace(/^git\+https:/, 'https:') : 
@@ -860,15 +866,15 @@ const generateNestedUpgradedSection = (upgraded, changelogs, ciStatus = {}) => {
     const oldVersionUrl = `https://www.npmjs.com/package/${dep.name}/v/${dep.oldVersion}`;
     const newVersionUrl = `https://www.npmjs.com/package/${dep.name}/v/${dep.newVersion}`;
     const parentUrl = `https://www.npmjs.com/package/${dep.parent}`;
-    const repoUrl = dep.repository ? 
+    const repoUrl = (dep.repository && typeof dep.repository === 'string') ? 
       dep.repository.replace(/\.git$/, '')
                    .replace(/^git@github\.com:/, 'https://github.com/')
                    .replace(/^git\+https:/, 'https:') : 
-      (changelogs[dep.name] && changelogs[dep.name].repoUrl ? 
+      (changelogs[dep.name] && changelogs[dep.name].repoUrl && typeof changelogs[dep.name].repoUrl === 'string') ? 
         changelogs[dep.name].repoUrl.replace(/\.git$/, '')
                             .replace(/^git@github\.com:/, 'https://github.com/')
                             .replace(/^git\+https:/, 'https:') : 
-        null);
+        null;
       
     const changeTypeBadge = `<span class="badge ${dep.changeType}">${dep.changeType}</span>`;
     

package/lib/index.mjs

@@ -932,11 +932,12 @@ const extractFromDependencies = (dependencies, packages) => {
  * @param {Object} dep - Dependency object
  * @param {string} newerVersionDir - Directory of the newer version
  * @param {string} reposDir - Repository directory
- * @param {Object} multibar - CLI multi progress bar instance
+ * @param {Object} multibar - CLI multi progress bar instance (null if disabled)
  * @param {number} maxNameLength - Maximum name length for consistent padding
+ * @param {boolean} useProgressBars - Whether to use progress bars
  * @returns {Promise<Object>} - Object with changelog, error, and CI status
  */
-const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar, maxNameLength) => {
+const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar, maxNameLength, useProgressBars = true) => {
   const result = {
     name: dep.name,
     changelog: null,
@@ -951,15 +952,23 @@ const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar,
   }
   displayName = displayName.padEnd(maxNameLength);
   
-  // Create individual progress bar for this dependency
-  const depBar = multibar.create(100, 0, { name: displayName, status: 'Starting...' });
-  
-  depBar.update(10, { status: 'Getting repo URL...' });
+  // Create individual progress bar for this dependency (if enabled)
+  let depBar = null;
+  if (useProgressBars && multibar) {
+    depBar = multibar.create(100, 0, { name: displayName, status: 'Starting...' });
+    depBar.update(10, { status: 'Getting repo URL...' });
+  } else if (!useProgressBars) {
+    console.log(`Processing ${dep.name}...`);
+  }
   const packageDir = join(newerVersionDir, 'node_modules', dep.name);
   const repoUrl = await getRepositoryUrl(packageDir);
   
   if (!repoUrl) {
-    depBar.update(100, { status: '❌ No repo URL' });
+    if (depBar) {
+      depBar.update(100, { status: '❌ No repo URL' });
+    } else if (!useProgressBars) {
+      console.log(`  ❌ ${dep.name}: No repository URL found`);
+    }
     result.error = {
       repoUrl: null,
       oldVersion: dep.oldVersion,
@@ -969,7 +978,9 @@ const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar,
     return result;
   }
   
-  depBar.update(20, { status: 'Cleaning repo URL...' });
+  if (depBar) {
+    depBar.update(20, { status: 'Cleaning repo URL...' });
+  }
   
   // Clean the repository URL and convert to git URL for authentication
   let cleanRepoUrl = repoUrl.replace(/^git\+/, '');
@@ -1005,13 +1016,19 @@ const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar,
     cleanRepoUrl += '.git';
   }
   
-  depBar.update(30, { status: 'Getting commits...' });
+  if (depBar) {
+    depBar.update(30, { status: 'Getting commits...' });
+  }
   
   let commits = [];
   try {
     commits = await getCommitHistory(cleanRepoUrl, dep.oldVersion, dep.newVersion, reposDir);
     if (commits.length > 0) {
-      depBar.update(70, { status: `Found ${commits.length} commits` });
+      if (depBar) {
+        depBar.update(70, { status: `Found ${commits.length} commits` });
+      } else if (!useProgressBars) {
+        console.log(`  ✅ ${dep.name}: Found ${commits.length} commits`);
+      }
       result.changelog = {
         repoUrl: cleanRepoUrl,
         oldVersion: dep.oldVersion,
@@ -1019,7 +1036,11 @@ const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar,
         commits
       };
     } else {
-      depBar.update(70, { status: '⚠️ No commits found' });
+      if (depBar) {
+        depBar.update(70, { status: '⚠️ No commits found' });
+      } else if (!useProgressBars) {
+        console.log(`  ⚠️ ${dep.name}: No commits found between versions`);
+      }
       result.error = {
         repoUrl: cleanRepoUrl,
         oldVersion: dep.oldVersion,
@@ -1028,7 +1049,11 @@ const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar,
       };
     }
   } catch (error) {
-    depBar.update(70, { status: '❌ Commit error' });
+    if (depBar) {
+      depBar.update(70, { status: '❌ Commit error' });
+    } else if (!useProgressBars) {
+      console.log(`  ❌ ${dep.name}: Error getting commits - ${error.message}`);
+    }
     result.error = {
       repoUrl: cleanRepoUrl,
       oldVersion: dep.oldVersion,
@@ -1038,19 +1063,38 @@ const processSingleDependency = async (dep, newerVersionDir, reposDir, multibar,
   }
   
   // Get GitHub Actions status for the new version
-  depBar.update(80, { status: 'Getting CI status...' });
+  if (depBar) {
+    depBar.update(80, { status: 'Getting CI status...' });
+  }
   try {
     const best = commits.length > 0 ? commits[0].hash : null;
     const actionsStatus = await getGitHubActionsStatus(cleanRepoUrl, dep.newVersion, best);
     if (actionsStatus) {
       result.ciStatus = actionsStatus;
-      depBar.update(100, { status: `✅ Complete (CI: ${actionsStatus.status})` });
+      if (depBar) {
+        depBar.update(90, { status: `CI: ${actionsStatus.status}` });
+      } else if (!useProgressBars) {
+        console.log(`  ✅ ${dep.name}: Complete (CI: ${actionsStatus.status})`);
+      }
     } else {
-      depBar.update(100, { status: '✅ Complete (no CI)' });
+      if (depBar) {
+        depBar.update(90, { status: 'No CI found' });
+      } else if (!useProgressBars) {
+        console.log(`  ✅ ${dep.name}: Complete (no CI)`);
+      }
     }
   } catch (error) {
     // Silently ignore CI status errors
-    depBar.update(100, { status: '✅ Complete (CI error)' });
+    if (depBar) {
+      depBar.update(90, { status: 'CI check failed' });
+    } else if (!useProgressBars) {
+      console.log(`  ✅ ${dep.name}: Complete (CI error)`);
+    }
+  }
+  
+  // Final completion status
+  if (depBar) {
+    depBar.update(100, { status: '✅ Complete' });
   }
   
   return result;
@@ -1073,33 +1117,46 @@ const processInParallel = async (dependencies, newerVersionDir, reposDir, concur
     return { changelogs, errors, ciStatus };
   }
   
+  // Detect small screen and disable progress bars if needed
+  const terminalHeight = process.stdout.rows || 24;
+  const terminalWidth = process.stdout.columns || 80;
+  const isSmallScreen = terminalHeight < 15 || terminalWidth < 60 || dependencies.length > (terminalHeight - 10);
+  
   // Calculate the maximum name length for consistent padding
   const maxNameLength = Math.min(
     Math.max(...dependencies.map(dep => dep.name.length)),
     40 // Reasonable maximum to prevent extremely long lines
   );
   
-  // Create multi progress bar
-  const multibar = new cliProgress.MultiBar({
-    clearOnComplete: false,
-    hideCursor: true,
-    format: '{name} |{bar}| {percentage}% | {status}'
-  }, {
-    barCompleteChar: '\u2588',
-    barIncompleteChar: '\u2591'
-  });
-  
-  // Register multibar for cleanup
-  globalCleanupState.multibar = multibar;
+  let multibar = null;
   
-  console.log(`\nProcessing ${dependencies.length} dependencies with concurrency limit of ${concurrency}:\n`);
+  if (isSmallScreen) {
+    console.log(`\nProcessing ${dependencies.length} dependencies with concurrency limit of ${concurrency}:`);
+    console.log('(Progress bars disabled for small screen or large dependency count)\n');
+  } else {
+    // Create multi progress bar
+    multibar = new cliProgress.MultiBar({
+      clearOnComplete: false,
+      hideCursor: true,
+      forceRedraw: true,
+      format: '{name} |{bar}| {percentage}% | {status}'
+    }, {
+      barCompleteChar: '\u2588',
+      barIncompleteChar: '\u2591'
+    });
+    
+    // Register multibar for cleanup
+    globalCleanupState.multibar = multibar;
+    
+    console.log(`\nProcessing ${dependencies.length} dependencies with concurrency limit of ${concurrency}:\n`);
+  }
   
   // Create queue with concurrency limit
   const queue = new PQueue({ concurrency });
   
   // Add all dependencies to the queue
   const promises = dependencies.map(dep => 
-    queue.add(() => processSingleDependency(dep, newerVersionDir, reposDir, multibar, maxNameLength))
+    queue.add(() => processSingleDependency(dep, newerVersionDir, reposDir, multibar, maxNameLength, !isSmallScreen))
   );
   
   // Wait for all tasks to complete
@@ -1118,13 +1175,15 @@ const processInParallel = async (dependencies, newerVersionDir, reposDir, concur
     }
   }
   
-  multibar.stop();
-  
-  // Unregister multibar from cleanup
-  globalCleanupState.multibar = null;
-  
-  // Ensure cursor is visible after progress bars
-  process.stdout.write('\x1b[?25h'); // Show cursor
+  if (multibar) {
+    multibar.stop();
+    
+    // Unregister multibar from cleanup
+    globalCleanupState.multibar = null;
+    
+    // Ensure cursor is visible after progress bars
+    process.stdout.write('\x1b[?25h'); // Show cursor
+  }
   
   console.log(`\n✅ Completed processing ${dependencies.length} dependencies\n`);
   

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-change-report",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "main": "index.mjs",
   "type": "module",
   "bin": {