OpenTelemetry dependencies in @sentry/node-core

[aivus]

Hello there,

As a part of #18502 @andreiborza made @sentry/opentelemetry and @opentelemetry/* dependencies optional.

Couple of days later in #19308 @sentry/opentelemetry dependency was moved back to prod dependencies. This made all the optionalities of @opentelemetry/* in #18502 useless, leading to always installing the "optional" dependencies.

@andreiborza maybe you have any plans to get rid of opentelemetry from direct deps there? Or maybe you have plans to create a separate package for lightweight version?

[andreiborza]

Hi @aivus,

You're right, this makes them not optional anymore. Curiously, if you use yarn v1 they do not get installed, but they do get installed when using npm or pnpm. Given we are all using yarn v1 in this repo this kind of flew under the radar.

We are planning to move away from OpenTelemetry dependencies as a whole with the next major and a lot of this will go away.

Even if these are pulled in, if you're just using @sentry/node-core/light, none of these dependencies are used so technically you could also delete them from you node_modules folder if that's a concern.

Is this causing any issues for you?

[aivus]

Thanks for the quick reply. Got it.

Is this causing any issues for you?

I'm maintaining an application for Homey (Smart Home Hub) which uses the library to interact with Sentry. The lib uses old raven module and I'm currently preparing the changes to switch from raven to @sentry/node-core.

After the switch I noticed that the Homey application package size increased from 3 MB to 31 MB because of the extra dependencies.

The Smart Home Hub is resource limited device, so I'm interested in the reducing the size of the dependencies.