fix(core): upgrade webpack-bundle-analyzer to 5.3.0 (#7389)

Author: colinaaa

e2e/cases/performance/bundle-analyzer/index.test.ts

@@ -32,3 +32,22 @@ test('should emit bundle analyze report correctly when build', async ({
   await rsbuild.expectLog('Webpack Bundle Analyzer saved report to');
   expect(filePaths.length).toBe(1);
 });
+
+test('should surface stats file write errors when statsFilename points to a directory', async ({
+  build,
+}) => {
+  const rsbuild = await build({
+    config: {
+      performance: {
+        bundleAnalyze: {
+          generateStatsFile: true,
+          statsFilename: '.',
+        },
+      },
+    },
+  });
+
+  await rsbuild.expectLog('Webpack Bundle Analyzer error saving stats file to');
+  await rsbuild.expectLog('EISDIR');
+  expect(rsbuild.buildError).toBeFalsy();
+});

packages/core/package.json

@@ -100,7 +100,7 @@
     "tinyglobby": "0.2.14",
     "typescript": "^5.9.3",
     "webpack": "^5.104.1",
-    "webpack-bundle-analyzer": "^4.10.2",
+    "webpack-bundle-analyzer": "^5.3.0",
     "webpack-merge": "6.0.1",
     "ws": "^8.18.3"
   },

packages/core/prebundle.config.ts

@@ -116,10 +116,34 @@ export default {
         webpack: 'webpack',
       },
       afterBundle(task) {
+        const distDir = join(task.distPath, 'dist');
+        const bundledEntryPath = join(task.distPath, 'index.js');
+        const distEntryPath = join(distDir, 'index.js');
+
+        // Keep the package entry at the root, but execute the bundled file from
+        // `dist/index.js` so webpack-bundle-analyzer still resolves `../public`
+        // relative to its own package directory.
+        fs.mkdirSync(distDir, { recursive: true });
+        fs.renameSync(bundledEntryPath, distEntryPath);
+        fs.copyFileSync(
+          join(task.distPath, 'package.json'),
+          join(distDir, 'package.json'),
+        );
+        fs.writeFileSync(
+          bundledEntryPath,
+          "module.exports = require('./dist/index.js');\n",
+        );
+
         // webpack type does not exist, use `@rspack/core` instead
         replaceFileContent(join(task.distPath, 'index.d.ts'), (content) =>
           content.replace("from 'webpack'", 'from "@rspack/core"'),
         );
+
+        // Static report generation in webpack-bundle-analyzer v5 resolves
+        // `public/viewer.js` from the package root at runtime.
+        fs.cpSync(join(task.depPath, 'public'), join(task.distPath, 'public'), {
+          recursive: true,
+        });
       },
     },
     {

packages/core/tests/__snapshots__/bundleAnalyzer.test.ts.snap

@@ -7,6 +7,7 @@ exports[`plugin-bundle-analyze > should add bundle analyze plugin 1`] = `
       "name": "RsbuildCorePlugin",
     },
     BundleAnalyzerPlugin {
+      "compiler": null,
       "logger": Logger {
         "activeLevels": Set {
           "info",
@@ -20,6 +21,7 @@ exports[`plugin-bundle-analyze > should add bundle analyze plugin 1`] = `
         "analyzerMode": "static",
         "analyzerPort": 8888,
         "analyzerUrl": [Function],
+        "compressionAlgorithm": "gzip",
         "defaultSizes": "parsed",
         "excludeAssets": null,
         "generateStatsFile": false,
@@ -29,7 +31,7 @@ exports[`plugin-bundle-analyze > should add bundle analyze plugin 1`] = `
         "reportTitle": [Function],
         "startAnalyzer": true,
         "statsFilename": "stats.json",
-        "statsOptions": null,
+        "statsOptions": undefined,
       },
       "server": null,
     },
@@ -44,6 +46,7 @@ exports[`plugin-bundle-analyze > should add bundle analyze plugin when bundle an
       "name": "RsbuildCorePlugin",
     },
     BundleAnalyzerPlugin {
+      "compiler": null,
       "logger": Logger {
         "activeLevels": Set {
           "info",
@@ -57,6 +60,7 @@ exports[`plugin-bundle-analyze > should add bundle analyze plugin when bundle an
         "analyzerMode": "static",
         "analyzerPort": 8888,
         "analyzerUrl": [Function],
+        "compressionAlgorithm": "gzip",
         "defaultSizes": "parsed",
         "excludeAssets": null,
         "generateStatsFile": false,
@@ -66,7 +70,7 @@ exports[`plugin-bundle-analyze > should add bundle analyze plugin when bundle an
         "reportTitle": [Function],
         "startAnalyzer": true,
         "statsFilename": "stats.json",
-        "statsOptions": null,
+        "statsOptions": undefined,
       },
       "server": null,
     },
@@ -81,6 +85,7 @@ exports[`plugin-bundle-analyze > should enable bundle analyze plugin when perfor
       "name": "RsbuildCorePlugin",
     },
     BundleAnalyzerPlugin {
+      "compiler": null,
       "logger": Logger {
         "activeLevels": Set {
           "info",
@@ -94,6 +99,7 @@ exports[`plugin-bundle-analyze > should enable bundle analyze plugin when perfor
         "analyzerMode": "disabled",
         "analyzerPort": 8888,
         "analyzerUrl": [Function],
+        "compressionAlgorithm": "gzip",
         "defaultSizes": "parsed",
         "excludeAssets": null,
         "generateStatsFile": true,
@@ -103,7 +109,7 @@ exports[`plugin-bundle-analyze > should enable bundle analyze plugin when perfor
         "reportTitle": [Function],
         "startAnalyzer": true,
         "statsFilename": "stats.json",
-        "statsOptions": null,
+        "statsOptions": undefined,
       },
       "server": null,
     },