chore(eslint-plugin): [no-unused-vars] remove unused nested TSModuleDeclaration rule listener (typescript-eslint#8279)

auvred · danvk · commit 1bc3c97d6478 · 2024-02-04T16:48:18.000-05:00
diff --git a/packages/eslint-plugin/src/rules/no-unused-vars.ts b/packages/eslint-plugin/src/rules/no-unused-vars.ts
@@ -312,23 +312,6 @@ export default createRule<Options, MessageIds>({
         markDeclarationChildAsUsed(node);
       },
 
-      // module declaration in module declaration should not report unused vars error
-      // this is workaround as this change should be done in better way
-      'TSModuleDeclaration > TSModuleDeclaration'(
-        node: TSESTree.TSModuleDeclaration,
-      ): void {
-        if (node.id.type === AST_NODE_TYPES.Identifier) {
-          let scope = getScope(context);
-          if (scope.upper) {
-            scope = scope.upper;
-          }
-          const superVar = scope.set.get(node.id.name);
-          if (superVar) {
-            superVar.eslintUsed = true;
-          }
-        }
-      },
-
       // children of a namespace that is a child of a declared namespace are auto-exported
       [ambientDeclarationSelector(
         'TSModuleDeclaration[declare = true] > TSModuleBlock TSModuleDeclaration > TSModuleBlock',
