From f1b28a6d3292e8d96f75a34cdc88b32861287e38 Mon Sep 17 00:00:00 2001 From: Lukas Stracke Date: Mon, 27 May 2024 13:18:33 +0200 Subject: [PATCH 1/2] fix(aws-serverless): Avoid minifying `Module._resolveFilename` in Lambda layer bundle --- dev-packages/rollup-utils/plugins/bundlePlugins.mjs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dev-packages/rollup-utils/plugins/bundlePlugins.mjs b/dev-packages/rollup-utils/plugins/bundlePlugins.mjs index 2404e3fc2d38..5344b8b0a21b 100644 --- a/dev-packages/rollup-utils/plugins/bundlePlugins.mjs +++ b/dev-packages/rollup-utils/plugins/bundlePlugins.mjs @@ -128,6 +128,9 @@ export function makeTerserPlugin() { '_sentrySpan', '_sentryScope', '_sentryIsolationScope', + // require-in-the-middle calls `Module._resolveFilename`. We cannot mangle this (AWS lambda layer bundle). + '_resolveFilename', + '__proto__', ], }, }, @@ -136,6 +139,7 @@ export function makeTerserPlugin() { }, }); } +‘ // We don't pass these plugins any options which need to be calculated or changed by us, so no need to wrap them in // another factory function, as they are themselves already factory functions. From 3b96745e9777befa9133c7e77921b1644c1eced2 Mon Sep 17 00:00:00 2001 From: Lukas Stracke Date: Mon, 27 May 2024 13:19:40 +0200 Subject: [PATCH 2/2] cleanup --- dev-packages/rollup-utils/plugins/bundlePlugins.mjs | 2 -- 1 file changed, 2 deletions(-) diff --git a/dev-packages/rollup-utils/plugins/bundlePlugins.mjs b/dev-packages/rollup-utils/plugins/bundlePlugins.mjs index 5344b8b0a21b..169062694d24 100644 --- a/dev-packages/rollup-utils/plugins/bundlePlugins.mjs +++ b/dev-packages/rollup-utils/plugins/bundlePlugins.mjs @@ -130,7 +130,6 @@ export function makeTerserPlugin() { '_sentryIsolationScope', // require-in-the-middle calls `Module._resolveFilename`. We cannot mangle this (AWS lambda layer bundle). '_resolveFilename', - '__proto__', ], }, }, @@ -139,7 +138,6 @@ export function makeTerserPlugin() { }, }); } -‘ // We don't pass these plugins any options which need to be calculated or changed by us, so no need to wrap them in // another factory function, as they are themselves already factory functions.