Add initial try at user level debugging for object visibility

Author: robtaylor

autoapi/_mapper.py

@@ -29,6 +29,7 @@
     PythonAttribute,
     PythonData,
     PythonException,
+    _trace_visibility,
 )
 from .settings import OWN_PAGE_LEVELS, TEMPLATE_DIR
 
@@ -499,6 +500,7 @@ def _skip_if_stdlib(self):
                     and not obj["inherited_from"]["is_abstract"]
                     and module not in documented_modules
                 ):
+                    _trace_visibility(self.app, f"Hiding {obj['qual_name']} as determined to be Python standard Library (found as {obj['full_name']})", verbose=2)
                     obj["hide"] = True
 
     def _resolve_placeholders(self):
@@ -518,12 +520,18 @@ def _hide_yo_kids(self):
         for module in self.paths.values():
             if module["all"] is not None:
                 all_names = set(module["all"])
+                _trace_visibility(self.app, f"{module['full_name']}: Found __all__ =")
+                for n in all_names:
+                    _trace_visibility(self.app, f"    {n}")
                 for child in module["children"]:
                     if child["qual_name"] not in all_names:
+                        _trace_visibility(self.app, f"Hiding {child['full_name']}, as {child['qual_name']} not in __all__")
                         child["hide"] = True
             elif module["type"] == "module":
+                _trace_visibility(self.app, f"Testing if any children of {module['full_name']} have already been documented")
                 for child in module["children"]:
                     if "original_path" in child:
+                        _trace_visibility(self.app, f"Hiding {child['full_name']} as documented at {child['original_path']}")
                         child["hide"] = True
 
     def map(self, options=None):
@@ -567,13 +575,17 @@ def _render_selection(self):
                 assert obj.type in self.own_page_types
                 self.objects_to_render[obj.id] = obj
             else:
+                if obj.subpackages or obj.submodules:
+                    _trace_visibility(self.app, f"Not rendering the following as {obj.id} set to not display:", verbose=2)
                 for module in itertools.chain(obj.subpackages, obj.submodules):
+                    _trace_visibility(self.app, f"    {module.obj['full_name']}", verbose=2)
                     module.obj["hide"] = True
 
         def _inner(parent):
             for child in parent.children:
                 self.all_objects[child.id] = child
                 if not parent.display:
+                    _trace_visibility(self.app, f"Hiding {child.id} as parent {parent.id} will not be displayed", verbose=2)
                     child.obj["hide"] = True
 
                 if child.display and child.type in self.own_page_types:

autoapi/_objects.py

@@ -7,10 +7,16 @@
 import sphinx.util
 import sphinx.util.logging
 
+from sphinx.util.console import colorize
+
 from .settings import OWN_PAGE_LEVELS
 
 LOGGER = sphinx.util.logging.getLogger(__name__)
 
+def _trace_visibility(app, msg: str, verbose=1) -> None:
+    if app.config.autoapi_verbose_visibility >= verbose:
+        LOGGER.info(colorize("bold", f"[AutoAPI] [Visibility] {msg}"))
+
 
 def _format_args(args_info, include_annotations=True, ignore_self=None):
     result = []
@@ -79,6 +85,7 @@ def __init__(
         # For later
         self._class_content = class_content
         self._display_cache: bool | None = None
+        self._skip_reason = None
 
     def __getstate__(self):
         """Obtains serialisable data for pickling."""
@@ -199,8 +206,13 @@ def display(self) -> bool:
         This attribute depends on the configuration options given in
         :confval:`autoapi_options` and the result of :event:`autoapi-skip-member`.
         """
+        skip = self._should_skip()
         if self._display_cache is None:
-            self._display_cache = not self._ask_ignore(self._should_skip())
+            self._display_cache = not self._ask_ignore(skip)
+            if self._display_cache is False:
+                _trace_visibility(self.app, self._skip_reason)
+        else:
+            _trace_visibility(self.app, f"Skipping {self.id} due to cache", verbose=2)
 
         return self._display_cache
 
@@ -231,6 +243,22 @@ def _should_skip(self) -> bool:
             self.inherited and "inherited-members" not in self.options
         )
 
+        reason = ""
+        if self.obj.get("hide", False):
+            reason = "marked hidden by mapper"
+        elif skip_undoc_member:
+            reason = "is undocumented"
+        elif skip_private_member:
+            reason = "is a private member"
+        elif skip_special_member:
+            reason = "is a special member"
+        elif skip_imported_member:
+            reason = "is an imported member"
+        elif  skip_inherited_member:
+            reason = "is an inherited member"
+
+        self._skip_reason = f"Skipping {self.id} as {reason}"
+
         return (
             self.obj.get("hide", False)
             or skip_undoc_member
@@ -241,10 +269,16 @@ def _should_skip(self) -> bool:
         )
 
     def _ask_ignore(self, skip: bool) -> bool:
+
         ask_result = self.app.emit_firstresult(
             "autoapi-skip-member", self.type, self.id, self, skip, self.options
         )
 
+        if ask_result is not None:
+            reason = f"Skipping as 'autoapi-skip-member' returned {ask_result}"
+            if self._skip_reason:
+                reason += f"Passed skip={skip} to 'autoapi-skip-member' as {self._skip_reason}"
+            self._skip_reason = reason
         return ask_result if ask_result is not None else skip
 
     def _children_of_type(self, type_: str) -> list[PythonObject]:
@@ -306,11 +340,13 @@ def __init__(self, *args, **kwargs):
         """
 
     def _should_skip(self) -> bool:
-        return super()._should_skip() or self.name in (
+        is_new_or_init = self.name in (
             "__new__",
             "__init__",
         )
-
+        if not super()._should_skip and is_new_or_init:
+            self._skip_reason = f"Skipping method {self.id} as is __new__ or __init__"
+        return super()._should_skip() or is_new_or_init
 
 class PythonProperty(PythonObject):
     """The representation of a property on a class."""

autoapi/extension.py

@@ -261,6 +261,7 @@ def setup(app):
     app.add_config_value("autoapi_generate_api_docs", True, "html")
     app.add_config_value("autoapi_prepare_jinja_env", None, "html")
     app.add_config_value("autoapi_own_page_level", "module", "html")
+    app.add_config_value("autoapi_verbose_visibility", 0, "html")
     app.add_autodocumenter(documenters.AutoapiFunctionDocumenter)
     app.add_autodocumenter(documenters.AutoapiPropertyDocumenter)
     app.add_autodocumenter(documenters.AutoapiDecoratorDocumenter)