feat: переработка NetwrokMessage и EventMapper

src/mind_reader/network/message.py

@@ -1,7 +1,8 @@
 import json
+import logging
 from dataclasses import asdict, dataclass, field
 from enum import StrEnum, auto
-from typing import Any
+from typing import Any, Optional
 
 from uuid_extensions import uuid7str
 
@@ -16,6 +17,8 @@ from ..state_machines.session.events import (
     SetterMoveEvent,
 )
 
+logger = logging.getLogger(__name__)
+
 
 class SessionMessageKind(StrEnum):
     PLAYER_HELLO = auto()
@@ -42,6 +45,14 @@ KIND_TO_EVENT_MAP = {
 }
 
 
+def serializer(obj: Any) -> Any:
+    if isinstance(obj, StrEnum):
+        return obj.value
+    raise TypeError(
+        f"Object of type {type(obj).__name__} is not JSON serializable"
+    )
+
+
 @dataclass(frozen=True)
 class NetworkMessage:
     kind: NetworkMessageKind
@@ -50,17 +61,22 @@ class NetworkMessage:
 
     @classmethod
     def from_json(cls, json_str: str) -> "NetworkMessage":
+        try:
             data = json.loads(json_str)
             return cls(
                 kind=NetworkMessageKind(data["kind"]),
                 payload=data.get("payload", {}),
                 id=data["id"],
             )
+        except (KeyError, ValueError, json.JSONDecodeError) as err:
+            logger.error(
+                "Failed to deserialize NetworkMessage from JSON: %s", err
+            )
+            raise
 
     def to_json(self) -> str:
-        data = asdict(self)
+        data = {"kind": self.kind.value, "payload": self.payload, "id": self.id}
-        data["kind"] = self.kind.value
+        return json.dumps(data, ensure_ascii=False, default=serializer)
-        return json.dumps(data, ensure_ascii=False)
 
 
 class EventMapper:
@@ -70,9 +86,14 @@ class EventMapper:
         kind = EVENT_TO_KIND_MAP.get(event_type)
 
         if not kind:
-            raise ValueError(f"Event {event_type} not registered")
+            logger.error(
+                "Attempted to serialize unregistered event: %s", event_type
+            )
+            raise ValueError(
+                f"Event {event_type} is not registered in EVENT_TO_KIND_MAP"
+            )
 
-        return NetworkMessage(kind=kind, payload=asdict(event))
+        return NetworkMessage(kind=kind, payload=event.to_payload())
 
     @staticmethod
     def from_message(message: NetworkMessage) -> Event | None:
@@ -82,33 +103,6 @@ class EventMapper:
             return None
 
         try:
-            payload = message.payload
+            return event_cls.from_payload(message.payload)
-
-            player = Player(
-                name=payload["player"]["name"],
-                role=PlayerRole(payload["player"]["role"]),
-            )
-
-            match message.kind:
-                case NetworkMessageKind.PLAYER_HELLO:
-                    return PlayerHelloEvent(player=player)
-                case NetworkMessageKind.SETTER_MOVED:
-                    return SetterMoveEvent(
-                        player=player,
-                        direction=ArrowDirection(payload["direction"]),
-                    )
-                case NetworkMessageKind.GUESSER_MOVED:
-                    return GuesserMoveEvent(
-                        player=player,
-                        direction=ArrowDirection(payload["direction"]),
-                    )
-                case NetworkMessageKind.SECRET_REVEALED:
-                    return RevealSecretEvent(
-                        player=player,
-                        direction=ArrowDirection(payload["direction"]),
-                    )
-                case NetworkMessageKind.NEXT_STEP_REQUESTED:
-                    return NextStepRequestedEvent(player=player)
-
         except (KeyError, ValueError, TypeError):
             return None

src/mind_reader/state_machines/base_event.py

@@ -0,0 +1,22 @@
+from dataclasses import asdict, dataclass, is_dataclass
+from enum import StrEnum
+from typing import Any
+
+
+def _factory(d: list[tuple[str, Any]]) -> dict[str, Any]:
+    def _convert(obj: Any) -> Any:
+        if isinstance(obj, StrEnum):
+            return obj.value
+        if isinstance(obj, list | tuple):
+            return [_convert(item) for item in obj]
+        if isinstance(obj, dict):
+            return {k: _convert(v) for k, v in obj.items()}
+        return obj
+
+    return {k: _convert(v) for k, v in d}
+
+
+@dataclass(frozen=True)
+class BaseEvent:
+    def to_payload(self) -> dict[str, Any]:
+        return asdict(self, dict_factory=_factory)

src/mind_reader/state_machines/lobby/domain.py

@@ -11,3 +11,9 @@ class PlayerRole(StrEnum):
 class Player:
     name: str
     role: PlayerRole
+
+    @classmethod
+    def from_dict(cls, d: dict) -> "Player":
+        return cls(
+            name=d["player"]["name"], role=PlayerRole(d["player"]["role"])
+        )

src/mind_reader/state_machines/session/events.py

@@ -1,36 +1,66 @@
 from dataclasses import dataclass
 
+from ..base_event import BaseEvent
 from ..lobby.domain import Player
 from ..session.domain import ArrowDirection
 
 
 @dataclass(frozen=True)
-class PlayerHelloEvent:
+class PlayerHelloEvent(BaseEvent):
     player: Player
 
+    @classmethod
+    def from_payload(cls, p: dict) -> "PlayerHelloEvent":
+        return cls(player=Player.from_dict(p))
+
 
 @dataclass(frozen=True)
-class SetterMoveEvent:
+class SetterMoveEvent(BaseEvent):
     player: Player
     direction: ArrowDirection
 
+    @classmethod
+    def from_payload(cls, p: dict) -> "SetterMoveEvent":
+        return cls(
+            player=Player.from_dict(p),
+            direction=ArrowDirection(p["arrow_direction"]),
+        )
+
 
 @dataclass(frozen=True)
-class GuesserMoveEvent:
+class GuesserMoveEvent(BaseEvent):
     player: Player
     direction: ArrowDirection
 
+    @classmethod
+    def from_payload(cls, p: dict) -> "GuesserMoveEvent":
+        return cls(
+            player=Player.from_dict(p),
+            direction=ArrowDirection(p["arrow_direction"]),
+        )
+
 
 @dataclass(frozen=True)
-class RevealSecretEvent:
+class RevealSecretEvent(BaseEvent):
     player: Player
     direction: ArrowDirection
 
+    @classmethod
+    def from_payload(cls, p: dict) -> "RevealSecretEvent":
+        return cls(
+            player=Player.from_dict(p),
+            direction=ArrowDirection(p["arrow_direction"]),
+        )
+
 
 @dataclass(frozen=True)
-class NextStepRequestedEvent:
+class NextStepRequestedEvent(BaseEvent):
     player: Player
 
+    @classmethod
+    def from_payload(cls, p: dict) -> "NextStepRequestedEvent":
+        return cls(player=Player.from_dict(p))
+
 
 SessionEvent = (
     PlayerHelloEvent