fixup! refactor: Use dataclasses for configuration/options and automate schema generation

Author: pawamoy

src/mkdocstrings_handlers/python/config.py

@@ -951,9 +951,9 @@ class PythonInputConfig:
     ] = field(default_factory=lambda: ["."])
 
     load_external_modules: Annotated[
-        bool,
+        bool | None,
         Field(description="Whether to always load external modules/packages."),
-    ] = False
+    ] = None
 
     options: Annotated[
         PythonInputOptions,
@@ -965,10 +965,31 @@ class PythonInputConfig:
         Field(description="The locale to use when translating template strings."),
     ] = None
 
+    @classmethod
+    def coerce(cls, **data: Any) -> MutableMapping[str, Any]:
+        """Coerce data."""
+        return data
+
+    @classmethod
+    def from_data(cls, **data: Any) -> Self:
+        """Create an instance from a dictionary."""
+        return cls(**cls.coerce(**data))
+
 
 # YORE: EOL 3.9: Replace `**_dataclass_options` with `frozen=True, kw_only=True` within line.
 @dataclass(**_dataclass_options)  # type: ignore[call-overload]
 class PythonConfig(PythonInputConfig):  # type: ignore[override,unused-ignore]
     """Python handler configuration."""
 
+    inventories: list[Inventory] = field(default_factory=list)
     options: dict[str, Any] = field(default_factory=dict)  # type: ignore[assignment]
+
+    @classmethod
+    def coerce(cls, **data: Any) -> MutableMapping[str, Any]:
+        """Coerce data."""
+        if "inventories" in data:
+            data["inventories"] = [
+                Inventory(url=inv) if isinstance(inv, str) else Inventory(**inv)
+                for inv in data["inventories"]
+            ]
+        return data

src/mkdocstrings_handlers/python/handler.py

@@ -138,7 +138,7 @@ def __init__(self, config: PythonConfig, base_dir: Path, **kwargs: Any) -> None:
 
     def get_inventory_urls(self) -> list[tuple[str, dict[str, Any]]]:
         """Return the URLs of the inventory files to download."""
-        return [(inv, {}) if isinstance(inv, str) else (inv.url, inv._config) for inv in self.config.inventories]
+        return [(inv.url, inv._config) for inv in self.config.inventories]
 
     @staticmethod
     def load_inventory(
@@ -352,7 +352,7 @@ def get_handler(
         warn("The 'import' key is renamed 'inventories' for the Python handler", FutureWarning, stacklevel=1)
         handler_config["inventories"] = handler_config.pop("import", [])
     return PythonHandler(
-        config=PythonConfig(**handler_config),
+        config=PythonConfig.from_data(**handler_config),
         base_dir=base_dir,
         **kwargs,
     )