stats include less-loader, css-loader about webpack-visualizer HOT 7 CLOSED

Same thing here.
The less-loader/css-loader output is effectively stripped from bundle by the ExtractTextPlugin

But stats.json still show them. Maybe an issue in the generation of json output? (i.e., using --json argument)

An possible workaround is to check the issuer field since it has the following pattern when is extracted from bundle:

\\node_modules\\extract-text-webpack-plugin\\loader.js?{\"remove\":true}

from webpack-visualizer.

@chrisbateman It seems less-loader is still affected (not seeing css-loader though) https://www.dropbox.com/s/9on4hjfafdjy9zv/Screenshot%202017-01-09%2014.40.25.png?dl=0
Using v 0.1.7 which was released 15 hours ago.

from webpack-visualizer.

@motin So right now I believe we're only removing CSS if it uses the ExtractTextPlugin - which moves the CSS to a separate file. Otherwise, the CSS is still included in your JS bundle.

from webpack-visualizer.

@chrisbateman ExtractTextPlugin is used in our config, and the CSS is placed in a separate file because of to that. The screenshot however shows that "less-loader" still is included, even though it is only a dev-dependency in our project.

from webpack-visualizer.

@motin Hmm, I don't think I'm seeing that in my test app.

Any chance you could share your loader config and maybe even your stats.json?

from webpack-visualizer.

I analyzed the stats of one of my projects, see below

Stats: https://1drv.ms/u/s!AhehaflTFeyS9CR_UiVT6Z7CVxaG
https://github.com/blikblum/marionette-wires-revisited/blob/master/src/main.js

All css is required by 'main.less' and extracted into a separate file (checked the actual bundle to make sure).

Tracking main.less entries in stats log i got the following:

• main.less entry as id 212 with name as "./src/main.less"
  extract-text-plugin is in identifier path, but NOT in issuer
  index !== null
  "chunks": [0]

• second main.less entry as id 213 with name as ".//css-loader!.//less-loader!./src/main.less"
  extract-text-plugin is NOT in identifier path, but is in issuer
  index !== null
  "chunks": []

later comes all *.less files required by main.less, the first being wells.less with the following pattern

• name: "./~ /less-loader/stringify.loader.js!./~/bootstrap/less/[name of the less file]"
  extract-text-plugin is NOT in identifier path and is NOT in issuer
  index: null
  "chunks": []

With this i modified the buildHierarchy to account these situations. Will do a PR to be reviewed

from webpack-visualizer.

@blikblum Awesome. I think it maybe it wasn't showing up in my example since none of my less files were in the node_modules. Thanks very much!

from webpack-visualizer.