fix: allow module config while using autoModules (egoist#281)

Refs developit/microbundle#653, developit/microbundle#617
lufrai · Jun 26, 2020 · 5e5c5b7 · 5e5c5b7
1 parent 9489eca
commit 5e5c5b7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -27,6 +27,7 @@ export default (options = {}) => {
     /** Extract CSS */
     extract: typeof options.extract === 'undefined' ? false : options.extract,
     /** CSS modules */
+    onlyModules: options.modules === true,
     modules: inferOption(options.modules, false),
     namedExports: options.namedExports,
     /** Automatically CSS modules for .module.xxx files */

diff --git a/src/postcss-loader.js b/src/postcss-loader.js
@@ -70,8 +70,9 @@ export default {
     const shouldInject = options.inject
 
     const modulesExported = {}
-    const autoModules = options.autoModules !== false && isModuleFile(this.id)
-    const supportModules = options.modules || autoModules
+    const autoModules = options.autoModules !== false && options.onlyModules !== true
+    const isAutoModule = autoModules && isModuleFile(this.id)
+    const supportModules = autoModules ? isAutoModule : options.modules
     if (supportModules) {
       plugins.unshift(
         require('postcss-modules')({

diff --git a/test/index.test.js b/test/index.test.js
@@ -198,6 +198,7 @@ snapshotMany('modules', [
     title: 'inject-object',
     input: 'css-modules/index.js',
     options: {
+      autoModules: false,
       modules: {
         getJSON() {
           //