[Bug]: Bundle Analyzer issue about rspack

[elsassph]

@LingyuCoder the screenshot was comparing only Rspack versions, you can ignore the Webpack configuration - sorry if this is causing confusion.
The issue is that bundle analizer report has "regressed" between 0.6.x and 0.7.x.

[LingyuCoder]

@LingyuCoder the screenshot was comparing only Rspack versions, you can ignore the Webpack configuration - sorry if this is causing confusion. The issue is that bundle analizer report has "regressed" between 0.6.x and 0.7.x.

In rspack 0.6.x, all module were wrapped in webpack_modules so that webpack-bundler-analyzer can easily parse the bundle artifact and get modules.

Rspack 0.7.x extract the entry module into bootstrap, which aligned with webpack, and then the bundle analyzer can not parse it without module.depth (0 means entry module). And also, lots of modules are concatenated into entry module, so they are lost in parsed viewer.

[LingyuCoder]

Unable to run your repro. There is no rule for jsx files in rspack.config.js. However, there are jsx files inside @lightningjs/solid/dist/source/components.

[elsassph]

@LingyuCoder thanks for checking - I indeed had a patched @lightningjs/solid library so I pushed a change to ignore that for now (BTW the nasty "panic" error still happens with a JSX rule). The issue reported now appears with a clean checkout.

[LingyuCoder]

@LingyuCoder thanks for checking - I indeed had a patched @lightningjs/solid library so I pushed a change to ignore that for now (BTW the nasty "panic" error still happens with a JSX rule). The issue reported now appears with a clean checkout.

Panic should has been fixed by #6778.

In your repro, webpack uses ts-loader and rspack uses builtin:swc-loader. Helpers of swc-loader are required as commonjs modules so that they can not be full concatenated. Use swc-loader in webpack is just the same.

But rspack does not support nestedModules and depth in stats, which is used to drill-down analysis of concatenated modules. I will add them in #6784.

[elsassph]

Thanks, does that mean that we need to specify a stats module depth in the rspack config? or will that "just work" now?

[LingyuCoder]

The stats module depth has been supported by #6784. I test your repro and rspack --analyze works well. You can wait for the latest nightly version, and we will release it later today.

You can use stats.depth: true and stats.modules: true or use stats.all: true directly to enable the stats module depth.

[elsassph]

Cool, no urgency.