refactor: update eslint configuration

Author: alexander-akait

examples/build-common.js

@@ -71,8 +71,8 @@ const doCompileAndReplace = (args, prefix, callback) => {
 
 	try {
 		require.resolve("webpack-cli");
-	} catch (_err) {
-		throw new Error("Please install webpack-cli at root.");
+	} catch (err) {
+		throw new Error("Please install webpack-cli at root.", { cause: err });
 	}
 
 	/**

jest.config.js

@@ -1,6 +1,6 @@
 "use strict";
 
-/** @type {import('jest').Config} */
+/** @type {import("jest").Config} */
 const config = {
 	testTimeout: 30000,
 	prettierPath: require.resolve("prettier-2"),

lib/Chunk.js

@@ -207,7 +207,7 @@ class Chunk {
 
 	/**
 	 * @param {Chunk} otherChunk the chunk to compare with
-	 * @returns {-1|0|1} the comparison result
+	 * @returns {-1 | 0 | 1} the comparison result
 	 */
 	compareTo(otherChunk) {
 		const chunkGraph = ChunkGraph.getChunkGraphForChunk(

lib/ChunkGraph.js

@@ -873,7 +873,7 @@ class ChunkGraph {
 	/**
 	 * @param {Chunk} chunkA first chunk
 	 * @param {Chunk} chunkB second chunk
-	 * @returns {-1|0|1} this is a comparator function like sort and returns -1, 0, or 1 based on sort order
+	 * @returns {-1 | 0 | 1} this is a comparator function like sort and returns -1, 0, or 1 based on sort order
 	 */
 	compareChunks(chunkA, chunkB) {
 		const cgcA = this._getChunkGraphChunk(chunkA);

lib/ChunkGroup.js

@@ -45,7 +45,7 @@ const getArray = (set) => [...set];
  * A convenience method used to sort chunks based on their id's
  * @param {ChunkGroup} a first sorting comparator
  * @param {ChunkGroup} b second sorting comparator
- * @returns {1|0|-1} a sorting index to determine order
+ * @returns {1 | 0 | -1} a sorting index to determine order
  */
 const sortById = (a, b) => {
 	if (a.id < b.id) return -1;
@@ -56,7 +56,7 @@ const sortById = (a, b) => {
 /**
  * @param {OriginRecord} a the first comparator in sort
  * @param {OriginRecord} b the second comparator in sort
- * @returns {1|-1|0} returns sorting order as index
+ * @returns {1 | -1| 0} returns sorting order as index
  */
 const sortOrigin = (a, b) => {
 	const aIdent = a.module ? a.module.identifier() : "";
@@ -488,7 +488,7 @@ class ChunkGroup {
 	 * Sorting values are based off of number of chunks in ChunkGroup.
 	 * @param {ChunkGraph} chunkGraph the chunk graph
 	 * @param {ChunkGroup} otherGroup the chunkGroup to compare this against
-	 * @returns {-1|0|1} sort position for comparison
+	 * @returns {-1 | 0 | 1} sort position for comparison
 	 */
 	compareTo(chunkGraph, otherGroup) {
 		if (this.chunks.length > otherGroup.chunks.length) return -1;

lib/CleanPlugin.js

@@ -391,7 +391,7 @@ class CleanPlugin {
 		// We assume that no external modification happens while the compiler is active
 		// So we can store the old assets and only diff to them to avoid fs access on
 		// incremental builds
-		/** @type {undefined|Assets} */
+		/** @type {undefined | Assets} */
 		let oldAssets;
 
 		compiler.hooks.emit.tapAsync(

lib/Compilation.js

@@ -710,8 +710,8 @@ class Compilation {
 			) => `Can't automatically convert plugin using Compilation.hooks.${name} to Compilation.hooks.processAssets because ${reason}.
 BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a single Compilation.hooks.processAssets hook.`;
 			/**
-			 * @param {string | (import("tapable").TapOptions & { name: string; } & ProcessAssetsAdditionalOptions)} options hook options
-			 * @returns {import("tapable").TapOptions & { name: string; } & ProcessAssetsAdditionalOptions} modified options
+			 * @param {string | (import("tapable").TapOptions & { name: string } & ProcessAssetsAdditionalOptions)} options hook options
+			 * @returns {import("tapable").TapOptions & { name: string } & ProcessAssetsAdditionalOptions} modified options
 			 */
 			const getOptions = (options) => {
 				if (typeof options === "string") options = { name: options };

lib/DefinePlugin.js

@@ -44,7 +44,7 @@ const createHash = require("./util/createHash");
  * @property {string[]=} contextDependencies
  * @property {string[]=} missingDependencies
  * @property {string[]=} buildDependencies
- * @property {string| (() => string)=} version
+ * @property {string | (() => string)=} version
  */
 
 /** @typedef {(value: { module: NormalModule, key: string, readonly version: ValueCacheVersion }) => CodeValuePrimitive} GeneratorFn */

lib/ExternalModule.js

@@ -87,7 +87,7 @@ const RUNTIME_REQUIREMENTS_FOR_SCRIPT = new Set([RuntimeGlobals.loadScript]);
 const RUNTIME_REQUIREMENTS_FOR_MODULE = new Set([
 	RuntimeGlobals.definePropertyGetters
 ]);
-const EMPTY_RUNTIME_REQUIREMENTS = new Set([]);
+const EMPTY_RUNTIME_REQUIREMENTS = new Set();
 
 /**
  * @param {string | string[]} variableName the variable name or path

lib/ExternalModuleFactoryPlugin.js

@@ -26,7 +26,7 @@ const { cachedSetProperty, resolveByProperty } = require("./util/cleverMerge");
 /** @typedef {import("./ModuleFactory").ModuleFactoryCreateDataContextInfo} ModuleFactoryCreateDataContextInfo */
 /** @typedef {import("./NormalModuleFactory")} NormalModuleFactory */
 
-/** @typedef {((context: string, request: string, callback: (err?: Error | null, result?: string | false, resolveRequest?: import('enhanced-resolve').ResolveRequest) => void) => void)} ExternalItemFunctionDataGetResolveCallbackResult */
+/** @typedef {((context: string, request: string, callback: (err?: Error | null, result?: string | false, resolveRequest?: import("enhanced-resolve").ResolveRequest) => void) => void)} ExternalItemFunctionDataGetResolveCallbackResult */
 /** @typedef {((context: string, request: string) => Promise<string>)} ExternalItemFunctionDataGetResolveResult */
 /** @typedef {(options?: ResolveOptions) => ExternalItemFunctionDataGetResolveCallbackResult | ExternalItemFunctionDataGetResolveResult} ExternalItemFunctionDataGetResolve */