[Bug?]: node-modules - Dependency with different major version in nested workspace not installed

Self-service

• I’d be willing to implement a fix

Describe the bug

I have two nested packages. The outer one depends on ESLint 7, the inner one on ESLint 8. Only ESLint 7 is installed.

To reproduce

const { writeFile } = require("fs").promises;

await packageJson({
  private: true,
  dependencies: {
    eslint: "^7.0.0"
  }
},{ cwd: "workspace" })

await packageJson({
  private: true,
  dependencies: {
    eslint: "8.0.0-beta.2"
  }
},{ cwd: "workspace/nested" })

await writeFile(".yarnrc.yml", `nodeLinker: node-modules`);
await writeFile("workspace/nested/index.js", `module.exports = require("eslint/package.json").version`);

await packageJsonAndInstall({
  workspaces: ["workspace", "workspace/nested"]
});

const version = await node("workspace/nested/index.js");

expect(version).toBe("8.0.0-beta.2");

If this sherlock reproduction doesn’t work, you can clone this branch: https://github.com/nicolo-ribaudo/babel/tree/yarn-bug-missing-dep. The last commit shows the package necessary to introduce the bug; you can run node eslint/babel-eslint-parser/test/eslint-8/index.js and it should print 8.0.0-beta.2.

Environment

System:
    OS: Linux 5.11 Ubuntu 21.04 (Hirsute Hippo)
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Binaries:
    Node: 16.8.0 - /tmp/xfs-cc5b8bc4/node
    Yarn: 2.4.1-git.20210125.0f31e6910 - /tmp/xfs-cc5b8bc4/yarn
    npm: 7.21.1 - ~/.npm-global/bin/npm

Additional context

No response

[larixer]

@nicolo-ribaudo Just a heads up, I’ve took a stab to fix this issue at #3438, checked your example and it works for me now. Gotta add integration tests to the PR to prevent future regressions in the following days.

[larixer]

Seems to be a valid issue; the workspace/nested copy of eslint shouldn’t be hoisted into its parent workspace (probably happens because of having a workspace inside a workspace).

The problem is actually different. The workspaces/nested copy is hoisted to the top-level, not to the parent workspace. The eslint copy in the parent workspace is correct. Looking…