fix(nm): prefer direct dependency binaries

.yarn/versions/a46c1d8d.yml

@@ -0,0 +1,5 @@
+releases:
+  "@yarnpkg/plugin-nm": patch
+
+declined:
+  - "@yarnpkg/cli"

packages/acceptance-tests/pkg-tests-fixtures/packages/one-dep-alias-bins-1.0.0/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "one-dep-alias-bins",
+  "version": "1.0.0",
+  "dependencies": {
+    "@fixture/old": "npm:has-bin-entries@1.0.0"
+  }
+}

packages/acceptance-tests/pkg-tests-specs/sources/node-modules.test.ts

@@ -1,5 +1,6 @@
 import {WindowsLinkType}                           from '@yarnpkg/core';
 import {xfs, npath, PortablePath, ppath, Filename} from '@yarnpkg/fslib';
+import {parseSyml}                                 from '@yarnpkg/parsers';
 
 
 const {
@@ -150,6 +151,34 @@ describe(`Node_Modules`, () => {
     ),
   );
 
+  test(`should prefer direct dependency bins over transitive dependency bins`,
+    makeTemporaryEnv(
+      {
+        dependencies: {
+          [`@fixture/native`]: `npm:has-bin-entries@2.0.0`,
+          [`has-bin-entries`]: `npm:one-dep-alias-bins@1.0.0`,
+        },
+      },
+      {
+        nodeLinker: `node-modules`,
+      },
+      async ({path, run}) => {
+        await run(`install`);
+
+        const installState = parseSyml(await xfs.readFilePromise(ppath.join(path, `node_modules/.yarn-state.yml` as PortablePath), `utf8`));
+        const rootState = Object.values<any>(installState).find(({locations}) => locations?.includes(``));
+
+        expect(rootState?.bin?.[`.`]?.[`has-bin-entries-with-relative-require`]).toEqual(`@fixture/native/bin-with-relative-require.js`);
+
+        if (process.platform !== `win32`) {
+          await expect(run(`node`, `${path}/node_modules/.bin/has-bin-entries-with-relative-require`)).resolves.toMatchObject({
+            stdout: `2.0.0\n`,
+          });
+        }
+      },
+    ),
+  );
+
   test(`should support dependency via link: protocol to a missing folder`,
     makeTemporaryEnv(
       {

packages/plugin-nm/sources/NodeModulesLinker.ts

@@ -1053,7 +1053,9 @@ async function createBinSymlinkMap(installState: NodeModulesLocatorMap, location
       for (const [childLocation, childNode] of node.children) {
         const childSymlinks = getBinSymlinks(ppath.join(location, childLocation), parentLocatorLocation, childNode);
         for (const [name, symlinkTarget] of childSymlinks) {
-          symlinks.set(name, symlinkTarget);
+          if (!symlinks.has(name)) {
+            symlinks.set(name, symlinkTarget);
+          }
         }
       }
     }