[2.0 Beta] Yarn PnP unable to apply resolve patch since resolve is now bundled in dist files.

[AlexandreBonaventure]

Describe the bug

Yarn PnP is patching the resolve lib : https://www.npmjs.com/package/resolve
Now with vite@2.0.0-beta being bundled by rollup, Yarn is unable to apply the patch when vite tries to resolve linked dependencies for instance here:

vite/packages/vite/src/node/optimizer/index.ts

Line 384 in 7858e62

const depRoot = path.dirname(resolveFrom(`${dep}/package.json`, config.root))

An easy fix would be to externalize it from the bundle in the rollup options. Otherwise, a resolve hook via config would be nice to be able ro modify internal module resolution in userland.

System Info

• required vite version: 2.0.0-beta
• required Operating System: Mac OS
• required Node version: 14

[AlexandreBonaventure]

NB: this is a regression

[yyx990803]

Can you confirm this is the only issue with Yarn PnP?

[AlexandreBonaventure]

Thanks ! @yyx990803, I've just tried the new release and I can see another problem with linked dependencies resolution.
Suppose we have a monorepo such as:

|_ Vite App  (require A)
|_ Dep A (require B)
|_ Dep B

We need to resolve link dep recursively by resolving the dep from the importer root as opposed to config.root.
I've made a quick change here to show what I mean:
AlexandreBonaventure@ca79884

[yyx990803]

Can you open a PR for that?