eslint 9.36.0 → 9.38.0

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-restricted-modules.js

@@ -52,7 +52,7 @@ module.exports = {
 			message: "Node.js rules were moved out of ESLint core.",
 			url: "https://eslint.org/docs/latest/use/migrating-to-7.0.0#deprecate-node-rules",
 			deprecatedSince: "7.0.0",
-			availableUntil: null,
+			availableUntil: "11.0.0",
 			replacedBy: [
 				{
 					message:

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-sync.js

@@ -17,7 +17,7 @@ module.exports = {
 			message: "Node.js rules were moved out of ESLint core.",
 			url: "https://eslint.org/docs/latest/use/migrating-to-7.0.0#deprecate-node-rules",
 			deprecatedSince: "7.0.0",
-			availableUntil: null,
+			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:

package/lib/rules/object-curly-newline.js

@@ -162,7 +162,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/object-curly-spacing.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:

package/lib/rules/object-property-newline.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/one-var-declaration-per-line.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/operator-linebreak.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/padded-blocks.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/padding-line-between-statements.js

@@ -393,7 +393,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/preserve-caught-error.js

@@ -82,11 +82,7 @@ function getErrorCause(throwStatement) {
 			}
 
 			const causeProperties = errorOptions.properties.filter(
-				prop =>
-					prop.type === "Property" &&
-					prop.key.type === "Identifier" &&
-					prop.key.name === "cause" &&
-					!prop.computed, // It is hard to accurately identify the value of computed props
+				prop => astUtils.getStaticPropertyName(prop) === "cause",
 			);
 
 			const causeProperty = causeProperties.at(-1);

package/lib/rules/quote-props.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/quotes.js

@@ -101,7 +101,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/rest-spread-spacing.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/semi-spacing.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/semi-style.js

@@ -89,7 +89,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/semi.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/space-before-blocks.js

@@ -42,7 +42,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/space-before-function-paren.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/space-in-parens.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:

package/lib/rules/space-infix-ops.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:

package/lib/rules/space-unary-ops.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/spaced-comment.js

@@ -147,7 +147,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/switch-colon-spacing.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/template-curly-spacing.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/template-tag-spacing.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/wrap-iife.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/wrap-regex.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/yield-star-spacing.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/shared/serialization.js

@@ -56,7 +56,7 @@ function isSerializable(val, seenObjects = new Set()) {
 						/*
 						 * We're creating a new Set of seen objects because we want to
 						 * ensure that `val` doesn't appear again in this path, but it can appear
-						 * in other paths. This allows for resuing objects in the graph, as long as
+						 * in other paths. This allows for reusing objects in the graph, as long as
 						 * there are no cycles.
 						 */
 						!isSerializable(

package/lib/types/config-api.d.ts

@@ -3,6 +3,10 @@
  * @author Nicholas C. Zakas
  */
 
-import { defineConfig, globalIgnores } from "@eslint/config-helpers";
+import {
+	type Config,
+	defineConfig,
+	globalIgnores,
+} from "@eslint/config-helpers";
 
-export { defineConfig, globalIgnores };
+export { type Config, defineConfig, globalIgnores };