Merge pull request #3 from Sjeff/update-3.0.4

Update 3.0.4

Author: Sjeff

.github/workflows/test.yml

@@ -1,6 +1,9 @@
 name: Tests
 
-on: [push, pull_request]
+on:
+  push:
+    branches: [master]
+  pull_request:
 
 jobs:
   backend-tests:

README.md

@@ -197,6 +197,20 @@ alembic upgrade head
 
 ## Changelog
 
+### v3.0.4
+
+All changes in this release were made in collaboration with [Claude](https://claude.ai) (Anthropic).
+
+#### Bug fixes
+
+- Fixed automation cycle crash when the Steam API returns no review data for a game — review fields (`review_score`, `total_positive`, `total_negative`, `total_reviews`) now fall back to `0` instead of `None` to satisfy the database `NOT NULL` constraint
+
+#### Code quality
+
+- Replaced all uses of the deprecated `datetime.utcnow()` (52 call sites across 21 files + 19 test files) with `datetime.now(timezone.utc)` — required for Python 3.12+ compatibility
+- Added `TZDateTime` SQLAlchemy TypeDecorator to re-attach `timezone.utc` when reading datetimes from SQLite, preventing naive/aware comparison errors
+- Fixed CI workflow running 4 jobs per PR instead of 2 — push trigger now limited to `master`
+
 ### v3.0.3
 
 All changes in this release were made in collaboration with [Claude](https://claude.ai) (Anthropic).

backend/src/api/routers/system.py

@@ -8,7 +8,7 @@
 import json
 from io import StringIO
 from typing import Dict, Any
-from datetime import datetime
+from datetime import datetime, timezone
 
 from fastapi import APIRouter, Query
 from fastapi.responses import StreamingResponse
@@ -45,7 +45,7 @@ async def health_check() -> Dict[str, Any]:
     return create_success_response(
         data={
             "status": "healthy",
-            "timestamp": datetime.utcnow().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "version": "0.1.0",
         }
     )
@@ -236,12 +236,12 @@ async def export_logs(
             writer.writerows(logs_data)
         content = output.getvalue()
         media_type = "text/csv"
-        filename = f"logs_{datetime.utcnow().strftime('%Y%m%d_%H%M%S')}.csv"
+        filename = f"logs_{datetime.now(timezone.utc).strftime('%Y%m%d_%H%M%S')}.csv"
     else:
         # Generate JSON
         content = json.dumps(logs_data, indent=2)
         media_type = "application/json"
-        filename = f"logs_{datetime.utcnow().strftime('%Y%m%d_%H%M%S')}.json"
+        filename = f"logs_{datetime.now(timezone.utc).strftime('%Y%m%d_%H%M%S')}.json"
 
     return StreamingResponse(
         iter([content]),

backend/src/api/schemas/common.py

@@ -5,7 +5,7 @@
 """
 
 from typing import Any, Optional, Generic, TypeVar
-from datetime import datetime
+from datetime import datetime, timezone
 from pydantic import BaseModel, Field
 
 
@@ -313,7 +313,7 @@ def create_success_response(
         True
     """
     meta = ResponseMeta(
-        timestamp=datetime.utcnow().isoformat() + "Z",
+        timestamp=datetime.now(timezone.utc).isoformat() + "Z",
         request_id=request_id,
         page=page,
         per_page=per_page,
@@ -356,7 +356,7 @@ def create_error_response(
         False
     """
     meta = ResponseMeta(
-        timestamp=datetime.utcnow().isoformat() + "Z",
+        timestamp=datetime.now(timezone.utc).isoformat() + "Z",
         request_id=request_id,
     )
 

backend/src/api/schemas/entry.py

@@ -5,7 +5,7 @@
 """
 
 from typing import Optional
-from datetime import datetime
+from datetime import datetime, timezone
 from pydantic import BaseModel, Field
 
 
@@ -59,7 +59,7 @@ class EntryResponse(EntryBase):
         ...     points_spent=50,
         ...     entry_type="manual",
         ...     status="success",
-        ...     entered_at=datetime.utcnow()
+        ...     entered_at=datetime.now(timezone.utc)
         ... )
     """
 

backend/src/api/schemas/game.py

@@ -5,7 +5,7 @@
 """
 
 from typing import Optional
-from datetime import datetime
+from datetime import datetime, timezone
 from pydantic import BaseModel, Field
 
 
@@ -102,7 +102,7 @@ class GameResponse(GameBase):
         ...     name="Portal 2",
         ...     type="game",
         ...     review_score=9,
-        ...     last_refreshed_at=datetime.utcnow()
+        ...     last_refreshed_at=datetime.now(timezone.utc)
         ... )
     """
 
@@ -214,7 +214,7 @@ class GameRefreshResponse(BaseModel):
         >>> response = GameRefreshResponse(
         ...     refreshed=True,
         ...     message="Game data refreshed successfully",
-        ...     last_refreshed_at=datetime.utcnow()
+        ...     last_refreshed_at=datetime.now(timezone.utc)
         ... )
     """
 

backend/src/api/schemas/giveaway.py

@@ -5,7 +5,7 @@
 """
 
 from typing import Optional
-from datetime import datetime
+from datetime import datetime, timezone
 from pydantic import BaseModel, Field, field_serializer
 
 
@@ -102,7 +102,7 @@ class GiveawayResponse(GiveawayBase):
         ...     url="https://www.steamgifts.com/giveaway/AbCd1/",
         ...     game_name="Portal 2",
         ...     price=50,
-        ...     discovered_at=datetime.utcnow()
+        ...     discovered_at=datetime.now(timezone.utc)
         ... )
     """
 

backend/src/api/schemas/settings.py

@@ -5,7 +5,7 @@
 """
 
 from typing import Optional
-from datetime import datetime
+from datetime import datetime, timezone
 from pydantic import BaseModel, Field, field_validator
 
 
@@ -163,8 +163,8 @@ class SettingsResponse(SettingsBase):
         ...     id=1,
         ...     user_agent="Mozilla/5.0...",
         ...     autojoin_enabled=True,
-        ...     created_at=datetime.utcnow(),
-        ...     updated_at=datetime.utcnow()
+        ...     created_at=datetime.now(timezone.utc),
+        ...     updated_at=datetime.now(timezone.utc)
         ... )
     """
 

backend/src/core/events.py

@@ -6,7 +6,7 @@
 
 import asyncio
 import json
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Dict, Any, Set
 from fastapi import WebSocket
 
@@ -123,7 +123,7 @@ async def broadcast_event(self, event_type: str, data: Dict[str, Any]) -> None:
         event = {
             "type": event_type,
             "data": data,
-            "timestamp": datetime.utcnow().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
         }
 
         # Track disconnected clients for removal

backend/src/models/account.py

@@ -4,7 +4,7 @@
 from sqlalchemy import String, Integer, Boolean, DateTime
 from sqlalchemy.orm import Mapped, mapped_column
 
-from models.base import Base, TimestampMixin
+from models.base import Base, TimestampMixin, TZDateTime
 
 
 class Account(Base, TimestampMixin):
@@ -167,7 +167,7 @@ class Account(Base, TimestampMixin):
 
     # ==================== Metadata ====================
     last_synced_at: Mapped[datetime | None] = mapped_column(
-        DateTime,
+        TZDateTime,
         nullable=True,
         comment="Last sync with SteamGifts",
     )