revert: keep get_games() as list[dict] — Statistics Center spec too incomplete for typed models

The generated Games model only documents 3 fields (match_id, home_team, away_team);
real API returns date, game_day, running etc. which land in additional_properties
without .get() support, breaking all existing callers.

The statistics_center_client is still vendored (Login/LoginSuccess models are
available) but response parsing stays as plain dicts until the spec is complete.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: mad4ms

AGENTS.md

@@ -10,7 +10,7 @@ Python wrapper for the Kinexon Handball API. Provides a user-friendly abstractio
   - `api.py`: Two-step authentication and `httpx` client initialization.
   - `handball.py`: Main entry point (`HandballAPI`). Convenience methods for common API operations.
   - `fetchers.py`: Static data loading (e.g., team IDs from `config/teams.yaml`).
-  - `statistics_center.py`: Separate wrapper for the Statistics Center REST + WebSocket API. Uses `Games` and `LoginSuccess` from the generated `statistics_center_client`.
+  - `statistics_center.py`: Separate wrapper for the Statistics Center REST + WebSocket API. All REST methods return plain `list[dict]` — the Statistics Center OpenAPI spec is incomplete (missing fields), so generated models are vendored but not used for response parsing.
 - **Generated Clients (`src/_vendor/`)**: Two auto-generated OpenAPI clients via `openapi-python-client`.
   - `kinexon_client/` — generated from `openapi/sports_app.json` (main Kinexon Cloud REST API).
   - `statistics_center_client/` — generated from `openapi/statistics_center.json` (Statistics Center REST API).

README.md

@@ -129,11 +129,10 @@ You can add new teams by modifying `config/teams.yaml`. Somehow there is no API
 
 ### Statistics Center (REST + Websocket)
 
-The package also provides a dedicated wrapper for the Statistics Center API. REST responses use typed models from the generated `statistics_center_client`:
+The package also provides a dedicated wrapper for the Statistics Center API:
 
 ```python
 from kinexon_handball_api import StatisticsCenterAPI
-from statistics_center_client.models import Games
 
 sc = StatisticsCenterAPI(
   username="<USERNAME>",
@@ -145,10 +144,10 @@ sc = StatisticsCenterAPI(
 # JWT login
 jwt = sc.login()
 
-# Typed REST responses
-games: list[Games] = sc.get_games(season="2025_2026")
-stats = sc.get_stats("<MATCH_ID>")        # list[dict]
-events = sc.get_events("<MATCH_ID>")      # list[dict]
+# REST endpoints — all return list[dict]
+games = sc.get_games(season="2025_2026")
+stats = sc.get_stats("<MATCH_ID>")
+events = sc.get_events("<MATCH_ID>")
 
 # Push endpoints
 all_endpoints = sc.list_endpoints()

docs/api-reference.md

@@ -197,28 +197,25 @@ sc = StatisticsCenterAPI(
 
 ### `login(force_refresh=False)`
 
-Authenticates and returns the JWT. Cached on the instance after first call. The response is parsed via the generated `LoginSuccess` model.
+Authenticates and returns the JWT. Cached on the instance after first call.
 
 ```python
 jwt = sc.login()
 ```
 
 ### `get_games(season="")`
 
-Returns all matches for the given season. Returns `list[Games]` (from `statistics_center_client.models`).
+Returns all matches for the given season from `/games`. Returns `list[dict[str, Any]]`.
 
 ```python
-from statistics_center_client.models import Games
-
-games: list[Games] = sc.get_games(season="2025_2026")
+games = sc.get_games(season="2025_2026")
 for g in games:
-    print(g.match_id, g.home_team, g.away_team)
-    # extra fields from the API are in g.additional_properties
+    print(g.get("match_id"), g.get("date"), g.get("home_team"))
 ```
 
 ### `get_stats(match_id)`
 
-Returns player statistics for a match from `/stats/{match_id}`. Returns `list[dict[str, Any]]` (the Statistics Center spec has no field-level schema for stats responses).
+Returns player statistics for a match from `/stats/{match_id}`. Returns `list[dict[str, Any]]`.
 
 ```python
 stats = sc.get_stats("123456")
@@ -234,10 +231,10 @@ events = sc.get_events("123456", event_type="shot")
 
 ### `get_games_via_websocket(season, timeout_s=10.0)`
 
-Discovers matches for a season via the WebSocket matches stream. Returns `list[Games]`.
+Discovers matches for a season via the WebSocket matches stream. Returns `list[dict[str, Any]]`.
 
 ```python
-games: list[Games] = sc.get_games_via_websocket("2025_2026")
+games = sc.get_games_via_websocket("2025_2026")
 ```
 
 ### `list_endpoints(jwt=None)`

docs/architecture.md

@@ -104,8 +104,7 @@ Standalone client (does not inherit `KinexonAPI`). Uses:
 
 - `httpx.Client` for REST calls.
 - JWT authentication via `/auth/login`. The login response is parsed into a `LoginSuccess` model from `statistics_center_client.models`.
-- `get_games()` and `get_games_via_websocket()` return `list[Games]` (typed attrs model from `statistics_center_client.models`).
-- `get_stats()` and `get_events()` return `list[dict[str, Any]]` — the Statistics Center spec has no field-level schema for these responses.
+- `get_games()`, `get_games_via_websocket()`, `get_stats()`, and `get_events()` all return `list[dict[str, Any]]` — the Statistics Center spec only documents 3 fields for games (`match_id`, `home_team`, `away_team`) while the real API returns many more; a generated model would bury the extra fields in `additional_properties` without `.get()` support, so plain dicts are used.
 - `python-socketio` for WebSocket subscriptions (lazy-imported so the dependency is optional at import time).
 
 ## Generated Clients

src/kinexon_handball_api/statistics_center.py

@@ -14,7 +14,6 @@
 from urllib.parse import urlsplit
 
 import httpx
-from statistics_center_client.models import Games, LoginSuccess
 
 SubscriptionType = Literal["matches", "stats", "events", "live_events"]
 HTTP_OK = 200
@@ -93,13 +92,7 @@ def login(self, *, force_refresh: bool = False) -> str:
             )
 
         payload = response.json()
-        raw = payload[0] if isinstance(payload, list) and payload else payload
-        if not isinstance(raw, dict):
-            raise StatisticsCenterAPIError(
-                "Statistics Center login response has no jwt."
-            )
-        result = LoginSuccess.from_dict(raw)
-        jwt = result.jwt if isinstance(result.jwt, str) else None
+        jwt = self._extract_jwt(payload)
         if not jwt:
             raise StatisticsCenterAPIError(
                 "Statistics Center login response has no jwt."
@@ -135,13 +128,13 @@ def _get_with_retry(
             )
         return response
 
-    def get_games(self, season: str = "") -> list[Games]:
+    def get_games(self, season: str = "") -> list[dict[str, Any]]:
         """Return all matches for the given season from /games."""
         params = {"season": season} if season else None
         payload = self._get_with_retry("/games", params=params).json()
         if not isinstance(payload, list):
             raise StatisticsCenterAPIError("Expected list from /games")
-        return [Games.from_dict(g) for g in payload if isinstance(g, dict)]
+        return payload
 
     def get_stats(self, match_id: str | int) -> list[dict[str, Any]]:
         """Return player statistics for a match from /stats/{match_id}."""
@@ -170,7 +163,7 @@ def get_games_via_websocket(
         season: str,
         timeout_s: float = 10.0,
         verbose: bool = False,
-    ) -> list[Games]:
+    ) -> list[dict[str, Any]]:
         """Discover matches for a season via the WebSocket matches stream.
 
         Connects, sends a matches subscription, waits up to timeout_s for the
@@ -185,7 +178,7 @@ def get_games_via_websocket(
             ) from exc
 
         done = threading.Event()
-        games: list[Games] = []
+        games: list[dict[str, Any]] = []
         token = self._jwt or self.login()
         sio = socketio.Client(
             reconnection=False,
@@ -215,7 +208,7 @@ def _on_message(data: Any) -> None:
             if isinstance(data, dict) and data.get("type") == "matches":
                 payload = data.get("payload")
                 if isinstance(payload, list):
-                    games = [Games.from_dict(r) for r in payload if isinstance(r, dict)]
+                    games = [r for r in payload if isinstance(r, dict)]
                 done.set()
 
         @sio.on("connect_error")
@@ -263,6 +256,18 @@ def delete_endpoint(self, endpoint_id: str, *, jwt: str | None = None) -> bool:
         del endpoint_id, jwt
         raise StatisticsCenterAPIError("delete_endpoint is disabled in read-only mode.")
 
+    @staticmethod
+    def _extract_jwt(payload: Any) -> str | None:
+        if isinstance(payload, dict):
+            value = payload.get("jwt")
+            return value if isinstance(value, str) else None
+        if isinstance(payload, list) and payload:
+            first = payload[0]
+            if isinstance(first, dict):
+                value = first.get("jwt")
+                return value if isinstance(value, str) else None
+        return None
+
     @staticmethod
     def _auth_headers(jwt: str) -> dict[str, str]:
         return {

test/test_statistics_center.py

@@ -5,7 +5,6 @@
 
 import httpx
 import pytest
-from statistics_center_client.models import Games
 
 from kinexon_handball_api.statistics_center import (
     StatisticsCenterAPI,
@@ -159,7 +158,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
     api = _make_rest_api(handler)
     games = api.get_games(season="2025_2026")
-    assert games == [Games(match_id="42")]
+    assert games == [{"match_id": "42"}]
     api.close()