fix: Resolve race condition in user stats recording and update navigation button behavior

- Updated UserStatsService to use newElo if available, preventing stale ELO reads during ranked duels.
- Modified Game class to pass newElo directly to user stats to ensure accurate recording.
- Changed navigation button in UserProfilePage to use window.location.href for improved reliability.

Author: codergautam

components/utils/userStatsService.js

@@ -18,9 +18,13 @@ class UserStatsService {
         return null;
       }
 
+      // Use provided newElo if available (for ranked duels where setElo may not have completed yet)
+      // This avoids race condition where we read stale ELO from DB before setElo writes complete
+      const eloToRecord = gameData.newElo ?? user.elo ?? 1000;
+
       // Get current rankings
       const xpRank = await this.calculateXPRank(user.totalXp);
-      const eloRankResult = await this.calculateELORank(user.elo || 1000);
+      const eloRankResult = await this.calculateELORank(eloToRecord);
       const eloRank = typeof eloRankResult === 'object' ? eloRankResult.rank : eloRankResult;
 
       // Record the stats snapshot
@@ -29,7 +33,7 @@ class UserStatsService {
         timestamp: new Date(),
         totalXp: user.totalXp || 0,
         xpRank: xpRank,
-        elo: user.elo || 1000,
+        elo: eloToRecord,
         eloRank: eloRank,
         triggerEvent: gameData?.triggerEvent || 'game_completed',
         gameId: gameId

pages/user.js

@@ -188,7 +188,9 @@ export default function UserProfilePage() {
         <div className="back-button-container">
           <button 
             className="back-to-wg-button"
-            onClick={() => router.push('/')}
+            // onClick={() => router.push('/')}
+            //  use window.location.href
+            onClick={() => window.location.href = '/'}
           >
             ← Go to WorldGuessr
           </button>

ws/classes/Game.js

@@ -1191,7 +1191,8 @@ export default class Game {
             opponent: this.accountIds.p2,
             eloChange: p1NewElo ? (p1NewElo - p1OldElo) : 0,
             finalScore: player1Data?.score || 0,
-            duration: this.endTime - this.startTime
+            duration: this.endTime - this.startTime,
+            newElo: p1NewElo // Pass new ELO directly to avoid race condition with setElo
           }
         );
         console.log(`Created userstats for player 1: ${this.accountIds.p1}`);
@@ -1208,7 +1209,8 @@ export default class Game {
             opponent: this.accountIds.p1,
             eloChange: p2NewElo ? (p2NewElo - p2OldElo) : 0,
             finalScore: player2Data?.score || 0,
-            duration: this.endTime - this.startTime
+            duration: this.endTime - this.startTime,
+            newElo: p2NewElo // Pass new ELO directly to avoid race condition with setElo
           }
         );
         console.log(`Created userstats for player 2: ${this.accountIds.p2}`);