chore: reduce Webpack log noise due to missing source maps in externa…

…l deps (#6793)
palantir · May 13, 2024 · 03b304f · 03b304f · svc-palantir-github · May 13, 2024
1 parent 3568b75
commit 03b304f
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/packages/webpack-build-scripts/webpack.config.base.mjs b/packages/webpack-build-scripts/webpack.config.base.mjs
@@ -157,7 +157,18 @@ export default {
         rules: [
             {
                 test: /\.js$/,
-                use: fileURLToPath(import.meta.resolve("source-map-loader")),
+                loader: fileURLToPath(import.meta.resolve("source-map-loader")),
+                options: {
+                    filterSourceMappingUrl: (_url, resourcePath) => {
+                        // These external modules (e.g. parse5) contain #sourceMappingUrl comments that point towards
+                        // non-existent files. Skip them to reduce Webpack noise.
+                        if (/\/node_modules\/(parse5|parse5-htmlparser2-tree-adapter)\//i.test(resourcePath)) {
+                            return "skip";
+                        }
+
+                        return true;
+                    },
+                },
             },
             {
                 test: /\.tsx?$/,

diff --git a/packages/webpack-build-scripts/webpack.config.karma.mjs b/packages/webpack-build-scripts/webpack.config.karma.mjs
@@ -35,7 +35,18 @@ export default {
         rules: [
             {
                 test: /\.js$/,
-                use: fileURLToPath(import.meta.resolve("source-map-loader")),
+                loader: fileURLToPath(import.meta.resolve("source-map-loader")),
+                options: {
+                    filterSourceMappingUrl: (_url, resourcePath) => {
+                        // These external modules (e.g. parse5) contain #sourceMappingUrl comments that point towards
+                        // non-existent files. Skip them to reduce Webpack noise.
+                        if (/\/node_modules\/(parse5|parse5-htmlparser2-tree-adapter)\//i.test(resourcePath)) {
+                            return "skip";
+                        }
+
+                        return true;
+                    },
+                },
             },
             {
                 test: /\.tsx?$/,