Store results

backend/main.py

@@ -49,7 +49,7 @@ from fastapi.middleware.cors import CORSMiddleware
 from fastapi.staticfiles import StaticFiles
 from fastapi.responses import FileResponse
 
-from game.bot import BOT_PLAYER_ID, BotPlayer
+from game.bot import BOT_PLAYER_ID, BOT_PLAYER_NAME, BotPlayer
 from game.commands import ActionType
 from game.engine import GameEngine
 from game.map import MAP_LANDMARKS
@@ -75,16 +75,33 @@ log = logging.getLogger(__name__)
 
 def _record_game_over(room, winner_id: str, winner_name: str) -> None:
     """Persist stats for tutorial completion or PvP win."""
-    # Never record a bot or tutorial dummy win in the leaderboard
-    if winner_id in (BOT_PLAYER_ID, TUTORIAL_DUMMY_ID):
+    # Never record tutorial dummy wins
+    if winner_id == TUTORIAL_DUMMY_ID:
         return
     import time as _time
     duration = int(_time.time() - room.started_at) if room and room.started_at else None
     engine = engines.get(room.room_id) if room else None
     is_tutorial = bool(engine and engine.state.is_tutorial) if engine else False
     event_type = "tutorial_complete" if is_tutorial else "game_won"
-    # Resolve opponent name for PvP matches
-    opponent_name: str | None = None
+
+    # When the bot wins, record the match with the human opponent so it appears
+    # in match history — bot wins are excluded from the leaderboard in stats.py
+    if winner_id == BOT_PLAYER_ID:
+        opponent_name: str | None = None
+        if engine:
+            for pid, pstate in engine.state.players.items():
+                if pid != BOT_PLAYER_ID:
+                    opponent_name = pstate.player_name
+                    break
+        try:
+            _stats.record(BOT_PLAYER_NAME, event_type, duration, opponent_name)
+            log.info("Stat recorded: %s %s vs %s %ss", BOT_PLAYER_NAME, event_type, opponent_name, duration)
+        except Exception:
+            log.exception("Failed to record stat")
+        return
+
+    # Resolve opponent name for PvP/bot matches
+    opponent_name = None
     if not is_tutorial and engine:
         for pid, pstate in engine.state.players.items():
             if pid != winner_id and pid != BOT_PLAYER_ID:
@@ -92,7 +109,7 @@ def _record_game_over(room, winner_id: str, winner_name: str) -> None:
                 break
         # If opponent was the bot, label it
         if opponent_name is None and any(pid == BOT_PLAYER_ID for pid in engine.state.players):
-            opponent_name = "Bot"
+            opponent_name = BOT_PLAYER_NAME
     try:
         _stats.record(winner_name, event_type, duration, opponent_name)
         log.info("Stat recorded: %s %s vs %s %ss", winner_name, event_type, opponent_name, duration)

backend/stats.py

@@ -87,8 +87,11 @@ def get_tutorial_leaderboard(limit: int = 50) -> list[dict]:
     return [dict(r) for r in rows]
 
 
+_BOT_PLAYER_NAME = "Bot IA"
+
+
 def get_wins_leaderboard(limit: int = 50) -> list[dict]:
-    """Most 1v1 wins, ordered by win count then best time."""
+    """Most 1v1 wins, ordered by win count then best time. Bots are excluded."""
     with _connect() as conn:
         rows = conn.execute(
             """
@@ -98,11 +101,12 @@ def get_wins_leaderboard(limit: int = 50) -> list[dict]:
                    MAX(recorded_at) AS last_at
             FROM player_stats
             WHERE event_type = 'game_won'
+              AND player_name != ?
             GROUP BY player_name
             ORDER BY wins DESC, best_s ASC NULLS LAST
             LIMIT ?
             """,
-            (limit,),
+            (_BOT_PLAYER_NAME, limit),
         ).fetchall()
     return [dict(r) for r in rows]