fix: allow to use module class name (#1649)

alexander-akait · web-flow · commit 01869bcace63 · 2026-01-27T19:01:48.000+03:00
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/utils.js b/src/utils.js
@@ -1183,6 +1183,7 @@ const keywords = new Set([
   "volatile",
   "while",
   "with",
+  "module",
 ]);
 
 function getExportCode(
diff --git a/test/__snapshots__/modules-option.test.js.snap b/test/__snapshots__/modules-option.test.js.snap
@@ -12431,6 +12431,29 @@ exports[`"modules" option should work with \`default\` class and without named e
 
 exports[`"modules" option should work with \`default\` class and without named export: warnings 1`] = `[]`;
 
+exports[`"modules" option should work with \`module\` class and with named export: errors 1`] = `[]`;
+
+exports[`"modules" option should work with \`module\` class and with named export: module 1`] = `
+"// Imports
+import ___CSS_LOADER_API_NO_SOURCEMAP_IMPORT___ from "../../../../src/runtime/noSourceMaps.js";
+import ___CSS_LOADER_API_IMPORT___ from "../../../../src/runtime/api.js";
+var ___CSS_LOADER_EXPORT___ = ___CSS_LOADER_API_IMPORT___(___CSS_LOADER_API_NO_SOURCEMAP_IMPORT___);
+// Module
+___CSS_LOADER_EXPORT___.push([module.id, \`.jeU4gq1x5FP0ehaW1cBG {
+  background: red
+}
+\`, ""]);
+// Exports
+var _1 = \`jeU4gq1x5FP0ehaW1cBG\`;
+export { _1 as "module" };
+export default ___CSS_LOADER_EXPORT___;
+"
+`;
+
+exports[`"modules" option should work with \`module\` class and with named export: result 1`] = `"jeU4gq1x5FP0ehaW1cBG"`;
+
+exports[`"modules" option should work with \`module\` class and with named export: warnings 1`] = `[]`;
+
 exports[`"modules" option should work with CSS nesting: errors 1`] = `[]`;
 
 exports[`"modules" option should work with CSS nesting: module 1`] = `
diff --git a/test/fixtures/modules/issue-1590/source.css b/test/fixtures/modules/issue-1590/source.css
@@ -0,0 +1,3 @@
+.module {
+  background: red
+}
diff --git a/test/fixtures/modules/issue-1590/source.js b/test/fixtures/modules/issue-1590/source.js
@@ -0,0 +1,5 @@
+import * as css from './source.css';
+
+__export__ = css.module;
+
+export default css;
diff --git a/test/modules-option.test.js b/test/modules-option.test.js
@@ -2692,4 +2692,20 @@ describe('"modules" option', () => {
     expect(getWarnings(stats)).toMatchSnapshot("warnings");
     expect(getErrors(stats)).toMatchSnapshot("errors");
   });
+
+  it("should work with `module` class and with named export", async () => {
+    const compiler = getCompiler("./modules/issue-1590/source.js", {
+      modules: true,
+    });
+    const stats = await compile(compiler);
+
+    expect(
+      getModuleSource("./modules/issue-1590/source.css", stats),
+    ).toMatchSnapshot("module");
+    expect(getExecutedCode("main.bundle.js", compiler, stats)).toMatchSnapshot(
+      "result",
+    );
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+  });
 });

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+.module {`
	`2`	`+ background: red`
	`3`	`+}`