no-array-for-each: Add ignoreNamedIdentifier option

docs/rules/no-array-for-each.md

@@ -61,3 +61,17 @@ for (const [index, element] of array.entries()) {
 	bar(element, index, array);
 }
 ```
+
+## Options
+
+### ignoreNamedIdentifier
+
+Type: `boolean`\
+Default: `false`
+
+Ignores the rule when the callback is function identifier.
+
+```js
+// Pass
+array.forEach(bar);
+```

rules/no-array-for-each.js

@@ -385,6 +385,7 @@ const create = context => {
 	const allIdentifiers = [];
 	const functionInfo = new Map();
 	const {sourceCode} = context;
+	const {ignoreNamedIdentifier} = context.options[0] || {};
 
 	context.on(functionTypes, node => {
 		functionStack.push(node);
@@ -420,6 +421,13 @@ const create = context => {
 				method: 'forEach',
 			})
 			|| isNodeMatches(node.callee.object, ignoredObjects)
+			|| (
+				isMethodCall(node, {
+					method: 'forEach',
+				})
+				&& ignoreNamedIdentifier
+				&& node.arguments[0].type === 'Identifier'
+			)
 		) {
 			return;
 		}
@@ -468,6 +476,19 @@ const create = context => {
 	});
 };
 
+const schema = [
+	{
+		type: 'object',
+		additionalProperties: false,
+		properties: {
+			ignoreNamedIdentifier: {
+				type: 'boolean',
+				default: false,
+			},
+		},
+	},
+];
+
 /** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
 	create,
@@ -479,6 +500,7 @@ module.exports = {
 		},
 		fixable: 'code',
 		hasSuggestions: true,
+		schema,
 		messages,
 	},
 };

test/no-array-for-each.mjs

@@ -680,3 +680,30 @@ test({
 		},
 	],
 });
+
+test({
+	valid: [
+		{
+			code: `foo.forEach(bar)`,
+			options: [
+				{
+					ignoreNamedIdentifier: true,
+				},
+			],
+		},
+	],
+	invalid: [
+		{
+			code: `foo.forEach(element => bar(element))`,
+			options: [
+				{
+					ignoreNamedIdentifier: true,
+				},
+			],
+			output: outdent`
+				for (const element of foo) bar(element)
+			`,
+			errors: 1,
+		},
+	],
+})