eslint 9.36.0 → 9.37.0

package/lib/rules/generator-star-spacing.js

@@ -33,7 +33,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/implicit-arrow-linebreak.js

@@ -17,7 +17,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/indent-legacy.js

@@ -35,6 +35,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "4.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/indent.js

@@ -512,7 +512,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/jsx-quotes.js

@@ -44,7 +44,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/key-spacing.js

@@ -151,7 +151,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/keyword-spacing.js

@@ -82,7 +82,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/line-comment-position.js

@@ -18,7 +18,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "9.3.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/linebreak-style.js

@@ -31,7 +31,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/lines-around-comment.js

@@ -60,7 +60,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/lines-around-directive.js

@@ -56,7 +56,7 @@ module.exports = {
 			message: "The rule was replaced with a more general rule.",
 			url: "https://eslint.org/blog/2017/06/eslint-v4.0.0-released/",
 			deprecatedSince: "4.0.0",
-			availableUntil: null,
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message: "The new rule moved to a plugin.",

package/lib/rules/lines-between-class-members.js

@@ -37,7 +37,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/max-len.js

@@ -71,7 +71,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/max-statements-per-line.js

@@ -22,7 +22,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/multiline-comment-style.js

@@ -18,7 +18,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "9.3.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/multiline-ternary.js

@@ -19,7 +19,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/new-parens.js

@@ -27,7 +27,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/newline-after-var.js

@@ -42,7 +42,7 @@ module.exports = {
 			message: "The rule was replaced with a more general rule.",
 			url: "https://eslint.org/blog/2017/06/eslint-v4.0.0-released/",
 			deprecatedSince: "4.0.0",
-			availableUntil: null,
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message: "The new rule moved to a plugin.",

package/lib/rules/newline-before-return.js

@@ -30,7 +30,7 @@ module.exports = {
 			message: "The rule was replaced with a more general rule.",
 			url: "https://eslint.org/blog/2017/06/eslint-v4.0.0-released/",
 			deprecatedSince: "4.0.0",
-			availableUntil: null,
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message: "The new rule moved to a plugin.",

package/lib/rules/newline-per-chained-call.js

@@ -20,7 +20,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-confusing-arrow.js

@@ -33,7 +33,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-extra-parens.js

@@ -21,7 +21,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-extra-semi.js

@@ -24,7 +24,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-floating-decimal.js

@@ -23,7 +23,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-loss-of-precision.js

@@ -124,14 +124,11 @@ module.exports = {
 		 * @returns {Object} the object containing the integer's coefficient and order of magnitude
 		 */
 		function normalizeInteger(stringInteger) {
-			const significantDigits = removeTrailingZeros(
-				removeLeadingZeros(stringInteger),
-			);
+			const trimmedInteger = removeLeadingZeros(stringInteger);
+			const significantDigits = removeTrailingZeros(trimmedInteger);
 
 			return {
-				magnitude: stringInteger.startsWith("0")
-					? stringInteger.length - 2
-					: stringInteger.length - 1,
+				magnitude: trimmedInteger.length - 1,
 				coefficient: addDecimalPointToNumber(significantDigits),
 			};
 		}
@@ -140,7 +137,7 @@ module.exports = {
 		 *
 		 * Converts a float to an object containing the floats's coefficient and order of magnitude
 		 * @param {string} stringFloat the string representation of the float being converted
-		 * @returns {Object} the object containing the integer's coefficient and order of magnitude
+		 * @returns {Object} the object containing the float's coefficient and order of magnitude
 		 */
 		function normalizeFloat(stringFloat) {
 			const trimmedFloat = removeLeadingZeros(stringFloat);
@@ -169,7 +166,7 @@ module.exports = {
 		 * @returns {string} the number converted to scientific notation
 		 */
 		function convertNumberToScientificNotation(stringNumber) {
-			const splitNumber = stringNumber.replace("E", "e").split("e");
+			const splitNumber = stringNumber.split("e");
 			const originalCoefficient = splitNumber[0];
 			const normalizedNumber = stringNumber.includes(".")
 				? normalizeFloat(originalCoefficient)

package/lib/rules/no-mixed-operators.js

@@ -90,7 +90,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-mixed-spaces-and-tabs.js

@@ -16,7 +16,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-multi-spaces.js

@@ -19,7 +19,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-multiple-empty-lines.js

@@ -17,7 +17,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-restricted-imports.js

@@ -41,6 +41,11 @@ const arrayOfStringsOrObjects = {
 							type: "string",
 						},
 					},
+					allowTypeImports: {
+						type: "boolean",
+						description:
+							"Whether to allow type-only imports for a path.",
+					},
 				},
 				additionalProperties: false,
 				required: ["name"],
@@ -105,6 +110,11 @@ const arrayOfStringsOrObjectPatterns = {
 					caseSensitive: {
 						type: "boolean",
 					},
+					allowTypeImports: {
+						type: "boolean",
+						description:
+							"Whether to allow type-only imports for a pattern.",
+					},
 				},
 				additionalProperties: false,
 				not: {
@@ -137,6 +147,8 @@ const arrayOfStringsOrObjectPatterns = {
 module.exports = {
 	meta: {
 		type: "suggestion",
+		dialects: ["typescript", "javascript"],
+		language: "javascript",
 
 		docs: {
 			description: "Disallow specified modules when loaded by `import`",
@@ -261,6 +273,7 @@ module.exports = {
 						message: importSource.message,
 						importNames: importSource.importNames,
 						allowImportNames: importSource.allowImportNames,
+						allowTypeImports: importSource.allowTypeImports,
 					});
 				}
 				return memo;
@@ -291,6 +304,7 @@ module.exports = {
 				importNamePattern,
 				allowImportNames,
 				allowImportNamePattern,
+				allowTypeImports,
 			}) => ({
 				...(group
 					? {
@@ -313,6 +327,7 @@ module.exports = {
 				importNamePattern,
 				allowImportNames,
 				allowImportNamePattern,
+				allowTypeImports,
 			}),
 		);
 
@@ -324,6 +339,48 @@ module.exports = {
 			return {};
 		}
 
+		/**
+		 * Check if the node is a type-only import
+		 * @param {ASTNode} node The node to check
+		 * @returns {boolean} Whether the node is a type-only import
+		 */
+		function isTypeOnlyImport(node) {
+			return (
+				node.importKind === "type" ||
+				(node.specifiers?.length > 0 &&
+					node.specifiers.every(
+						specifier => specifier.importKind === "type",
+					))
+			);
+		}
+
+		/**
+		 * Check if a specifier is type-only
+		 * @param {ASTNode} specifier The specifier to check
+		 * @returns {boolean} Whether the specifier is type-only
+		 */
+		function isTypeOnlySpecifier(specifier) {
+			return (
+				specifier.importKind === "type" ||
+				specifier.exportKind === "type"
+			);
+		}
+
+		/**
+		 * Check if the node is a type-only export
+		 * @param {ASTNode} node The node to check
+		 * @returns {boolean} Whether the node is a type-only export
+		 */
+		function isTypeOnlyExport(node) {
+			return (
+				node.exportKind === "type" ||
+				(node.specifiers?.length > 0 &&
+					node.specifiers.every(
+						specifier => specifier.exportKind === "type",
+					))
+			);
+		}
+
 		/**
 		 * Report a restricted path.
 		 * @param {string} importSource path of the import
@@ -344,6 +401,28 @@ module.exports = {
 						restrictedPathEntry.importNames;
 					const allowedImportNames =
 						restrictedPathEntry.allowImportNames;
+					const allowTypeImports =
+						restrictedPathEntry.allowTypeImports;
+
+					// Skip if this is a type-only import and it's allowed for this specific entry
+					if (
+						allowTypeImports &&
+						(node.type === "ImportDeclaration" ||
+							node.type === "TSImportEqualsDeclaration") &&
+						isTypeOnlyImport(node)
+					) {
+						return;
+					}
+
+					// Skip if this is a type-only export and it's allowed for this specific entry
+					if (
+						allowTypeImports &&
+						(node.type === "ExportNamedDeclaration" ||
+							node.type === "ExportAllDeclaration") &&
+						isTypeOnlyExport(node)
+					) {
+						return;
+					}
 
 					if (!restrictedImportNames && !allowedImportNames) {
 						context.report({
@@ -400,6 +479,14 @@ module.exports = {
 							restrictedImportNames.includes(importName)
 						) {
 							specifiers.forEach(specifier => {
+								// Skip if this is a type-only import specifier and type imports are allowed
+								if (
+									allowTypeImports &&
+									isTypeOnlySpecifier(specifier.specifier)
+								) {
+									return;
+								}
+
 								context.report({
 									node,
 									messageId: customMessage
@@ -420,6 +507,14 @@ module.exports = {
 							!allowedImportNames.includes(importName)
 						) {
 							specifiers.forEach(specifier => {
+								// Skip if this is a type-only import specifier and type imports are allowed
+								if (
+									allowTypeImports &&
+									isTypeOnlySpecifier(specifier.specifier)
+								) {
+									return;
+								}
+
 								context.report({
 									node,
 									loc: specifier.loc,
@@ -446,11 +541,30 @@ module.exports = {
 		 * @param {Object} group contains an Ignore instance for paths, the customMessage to show on failure,
 		 * and any restricted import names that have been specified in the config
 		 * @param {Map<string,Object[]>} importNames Map of import names that are being imported
+		 * @param {string} importSource the import source string
 		 * @returns {void}
 		 * @private
 		 */
-		function reportPathForPatterns(node, group, importNames) {
-			const importSource = node.source.value.trim();
+		function reportPathForPatterns(node, group, importNames, importSource) {
+			// Skip if this is a type-only import and it's allowed
+			if (
+				group.allowTypeImports &&
+				(node.type === "ImportDeclaration" ||
+					node.type === "TSImportEqualsDeclaration") &&
+				isTypeOnlyImport(node)
+			) {
+				return;
+			}
+
+			// Skip if this is a type-only export and it's allowed
+			if (
+				group.allowTypeImports &&
+				(node.type === "ExportNamedDeclaration" ||
+					node.type === "ExportAllDeclaration") &&
+				isTypeOnlyExport(node)
+			) {
+				return;
+			}
 
 			const customMessage = group.customMessage;
 			const restrictedImportNames = group.importNames;
@@ -553,6 +667,14 @@ module.exports = {
 						restrictedImportNamePattern.test(importName))
 				) {
 					specifiers.forEach(specifier => {
+						// Skip if this is a type-only import specifier and type imports are allowed
+						if (
+							group.allowTypeImports &&
+							isTypeOnlySpecifier(specifier.specifier)
+						) {
+							return;
+						}
+
 						context.report({
 							node,
 							messageId: customMessage
@@ -573,6 +695,14 @@ module.exports = {
 					!allowedImportNames.includes(importName)
 				) {
 					specifiers.forEach(specifier => {
+						// Skip if this is a type-only import specifier and type imports are allowed
+						if (
+							group.allowTypeImports &&
+							isTypeOnlySpecifier(specifier.specifier)
+						) {
+							return;
+						}
+
 						context.report({
 							node,
 							messageId: customMessage
@@ -592,6 +722,14 @@ module.exports = {
 					!allowedImportNamePattern.test(importName)
 				) {
 					specifiers.forEach(specifier => {
+						// Skip if this is a type-only import specifier and type imports are allowed
+						if (
+							group.allowTypeImports &&
+							isTypeOnlySpecifier(specifier.specifier)
+						) {
+							return;
+						}
+
 						context.report({
 							node,
 							messageId: customMessage
@@ -640,7 +778,7 @@ module.exports = {
 			} else if (node.specifiers) {
 				for (const specifier of node.specifiers) {
 					let name;
-					const specifierData = { loc: specifier.loc };
+					const specifierData = { loc: specifier.loc, specifier };
 
 					if (specifier.type === "ImportDefaultSpecifier") {
 						name = "default";
@@ -665,7 +803,12 @@ module.exports = {
 			checkRestrictedPathAndReport(importSource, importNames, node);
 			restrictedPatternGroups.forEach(group => {
 				if (isRestrictedPattern(importSource, group)) {
-					reportPathForPatterns(node, group, importNames);
+					reportPathForPatterns(
+						node,
+						group,
+						importNames,
+						importSource,
+					);
 				}
 			});
 		}
@@ -678,6 +821,30 @@ module.exports = {
 				}
 			},
 			ExportAllDeclaration: checkNode,
+			// Add support for TypeScript import equals declarations
+			TSImportEqualsDeclaration(node) {
+				if (node.moduleReference.type === "TSExternalModuleReference") {
+					const importSource = node.moduleReference.expression.value;
+					const importNames = new Map();
+
+					// Use existing logic with the actual node
+					checkRestrictedPathAndReport(
+						importSource,
+						importNames,
+						node,
+					);
+					restrictedPatternGroups.forEach(group => {
+						if (isRestrictedPattern(importSource, group)) {
+							reportPathForPatterns(
+								node,
+								group,
+								importNames,
+								importSource,
+							);
+						}
+					});
+				}
+			},
 		};
 	},
 };

package/lib/rules/no-spaced-func.js

@@ -26,7 +26,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2016/08/eslint-v3.3.0-released/#deprecated-rules",
 			deprecatedSince: "3.3.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-tabs.js

@@ -24,7 +24,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-trailing-spaces.js

@@ -30,7 +30,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/no-whitespace-before-property.js

@@ -22,7 +22,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

package/lib/rules/nonblock-statement-body-position.js

@@ -18,7 +18,7 @@ module.exports = {
 			message: "Formatting rules are being moved out of ESLint core.",
 			url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
 			deprecatedSince: "8.53.0",
-			availableUntil: "10.0.0",
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message: