fix(unused-imported-specifier): fix unused specifier rule - wrong det…

…ection
tsedio · Mar 30, 2024 · 6bed750 · 6bed750
1 parent 86bff13
commit 6bed750
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/rules/unused-imported-specifier/unusedImportedSpecifier.spec.ts b/src/rules/unused-imported-specifier/unusedImportedSpecifier.spec.ts
@@ -5,7 +5,7 @@ const ruleTester = new RuleTester({
   parser: "@typescript-eslint/parser"
 });
 
-describe("missingImportSpecifiers", () => {
+describe("unusedImportedSpecifier", () => {
   ruleTester.run(name, rule, {
     valid: [
       {

diff --git a/src/rules/unused-imported-specifier/unusedImportedSpecifier.ts b/src/rules/unused-imported-specifier/unusedImportedSpecifier.ts
@@ -29,21 +29,21 @@ export const rule = createRule<[], RULES>({
         const program = context.sourceCode.ast;
         const imports = getImportDeclarations(program);
 
-        const isUsed = program.tokens.reduce((isUsed, token, currentIndex) => {
-          if (token.type === "Identifier" && program.tokens[currentIndex + 1].type === "Punctuator" && program.tokens[currentIndex - 1].value === "@") {
-            isUsed.add(token.value);
+        const tokens = program.tokens.reduce((isUsed, token, currentIndex) => {
+          if (token.type === "Identifier") {
+            isUsed.set(token.value, (isUsed.get(token.value) || 0) + 1);
           }
 
           return isUsed;
-        }, new Set());
+        }, new Map());
 
         imports
           .filter((importDeclaration) => {
             return importDeclaration.source.value.startsWith("@tsed/");
           })
           .map((importDeclaration) => {
             importDeclaration.specifiers.forEach((specifier, index, specifiers) => {
-              if (!isUsed.has(specifier.local.name)) {
+              if (tokens.get(specifier.local.name) === 1) {
                 context.report({
                   node: specifier,
                   messageId: "unused-imported-specifier",