Unique game ID

app.py

@@ -326,6 +326,8 @@ def build_street_html(image_url: str) -> str:
 def gr_start_game(difficulty: str, username: str, request: gr.Request):
     rounds: List[Dict[str, Any]] = []
     date_str = datetime.now(timezone.utc).date().isoformat()
+    game_id = str(uuid.uuid4())  # Generate unique game ID
+    
     for _ in range(3):
         loc = pick_random_location(difficulty)
         round_id = generate_id()
@@ -341,6 +343,7 @@ def gr_start_game(difficulty: str, username: str, request: gr.Request):
         })
 
     user_sessions[username] = {
+        'game_id': game_id,
         'difficulty': difficulty,
         'rounds': rounds,
         'total_score': 0.0,
@@ -881,6 +884,7 @@ with gr.Blocks(css=APP_CSS, title="LLM GeoGuessr") as demo:
             
             # Record this round's data immediately to prevent abuse
             sess = user_sessions.get(username, {})
+            game_id = sess.get('game_id', '')
             round_idx = next((i for i, rr in enumerate(sess.get('rounds', [])) if rr['id'] == rid), 0) + 1
             round_record = {
                 "round_number": round_idx,
@@ -893,7 +897,7 @@ with gr.Blocks(css=APP_CSS, title="LLM GeoGuessr") as demo:
                 "ai_score": float(round(rnd.get('ai_score', 0))) if rnd.get('ai_score') else 0.0,
                 "ai_analysis": rnd.get('ai_analysis', ''),
             }
-            data_manager.update_game_record(username, round_data=round_record)
+            data_manager.update_game_record(username, game_id, round_data=round_record)
         except Exception as e:
             yield "", txt + f"\n\n[Error] {e}"
             return
@@ -966,9 +970,10 @@ with gr.Blocks(css=APP_CSS, title="LLM GeoGuessr") as demo:
         if not sess or idx >= len(sess['rounds']):
             total_human = sum(float(r.get('human_score', 0.0)) for r in sess.get('rounds', []))
             total_ai = sum(float(r.get('ai_score', 0.0)) for r in sess.get('rounds', []))
+            game_id = sess.get('game_id', '')
             
             # Update the final scores (rounds already recorded incrementally)
-            data_manager.update_game_record(username, final_score=total_human, final_ai_score=total_ai)
+            data_manager.update_game_record(username, game_id, final_score=total_human, final_ai_score=total_ai)
 
             winner_message = "It's a tie!"
             if total_human > total_ai:

data_manager.py

@@ -42,25 +42,47 @@ def get_todays_games(token: str) -> list:
 
 def has_user_played_today(username: str, todays_games: list) -> bool:
     """Checks if a user has completed a game today."""
+    today_date = datetime.now(timezone.utc).date().isoformat()
+    
     for game in todays_games:
         if game.get("username") == username and game.get("completed", False):
-            return True
+            # Additional check: ensure the game's timestamp is from today
+            game_timestamp = game.get("timestamp", "")
+            if game_timestamp:
+                try:
+                    game_date = datetime.fromisoformat(game_timestamp.replace('Z', '+00:00')).date().isoformat()
+                    if game_date == today_date:
+                        return True
+                except:
+                    # If we can't parse the timestamp, assume it's from today (since it's in todays_games)
+                    return True
+            else:
+                # If no timestamp, assume it's from today (since it's in todays_games)
+                return True
     return False
 
-def get_user_game_today(username: str, todays_games: list) -> dict:
-    """Gets the user's game record for today, if it exists."""
+def get_user_game_today(username: str, todays_games: list, game_id: str = None) -> dict:
+    """
+    Gets the user's game record for today, if it exists.
+    If game_id is provided, returns the specific game with that ID.
+    Otherwise, returns the first game found for the user.
+    """
     for game in todays_games:
         if game.get("username") == username:
-            return game
+            if game_id and game.get("game_id") == game_id:
+                return game
+            elif not game_id:
+                return game
     return None
 
-def update_game_record(username: str, round_data: dict = None, final_score: float = None, final_ai_score: float = None):
+def update_game_record(username: str, game_id: str, round_data: dict = None, final_score: float = None, final_ai_score: float = None):
     """
     Updates or creates a game record for a user after each round.
     This ensures data is recorded incrementally and prevents abuse.
     
     Args:
         username: The player's username
+        game_id: Unique identifier for this game session
         round_data: Single round details to append to the record
         final_score: Final total human score (only set when game is complete)
         final_ai_score: Final total AI score (only set when game is complete)
@@ -74,8 +96,8 @@ def update_game_record(username: str, round_data: dict = None, final_score: floa
         # Fetch the latest records
         todays_games = get_todays_games(token=write_token)
         
-        # Find existing game record for this user today
-        existing_game = get_user_game_today(username, todays_games)
+        # Find existing game record for this user today with this specific game_id
+        existing_game = get_user_game_today(username, todays_games, game_id)
         
         if existing_game:
             # Update existing record
@@ -92,6 +114,7 @@ def update_game_record(username: str, round_data: dict = None, final_score: floa
             # Create new game record
             game_record = {
                 "username": username,
+                "game_id": game_id,
                 "human_score": 0.0,  # Will be updated when game completes
                 "ai_score": 0.0,  # Will be updated when game completes
                 "timestamp": datetime.now(timezone.utc).isoformat(),
@@ -122,9 +145,12 @@ def update_game_record(username: str, round_data: dict = None, final_score: floa
         if 'tmp_file_path' in locals() and os.path.exists(tmp_file_path):
             os.remove(tmp_file_path)
 
-def record_game(username: str, score: float, rounds_data: list = None, ai_score: float = None):
+def record_game(username: str, score: float, rounds_data: list = None, ai_score: float = None, game_id: str = None):
     """
     Legacy function - now just calls update_game_record with final score.
     Kept for backwards compatibility.
     """
-    update_game_record(username, final_score=score, final_ai_score=ai_score)
+    if not game_id:
+        import uuid
+        game_id = str(uuid.uuid4())
+    update_game_record(username, game_id, final_score=score, final_ai_score=ai_score)