Update app.py

app.py

@@ -921,23 +921,25 @@ def create_game(host=None, topics=[], num_questions=5, auto_close=True, ai_topic
 # PLAY PAGE
 # -------------------------
 def play_page():
-    import time
+    import time, json
     import streamlit as st
 
     gid = st.session_state.get("active_game_id")
     uname = st.session_state.get("username")
-    
+
     if not gid or not uname:
         st.error("No active game or username found. Please join or create a game first.")
         return
 
-    # Get game and questions
+    # ---------------- Load game ----------------
     games = unified_get("games") or {}
     game = games.get(gid)
+
     if not game:
         st.error("Game not found.")
         return
-    if game.get('closed'):
+
+    if game.get("closed"):
         st.warning("This game is closed.")
         return
 
@@ -946,35 +948,33 @@ def play_page():
         st.info("No questions loaded for this game.")
         return
 
-    # Initialize session state for answers and times
-    if 'answers' not in st.session_state:
-        st.session_state['answers'] = [""] * len(questions)
-    if 'answer_times' not in st.session_state:
-        st.session_state['answer_times'] = [0] * len(questions)
-    if 'current_index' not in st.session_state:
-        st.session_state['current_index'] = 0
-    if 'question_started_at' not in st.session_state:
-        st.session_state['question_started_at'] = time.time()
+    # ---------------- Init session state ----------------
+    if "answers" not in st.session_state:
+        st.session_state.answers = [""] * len(questions)
+    if "answer_times" not in st.session_state:
+        st.session_state.answer_times = [0] * len(questions)
+    if "current_index" not in st.session_state:
+        st.session_state.current_index = 0
+    if "question_started_at" not in st.session_state:
+        st.session_state.question_started_at = time.time()
 
-    idx = st.session_state['current_index']
+    idx = st.session_state.current_index
 
-    # All done
+    # ---------------- All done ----------------
     if idx >= len(questions):
         st.success("All done — submit your answers!")
         return
 
-    # ---------------- QUESTION UI ----------------
+    # ---------------- Question UI ----------------
     q = questions[idx]
     st.subheader(f"Question {idx + 1}/{len(questions)}")
     st.write(q["question"])
 
-    # Safe elapsed calculation
-    start_time = st.session_state.get('question_started_at') or time.time()
+    start_time = st.session_state.question_started_at
     elapsed = int(time.time() - start_time)
     time_limit = 15
-    st.markdown(f"**Time left:** {max(0, time_limit - elapsed)} seconds")
+    st.markdown(f"**⏱ Time left:** {max(0, time_limit - elapsed)} seconds")
 
-    # Stable radio buttons
     choice = st.radio(
         "Choose an answer:",
         q["options"],
@@ -986,28 +986,29 @@ def play_page():
     # ---------------- NEXT ----------------
     with col1:
         if st.button("Next", key=f"next_{gid}_{idx}"):
-            st.session_state['answers'][idx] = choice
-            st.session_state['answer_times'][idx] = time.time() - start_time
-            st.session_state['current_index'] = idx + 1
-            st.session_state['question_started_at'] = time.time()
+            st.session_state.answers[idx] = choice
+            st.session_state.answer_times[idx] = time.time() - start_time
+            st.session_state.current_index += 1
+            st.session_state.question_started_at = time.time()
             st.rerun()
 
     # ---------------- SUBMIT ----------------
     with col2:
         if idx == len(questions) - 1:
             if st.button("Submit All Answers", key=f"submit_{gid}_{idx}"):
-                # Record last question
-                st.session_state['answers'][idx] = choice
-                st.session_state['answer_times'][idx] = time.time() - start_time
 
-                answers = st.session_state['answers']
-                times = st.session_state['answer_times']
+                # Save last answer
+                st.session_state.answers[idx] = choice
+                st.session_state.answer_times[idx] = time.time() - start_time
+
+                answers = st.session_state.answers
+                times = st.session_state.answer_times
 
                 # Compute score
                 score, flags = compute_score(questions, answers, times)
                 percentage = int(score / (len(questions) * 15) * 100)
 
-                # ---------------- Update players dict ----------------
+                # ---------------- Update players ----------------
                 players = unified_get("players") or {}
                 players.setdefault(gid, {})
                 players[gid][uname] = {
@@ -1020,37 +1021,35 @@ def play_page():
                 }
                 unified_set("players", players)
 
-                # ---------------- Push leaderboard ----------------
+                # ---------------- Save leaderboard ----------------
                 row = {
                     "name": uname,
-                    "avatar": st.session_state.get("avatar", "🎮"),
                     "score": score,
-                    "percentage": percentage,
                     "game_id": gid,
-                    "topics": game.get("topics", []),
+                    "topics": ",".join(game.get("topics", [])),
                     "timestamp": now_iso(),
+                    "avatar": st.session_state.get("avatar", "🎮"),
                     "questions": len(questions),
-                    "answers": answers,
-                    "correct_flags": flags
+                    "answers": json.dumps(answers),
+                    "correct_flags": json.dumps(flags),
                 }
+
                 unified_push_leaderboard(row)
                 save_score_to_csv(row)
 
-                # Auto-close game if needed
-                if game.get('auto_close', True):
-                    games[gid]['closed'] = True
-                    games[gid]['closed_at'] = now_iso()
+                # ---------------- Auto close game ----------------
+                if game.get("auto_close", True):
+                    games[gid]["closed"] = True
+                    games[gid]["closed_at"] = now_iso()
                     unified_set("games", games)
 
-                # Reset session state
-                st.success(f"Submitted! Score: {score} / {len(questions)*15} ({percentage}%)")
+                st.success(f"🎉 Submitted! Score: {score} / {len(questions)*15} ({percentage}%)")
                 st.balloons()
-                st.session_state['last_score'] = score
-                st.session_state['last_game'] = gid
-                st.session_state['current_index'] = 0
-                st.session_state['answers'] = []
-                st.session_state['answer_times'] = []
+
+                # Reset state
+                st.session_state.clear()
                 st.rerun()
+
 # Join game
 def join_game_page():
     st.header("Join Game")