-
Notifications
You must be signed in to change notification settings - Fork 27k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
12.0.2 regression: can no longer import uuid package (a package with exports) with yarn pnp #31226
Comments
duplicate of #30873 |
Yes, I missed that one. Could keep this one tho since it has a reproduction to test when yarn deploys their fix. |
This may not be related, but yarn recently released their fix to canary and I tried it in the reproduction. It fixed the issue during page compilation when running Again, this is not using a stable version of yarn pnp, but it may be worth a look. |
that gives me "Error: Failed to read source code "
|
@balazsorban44 My issue is fixed on canary, and using |
@balazsorban44 seems like it's still there :/ I'm using yarn-3.2.0-rc.12.cjs trace
|
This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you. |
What version of Next.js are you using?
12.0.2, 12.0.3, and 12.0.4-canary.3
What version of Node.js are you using?
16.13.0
What browser are you using?
Chrome, Safari, Vivaldi, Brave
What operating system are you using?
macOS and Windows
How are you deploying your application?
N/A (yarn dev and yarn build)
Describe the Bug
There seems to be a regression in 12.0.2 where I can no longer import the package "uuid" (and some others) when using yarn pnp. The full error looks like this:
That error text is from windows, but it's the same on mac.
At first, #30393 seemed like it might be related, but I only started experiencing my issue in 12.0.2. If I revert to 12.0.1, everything works as expected.
I'm not sure what it is about this package that next doesn't like. It has exports set in its package.json, but that's not a new thing.
Expected Behavior
No error when compiling the page.
To Reproduce
Minimal reproduction: https://github.com/timmywil/test-nextjs-yarn-uuid
Note that this repo does not include typescript.
Also note that you still get this error when swc is disabled.
The text was updated successfully, but these errors were encountered: