-
-
Notifications
You must be signed in to change notification settings - Fork 46
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
triggerUncaughtException with react-native 0.71.2 #116
Comments
Hi @frankcalise, Thanks for the information. Could you please add your setup details for eg: react-native version and node version? Also if you could share package.json probably dependencies and dev deps only. |
hey @hurali97 sure thing! Here is a reproducible repo (with the dep list below). It also did this for me on version https://github.com/frankcalise/react-native-vanilla-reanimated
|
Hi @frankcalise, thanks for sharing this. I tried on react-native version 0.70.6 and below, it works fine. Seems there's something that's conflicting in later versions. I should see what's the cause of it. |
UPDATE @frankcalise I cloned your provided repro and ran See the last one, which is successful output with dev bundle. If visualising dev bundle helps unblocking you, please go with that. As I don't have any timeline to get this working for release bundle. I will try to prioritise this. 🤝 |
Ok I will try dev again, but I thought I had the same issue. Maybe I'll try with clearing the cache first. Are there any temp files I should clear? Thanks for looking into it! |
You can try running the visualiser command with |
@hurali97 visualizer:dev script ran on 0.71.1 And also was able to run |
Cool, thanks for checking 😃 My wild guess is there's probably something changed in the internals of RN or any third party library. I will try to keep looking for the root cause of this meanwhile, to get this working on recent RN versions in release mode. |
Hi @frankcalise, I pushed a fix on another branch, you can try it by running following command:
I tried to locate the source of issue in a couple of third party packages, including RN-CLI, React-Native and Metro. It turned out that recently metro has defaulted Now For future, I will try to remove this more of a patch fix, and allow us using the minification. As I am not really much familiar with these minifier, transformers stuff, I can't promise a timeline for that, but I will try keep looking for ideas around it. As of now, this fix/ patch should unblock us even on latest RN versions on both dev and release. Let me know, if this fix works for you. I will try to publish this fix soon. ✨ |
@hurali97 Thanks for your hard work! Yeah they both run now with that patch. With regards to the minification, does skipping that impact the bundle size that is being reported now? Or is release mode still a true release bundle size? |
Hi @frankcalise, Yes it impacts the bundle size to some extent as lines, spaces, function names, variable names etc won't be minified due to this. Other than this, release level optimisations must be applied on the bundle. So if you visualize bundle with dev true and later on with dev false, you should see a green line in terminal which would say: "bundle size decreased by __". |
I'm trying to utilize bundle visualizer with the latest react-native and I'm running into this output:
Any suggestions or would you like any other information attached? Open to trying patches and other solutions
The text was updated successfully, but these errors were encountered: