fix(resolve): takes 'workspaces' field into account when merging pack…

…age manifests
sverweij · Nov 15, 2023 · fe0a831 · fe0a831
1 parent 8b37fe3
commit fe0a831
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/extract/resolve/merge-manifests.mjs b/src/extract/resolve/merge-manifests.mjs
@@ -24,12 +24,12 @@ function mergeDependencyArray(pClosestDependencyKey, pFurtherDependencyKey) {
   return uniq(pClosestDependencyKey.concat(pFurtherDependencyKey));
 }
 
-function isDependencyKey(pKey) {
-  return pKey.endsWith("ependencies");
+function isInterestingKey(pKey) {
+  return pKey.endsWith("ependencies") || pKey === "workspaces";
 }
 
 function getDependencyKeys(pPackage) {
-  return Object.keys(pPackage).filter(isDependencyKey);
+  return Object.keys(pPackage).filter(isInterestingKey);
 }
 
 function getJointUniqueDependencyKeys(pClosestPackage, pFurtherPackage) {
@@ -40,6 +40,10 @@ function getJointUniqueDependencyKeys(pClosestPackage, pFurtherPackage) {
   );
 }
 
+function isAnArrayKey(pKey) {
+  return pKey.startsWith("bundle") || pKey === "workspaces";
+}
+
 /**
  * returns an object with
  * - the *dependencies keys from both passed packages
@@ -63,7 +67,7 @@ export default function mergeManifests(pClosestManifest, pFurtherManifest) {
   )
     .map((pKey) => ({
       key: pKey,
-      value: pKey.startsWith("bundle")
+      value: isAnArrayKey(pKey)
         ? mergeDependencyArray(
             pClosestManifest?.[pKey] ?? [],
             pFurtherManifest?.[pKey] ?? [],