refactor: Remove base visitor and inspector

They are not useful and make reasoning
about class hierarchy more difficult.

Author: pawamoy

src/griffe/agents/base.py

@@ -28,7 +28,6 @@
 from inspect import signature as getsignature
 from typing import TYPE_CHECKING, Any
 
-from griffe.agents.base import BaseInspector
 from griffe.agents.nodes import ObjectKind, ObjectNode, safe_get_annotation
 from griffe.collections import LinesCollection, ModulesCollection
 from griffe.dataclasses import (
@@ -152,7 +151,7 @@ def _should_create_alias(parent: ObjectNode, child: ObjectNode, current_module_p
     return None
 
 
-class Inspector(BaseInspector):
+class Inspector:
     """This class is used to instantiate an inspector.
 
     Inspectors iterate on objects members to extract data from them.
@@ -241,7 +240,7 @@ def inspect(self, node: ObjectNode) -> None:
         """
         for before_inspector in self.extensions.before_inspection:
             before_inspector.inspect(node)
-        super().inspect(node)
+        getattr(self, f"inspect_{node.kind}", self.generic_inspect)(node)
         for after_inspector in self.extensions.after_inspection:
             after_inspector.inspect(node)
 

src/griffe/agents/inspector.py

@@ -14,7 +14,6 @@
 from itertools import zip_longest
 from typing import TYPE_CHECKING, Any, Iterable
 
-from griffe.agents.base import BaseVisitor
 from griffe.agents.nodes import (
     ASTNode,
     get_docstring,
@@ -111,7 +110,7 @@ def visit(
     ).get_module()
 
 
-class Visitor(BaseVisitor):
+class Visitor:
     """This class is used to instantiate a visitor.
 
     Visitors iterate on AST nodes to extract data from them.
@@ -189,7 +188,7 @@ def visit(self, node: ast.AST) -> None:
         """
         for before_visitor in self.extensions.before_visit:
             before_visitor.visit(node)
-        super().visit(node)
+        getattr(self, f"visit_{node.kind}", self.generic_visit)(node)
         for after_visitor in self.extensions.after_visit:
             after_visitor.visit(node)
 

src/griffe/agents/visitor.py

@@ -10,7 +10,6 @@
 from inspect import isclass
 from typing import TYPE_CHECKING, Any, Sequence, Union
 
-from griffe.agents.base import BaseInspector, BaseVisitor
 from griffe.exceptions import ExtensionNotLoadedError
 from griffe.importer import dynamic_import
 
@@ -39,7 +38,7 @@ class When(enum.Enum):
     after_all: int = 4
 
 
-class VisitorExtension(BaseVisitor):
+class VisitorExtension:
     """The node visitor extension base class, to inherit from."""
 
     when: When = When.after_all
@@ -66,7 +65,7 @@ def visit(self, node: ast.AST) -> None:
         getattr(self, f"visit_{node.kind}", lambda _: None)(node)  # type: ignore[attr-defined]
 
 
-class InspectorExtension(BaseInspector):
+class InspectorExtension:
     """The object inspector extension base class, to inherit from."""
 
     when: When = When.after_all