fix: Properly raise on errors (respect strict mode)

src/mkdocstrings/extension.py

@@ -147,7 +147,7 @@ def run(self, parent: Element, blocks: Element) -> None:
                 data: Any = handler.collector.collect(identifier, selection)
             except CollectionError:
                 log.error(f"mkdocstrings.extension: Could not collect '{identifier}'")
-                return
+                raise
 
             log.debug("mkdocstrings.extension: Updating renderer's env")
             handler.renderer.update_env(self.md, self._config)
@@ -161,7 +161,7 @@ def run(self, parent: Element, blocks: Element) -> None:
                     f"mkdocstrings.extension: Template '{error.name}' not found "
                     f"for '{handler_name}' handler and theme '{theme_name}'."
                 )
-                return
+                raise
 
             log.debug("mkdocstrings.extension: Loading HTML back into XML tree")
             try:
@@ -182,7 +182,7 @@ def run(self, parent: Element, blocks: Element) -> None:
                         f"or replace them with < and >"
                     )
                 log.error(message)
-                return
+                raise
 
             as_xml = atomic_brute_cast(as_xml)  # type: ignore
             parent.append(as_xml)

src/mkdocstrings/plugin.py

@@ -142,6 +142,7 @@ def on_config(self, config: Config, **kwargs) -> Config:
         """
         if not config["site_url"]:
             log.error("mkdocstrings.plugin: configuration item 'site_url' is required for cross-references")
+            raise ValueError("site_url is required")
 
         log.debug("mkdocstrings.plugin: Adding extension to the list")
 
@@ -165,15 +166,9 @@ def on_page_content(self, html: str, page: Page, config: Config, files: Files, *
         `[identifier][]`.
         """
         log.debug(f"mkdocstrings.plugin: Mapping identifiers to URLs for page {page.file.src_path}")
-        try:
-            for item in page.toc.items:
-                self.map_urls(page.canonical_url, item)
-            return html
-        except TypeError:
-            # page.canonical_url is None, fail silently.
-            # An error already has been logged in the on_config hook,
-            # and warnings will be logged later, in the on_post_page hook.
-            pass
+        for item in page.toc.items:
+            self.map_urls(page.canonical_url, item)
+        return html
 
     def map_urls(self, base_url: str, anchor: AnchorLink) -> None:
         """