fix typescript-eslint#6188

Maxim-Mazurok · Apr 9, 2023 · 36195e6 · 36195e6
1 parent f0f83ce
commit 36195e6
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 11 deletions.
diff --git a/packages/eslint-plugin/src/util/collectUnusedVariables.ts b/packages/eslint-plugin/src/util/collectUnusedVariables.ts
@@ -423,20 +423,21 @@ function isMergableExported(variable: TSESLint.Scope.Variable): boolean {
  * @returns True if the variable is exported, false if not.
  */
 function isExported(variable: TSESLint.Scope.Variable): boolean {
-  const definition = variable.defs[0];
+  const exportedDefinition = variable.defs.find(definition => {
+    if (definition) {
+      let node = definition.node;
 
-  if (definition) {
-    let node = definition.node;
+      if (node.type === AST_NODE_TYPES.VariableDeclarator) {
+        node = node.parent!;
+      } else if (definition.type === TSESLint.Scope.DefinitionType.Parameter) {
+        return false;
+      }
 
-    if (node.type === AST_NODE_TYPES.VariableDeclarator) {
-      node = node.parent!;
-    } else if (definition.type === TSESLint.Scope.DefinitionType.Parameter) {
-      return false;
+      return node.parent!.type.indexOf('Export') === 0;
     }
-
-    return node.parent!.type.indexOf('Export') === 0;
-  }
-  return false;
+    return false;
+  });
+  return exportedDefinition !== undefined;
 }
 
 /**

diff --git a/packages/eslint-plugin/tests/rules/no-unused-vars/no-unused-vars.test.ts b/packages/eslint-plugin/tests/rules/no-unused-vars/no-unused-vars.test.ts
@@ -1072,6 +1072,12 @@ export class Foo {
         typescript: '4.4',
       },
     },
+    `
+interface Foo {
+  bar: string;
+}
+export const Foo = 'bar';
+    `,
   ],
 
   invalid: [