Remove @cached decorator and CACHE_MANAGER

ChangeLog

@@ -6,6 +6,9 @@ What's New in astroid 2.16.0?
 =============================
 Release date: TBA
 
+* Remove ``@cached`` decorator (just use ``@cached_property`` from the stdlib).
+
+  Closes #1780
 
 
 What's New in astroid 2.15.2?

astroid/bases.py

@@ -13,7 +13,7 @@
 from collections.abc import Sequence
 from typing import TYPE_CHECKING, Any, ClassVar
 
-from astroid import decorators, nodes
+from astroid import nodes
 from astroid.const import PY310_PLUS
 from astroid.context import (
     CallContext,
@@ -634,7 +634,6 @@ def __init__(
         self.parent = parent
         self._call_context = copy_context(generator_initial_context)
 
-    @decorators.cached
     def infer_yield_types(self):
         yield from self.parent.infer_yield_result(self._call_context)
 

astroid/decorators.py

@@ -15,7 +15,7 @@
 
 import wrapt
 
-from astroid import _cache, util
+from astroid import util
 from astroid.context import InferenceContext
 from astroid.exceptions import InferenceError
 
@@ -28,20 +28,6 @@
 _P = ParamSpec("_P")
 
 
-@wrapt.decorator
-def cached(func, instance, args, kwargs):
-    """Simple decorator to cache result of method calls without args."""
-    cache = getattr(instance, "__cache", None)
-    if cache is None:
-        instance.__cache = cache = {}
-        _cache.CACHE_MANAGER.add_dict_cache(cache)
-    try:
-        return cache[func]
-    except KeyError:
-        cache[func] = result = func(*args, **kwargs)
-        return result
-
-
 # TODO: Remove when support for 3.7 is dropped
 # TODO: astroid 3.0 -> move class behind sys.version_info < (3, 8) guard
 class cachedproperty:

astroid/manager.py

@@ -18,7 +18,6 @@
 from typing import Any, ClassVar
 
 from astroid import nodes
-from astroid._cache import CACHE_MANAGER
 from astroid.const import BRAIN_MODULES_DIRECTORY
 from astroid.context import InferenceContext, _invalidate_cache
 from astroid.exceptions import AstroidBuildingError, AstroidImportError
@@ -424,8 +423,6 @@ def clear_cache(self) -> None:
         # NB: not a new TransformVisitor()
         AstroidManager.brain["_transform"].transforms = collections.defaultdict(list)
 
-        CACHE_MANAGER.clear_all_caches()
-
         for lru_cache in (
             LookupMixIn.lookup,
             _cache_normalize_path_,

astroid/nodes/scoped_nodes/scoped_nodes.py

@@ -2937,8 +2937,8 @@ def _slots(self):
         return [first, *slots]
 
     # Cached, because inferring them all the time is expensive
-    @decorators_mod.cached
-    def slots(self):
+    @cached_property
+    def _all_slots(self):
         """Get all the slots for this node.
 
         :returns: The names of slots for this class.
@@ -2982,6 +2982,9 @@ def grouped_slots(
 
         return sorted(set(slots), key=lambda item: item.value)
 
+    def slots(self):
+        return self._all_slots
+
     def _inferred_bases(self, context: InferenceContext | None = None):
         # Similar with .ancestors, but the difference is when one base is inferred,
         # only the first object is wanted. That's because