Remove Renderer.builder, pass builder as required

Author: has2k1

quartodoc/autosummary.py

@@ -507,7 +507,6 @@ def __init__(
         self.parser = parser
 
         self.renderer = Renderer.from_config(renderer)
-        self.renderer.builder = self
         if render_interlinks:
             # this is a top-level option, but lives on the renderer
             # so we just manually set it there for now.
@@ -567,7 +566,7 @@ def build(self, filter: str = "*"):
 
         _log.info("Writing docs pages")
         self.write_doc_pages(pages, filter)
-        self.renderer.pages_written()
+        self.renderer._pages_written(self)
 
         # inventory ----
 

quartodoc/renderers/base.py

@@ -40,9 +40,6 @@ class Renderer:
     style: str
     _registry: "dict[str, Renderer]" = {}
 
-    builder: "Builder"
-    """The builder that uses this renderer for generating docs"""
-
     def __init_subclass__(cls, **kwargs):
         super().__init_subclass__(**kwargs)
 
@@ -88,12 +85,22 @@ def from_config(cls, cfg: "dict | Renderer | str"):
     def render(self, el):
         raise NotImplementedError(f"render method does not support type: {type(el)}")
 
-    def pages_written(self):
+    def _pages_written(self, builder: "Builder"):
         """
         Called after all the qmd pages have been render and written to disk
 
         It is called before the documented items are written to an inventory
         file. This is a chance for the renderer to add to the documented items
         and write the pages to them to disk.
+
+        Parameters
+        ----------
+        builder :
+            There builder using this renderer to generate documentation.
+
+        Notes
+        -----
+        This method is provided for experimental purposes and it is not bound
+        to be available for long, or have the same form.
         """
         ...