v4 breaks css-module-loader

[jquense]

Hey @evilebottnawi the change that only adds the ICSS plugins breaks the strategy used by https://github.com/jquense/css-module-loader to convert modules into :import and :export rules which depended on the (admittedly weird) behavior of css-loader always including the icss plugin.

It's not enough to turn on modules tho b/c css-loader will then rehash everything. For this sort of thing to work we need an api to handle downstream import/exports without css-loader trying to process css-modules. This is also true for the new auto default. it's not safe to assume any .module.css file should be processed by css-loader when there are loaders ahead of it.

Would love if we could come up with a first class API for this!

[alexander-akait]

I don't understand the problem, please clarify

This is also true for the new auto default. it's not safe to assume any .module.css file should be processed by css-loader when there are loaders ahead of it.

It is breaking change, you can disable this behavior

[jquense]

css-module-loader converts css to ICSS :export and :import rules, css-loader would process those without also processing @value and classes to create the JS exports. Now it doesn't because the ICSS plugin is only included when modules is true.

css-module-loader is incompatible with css-loader with modules turned on because both loaders would process the file for css modules when it is already done

At the moment there is no way to have a loader above css-loader process css-modules, in v3 there was a hacky way to do it.

[alexander-akait]

@jquense Maybe you need icss: boolean (false by default), if you specify icss: true, it is unable only icssParser (no css modules plugins), if you set modules: true we enable icss: true by default. Is this what you are looking for? Should be very easy to implement

[jquense]

if you set modules: true we enable icss: true by default.

I think that would work, let me get a PR together if that sounds reasonable

[alexander-akait]

@jquense Go ahead, it is allow fix hacky setup for css-module-loader

[alexander-akait]

Fixed, now you can add { modules: { auto: false }, icss: true } and all should be work fine, feel free to feelback