Merge pull request #97 from mansona/optional-peer-deps

don't error about missing optional peer deps
stefanpenner · May 14, 2024 · 698f313 · 698f313
2 parents c89d99a + 7654463
commit 698f313
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -567,8 +567,22 @@ export class Project {
       }
     }
 
+    // collect a list of optional peers
+    let optionalPeers = [];
+    for (let depName of Object.keys(targetPkg['peerDependenciesMeta'] ?? {})) {
+      if (targetPkg['peerDependenciesMeta'][depName]?.optional === true) {
+        optionalPeers.push(depName);
+      }
+    }
+
     for (let depName of depsToLink) {
       let depTarget = resolvePackagePath(depName, target, this.resolutionCache);
+
+      if (!depTarget && optionalPeers.includes(depName)) {
+        // don't link or error about missing optional peer deps
+        continue;
+      }
+
       if (!depTarget) {
         throw new Error(
           `[FixturifyProject] package ${name} in ${target} depends on ${depName} but we could not resolve it`