Make this work with yarn berry (aka yarn 2/3/4) about import-meta-resolve HOT 7 CLOSED

Can you check the issues, it's been discussed. It's impossible. Or: feel free to spend like a month on this if you really want it.

Also: I think svelte itself also is like: yeah, no, not going to happen.

from import-meta-resolve.

I have made it work with the above code, so it's not impossible. It you can change the resolver to add those few lines of code to detect pnp, then it will work fine. I'm happy to help here.

Old comments about yarn pnp not working with svelte are no longer true, as bugs have been fixed.

from import-meta-resolve.

This pkg is a ton of code, it’s a copy of the actual node internals. Just using require isn’t going to work on ESM files for this project. If that’s all your use case, then that’s fine for you?

I meant Rich’s comments on not being interested in supporting PnP

from import-meta-resolve.

Please see #23 (comment) and the above and related threads, which include the yarn maintainer

from import-meta-resolve.

This pkg is a ton of code, it’s a copy of the actual node internals. Just using require isn’t going to work on ESM files for this project. If that’s all your use case, then that’s fine for you?

This package will only be loaded in the PnP case, so it won't penalize anyone else.

I have not investigated how to do the fix with the ESM portions of your code, as I don't require that, but I'm sure something similar can be done in your code to handle this. If necessary we can get help from the yarn folks like @arcanis who have been very active in helping with exactly these situation. I'm willing to help too.

As I said before, I have resolved my problem by patching Prettier and bypassing the call to this package, but I'm trying to advocate for a broader fix which will help more people.

I meant Rich’s comments on not being interested in supporting PnP

Yes, I have seen this. I find it not very useful or helpful. Svelte does not need to do anything directly to support PnP, the issues are with underlying packages (like this one). His comment is also quite old. yarn pnp is very actively developed and constantly improving.

from import-meta-resolve.

Ok well #23 (comment) still stands. I think it’s going to take several weeks for someone to implement. It would be a fork of this project or a completely different project. If someone wants to make that, we can talk again after that.
It basically means copy/pasting a lot of Node into a project. At that point, using Node itself seems simpler to me.

from import-meta-resolve.

See PR #28

from import-meta-resolve.