aws_lambda_nodejs: exclude node_modules option from sourcemaps

[rehanvdm]

Describe the feature

Excluding sourcemaps and shipping them to your third-party provider is great, but for Lambdas using CloudWatch, you have to include them to get meaningful traces.

It is possible to exclude node_modules sourcemaps to drastically save on final zip size.

Use Case

Reduce lambda cold starts without sacraficing on the sourcemaps for your own code, which should be enough to properly debug in 90% of the cases.

Proposed Solution

This PR shows an improvement from 34K to 2.2K for the sourcemap file.

Esbuild plugins do not work in sync mode, only async. I know we can not do async work in the try bundle, but there are ways around it 😅, using deasync. Yes this is a controversial package and should not be used in any production runtime code, but seeing as this happens on build time, I feel it is an acceptable "risk" worst case the user can disable the flag if there is really an issue.

You can see it in action in this Gist I proposed some time ago.

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

CDK version used

Environment details (OS name and version, etc.)

[pahud]

Thank you for your feedback. Feel free to submit your PR when it's ready and we welcome any further discussions.

[rehanvdm]

Okay, I will try to implement this, first time contributor :)

[rehanvdm]

I looked into it and I don't think this can be done easily without a considerable rewrite. This is because this package runs the esbuild command natively instead of using the esbuild API as in my example and we need to use the API to use plugins.

If done, then my premise (below) is wrong, this change will not be as isolated as I hoped.

Yes this is a controversial package and should not be used in any production runtime code, but seeing as this happens on build time, I feel it is an acceptable "risk" worst case the user can disable the flag if there is an issue.

So I believe we can close this. I will be publishing an external package for this, something that does not have docker bundling as an option and can use esbuild plugins. I'll also be including this #28087 in that package, so I don't know if we want to close it as well.