-
-
Notifications
You must be signed in to change notification settings - Fork 212
Can't find module entry since v6 #219
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
Comments
Upgraded error boundary from 5 to 6 while upgrading to expo sdk 53 and ran into the same issue. Downgrading for the time being helped get past it for now: "react-error-boundary": "^5.0.0", I wonder if it's at all related to expo/expo#36375 |
Hmm 🤔 Sorry to hear you ran into this! I think it might be an issue with RN/Expo tooling. (I'm not sure what the state of ESM support is.) Seems potentially related to expo/expo#30323 Best bet for you might be to use version 5 of this library (for now). Or you could try one of the workarounds mentioned on that thread. |
I'm having the same issue but with a standard react web app. |
Without more details, I can’t help. I do suspect it has to do with your bundler/build tooling not supporting ESM though- in which case the recommended fix is to upgrade your tooling or downgrade to version 5 of this library |
It seems that Hermes doesn't support natively ESM.
Maybe one day: |
Hello, same here on standard react app project.
|
Please take the time to read comments above before adding a comment 🙇🏼 |
Yeah @TomCorvus, that’s unfortunate. But in this case you aren’t missing out on anything by sticking with version 5. Same features. This package has been stable for a long time, feature wise. |
I'm using parcel as bundler/build. It supports ESM as all my libraries are ESM builds. Of course I already downgraded to v5, my comment was just to tell that it's not only a react-native issue. |
Agreed that it doesn’t really have anything to do with React native, no. Just what features tooling supports. But without specific details about the tooling (including version numbers, config files) it’s difficult to do anything meaningful here. I think Parcel requires explicit opt-in: |
One thing that I noticed is that v6 installed itself in the app folder and not in the repo folder. I have a monorepo with the following structure:
There is a root package.json, and every lib/app has its own package.json. All packages are installed in the root node_modules. When I upgrade react-error-boundary in, let's say, app1, a new node_modules folder is created inside app1. And I believe that's the problem. The bundler is looking for the library in the wrong path (./node_modules instead of ./app1/node_modules). Not sure why this is happening. This is the first package that does that. |
For what it's worth, I followed the "getting started" guides for both Vite and Parcel and version 6 of this package worked fine with both out of the box. No custom configuration needed. |
Hi!
I'm trying to build without success my app with the v6 but the bundler seems to don't find the entry :
Any idea?
It is a react-native project.
Thank you.
The text was updated successfully, but these errors were encountered: