@yarnpkg/nm 3.0.3 → 3.1.0

package/lib/buildNodeModulesTree.d.ts

@@ -10,10 +10,10 @@ export declare enum NodeModulesHoistingLimits {
     DEPENDENCIES = "dependencies",
     NONE = "none"
 }
-export declare type NodeModulesBaseNode = {
+export type NodeModulesBaseNode = {
     dirList: Set<Filename>;
 };
-export declare type NodeModulesPackageNode = {
+export type NodeModulesPackageNode = {
     locator: LocatorKey;
     target: PortablePath;
     linkType: LinkType;
@@ -30,8 +30,8 @@ export declare type NodeModulesPackageNode = {
  * /home/user/project/node_modules/foo -> {target: '/home/user/project/.yarn/.cache/foo.zip/node_modules/foo', linkType: 'HARD'}
  * /home/user/project/node_modules/bar -> {target: '/home/user/project/packages/bar', linkType: 'SOFT'}
  */
-export declare type NodeModulesTree = Map<PortablePath, NodeModulesBaseNode | NodeModulesPackageNode>;
-export declare type NodeModulesTreeErrors = Array<{
+export type NodeModulesTree = Map<PortablePath, NodeModulesBaseNode | NodeModulesPackageNode>;
+export type NodeModulesTreeErrors = Array<{
     messageName: MessageName;
     text: string;
 }>;
@@ -43,7 +43,7 @@ export interface NodeModulesTreeOptions {
     project?: Project;
 }
 /** Package locator key for usage inside maps */
-declare type LocatorKey = string;
+type LocatorKey = string;
 /**
  * Returns path to archive, if package location is inside the archive.
  *
@@ -65,7 +65,7 @@ export declare const buildNodeModulesTree: (pnp: PnpApi, options: NodeModulesTre
     errors: NodeModulesTreeErrors;
     preserveSymlinksRequired: boolean;
 };
-export declare type NodeModulesLocatorMap = Map<LocatorKey, {
+export type NodeModulesLocatorMap = Map<LocatorKey, {
     target: PortablePath;
     linkType: LinkType;
     locations: Array<PortablePath>;

package/lib/buildNodeModulesTree.js

@@ -410,9 +410,11 @@ const populateNodeModulesTree = (pnp, hoistedTree, options) => {
         if (seenNodes.has(pkg))
             return;
         seenNodes.add(pkg);
+        const pkgReferences = Array.from(pkg.references).sort().join(`#`);
         for (const dep of pkg.dependencies) {
+            const depReferences = Array.from(dep.references).sort().join(`#`);
             // We do not want self-references in node_modules, since they confuse existing tools
-            if (dep === pkg)
+            if (dep.identName === pkg.identName && depReferences === pkgReferences)
                 continue;
             const references = Array.from(dep.references).sort();
             const locator = { name: dep.identName, reference: references[0] };
@@ -430,7 +432,8 @@ const populateNodeModulesTree = (pnp, hoistedTree, options) => {
                 const workspace = options.project.workspacesByCwd.get(leafNode.target.slice(0, -1));
                 isAnonymousWorkspace = !!(workspace && !workspace.manifest.name);
             }
-            if (!dep.name.endsWith(WORKSPACE_NAME_SUFFIX) && !isAnonymousWorkspace) {
+            const isCircularSymlink = leafNode.linkType === LinkType.SOFT && nodeModulesLocation.startsWith(leafNode.target);
+            if (!dep.name.endsWith(WORKSPACE_NAME_SUFFIX) && !isAnonymousWorkspace && !isCircularSymlink) {
                 const prevNode = tree.get(nodeModulesLocation);
                 if (prevNode) {
                     if (prevNode.dirList) {

package/lib/hoist.d.ts

@@ -45,13 +45,13 @@
  * the next node as current tree root and run the algorithm again
  * until you run out of candidates for current tree root.
  */
-declare type PackageName = string;
+type PackageName = string;
 export declare enum HoisterDependencyKind {
     REGULAR = 0,
     WORKSPACE = 1,
     EXTERNAL_SOFT_LINK = 2
 }
-export declare type HoisterTree = {
+export type HoisterTree = {
     name: PackageName;
     identName: PackageName;
     reference: string;
@@ -60,13 +60,13 @@ export declare type HoisterTree = {
     hoistPriority?: number;
     dependencyKind?: HoisterDependencyKind;
 };
-export declare type HoisterResult = {
+export type HoisterResult = {
     name: PackageName;
     identName: PackageName;
     references: Set<string>;
     dependencies: Set<HoisterResult>;
 };
-declare type Locator = string;
+type Locator = string;
 declare enum DebugLevel {
     NONE = -1,
     PERF = 0,
@@ -74,7 +74,7 @@ declare enum DebugLevel {
     REASONS = 2,
     INTENSIVE_CHECK = 9
 }
-export declare type HoistOptions = {
+export type HoistOptions = {
     /** Runs self-checks after hoisting is finished */
     check?: boolean;
     /** Debug level */

package/lib/index.d.ts

@@ -4,3 +4,5 @@ export type { NodeModulesBaseNode, NodeModulesPackageNode, NodeModulesTreeOption
 export { NodeModulesHoistingLimits, } from './buildNodeModulesTree';
 export { buildNodeModulesTree, buildLocatorMap, getArchivePath, };
 export type { NodeModulesLocatorMap };
+export type { HoisterTree, HoisterResult } from './hoist';
+export { hoist, HoisterDependencyKind } from './hoist';

package/lib/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getArchivePath = exports.buildLocatorMap = exports.buildNodeModulesTree = exports.NodeModulesHoistingLimits = void 0;
+exports.HoisterDependencyKind = exports.hoist = exports.getArchivePath = exports.buildLocatorMap = exports.buildNodeModulesTree = exports.NodeModulesHoistingLimits = void 0;
 const buildNodeModulesTree_1 = require("./buildNodeModulesTree");
 Object.defineProperty(exports, "getArchivePath", { enumerable: true, get: function () { return buildNodeModulesTree_1.getArchivePath; } });
 const buildNodeModulesTree_2 = require("./buildNodeModulesTree");
@@ -8,3 +8,6 @@ Object.defineProperty(exports, "buildNodeModulesTree", { enumerable: true, get:
 Object.defineProperty(exports, "buildLocatorMap", { enumerable: true, get: function () { return buildNodeModulesTree_2.buildLocatorMap; } });
 var buildNodeModulesTree_3 = require("./buildNodeModulesTree");
 Object.defineProperty(exports, "NodeModulesHoistingLimits", { enumerable: true, get: function () { return buildNodeModulesTree_3.NodeModulesHoistingLimits; } });
+var hoist_1 = require("./hoist");
+Object.defineProperty(exports, "hoist", { enumerable: true, get: function () { return hoist_1.hoist; } });
+Object.defineProperty(exports, "HoisterDependencyKind", { enumerable: true, get: function () { return hoist_1.HoisterDependencyKind; } });

package/package.json

@@ -1,16 +1,14 @@
 {
   "name": "@yarnpkg/nm",
-  "version": "3.0.3",
+  "version": "3.1.0",
   "license": "BSD-2-Clause",
   "main": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "sideEffects": false,
   "dependencies": {
-    "@yarnpkg/core": "^3.2.2",
-    "@yarnpkg/fslib": "^2.6.2"
-  },
-  "devDependencies": {
-    "@yarnpkg/pnp": "^3.2.1"
+    "@yarnpkg/core": "^3.3.0",
+    "@yarnpkg/fslib": "^2.9.0",
+    "@yarnpkg/pnp": "^3.2.5"
   },
   "scripts": {
     "postpack": "rm -rf lib",