eslint
diff --git a/‎lib/rules/no-array-constructor.js
Copy file name to clipboard
+51-1Lines changed: 51 additions & 1 deletion b/‎lib/rules/no-array-constructor.js
Copy file name to clipboard
+51-1Lines changed: 51 additions & 1 deletion
@@ -34,6 +34,8 @@ module.exports = {
 			url: "https://eslint.org/docs/latest/rules/no-array-constructor",
 		},
 
+		fixable: "code",
+
 		hasSuggestions: true,
 
 		schema: [],
@@ -49,6 +51,30 @@ module.exports = {
 	create(context) {
 		const sourceCode = context.sourceCode;
 
+		/**
+		 * Checks if there are comments in Array constructor expressions.
+		 * @param {ASTNode} node A CallExpression or NewExpression node.
+		 * @returns {boolean} True if there are comments, false otherwise.
+		 */
+		function hasCommentsInArrayConstructor(node) {
+			const firstToken = sourceCode.getFirstToken(node);
+			const lastToken = sourceCode.getLastToken(node);
+
+			let lastRelevantToken = sourceCode.getLastToken(node.callee);
+
+			while (
+				lastRelevantToken !== lastToken &&
+				!isOpeningParenToken(lastRelevantToken)
+			) {
+				lastRelevantToken = sourceCode.getTokenAfter(lastRelevantToken);
+			}
+
+			return sourceCode.commentsExistBetween(
+				firstToken,
+				lastRelevantToken,
+			);
+		}
+
 		/**
 		 * Gets the text between the calling parentheses of a CallExpression or NewExpression.
 		 * @param {ASTNode} node A CallExpression or NewExpression node.
@@ -107,6 +133,17 @@ module.exports = {
 				let fixText;
 				let messageId;
 
+				const nonSpreadCount = node.arguments.reduce(
+					(count, arg) =>
+						arg.type !== "SpreadElement" ? count + 1 : count,
+					0,
+				);
+
+				const shouldSuggest =
+					node.optional ||
+					(node.arguments.length > 0 && nonSpreadCount < 2) ||
+					hasCommentsInArrayConstructor(node);
+
 				/*
 				 * Check if the suggested change should include a preceding semicolon or not.
 				 * Due to JavaScript's ASI rules, a missing semicolon may be inserted automatically
@@ -127,10 +164,23 @@ module.exports = {
 				context.report({
 					node,
 					messageId: "preferLiteral",
+					fix(fixer) {
+						if (shouldSuggest) {
+							return null;
+						}
+
+						return fixer.replaceText(node, fixText);
+					},
 					suggest: [
 						{
 							messageId,
-							fix: fixer => fixer.replaceText(node, fixText),
+							fix(fixer) {
+								if (shouldSuggest) {
+									return fixer.replaceText(node, fixText);
+								}
+
+								return null;
+							},
 						},
 					],
 				});