Exclude optional performance instrumentation by default for AWS Serverless

[jd-carroll]

Exclude optional performance instrumentation by default for AWS Serverless

[mydea]

Thank you for raising this, this is a valid point. We will adjust this so that we do not include all integrations by default for serverless, meaning you'll have to manually add the DB integrations etc. you care about.

[mydea]

We just shipped v8.0.0, where we do not automatically add all auto Instrumentation anymore. This should help to reduce the default bundle size!

[jd-carroll]

Amazing, thank you for getting this in last minute ❤️

Separately, it would be nice to have a little more detail to the instrumentation customization guide.

[jd-carroll]

@mydea I just tried v8.0.0 🎊 ... and the problem still exists.

The core of the issue is that they are "statically" imported and still on a valid code path:

Here is the latest screenshot from esbuild output:

So there is no difference, despite the changes you included in v8.

Would it be possible to create a "node-core" package that is most of the current "node" package? The difference being, using @sentry/node would be an instantiation of "node-core" and contain references to all of the standard instrumentation. But then "serverless" would be an instantiation of "node-core" (instead of "node") and only have reference to:

• instrumentation-aws-sdk
• instrumentation-http
• instrumentation-aws-lambda

Then you would be required to add any additional if you wanted.

[mydea]

So just the presence/exporting of certain things should normally not affect the bundle size. As long as the code is not used, it should be tree-shaken away. How do you build your lambda function, if I may ask? Maybe there is some config option missing to ensure tree shaking works as expected 🤔

EDIT: Ignore this, I've seen there is a new issue with newer info.

[mydea]

Actually, I just noticed my "fix" did not really fix this. I will make sure to actually fix the bundle size, sorry about that...