Huge bundle size in react-native for sentry/react-native (264kb)

[jayshah123]

Environment

How do you use Sentry?
Sentry SaaS (sentry.io)

Which SDK and version?
sentry-react-native 3.0.0
React native 0.63.2

Steps to Reproduce

1. analyze using npx react-native-bundle-visualizer in a project that contains sentry/react-native

Expected Result

Smaller bundle size, I see packages like browser etc included which would probably not be needed.

Actual Result

264kb bundle

[shamilovtim]

Sentry team can treeshake in react native with require + try/catch. Dev dependencies should not be imported and should be behind an env var for imports.

[marandaneto]

@jayshah123 is 264kb before or after treeshake?

browser and integrations are vitals for the RN SDK so we can't remove them, but only used code should be bundled anyway.

[jayshah123]

Metro does not have treeshaking as far as I know

[shamilovtim]

Treeshaking in metro is not automatic but can be done manually with require + try/catch. if you ever import a dev dependency you must treeshake it or else you will bloat the production bundle with a bunch of useless code. For instance, this is how you can treeshake reactotron:

if (__DEV__) {
  try {
    const Reactotron = require('reactotron-react-native');
    Reactotron.configure() // controls connection & communication settings
      .useReactNative() // add all built-in react native plugins
      .connect(); // let's connect!
  } catch (e) {}
}

[marandaneto]

dup of #1733