perf: parse options only one time

Author: ricardogobbosouza

src/index.js

@@ -33,16 +33,30 @@ class ESLintWebpackPlugin {
     // this differentiates one from the other when being cached.
     this.key = compiler.name || `${this.key}_${(counter += 1)}`;
 
+    const options = {
+      ...this.options,
+      exclude: parseFiles(
+        this.options.exclude || [],
+        this.getContext(compiler)
+      ),
+      extensions: arrify(this.options.extensions),
+      files: parseFiles(this.options.files || '', this.getContext(compiler)),
+    };
+
+    const wanted = parseFoldersToGlobs(options.files, options.extensions);
+    const exclude = parseFoldersToGlobs(
+      this.options.exclude ? options.exclude : '**/node_modules/**',
+      []
+    );
+
     // If `lintDirtyModulesOnly` is disabled,
     // execute the linter on the build
     if (!this.options.lintDirtyModulesOnly) {
-      compiler.hooks.run.tapPromise(this.key, this.run);
+      compiler.hooks.run.tapPromise(this.key, (c) =>
+        this.run(c, options, wanted, exclude)
+      );
     }
 
-    // TODO: Figure out want `compiler.watching` is and how to use it in Webpack5.
-    // From my testing of compiler.watch() ... compiler.watching is always
-    // undefined (webpack 4 doesn't define it either) I'm leaving it out
-    // for now.
     let isFirstRun = this.options.lintDirtyModulesOnly;
     compiler.hooks.watchRun.tapPromise(this.key, (c) => {
       if (isFirstRun) {
@@ -51,39 +65,26 @@ class ESLintWebpackPlugin {
         return Promise.resolve();
       }
 
-      return this.run(c);
+      return this.run(c, options, wanted, exclude);
     });
   }
 
   /**
    * @param {Compiler} compiler
+   * @param {Options} options
+   * @param {string[]} wanted
+   * @param {string[]} exclude
    */
-  async run(compiler) {
+  async run(compiler, options, wanted, exclude) {
     // Do not re-hook
     if (
       // @ts-ignore
-      compiler.hooks.compilation.taps.find(({ name }) => name === this.key)
+      compiler.hooks.thisCompilation.taps.find(({ name }) => name === this.key)
     ) {
       return;
     }
 
-    const options = {
-      ...this.options,
-      exclude: parseFiles(
-        this.options.exclude || [],
-        this.getContext(compiler)
-      ),
-      extensions: arrify(this.options.extensions),
-      files: parseFiles(this.options.files || '', this.getContext(compiler)),
-    };
-
-    const wanted = parseFoldersToGlobs(options.files, options.extensions);
-    const exclude = parseFoldersToGlobs(
-      this.options.exclude ? options.exclude : '**/node_modules/**',
-      []
-    );
-
-    compiler.hooks.compilation.tap(this.key, (compilation) => {
+    compiler.hooks.thisCompilation.tap(this.key, (compilation) => {
       /** @type {import('./linter').Linter} */
       let lint;
       /** @type {import('./linter').Reporter} */