version bump 0.6.0

Author: fqborges

.gitignore

@@ -0,0 +1,3 @@
+.idea
+_ignored
+node_modules

.prettierrc

@@ -0,0 +1,3 @@
+{
+  "trailingComma": "es5"
+}

CHANGELOG.md

@@ -0,0 +1,22 @@
+## 0.6.0 (Oct 13, 2020)
+ Being overly careful here, this version is not breaking for almost all the existing users. It could possibly break in some edge cases, since it changes how modules are collected (from global to one each compilation) or if you have a workaround for a working webpack multi configuration.
+ * BREAKING (POSSIBLY): Use a dedicated cache for every compilation (Prevent arbitrary files deletion when using Webpack with multi configurations) (PR [#39](https://github.com/fqborges/webpack-fix-style-only-entries/pull/39))
+
+## 0.5.2 (Oct 07, 2020)
+ * Webpack 5 support using ModuleGraph API (PR [#38](https://github.com/fqborges/webpack-fix-style-only-entries/pull/38))
+ * npm audit fix: ([ea9dd7](https://github.com/fqborges/webpack-fix-style-only-entries/commit/ea9dd7bd7ade5b16623064a4850de39545e1e18e))
+
+## 0.5.1 (Jun 13, 2020)
+ * Fix Maximum call stack size exceeded (PR [#34](https://github.com/fqborges/webpack-fix-style-only-entries/pull/34))
+ * Added CHANGELOG ([3e3767](https://github.com/fqborges/webpack-fix-style-only-entries/commit/3e3767d8998a53d807b5d5b10643d05b800aa79a))
+
+## 0.5.0 (May 18, 2020)
+
+ * fix for [#23](https://github.com/fqborges/webpack-fix-style-only-entries/issues/23) ([37d350](https://github.com/fqborges/webpack-fix-style-only-entries/commit/37d350eec83f49c0b12729a93aa6cf2f96d92d0b))
+ * fix for [#24](https://github.com/fqborges/webpack-fix-style-only-entries/issues/24) ([d92bec](https://github.com/fqborges/webpack-fix-style-only-entries/commit/d92bec4be5fe4b97a8651a9206fa2281ce1424be))
+ * BREAKING: added styl and sass to default extensions ([247a5c](https://github.com/fqborges/webpack-fix-style-only-entries/commit/247a5c9f963e4d7598539056ab3f709b0558b4ec))
+ * added LICENSE.txt ([220e20](https://github.com/fqborges/webpack-fix-style-only-entries/commit/220e20eeb9bc95652a942812a424aadd92ec7d1f))
+
+## 0.4.0 (Sep 8, 2019)
+ * feat: add support for module js files (PR [#21](https://github.com/fqborges/webpack-fix-style-only-entries/pull/21))
+

LICENSE.txt

@@ -0,0 +1,15 @@
+ISC License
+
+Copyright (c) 2020, fqborges
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

README.md

@@ -1,16 +1,24 @@
-# webpack-fix-style-only-entries
+[![npm version](https://badge.fury.io/js/webpack-fix-style-only-entries.svg)](https://www.npmjs.com/package/webpack-fix-style-only-entries)
 
-This is a small plugin developed to solve the problem of having a style only entry (css/sass/less) generating an extra [name].js file.
+# [webpack-fix-style-only-entries](https://www.npmjs.com/package/webpack-fix-style-only-entries)
+
+This is a small plugin developed to solve the problem of having a style only entry (css/sass/less/stylus) generating an extra js file.
 
 You can find more info by reading the following issues:
 
  - https://github.com/webpack-contrib/extract-text-webpack-plugin/issues/518
  - https://github.com/webpack-contrib/mini-css-extract-plugin/issues/151
 
+View on: [Github](https://github.com/fqborges/webpack-fix-style-only-entries) - [npm](https://www.npmjs.com/package/webpack-fix-style-only-entries)
+
 ## How it works
-It just find js files from chunks of css only entries and remove the js file rom the compilation.
+It just find js files from chunks of css only entries and remove the js file from the compilation.
 
 ## How to use
+install using your package manager of choice:
+ - npm: `npm install -D webpack-fix-style-only-entries`
+ - yarn: `yarn add -D webpack-fix-style-only-entries`
+
 Require and add to webpack.config plugins.
 
 Warning: this plugin does not load styles or split your bundles, it just fix chunks of css only entries by removing the (almost) empty js file.
@@ -44,9 +52,22 @@ module.exports = {
 };
 ```
 
-### Options
- - extensions: file extensions for styles.
-   - type: Array[string]
-   - default: ["less", "scss", "css"]
-   - optional
-   - Example: to identify only 'foo' and 'bar' extensions as styles: `new FixStyleOnlyEntriesPlugin({ extensions:['foo', 'bar'] }),`
+## Options
+ 
+| Name       | Type             | Default                                | Description |
+|------------|---------------   |----------------------------------------|-------------|
+| extensions | Array[string]    | ["less", "scss", "css", "styl","sass"] | file extensions for styles      |
+| silent     | boolean          | false                                  | supress logs to console         |
+| ignore     | string or RegExp | undefined                              | match resource to be ignored    |
+
+### Example config:
+    // to identify only 'foo' and 'bar' extensions as styles
+    new FixStyleOnlyEntriesPlugin({ extensions:['foo', 'bar'] }),
+
+## Recipes
+
+### I use a javascript entry to styles:
+Give an especial extension to your file (`.css.js` for example) and configure `new FixStyleOnlyEntriesPlugin({ extensions:['css.js'] })`. See: https://github.com/fqborges/webpack-fix-style-only-entries/issues/8.
+
+### I use webpack-hot-middleware:
+Configure this plugin as `new FixStyleOnlyEntriesPlugin({ ignore: 'webpack-hot-middleware' })`. See: https://github.com/fqborges/webpack-fix-style-only-entries/issues/12 and https://github.com/fqborges/webpack-fix-style-only-entries/blob/master/test/cases/css-entry-with-ignored-hmr/webpack.config.js.

index.js

@@ -1,50 +1,103 @@
 const NAME = "webpack-fix-style-only-entries";
 
+const defaultOptions = {
+  extensions: ["less", "scss", "css", "sass", "styl"],
+  silent: false,
+  ignore: undefined,
+};
+
 class WebpackFixStyleOnlyEntriesPlugin {
   constructor(options) {
     this.apply = this.apply.bind(this);
 
-    options = options || {};
-    let extensions = options.extensions || ["less", "scss", "css"];
-    this.extensions = extensions.map(e => (e[0] === "." ? e : "." + e));
+    this.options = Object.assign({}, defaultOptions, options);
   }
 
   apply(compiler) {
+    const extensionsWithoutDots = this.options.extensions.map(e =>
+      e[0] === "." ? e.substring(1) : e
+    );
+
+    const patternOneOfExtensions = extensionsWithoutDots
+      .map(ext => escapeRegExp(ext))
+      .join("|");
+
+    const reStylesResource = new RegExp(
+      `[.](${patternOneOfExtensions})([?].*)?$`
+    );
+
     compiler.hooks.compilation.tap(NAME, compilation => {
+      const resourcesCache = [];
       compilation.hooks.chunkAsset.tap(NAME, (chunk, file) => {
-        if (chunk.hasEntryModule()) {
-          let resources;
-          if (typeof chunk.entryModule.resource == "string") {
-            resources = [chunk.entryModule.resource];
-          } else {
-            if (
-              chunk.entryModule.dependencies &&
-              chunk.entryModule.dependencies.length
-            ) {
-              const modules = chunk.entryModule.dependencies.map(
-                dep => dep.module
-              );
-              resources = modules.map(m => m.resource);
-            }
-          }
+        if (!file.endsWith(".js") && !file.endsWith(".mjs")) return;
+        if (!chunk.hasEntryModule()) return;
 
-          if (resources) {
-            if (
-              resources.every(resource =>
-                this.extensions.find(ext => resource.endsWith(ext))
-              )
-            ) {
-              if (file.endsWith(".js")) {
-                console.log("removing js from style only module: " + file);
-                chunk.files = chunk.files.filter(f => f != file);
-                delete compilation.assets[file];
-              }
-            }
+        const rawResources = collectEntryResources(compilation, chunk.entryModule, resourcesCache);
+        const resources = this.options.ignore
+          ? rawResources.filter(r => !r.match(this.options.ignore))
+          : rawResources;
+
+        const isStyleOnly =
+          resources.length &&
+          resources.every(resource => reStylesResource.test(resource));
+        if (isStyleOnly) {
+          if (!this.options.silent) {
+            console.error(
+              "webpack-fix-style-only-entries: removing js from style only module: " +
+                file
+            );
           }
+          chunk.files = chunk.files.filter(f => f != file);
+          delete compilation.assets[file];
         }
       });
     });
   }
 }
 
+function collectEntryResources(compilation, module, cache) {
+  // module.index is unique per compilation
+  // module.id can be null, not used here
+  if (cache[module.index] !== undefined) {
+    return cache[module.index];
+  }
+
+  if (typeof module.resource == "string") {
+    const resources = [module.resource];
+    cache[module.index] = resources;
+    return resources;
+  }
+
+  const resources = [];
+  if (module.dependencies) {
+    const hasModuleGraphSupport = compilation.hasOwnProperty('moduleGraph');
+    module.dependencies.forEach(dep => {
+      if(dep) {
+        const module = hasModuleGraphSupport ? compilation.moduleGraph.getModule(dep) : dep.module;
+        const originModule = hasModuleGraphSupport ? compilation.moduleGraph.getParentModule(dep) : dep.originModule;
+        const nextModule = module || originModule;
+        if (nextModule) {
+          const depResources = collectEntryResources(compilation, nextModule, cache);
+          for (let index = 0, length = depResources.length; index !== length; index++) {
+            resources.push(depResources[index]);
+          }
+        }
+      }
+    });
+  }
+
+  cache[module.index] = resources;
+  return resources;
+}
+
+// https://github.com/lodash/lodash/blob/4.17.11/lodash.js#L14274
+const reRegExpChar = /[\\^$.*+?()[\]{}|]/g;
+const reHasRegExpChar = RegExp(reRegExpChar.source);
+function escapeRegExp(string) {
+  string = String(string);
+  return string && reHasRegExpChar.test(string)
+    ? string.replace(reRegExpChar, "\\$&")
+    : string;
+}
+
 module.exports = WebpackFixStyleOnlyEntriesPlugin;