ci: A few quality fixes

Author: pawamoy

src/griffe_typedoc/decoder.py

@@ -4,6 +4,7 @@
 
 import json
 import re
+from contextlib import suppress
 from functools import wraps
 from typing import Any, Callable
 
@@ -72,10 +73,8 @@ def wrapper(obj_dict: dict[str, Any], symbol_id_map: dict[int, Any]) -> Any:
         # Assign object as parent on children.
         if "children" in obj_dict:
             for child in obj.children:
-                try:
+                with suppress(AttributeError):  # ints in groups
                     child.parent = obj
-                except AttributeError:
-                    pass  # ints in groups
 
         return obj
 
@@ -505,7 +504,15 @@ def _load_group(obj_dict: dict) -> Group:
 
 
 class TypedocDecoder(json.JSONDecoder):
-    def __init__(self, *args, **kwargs) -> None:
+    """JSON decoder."""
+
+    def __init__(self, *args: Any, **kwargs: Any) -> None:
+        """Initialize the decoder.
+
+        Parameters:
+            *args: Arguments passed to parent init method.
+            *kwargs: Keyword arguments passed to parent init method.
+        """
         kwargs["object_hook"] = self._object_hook
         super().__init__(*args, **kwargs)
         self._symbol_map: dict[int, Any] = {}
@@ -515,11 +522,6 @@ def _object_hook(self, obj_dict: dict[str, Any]) -> dict[str, Any] | str:
 
         The [`json.loads`][] method walks the tree from bottom to top.
 
-        Examples:
-            >>> import json
-            >>> from griffe.encoders import json_decoder
-            >>> json.loads(..., object_hook=json_decoder)
-
         Parameters:
             obj_dict: The dictionary to decode.
 
@@ -561,5 +563,3 @@ def _object_hook(self, obj_dict: dict[str, Any]) -> dict[str, Any] | str:
 
         # Return dict as is.
         return obj_dict
-
-    __all__ = ["json_decoder"]

src/griffe_typedoc/loader.py

@@ -5,11 +5,14 @@
 import re
 import subprocess
 from tempfile import NamedTemporaryFile
+from typing import TYPE_CHECKING
 
-from griffe_typedoc.dataclasses import Project
 from griffe_typedoc.decoder import TypedocDecoder
 from griffe_typedoc.logger import get_logger
 
+if TYPE_CHECKING:
+    from griffe_typedoc.dataclasses import Project
+
 logger = get_logger(__name__)
 
 
@@ -18,6 +21,15 @@ def _double_brackets(message: str) -> str:
 
 
 def load(typedoc_command: str | list[str], working_directory: str = ".") -> Project:
+    """Load TypeScript API data using TypeDoc.
+
+    Parameters:
+        typedoc_command: Name/path of the 1`typedoc` executable, or a command as list.
+        working_directory: Where to execute the command.
+
+    Returns:
+        Top-level project object containing API data.
+    """
     with NamedTemporaryFile("r+") as tmpfile:
         if isinstance(typedoc_command, str):
             typedoc_command += f" --json {tmpfile.name}"
@@ -27,7 +39,7 @@ def load(typedoc_command: str | list[str], working_directory: str = ".") -> Proj
             shell = False
         env = os.environ.copy()
         env["NO_COLOR"] = "1"
-        process = subprocess.Popen(
+        process = subprocess.Popen(  # noqa: S603
             typedoc_command,
             shell=shell,
             text=True,

src/griffe_typedoc/logger.py

@@ -1,33 +1,4 @@
-"""This module contains logging utilities.
-
-We provide the [`patch_loggers`][griffe_typedoc.logger.patch_loggers]
-function so dependant libraries can patch loggers as they see fit.
-
-For example, to fit in the MkDocs logging configuration
-and prefix each log message with the module name:
-
-```python
-import logging
-from griffe.logger import patch_loggers
-
-
-class LoggerAdapter(logging.LoggerAdapter):
-    def __init__(self, prefix, logger):
-        super().__init__(logger, {})
-        self.prefix = prefix
-
-    def process(self, msg, kwargs):
-        return f"{self.prefix}: {msg}", kwargs
-
-
-def get_logger(name):
-    logger = logging.getLogger(f"mkdocs.plugins.{name}")
-    return LoggerAdapter(name, logger)
-
-
-patch_loggers(get_logger)
-```
-"""
+"""This module contains logging utilities."""
 
 from __future__ import annotations