From 8dd322caaa8acf431f9d80d0861078f9c86b2224 Mon Sep 17 00:00:00 2001 From: Alexander Akait <4567934+alexander-akait@users.noreply.github.com> Date: Wed, 30 Jul 2025 15:03:50 +0300 Subject: [PATCH 1/2] allow to catch parsing errors Now you can see: ``` Error parsing bundle asset "/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/dist/public/index.mjs": Cannot read properties of undefined (reading 'arguments') { cause: TypeError: Cannot read properties of undefined (reading 'arguments') at getModulesLocations (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/webpack-bundle-analyzer/lib/parseUtils.js:254:24) at Object.ExpressionStatement (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/webpack-bundle-analyzer/lib/parseUtils.js:44:35) at c (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/acorn-walk/dist/walk.js:58:37) at Object.skipThrough (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/acorn-walk/dist/walk.js:186:39) at c (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/acorn-walk/dist/walk.js:58:37) at base.Program.base.BlockStatement.base.StaticBlock (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/acorn-walk/dist/walk.js:198:7) at c (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/acorn-walk/dist/walk.js:58:37) at Object.recursive (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/acorn-walk/dist/walk.js:59:7) at parseBundle (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/webpack-bundle-analyzer/lib/parseUtils.js:26:8) at Object.getViewerData (/home/alexander-akait/IdeaProjects/webpack-bundle-analyzer-reproducer-409/node_modules/webpack-bundle-analyzer/lib/analyzer.js:87:22) } ``` Useful for https://github.com/webpack-contrib/webpack-bundle-analyzer/issues/409#issuecomment-3134857678 --- src/analyzer.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/analyzer.js b/src/analyzer.js index 52a1cf44..54375f95 100644 --- a/src/analyzer.js +++ b/src/analyzer.js @@ -81,7 +81,7 @@ function getViewerData(bundleStats, bundleDir, opts) { bundleInfo = parseBundle(assetFile, {sourceType: statAsset.info.javascriptModule ? 'module' : 'script'}); } catch (err) { const msg = (err.code === 'ENOENT') ? 'no such file' : err.message; - logger.warn(`Error parsing bundle asset "${assetFile}": ${msg}`); + logger.warn(`Error parsing bundle asset "${assetFile}": ${msg}`, { cause: err }); continue; } From 4a00c61d376b2ec35b56f20b213cfb268bbfb9d8 Mon Sep 17 00:00:00 2001 From: Vesa Laakso <482561+valscion@users.noreply.github.com> Date: Wed, 30 Jul 2025 20:16:22 +0300 Subject: [PATCH 2/2] Fix lint --- src/analyzer.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/analyzer.js b/src/analyzer.js index 54375f95..ddcbef7f 100644 --- a/src/analyzer.js +++ b/src/analyzer.js @@ -81,7 +81,7 @@ function getViewerData(bundleStats, bundleDir, opts) { bundleInfo = parseBundle(assetFile, {sourceType: statAsset.info.javascriptModule ? 'module' : 'script'}); } catch (err) { const msg = (err.code === 'ENOENT') ? 'no such file' : err.message; - logger.warn(`Error parsing bundle asset "${assetFile}": ${msg}`, { cause: err }); + logger.warn(`Error parsing bundle asset "${assetFile}": ${msg}`, {cause: err}); continue; }