removed extra conversion function from replay api

Author: HardMax71

backend/app/api/routes/replay.py

@@ -6,7 +6,7 @@
 
 from app.api.dependencies import admin_user
 from app.domain.enums.replay import ReplayStatus
-from app.domain.replay import ReplayConfig, ReplaySessionState
+from app.domain.replay import ReplayConfig
 from app.schemas_pydantic.replay import (
     CleanupResponse,
     ReplayRequest,
@@ -16,22 +16,6 @@
 from app.schemas_pydantic.replay_models import ReplaySession
 from app.services.replay_service import ReplayService
 
-
-def _session_to_summary(state: ReplaySessionState) -> SessionSummary:
-    """Convert ReplaySessionState to SessionSummary."""
-    state_data = asdict(state)
-    state_data.update(state_data.pop("config"))  # flatten config fields
-
-    duration = None
-    throughput = None
-    if state.started_at and state.completed_at:
-        duration = (state.completed_at - state.started_at).total_seconds()
-        if state.replayed_events > 0 and duration > 0:
-            throughput = state.replayed_events / duration
-
-    return SessionSummary(**state_data, duration_seconds=duration, throughput_events_per_second=throughput)
-
-
 router = APIRouter(prefix="/replay", tags=["Event Replay"], route_class=DishkaRoute, dependencies=[Depends(admin_user)])
 
 
@@ -80,8 +64,10 @@ async def list_replay_sessions(
     status: ReplayStatus | None = Query(None),
     limit: int = Query(100, ge=1, le=1000),
 ) -> list[SessionSummary]:
-    states = service.list_sessions(status=status, limit=limit)
-    return [_session_to_summary(s) for s in states]
+    return [
+        SessionSummary.model_validate({**asdict(s), **asdict(s)["config"]})
+        for s in service.list_sessions(status=status, limit=limit)
+    ]
 
 
 @router.get("/sessions/{session_id}", response_model=ReplaySession)

backend/app/schemas_pydantic/replay.py

@@ -1,7 +1,7 @@
 from datetime import datetime
 from typing import Dict
 
-from pydantic import BaseModel, ConfigDict, Field
+from pydantic import BaseModel, ConfigDict, Field, computed_field
 
 from app.domain.enums.events import EventType
 from app.domain.enums.replay import ReplayStatus, ReplayTarget, ReplayType
@@ -51,8 +51,20 @@ class SessionSummary(BaseModel):
     created_at: datetime
     started_at: datetime | None
     completed_at: datetime | None
-    duration_seconds: float | None = None
-    throughput_events_per_second: float | None = None
+
+    @computed_field  # type: ignore[prop-decorator]
+    @property
+    def duration_seconds(self) -> float | None:
+        if self.started_at and self.completed_at:
+            return (self.completed_at - self.started_at).total_seconds()
+        return None
+
+    @computed_field  # type: ignore[prop-decorator]
+    @property
+    def throughput_events_per_second(self) -> float | None:
+        if self.duration_seconds and self.duration_seconds > 0 and self.replayed_events > 0:
+            return self.replayed_events / self.duration_seconds
+        return None
 
 
 class CleanupResponse(BaseModel):