Optimization problem with @sentry/react-native within a react-native-web app - lodash gots fully imported

[kopax-polyconseil]

OS:

• Windows
• MacOS
• Linux

Platform:

• iOS
• Android

SDK:

• @sentry/react-native (>= 1.0.0)
• react-native-sentry (<= 0.43.2)

SDK version: 4.2.2

react-native version: 0.68.2

Are you using Expo?

• Yes
• No

Are you using sentry.io or on-premise?

• sentry.io (SaaS)
• on-premise 22.5.0

Configuration:

---

I have following issue:

We use lodash.* do target exactly which module we need from lodash, but after buildeing our react-native-web app (which is also a react-native app), it appears looking at webpack-bundle-analyzer that one of our dependency (which is sentry-react-native) import the whole lodash. Is this necessary ?

see in red the multiple lodash we have now

Steps to reproduce:

• yarn add @sentry/react-native
• npm ls lodash

Actual result:

├─┬ @sentry/react-native@4.2.2
│ └─┬ @sentry/wizard@1.2.17
│   ├─┬ inquirer@6.5.2
│   │ └── lodash@4.17.21 deduped
│   └── lodash@4.17.21 deduped

Expected result:

├─┬ @sentry/react-native@4.2.2
│ └─┬ @sentry/wizard@1.2.17
│   ├─┬ inquirer@6.5.2
│   │ └── lodash.something@4.17.21
│   └── lodash.something@4.17.21

Related issues

• https://stackoverflow.com/questions/73222347/why-webpack-bundle-analyzer-show-lodash-bundled-more-than-once
• Remove this plugin from production sentry-wizard#182

[kopax-polyconseil]

Also, I do see that https://www.npmjs.com/package/inquirer is a cli tool, what's the point of shipping it into our iOS/Android app ? Could this dependence be removed?

[marandaneto]

It was removed but it's back because of #2015
We can remove it again for the 5.x release since we don't need to support older RN versions in that major bump.

[kopax-polyconseil]

Hi @marandaneto , it's great, do you have an ETA for the 5.x release ? Meanwhile I presume we must switch to lodash the whole instead of picking everything.