fix: false positives in 5.4.0 for functions that aren't ESLint rules

fixes #450

Author: aladdin-add

lib/utils.js

@@ -139,15 +139,18 @@ function getRuleExportsESM(ast, scopeManager) {
           possibleNodes.push(specifier.local);
         }
         if (statement.declaration) {
+          let nodes = []
+
           if (statement.declaration.type === 'VariableDeclaration') {
-            for (const declarator of statement.declaration.declarations) {
-              if (declarator.init) {
-                possibleNodes.push(declarator.init);
-              }
-            }
+            nodes = statement.declaration.declarations.map((declarator) => declarator.init);
           } else {
-            possibleNodes.push(statement.declaration);
+            nodes = [statement.declaration];
           }
+
+          // named exports like `export const rule = { ... };`
+          // skip if it's function-style rule to avoid false positives
+          // refs: https://github.com/eslint-community/eslint-plugin-eslint-plugin/issues/450
+          possibleNodes.push(...nodes.filter((node) => node && node.type !== 'FunctionDeclaration'));
         }
         break;
       }

tests/lib/utils.js

@@ -86,6 +86,12 @@ describe('utils', () => {
         'export default function () { return {}; }',
         'export default function (foo, bar) { return {}; }',
 
+        // named export of functions
+        // refs: https://github.com/eslint-community/eslint-plugin-eslint-plugin/issues/450
+        'export function foo(options) { return {}; }',
+        'export function foo(options) { return; }',
+        'export function foo({opt1, opt2}) { return {}; }',
+
         // Incorrect TypeScript helper structure:
         'export default foo()({ create() {}, meta: {} });',
         'export default foo().bar({ create() {}, meta: {} });',