remove markVariableAsUsed

Author: lilnasy

apps/oxlint/src-js/plugins/scope.ts

@@ -226,49 +226,3 @@ export function getScope(node: ESTree.Node): Scope {
 
   return scopeManager.scopes[0];
 }
-
-/**
- * Mark a variable as used in the current scope
- * @param name - The name of the variable to mark as used.
- * @param refNode? - The closest node to the variable reference.
- * @returns `true` if the variable was found and marked as used, `false` if not.
- */
-export function markVariableAsUsed(name: string, refNode?: ESTree.Node): boolean {
-  // ref: https://github.com/eslint/eslint/blob/e7cda3bdf1bdd664e6033503a3315ad81736b200/lib/languages/js/source-code/source-code.js#L991-L1023
-  const currentScope = getScope(refNode ?? SOURCE_CODE.ast);
-  let initialScope = currentScope;
-
-  /*
-   * When we are in an ESM or CommonJS module, we need to start searching
-   * from the top-level scope, not the global scope. For ESM the top-level
-   * scope is the module scope; for CommonJS the top-level scope is the
-   * outer function scope.
-   *
-   * Without this check, we might miss a variable declared with `var` at
-   * the top-level because it won't exist in the global scope.
-   */
-  if (
-    currentScope.type === 'global' &&
-    currentScope.childScopes.length > 0 &&
-    // top-level scopes refer to a `Program` node
-    currentScope.childScopes[0].block === SOURCE_CODE.ast
-  ) {
-    initialScope = currentScope.childScopes[0];
-  }
-
-  for (let scope = initialScope; scope; scope = scope.upper) {
-    const { variables } = scope;
-    for (let i = 0; i < variables.length; i++) {
-      const variable = variables[i];
-      if (variable.name === name) {
-        // This probably will not have the intended effect since mutation here does not
-        // have any bearing on the rust side analysis.
-        // @ts-expect-error - eslintUsed is not part of the public API.
-        variable.eslintUsed = true;
-        return true;
-      }
-    }
-  }
-
-  return false;
-}

apps/oxlint/src-js/plugins/source_code.ts

@@ -221,7 +221,6 @@ export const SOURCE_CODE = Object.freeze({
   isGlobalReference: scopeMethods.isGlobalReference,
   getDeclaredVariables: scopeMethods.getDeclaredVariables,
   getScope: scopeMethods.getScope,
-  markVariableAsUsed: scopeMethods.markVariableAsUsed,
 
   // Token methods
   getTokens: tokenMethods.getTokens,