Require using built-in dataclasses in load_config()

This is just the last step to be able to enforce via the type system
that only built-in dataclasses are used.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

src/frequenz/sdk/config/_util.py

@@ -4,22 +4,30 @@
 """Utilities to deal with configuration."""
 
 from collections.abc import Mapping
-from typing import Any, TypeVar, cast
+from typing import Any, ClassVar, Protocol, TypeVar, cast
 
 from marshmallow import Schema
 from marshmallow_dataclass import class_schema
 
-T = TypeVar("T")
+
+class Dataclass(Protocol):
+    """A protocol for dataclasses."""
+
+    __dataclass_fields__: ClassVar[dict[str, Any]]
+    """The fields of the dataclass."""
+
+
+DataclassT = TypeVar("DataclassT", bound=Dataclass)
 """Type variable for configuration classes."""
 
 
 def load_config(
-    cls: type[T],
+    cls: type[DataclassT],
     config: Mapping[str, Any],
     /,
     base_schema: type[Schema] | None = None,
     **marshmallow_load_kwargs: Any,
-) -> T:
+) -> DataclassT:
     """Load a configuration from a dictionary into an instance of a configuration class.
 
     The configuration class is expected to be a [`dataclasses.dataclass`][], which is
@@ -56,4 +64,4 @@ def load_config(
     instance = class_schema(cls, base_schema)().load(config, **marshmallow_load_kwargs)
     # We need to cast because `.load()` comes from marshmallow and doesn't know which
     # type is returned.
-    return cast(T, instance)
+    return cast(DataclassT, instance)