fix: improve esMododule exports

Author: jantimon

README.md

@@ -407,7 +407,6 @@ module.exports = {
 - **[`publicPath`](#publicPath)**
 - **[`emit`](#emit)**
 - **[`esModule`](#esModule)**
-- **[`defaultExport`](#defaultExport)**
 
 #### `publicPath`
 
@@ -550,47 +549,6 @@ module.exports = {
 };
 ```
 
-#### `defaultExport`
-
-Type:
-
-```ts
-type defaultExport = boolean;
-```
-
-Default: `false`
-
-By default, `mini-css-extract-plugin` generates JS modules with a default export.
-However for name exports, each local is exported as a named export.
-
-In case you need both default and named exports, you can enable this option:
-
-**webpack.config.js**
-
-```js
-const MiniCssExtractPlugin = require("mini-css-extract-plugin");
-
-module.exports = {
-  plugins: [new MiniCssExtractPlugin()],
-  module: {
-    rules: [
-      {
-        test: /\.css$/i,
-        use: [
-          {
-            loader: MiniCssExtractPlugin.loader,
-            options: {
-              defaultExport: true,
-            },
-          },
-          "css-loader",
-        ],
-      },
-    ],
-  },
-};
-```
-
 ## Examples
 
 ### Recommended

src/index.js

@@ -36,7 +36,6 @@ const {
  * @property {string | ((resourcePath: string, rootContext: string) => string)} [publicPath]
  * @property {boolean} [emit]
  * @property {boolean} [esModule]
- * @property {boolean} [defaultExport]
  * @property {string} [layer]
  */
 

src/loader-options.json

@@ -25,11 +25,6 @@
       "description": "Generates JS modules that use the ES modules syntax.",
       "link": "https://github.com/webpack-contrib/mini-css-extract-plugin#esmodule"
     },
-    "defaultExport": {
-      "type": "boolean",
-      "description": "Generates JS modules with the default export syntax even for named exports.",
-      "link": "https://github.com/webpack-contrib/mini-css-extract-plugin#defaultexports"
-    },
     "layer": {
       "type": "string"
     }

src/loader.js

@@ -124,11 +124,6 @@ function pitch(request) {
     const esModule =
       typeof options.esModule !== "undefined" ? options.esModule : true;
 
-    const defaultExport =
-      typeof options.defaultExport !== "undefined"
-        ? options.defaultExport
-        : false;
-
     /**
      * @param {Dependency[] | [null, object][]} dependencies
      */
@@ -276,10 +271,10 @@ function pitch(request) {
           const exportsString = `export { ${identifiers
             .map(([id, key]) => `${id} as ${JSON.stringify(key)}`)
             .join(", ")} }`;
-          const exportDefaultString = defaultExport
-            ? `export default ${JSON.stringify(locals)}\n`
-            : "";
-          return `${localsString}\n${exportsString}\n${exportDefaultString}`;
+          const exportDefaultString = `export default ${JSON.stringify(
+            locals
+          )}`;
+          return `${localsString}\n${exportsString}\n${exportDefaultString}\n`;
         }
 
         return `\n${

test/cases/custom-loader-with-functional-exports/expected/main.js

@@ -8,12 +8,14 @@
 __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
 /* harmony export */   cnA: () => (/* binding */ _1),
-/* harmony export */   cnB: () => (/* binding */ _2)
+/* harmony export */   cnB: () => (/* binding */ _2),
+/* harmony export */   "default": () => (__WEBPACK_DEFAULT_EXPORT__)
 /* harmony export */ });
 // extracted by mini-css-extract-plugin
 var _1 = () => "class-name-a";
 var _2 = () => "class-name-b";
 
+/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = ({});
 
 
 /***/ })

test/cases/es-module-concatenation-modules/expected/main.js

@@ -48,14 +48,16 @@ __webpack_require__.d(__webpack_exports__, {
 var a_namespaceObject = {};
 __webpack_require__.r(a_namespaceObject);
 __webpack_require__.d(a_namespaceObject, {
-  a: () => (_1)
+  a: () => (_1),
+  "default": () => (a)
 });
 
 // NAMESPACE OBJECT: ./b.css
 var b_namespaceObject = {};
 __webpack_require__.r(b_namespaceObject);
 __webpack_require__.d(b_namespaceObject, {
-  b: () => (b_1)
+  b: () => (b_1),
+  "default": () => (b)
 });
 
 // NAMESPACE OBJECT: ./index.js
@@ -71,16 +73,19 @@ __webpack_require__.d(index_namespaceObject, {
 // extracted by mini-css-extract-plugin
 var _1 = "foo__a";
 
+/* harmony default export */ const a = ({"a":"foo__a"});
 
 ;// CONCATENATED MODULE: ./b.css
 // extracted by mini-css-extract-plugin
 var b_1 = "foo__b";
 
+/* harmony default export */ const b = ({"b":"foo__b"});
 
 ;// CONCATENATED MODULE: ./c.css
 // extracted by mini-css-extract-plugin
 var c_1 = "foo__c";
 
+/* harmony default export */ const c = ({"c":"foo__c"});
 
 ;// CONCATENATED MODULE: ./index.js
 /* eslint-disable import/no-namespace */

test/cases/es-named-export-as-is-output-module/expected/main.mjs

@@ -7,13 +7,15 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
 /* harmony export */   "a-class": () => (/* binding */ _1),
 /* harmony export */   b__class: () => (/* binding */ _2),
-/* harmony export */   cClass: () => (/* binding */ _3)
+/* harmony export */   cClass: () => (/* binding */ _3),
+/* harmony export */   "default": () => (__WEBPACK_DEFAULT_EXPORT__)
 /* harmony export */ });
 // extracted by mini-css-extract-plugin
 var _1 = "Xh041yLR4iCP4RGjge50";
 var _2 = "NMuRsxoDwvW8BhSXhFAY";
 var _3 = "ayWIv09rPsAqE2JznIsI";
 
+/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = ({"a-class":"Xh041yLR4iCP4RGjge50","b__class":"NMuRsxoDwvW8BhSXhFAY","cClass":"ayWIv09rPsAqE2JznIsI"});
 
 
 /***/ })

test/cases/es-named-export-as-is/expected/main.js

@@ -9,13 +9,15 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
 /* harmony export */   "a-class": () => (/* binding */ _1),
 /* harmony export */   b__class: () => (/* binding */ _2),
-/* harmony export */   cClass: () => (/* binding */ _3)
+/* harmony export */   cClass: () => (/* binding */ _3),
+/* harmony export */   "default": () => (__WEBPACK_DEFAULT_EXPORT__)
 /* harmony export */ });
 // extracted by mini-css-extract-plugin
 var _1 = "Xh041yLR4iCP4RGjge50";
 var _2 = "NMuRsxoDwvW8BhSXhFAY";
 var _3 = "ayWIv09rPsAqE2JznIsI";
 
+/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = ({"a-class":"Xh041yLR4iCP4RGjge50","b__class":"NMuRsxoDwvW8BhSXhFAY","cClass":"ayWIv09rPsAqE2JznIsI"});
 
 
 /***/ })

test/cases/es-named-export-output-module/expected/main.mjs

@@ -7,13 +7,15 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
 /* harmony export */   aClass: () => (/* binding */ _1),
 /* harmony export */   bClass: () => (/* binding */ _2),
-/* harmony export */   cClass: () => (/* binding */ _3)
+/* harmony export */   cClass: () => (/* binding */ _3),
+/* harmony export */   "default": () => (__WEBPACK_DEFAULT_EXPORT__)
 /* harmony export */ });
 // extracted by mini-css-extract-plugin
 var _1 = "foo__style__a-class";
 var _2 = "foo__style__b__class";
 var _3 = "foo__style__cClass";
 
+/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = ({"aClass":"foo__style__a-class","bClass":"foo__style__b__class","cClass":"foo__style__cClass"});
 
 
 /***/ })