refactor: cleanup markVariableAsUsed

Author: 9romise

lib/rules/jsx-uses-vars.js

@@ -65,7 +65,7 @@ module.exports = {
           return
         }
 
-        utils.markVariableAsUsed(context, name, node)
+        context.sourceCode.markVariableAsUsed(name, node)
       }
     }
   }

lib/utils/index.js

@@ -2015,10 +2015,6 @@ module.exports = {
   withinTypeNode,
   findVariableByIdentifier,
   getScope,
-  /**
-   * Marks a variable with the given name in the current scope as used. This affects the no-unused-vars rule.
-   */
-  markVariableAsUsed,
   /**
    * Checks whether the given node is in export default.
    * @param {ASTNode} node
@@ -2664,26 +2660,6 @@ function isTypeScriptFile(path) {
   return path.endsWith('.ts') || path.endsWith('.tsx') || path.endsWith('.mts')
 }
 
-// ------------------------------------------------------------------------------
-// ESLint Helpers
-// ------------------------------------------------------------------------------
-/**
- * Marks a variable with the given name in the current scope as used. This affects the no-unused-vars rule.
- * @param {RuleContext} context
- * @param {string} name
- * @param {ASTNode} node The node to get the current scope.
- */
-function markVariableAsUsed(context, name, node) {
-  const sourceCode = context.sourceCode
-  if (sourceCode.markVariableAsUsed) {
-    sourceCode.markVariableAsUsed(name, node)
-  } else {
-    // This function does not use the given node, but the currently visited node.
-    // If we need to determine the scope of a given node, we need to implement it yourself.
-    context.markVariableAsUsed?.(name)
-  }
-}
-
 // ------------------------------------------------------------------------------
 // Vue Helpers
 // ------------------------------------------------------------------------------

typings/eslint/index.d.ts

@@ -233,9 +233,9 @@ export class SourceCode /*extends ESLintSourceCode*/ {
   getCommentsInside(node: VNODE.HasLocation): VNODE.Comment[]
 
   /** @since ESLint v8.39.0 */
-  markVariableAsUsed?(name: string, node?: VNODE.HasLocation): void
+  markVariableAsUsed(name: string, node?: VNODE.HasLocation): void
   /** @since ESLint v8.37.0 */
-  getScope?(node: VNODE.HasLocation): Scope.Scope
+  getScope(node: VNODE.HasLocation): Scope.Scope
 }
 export namespace SourceCode {
   interface Config {
@@ -351,7 +351,7 @@ export namespace Rule {
     /** @since ESLint v8.40.0 */
     sourceCode: SourceCode
     /** @deprecated removed in ESLint v9 */
-    markVariableAsUsed?(name: string): boolean
+    markVariableAsUsed(name: string): boolean
     report(descriptor: ReportDescriptor): void
 
     /** @deprecated removed in ESLint v10 */