Add clean_lru_cache following review (and remove a circular import)

Author: Pierre-Sassoulas

pylint/checkers/clear_lru_cache.py

@@ -0,0 +1,37 @@
+# Licensed under the GPL: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html
+# For details: https://github.com/pylint-dev/pylint/blob/main/LICENSE
+# Copyright (c) https://github.com/pylint-dev/pylint/blob/main/CONTRIBUTORS.txt
+
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, Any
+
+from pylint.checkers.typecheck import _similar_names
+from pylint.checkers.utils import (
+    class_is_abstract,
+    in_for_else_branch,
+    infer_all,
+    is_overload_stub,
+    overridden_method,
+    safe_infer,
+    unimplemented_abstract_methods,
+)
+
+if TYPE_CHECKING:
+    from functools import _lru_cache_wrapper
+
+
+def clear_lru_caches() -> None:
+    """Clear caches holding references to AST nodes."""
+    caches_holding_node_references: list[_lru_cache_wrapper[Any]] = [
+        class_is_abstract,
+        in_for_else_branch,
+        infer_all,
+        is_overload_stub,
+        overridden_method,
+        unimplemented_abstract_methods,
+        safe_infer,
+        _similar_names,
+    ]
+    for lru in caches_holding_node_references:
+        lru.cache_clear()

pylint/checkers/utils.py

@@ -16,7 +16,7 @@
 from collections.abc import Callable, Iterable, Iterator
 from functools import lru_cache, partial
 from re import Match
-from typing import TYPE_CHECKING, Any, TypeVar
+from typing import TYPE_CHECKING, TypeVar
 
 import astroid.objects
 from astroid import TooManyLevelsError, nodes, util
@@ -28,7 +28,6 @@
 from pylint.constants import TYPING_NEVER, TYPING_NORETURN
 
 if TYPE_CHECKING:
-    from functools import _lru_cache_wrapper
 
     from pylint.checkers import BaseChecker
 
@@ -2327,21 +2326,6 @@ def overridden_method(
     return None  # pragma: no cover
 
 
-def clear_lru_caches() -> None:
-    """Clear caches holding references to AST nodes."""
-    caches_holding_node_references: list[_lru_cache_wrapper[Any]] = [
-        class_is_abstract,
-        in_for_else_branch,
-        infer_all,
-        is_overload_stub,
-        overridden_method,
-        unimplemented_abstract_methods,
-        safe_infer,
-    ]
-    for lru in caches_holding_node_references:
-        lru.cache_clear()
-
-
 def is_enum_member(node: nodes.AssignName) -> bool:
     """Return `True` if `node` is an Enum member (is an item of the
     `__members__` container).

pylint/lint/run.py

@@ -12,7 +12,7 @@
 from typing import ClassVar
 
 from pylint import config
-from pylint.checkers.utils import clear_lru_caches
+from pylint.checkers.clear_lru_cache import clear_lru_caches
 from pylint.config._pylint_config import (
     _handle_pylint_config_commands,
     _register_generate_config_options,