Safely handle components without names (#1464)

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
CycloneDX · Nov 18, 2024 · 4c70540 · 4c70540
1 parent d83d58a
commit 4c70540
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/lib/helpers/display.js b/lib/helpers/display.js
@@ -289,8 +289,11 @@ export function printDependencyTree(
     console.log(
       table([[highlightStr(treeGraphics.join("\n"), highlight)]], config),
     );
-  } else {
+  } else if (treeGraphics.length < 500) {
+    // https://github.com/nodejs/node/issues/35973
     console.log(highlightStr(treeGraphics.join("\n"), highlight));
+  } else {
+    console.log("Dependency tree is too large to print.");
   }
 }
 

diff --git a/lib/stages/postgen/annotator.js b/lib/stages/postgen/annotator.js
@@ -33,11 +33,11 @@ function joinArray(arr) {
 }
 
 function cleanNames(s) {
-  return s.replace(/[+]/g, " ");
+  return s?.replace(/[+]/g, " ");
 }
 
 function cleanTypes(s) {
-  return s.replace(/[+-_]/g, " ");
+  return s?.replace(/[+-_]/g, " ");
 }
 
 /**

diff --git a/types/lib/helpers/display.d.ts.map b/types/lib/helpers/display.d.ts.map