Update app.py (#104)

- Update app.py (d5b3858c95dd43e2596968600625436fd9da1fca)

app.py

@@ -1237,9 +1237,18 @@ def inbox_page():
 # Leaderboard page
 # ----------------- Leaderboard Page -----------------
 def leaderboard_page():
+    import os, csv
+    import streamlit as st
+    from datetime import datetime
+
     st.header("🏆 Leaderboard")
 
-    # Get leaderboard data
+    # ---------------- File check ----------------
+    if not os.path.exists(LEADERBOARD_FILE):
+        st.info("No scores yet.")
+        return
+
+    # ---------------- Download CSV ----------------
     with open(LEADERBOARD_FILE, "rb") as f:
         st.download_button(
             "⬇ Download Leaderboard CSV",
@@ -1248,35 +1257,53 @@ def leaderboard_page():
             mime="text/csv"
         )
 
-    leaderboard = unified_get("leaderboard") or []
+    # ---------------- Read CSV ----------------
+    leaderboard = []
+    with open(LEADERBOARD_FILE, "r", encoding="utf-8") as f:
+        reader = csv.DictReader(f)
+        leaderboard = list(reader)
 
     if not leaderboard:
         st.info("No scores yet.")
         return
 
-    # Sort by score descending
-    leaderboard = sorted(leaderboard, key=lambda x: x.get("score", 0), reverse=True)
+    # Convert score to int (CSV reads as string)
+    for r in leaderboard:
+        r["score"] = int(r.get("score", 0))
+
+    # ---------------- Sort by score ----------------
+    leaderboard = sorted(leaderboard, key=lambda x: x["score"], reverse=True)
 
-    # Display top 10
+    # ---------------- Top 10 ----------------
     st.subheader("Top Players")
     for i, row in enumerate(leaderboard[:10], 1):
         st.write(
             f"{i}. {row.get('avatar','🎮')} **{row.get('name','Guest')}** "
-            f"(Game {row.get('game_id','')}) — {row.get('score',0)} pts — {row.get('percentage',0)}%"
+            f"(Game {row.get('game_id','')}) — {row.get('score',0)} pts"
         )
 
-    # Weekly leaderboard (optional)
+    # ---------------- Weekly Leaderboard ----------------
     st.subheader("Weekly Leaderboard (Top 10)")
-    week_leaderboard = [r for r in leaderboard if datetime.fromisoformat(r.get("timestamp", now_iso())).isocalendar()[1] == datetime.now().isocalendar()[1]]
-    if not week_leaderboard:
+    current_week = datetime.now().isocalendar()[1]
+
+    weekly = []
+    for r in leaderboard:
+        try:
+            ts = datetime.fromisoformat(r.get("timestamp", ""))
+            if ts.isocalendar()[1] == current_week:
+                weekly.append(r)
+        except Exception:
+            pass
+
+    if not weekly:
         st.info("No scores yet this week.")
     else:
-        week_leaderboard = sorted(week_leaderboard, key=lambda x: x.get("score",0), reverse=True)
-        for i, r in enumerate(week_leaderboard[:10], 1):
+        for i, r in enumerate(weekly[:10], 1):
             st.write(
                 f"{i}. {r.get('avatar','🎮')} **{r.get('name','Guest')}** "
                 f"(Game {r.get('game_id','')}) — {r.get('score',0)} pts"
             )
+
             
 def get_weekly_leaderboard(limit=10):
     rows = unified_get("/leaderboard") or []