chore: use eslint-config-webpack

Author: alexander-akait

.eslintrc.js

@@ -23,7 +23,7 @@ jobs:
         uses: actions/cache@v4
         with:
           path: .eslintcache
-          key: lint-eslint-${{ runner.os }}-node-${{ hashFiles('**/yarn.lock', '**/.eslintrc.js') }}
+          key: lint-eslint-${{ runner.os }}-node-${{ hashFiles('**/yarn.lock', '**/eslint.config.mjs') }}
           restore-keys: lint-eslint-
       - run: yarn lint
   test:
@@ -52,9 +52,9 @@ jobs:
           architecture: ${{ steps.calculate_architecture.outputs.result }}
           cache: "yarn"
       - run: yarn --frozen-lockfile --ignore-engines
-        if: matrix.node-version == '10.x' || matrix.node-version == '12.x' || matrix.node-version == '14.x'
+        if: matrix.node-version == '10.x' || matrix.node-version == '12.x' || matrix.node-version == '14.x' || matrix.node-version == '16.x' || matrix.node-version == '18.x'
       - run: yarn --frozen-lockfile
-        if: matrix.node-version != '10.x' && matrix.node-version != '12.x' && matrix.node-version != '14.x'
+        if: matrix.node-version != '10.x' && matrix.node-version != '12.x' && matrix.node-version != '14.x' && matrix.node-version != '16.x' && matrix.node-version != '18.x'
       - run: yarn cover
       - uses: codecov/codecov-action@v5
         with:

.github/workflows/test.yml

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = {
 	printWidth: 80,
 	useTabs: true,

.prettierrc.js

@@ -0,0 +1,9 @@
+// eslint-disable-next-line import/no-unresolved
+import { defineConfig } from "eslint/config";
+import config from "eslint-config-webpack";
+
+export default defineConfig([
+	{
+		extends: [config]
+	}
+]);

eslint.config.mjs

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

jest.config.js

@@ -26,13 +26,13 @@ const {
 
 /**
  * @typedef {object} BufferedMap
- * @property {number} version
- * @property {string[]} sources
- * @property {string[]} names
- * @property {string=} sourceRoot
- * @property {(Buffer | "")[]=} sourcesContent
- * @property {Buffer=} mappings
- * @property {string} file
+ * @property {number} version version
+ * @property {string[]} sources sources
+ * @property {string[]} names name
+ * @property {string=} sourceRoot source root
+ * @property {(Buffer | "")[]=} sourcesContent sources content
+ * @property {Buffer=} mappings mappings
+ * @property {string} file file
  */
 
 /**
@@ -41,14 +41,15 @@ const {
  */
 const mapToBufferedMap = (map) => {
 	if (typeof map !== "object" || !map) return map;
-	/** @type {BufferedMap} */
-	const bufferedMap = Object.assign(/** @type {BufferedMap} */ ({}), map);
+	const bufferedMap =
+		/** @type {BufferedMap} */
+		(/** @type {unknown} */ ({ ...map }));
 	if (map.mappings) {
-		bufferedMap.mappings = Buffer.from(map.mappings, "utf-8");
+		bufferedMap.mappings = Buffer.from(map.mappings, "utf8");
 	}
 	if (map.sourcesContent) {
 		bufferedMap.sourcesContent = map.sourcesContent.map(
-			(str) => str && Buffer.from(str, "utf-8")
+			(str) => str && Buffer.from(str, "utf8")
 		);
 	}
 	return bufferedMap;
@@ -60,14 +61,15 @@ const mapToBufferedMap = (map) => {
  */
 const bufferedMapToMap = (bufferedMap) => {
 	if (typeof bufferedMap !== "object" || !bufferedMap) return bufferedMap;
-	/** @type {RawSourceMap} */
-	const map = Object.assign(/** @type {RawSourceMap} */ ({}), bufferedMap);
+	const map =
+		/** @type {RawSourceMap} */
+		(/** @type {unknown} */ ({ ...bufferedMap }));
 	if (bufferedMap.mappings) {
-		map.mappings = bufferedMap.mappings.toString("utf-8");
+		map.mappings = bufferedMap.mappings.toString("utf8");
 	}
 	if (bufferedMap.sourcesContent) {
 		map.sourcesContent = bufferedMap.sourcesContent.map(
-			(buffer) => buffer && buffer.toString("utf-8")
+			(buffer) => buffer && buffer.toString("utf8")
 		);
 	}
 	return map;
@@ -78,15 +80,14 @@ const bufferedMapToMap = (bufferedMap) => {
 
 /**
  * @typedef {object} CachedData
- * @property {boolean=} source
- * @property {Buffer} buffer
- * @property {number=} size
- * @property {BufferedMaps} maps
- * @property {(string | Buffer)[]=} hash
+ * @property {boolean=} source source
+ * @property {Buffer} buffer buffer
+ * @property {number=} size size
+ * @property {BufferedMaps} maps maps
+ * @property {(string | Buffer)[]=} hash hash
  */
 
 class CachedSource extends Source {
-	// eslint-disable-next-line valid-jsdoc
 	/**
 	 * @param {Source | (() => Source)} source source
 	 * @param {CachedData=} cachedData cached data
@@ -117,7 +118,7 @@ class CachedSource extends Source {
 		/** @type {BufferedMaps} */
 		const bufferedMaps = new Map();
 		for (const pair of this._cachedMaps) {
-			let cacheEntry = pair[1];
+			const [, cacheEntry] = pair;
 			if (cacheEntry.bufferedMap === undefined) {
 				cacheEntry.bufferedMap = mapToBufferedMap(
 					this._getMapFromCacheEntry(cacheEntry)
@@ -140,10 +141,10 @@ class CachedSource extends Source {
 				this._cachedSourceType !== undefined
 					? this._cachedSourceType
 					: typeof this._cachedSource === "string"
-					? true
-					: Buffer.isBuffer(this._cachedSource)
-					? false
-					: undefined,
+						? true
+						: Buffer.isBuffer(this._cachedSource)
+							? false
+							: undefined,
 			size: this._cachedSize,
 			maps: bufferedMaps,
 			hash: this._cachedHashUpdate
@@ -193,7 +194,7 @@ class CachedSource extends Source {
 		if (this._cachedSource !== undefined) return this._cachedSource;
 		if (this._cachedBuffer && this._cachedSourceType !== undefined) {
 			const value = this._cachedSourceType
-				? this._cachedBuffer.toString("utf-8")
+				? this._cachedBuffer.toString("utf8")
 				: this._cachedBuffer;
 			if (isDualStringBufferCachingEnabled()) {
 				this._cachedSource = /** @type {string} */ (value);
@@ -210,7 +211,7 @@ class CachedSource extends Source {
 		if (this._cachedSource !== undefined) {
 			const value = Buffer.isBuffer(this._cachedSource)
 				? this._cachedSource
-				: Buffer.from(this._cachedSource, "utf-8");
+				: Buffer.from(this._cachedSource, "utf8");
 			if (isDualStringBufferCachingEnabled()) {
 				this._cachedBuffer = value;
 			}
@@ -223,7 +224,7 @@ class CachedSource extends Source {
 		if (Buffer.isBuffer(bufferOrString)) {
 			return (this._cachedBuffer = bufferOrString);
 		}
-		const value = Buffer.from(bufferOrString, "utf-8");
+		const value = Buffer.from(bufferOrString, "utf8");
 		if (isDualStringBufferCachingEnabled()) {
 			this._cachedBuffer = value;
 		}
@@ -302,33 +303,32 @@ class CachedSource extends Source {
 					onChunk,
 					onSource,
 					onName,
-					!!(options && options.finalSource),
+					Boolean(options && options.finalSource),
 					true
 				);
-			} else {
-				return streamChunksOfRawSource(
-					/** @type {string} */
-					(source),
-					onChunk,
-					onSource,
-					onName,
-					!!(options && options.finalSource)
-				);
 			}
+			return streamChunksOfRawSource(
+				/** @type {string} */
+				(source),
+				onChunk,
+				onSource,
+				onName,
+				Boolean(options && options.finalSource)
+			);
 		}
-		const { result, source, map } = streamAndGetSourceAndMap(
+		const sourceAndMap = streamAndGetSourceAndMap(
 			this.original(),
 			options,
 			onChunk,
 			onSource,
 			onName
 		);
-		this._cachedSource = source;
+		this._cachedSource = sourceAndMap.source;
 		this._cachedMaps.set(key, {
-			map: /** @type {RawSourceMap} */ (map),
+			map: /** @type {RawSourceMap} */ (sourceAndMap.map),
 			bufferedMap: undefined
 		});
-		return result;
+		return sourceAndMap.result;
 	}
 
 	/**
@@ -361,7 +361,7 @@ class CachedSource extends Source {
 		/** @type {(string | Buffer)[]} */
 		const update = [];
 		/** @type {string | undefined} */
-		let currentString = undefined;
+		let currentString;
 		const tracker = {
 			/**
 			 * @param {string | Buffer} item item

lib/CachedSource.js

@@ -15,12 +15,12 @@ const Source = require("./Source");
 
 /**
  * @typedef {object} SourceLike
- * @property {() => SourceValue} source
- * @property {(() => Buffer)=} buffer
- * @property {(() => number)=} size
- * @property {((options?: MapOptions) => RawSourceMap | null)=} map
- * @property {((options?: MapOptions) => SourceAndMap)=} sourceAndMap
- * @property {((hash: HashLike) => void)=} updateHash
+ * @property {() => SourceValue} source source
+ * @property {(() => Buffer)=} buffer buffer
+ * @property {(() => number)=} size size
+ * @property {((options?: MapOptions) => RawSourceMap | null)=} map map
+ * @property {((options?: MapOptions) => SourceAndMap)=} sourceAndMap source and map
+ * @property {((hash: HashLike) => void)=} updateHash hash updater
  */
 
 class CompatSource extends Source {