Cannot load CSS module file which has a class named "default" after 7.x upgrade about css-loader HOT 6 CLOSED

You can't use default as a class name, it is a limitation, because we have default export in ECMA modules, we need to document it

from css-loader.

Fix - #1590, I improved logic and now we rename default class to _default automatically to avoid crash, also updated docs

from css-loader.

Anyway I added test case with:

{
  test: /\.module\.css$/,
  use: [
    'style-loader',
    {
      loader: 'css-loader',
      options: {
        modules: {
          exportLocalsConvention: 'as-is',
          namedExport: false,
        },
      },
    },
  ],
}

And can't reproduce the problem, maybe you forget to apply these options in another place

Anyway feel free to feedback (or provide reproducible test repo)

from css-loader.

@alexander-akait Thanks for making that change. Can you help me understand why this change was made in v7? Since a CSS Modules file is returned as an object I don't understand how a class called default would ever conflict with any ES keywords. It worked just fine in v6.

I would understand if the class name was something like .isPrototypeOf or .constructor, but .default isn't one of the reserved keywords for JS objects:

from css-loader.

You should not have problems with namedExport: false,, I added test and all everything fine, maybe you have more loaders/plugins, can you create minimum reproducible test repo?

from css-loader.

Ah, you're right. My bad. I saw the docs still said .default gets remapped to _default, but it's not the case with namedExport: false.

from css-loader.