fix: add null check for link's target when calling matches to dedupe …

…deps
npm · Jun 5, 2024 · eb982f4 · eb982f4
1 parent 93883bb
commit eb982f4
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/workspaces/arborist/lib/node.js b/workspaces/arborist/lib/node.js
@@ -1113,8 +1113,12 @@ class Node {
  }
 
  // if they're links, they match if the targets match
- if (this.isLink) {
- return node.isLink && this.target.matches(node.target)
+ if (node.isLink) {
+ // Linked to nothing, cannot matches
+ if (!this.target || !node.target) {
+ return false;
+ }
+ return this.target.matches(node.target);
  }
 
  // if they're two project root nodes, they're different if the paths differ