@yarnpkg/nm 4.0.1 → 4.0.3

package/lib/buildNodeModulesTree.js

@@ -123,7 +123,7 @@ const buildWorkspaceMap = (pnp) => {
         const pkg = pnp.getPackageInformation(locator);
         if (pkg) {
             const parentLocatorKey = parentLocator ? stringifyLocator(parentLocator) : ``;
-            if (stringifyLocator(locator) !== parentLocatorKey && pkg.linkType === LinkType.SOFT && !isExternalSoftLink(pkg, locator, pnp, topPkgPortableLocation)) {
+            if (stringifyLocator(locator) !== parentLocatorKey && pkg.linkType === LinkType.SOFT && !locator.reference.startsWith(`link:`) && !isExternalSoftLink(pkg, locator, pnp, topPkgPortableLocation)) {
                 const location = getRealPackageLocation(pkg, locator, pnp);
                 const prevLocator = workspaceLikeLocators.get(location);
                 // Give workspaces a priority over portals and other protocols pointing to the same location

package/lib/hoist.js

@@ -50,7 +50,8 @@ const hoist = (tree, opts = {}) => {
     let anotherRoundNeeded = false;
     let round = 0;
     do {
-        anotherRoundNeeded = hoistTo(treeCopy, [treeCopy], new Set([treeCopy.locator]), new Map(), options).anotherRoundNeeded;
+        const result = hoistTo(treeCopy, [treeCopy], new Set([treeCopy.locator]), new Map(), options);
+        anotherRoundNeeded = result.anotherRoundNeeded || result.isGraphChanged;
         options.fastLookupPossible = false;
         round++;
     } while (anotherRoundNeeded);
@@ -351,23 +352,6 @@ const getNodeHoistInfo = (rootNode, rootNodePathLocators, nodePath, node, usedDe
             reason = `- external soft link with unhoisted dependencies`;
         }
     }
-    if (isHoistable) {
-        // Direct workspace dependencies must be hoisted to any common ancestor workspace of all the
-        // graph paths that include the dependency, because otherwise running app with
-        // `--preserve-symlinks` will become broken (without this flag the Node.js will pick dependency
-        // from the ancestor on the file system and with this flag it will pick ancestor from the graph
-        // and if these ancestors are different, the behavious of the application will be different).
-        // Another problem, which is prevented - is a creation of multiple hoisting layouts
-        // for the same workspace, because different dependencies of the same workspace might be hoisted
-        // differently, depending on the recepient workspace.
-        // It is difficult to find all common ancestors, but there is one easy to find common ancestor -
-        // the root workspace, so, for now, we either hoist direct dependencies into the root workspace, or we keep them
-        // unhoisted, thus we are safe from various pathological cases with `--preserve-symlinks`
-        isHoistable = parentNode.dependencyKind !== HoisterDependencyKind.WORKSPACE || parentNode.hoistedFrom.has(node.name) || rootNodePathLocators.size === 1;
-        if (outputReason && !isHoistable) {
-            reason = parentNode.reasons.get(node.name);
-        }
-    }
     if (isHoistable) {
         isHoistable = !rootNode.peerNames.has(node.name);
         if (outputReason && !isHoistable) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yarnpkg/nm",
-  "version": "4.0.1",
+  "version": "4.0.3",
   "license": "BSD-2-Clause",
   "main": "./lib/index.js",
   "exports": {
@@ -9,9 +9,9 @@
   },
   "sideEffects": false,
   "dependencies": {
-    "@yarnpkg/core": "^4.0.2",
-    "@yarnpkg/fslib": "^3.0.1",
-    "@yarnpkg/pnp": "^4.0.1"
+    "@yarnpkg/core": "^4.1.3",
+    "@yarnpkg/fslib": "^3.1.0",
+    "@yarnpkg/pnp": "^4.0.6"
   },
   "scripts": {
     "postpack": "rm -rf lib",