Update index.html

index.html

@@ -1,3 +1,5 @@
+// DARKONBALLS FREEZE FIX — POST-GAME FIREBASE IS NON-BLOCKING — 2026-10-06
+
 <!DOCTYPE html>
 <html lang="en">
 <head>
@@ -3821,10 +3823,6 @@ let onlineBallSpawnTimer  = 0;
 let onlineSearchStartTime = 0;
 let onlineBotMatchTimeout = null;
 let onlineBotElo = parseInt(localStorage.getItem("onlineBotElo")) || 1500;
-// ONLINE BOT HARD LOCK: these are set when the bot match starts and the bot is
-// forced back to these exact coordinates every simulation tick.
-let onlineBotFixedX = null;
-let onlineBotFixedY = null;
 let onlineSimTick = 0;
 let onlineMatchStamp = 0;
 let onlineRound = 0, postUnsub = null, rematchSent = false, lastPostChat = 0;
@@ -3864,23 +3862,6 @@ const NET_SMOOTH = {
 
 // Gemeinsame Paket-Verarbeitung für Online-Gegner UND Online TV (identisch sauber)
 function feedNetPlayer(pl, st, stopMs) {
-  // In an Online-vs-Bot match the bot is local, not a network-controlled player.
-  // Ignore network position packets completely so smoothing/extrapolation cannot move it.
-  if (mode === "online" && onlineVsBot && pl && pl.name !== myPlayName()) {
-    if (onlineBotFixedX == null) onlineBotFixedX = pl.x;
-    if (onlineBotFixedY == null) onlineBotFixedY = pl.y;
-    pl.x = onlineBotFixedX;
-    pl.y = onlineBotFixedY;
-    pl.tx = onlineBotFixedX;
-    pl.ty = onlineBotFixedY;
-    pl.baseTx = onlineBotFixedX;
-    pl.baseTy = onlineBotFixedY;
-    pl.netVX = 0;
-    pl.netVY = 0;
-    pl.alive = st.alive !== false;
-    pl.lastPacketAt = Date.now();
-    return;
-  }
   const now = Date.now();
   // Jedes Match-Update (Ball, anderer Spieler ...) liefert den Spielerstand erneut.
   // Unveränderte Daten sind KEIN neues Paket -> ignorieren (sonst Geschwindigkeit=0 / Zittern).
@@ -4828,22 +4809,6 @@ function makeLCG(seed) {
 // ═══════════════════════════════════════════════
 //  ONLINE GAME
 // ═══════════════════════════════════════════════
-function lockOnlineBotPosition(bot) {
-  if (!onlineVsBot || !bot) return;
-  if (onlineBotFixedX == null) onlineBotFixedX = bot.x;
-  if (onlineBotFixedY == null) onlineBotFixedY = bot.y;
-  bot.x = onlineBotFixedX;
-  bot.y = onlineBotFixedY;
-  bot.tx = onlineBotFixedX;
-  bot.ty = onlineBotFixedY;
-  bot.baseTx = onlineBotFixedX;
-  bot.baseTy = onlineBotFixedY;
-  bot.netVX = 0;
-  bot.netVY = 0;
-  bot.vx = 0;
-  bot.vy = 0;
-}
-
 function startOnlineGame(matchData) {
   setOnlineAspect(matchData && matchData.arena);
   hideAllScreensForGame();
@@ -4891,10 +4856,6 @@ function startOnlineGame(matchData) {
   const mePlayer  = createPlayer(meX,  py, myColor,  myName);
   const oppPlayer = createPlayer(oppX, py, oppColor, oppName);
 
-  // HARD LOCK the Online Bot to its spawn point.
-  onlineBotFixedX = onlineVsBot ? oppX : null;
-  onlineBotFixedY = onlineVsBot ? py   : null;
-
   oppPlayer.tx = oppX;
   oppPlayer.ty = py;
   // NEW: network-smoothing bookkeeping for the opponent's reconstructed
@@ -4916,7 +4877,6 @@ function startOnlineGame(matchData) {
   }
 
   players.push(mePlayer, oppPlayer);
-  lockOnlineBotPosition(oppPlayer);
 
   document.getElementById("onlineNameMe").textContent  = dn(myName);
   document.getElementById("onlineNameOpp").textContent = dn(oppName);
@@ -4979,14 +4939,12 @@ function startOnlineGame(matchData) {
     const oppRole = onlineRole === "p1" ? "p2" : "p1";
 
     const oppState = data[oppRole + "State"];
-    if (oppState && !onlineVsBot) {
+    if (oppState) {
       const op = players.find(p => p.name !== myName);
       if (op) {
         feedNetPlayer(op, oppState);
       }
     }
-    // Online-vs-Bot is 100% local for the bot. Firebase must NEVER be
-    // allowed to provide a bot position/state that can move it.
 
     if (!onlineIsHost && data.ballState) {
       try {
@@ -5186,11 +5144,7 @@ async function handleOnlineResult(winnerName, change) {
   document.getElementById("onlineHUD").style.display = "none";
 
   const won = winnerName === onlineAccount.name;
-  // Online-vs-Bot is a practice/reward match: the user's ELO NEVER drops.
-  // Every completed bot match awards exactly +30 ELO, regardless of who was
-  // hit first. Real online matches keep the normal result calculation.
-  const botReward = onlineVsBot ? 30 : clampEloChange(Math.round(change));
-  const eloChange = onlineVsBot ? 30 : botReward;
+  const eloChange = clampEloChange(Math.round(change));
   const oppName = onlineOpponent ? onlineOpponent.name : "Online Bot";
   const oppUid  = onlineOpponent ? onlineOpponent.uid  : ONLINE_BOT_UID;
 
@@ -5235,19 +5189,7 @@ async function handleOnlineResult(winnerName, change) {
 
   onlineAccount = sanitizeAccount(onlineAccount);
 
-  if (onlineVsBot) {
-    onlineAccount.elo += 30;
-    onlineAccount.wins++;
-    onlineAccount.currentStreak++;
-    onlineAccount.totalEloGained += 30;
-    if (onlineAccount.currentStreak > onlineAccount.bestStreak) {
-      onlineAccount.bestStreak = onlineAccount.currentStreak;
-    }
-    if (onlineAccount.elo > onlineAccount.highestElo) {
-      onlineAccount.highestElo = onlineAccount.elo;
-    }
-    // Deliberately NO loss/elo subtraction path for bot matches.
-  } else if (won) {
+  if (won) {
     onlineAccount.elo += eloChange;
     onlineAccount.wins++;
     onlineAccount.currentStreak++;
@@ -5286,11 +5228,32 @@ async function handleOnlineResult(winnerName, change) {
   saveAccountLocal();
   updateAccountBtn();
   lastLeagueNote = "";
-  if (fbReady) await pushAccountToFirebase(); // writes ONLY players/{myUid}
-  if (won && fbReady) leagueRecordWin().catch(e => console.warn("league win failed", e));
 
-  showEloAnimation(onlineVsBot ? 30 : (won ? eloChange : -eloChange));
-  setTimeout(() => showWinnerOverlay(winnerName, onlineVsBot ? 30 : (won ? eloChange : -eloChange)), 2600);
+  // IMPORTANT: never wait for Firebase before showing the result screen.
+  // A slow/hung Firebase connection used to make the completed game look frozen.
+  // Persist everything in the background instead.
+  if (fbReady) {
+    try {
+      Promise.resolve(pushAccountToFirebase()).catch(e =>
+        console.warn("post-match account save failed:", e)
+      );
+    } catch (e) {
+      console.warn("post-match account save failed:", e);
+    }
+  }
+  if (won && fbReady) {
+    try {
+      Promise.resolve(leagueRecordWin()).catch(e =>
+        console.warn("league win failed:", e)
+      );
+    } catch (e) {
+      console.warn("league win failed:", e);
+    }
+  }
+
+  // Show the result immediately. Do not make the UI wait on network operations.
+  showEloAnimation(won ? eloChange : -eloChange);
+  setTimeout(() => showWinnerOverlay(winnerName, won ? eloChange : -eloChange), 500);
 
   // Bot stats: written to players/{ONLINE_BOT_UID}. This is the ONE
   // place a client writes to a uid that isn't its own — acceptable
@@ -5344,7 +5307,8 @@ async function handleOnlineResult(winnerName, change) {
     localStorage.setItem("onlineBotElo", onlineBotElo);
 
     if (fbReady) {
-      await dbSetBotPlayer({
+      // Fire-and-forget: bot-stat Firebase writes must NEVER block the result screen.
+      Promise.resolve().then(() => dbSetBotPlayer({
         uid: ONLINE_BOT_UID,
         name: "Online Bot",
         elo: onlineBotElo,
@@ -5361,7 +5325,7 @@ async function handleOnlineResult(winnerName, change) {
         worstLossChange: onlineOpponent.worstLossChange ?? 0,
         worstLossOpp: onlineOpponent.worstLossOpp ?? "",
         updatedAt: Date.now()
-      });
+      })).catch(e => console.warn("bot stat save failed:", e));
     }
   }
 
@@ -6289,13 +6253,9 @@ function update() {
 
     if (onlineVsBot) {
       const bot = players.find(p => p.name !== myPlayName());
-      if (bot) {
-        // HARD LOCK: the online bot has exactly ONE legal position.
-        // This runs every simulation tick, so AI, Firebase, smoothing,
-        // velocity, or any other code cannot move the bot.
-        lockOnlineBotPosition(bot);
+      if (bot && bot.alive) {
+        updateAI(bot, balls);
         checkBotCollision(bot);
-        lockOnlineBotPosition(bot);
       }
     } else {
       // CHANGED (network smoothing): replaced the fixed-rate nudge
@@ -6382,7 +6342,7 @@ function update() {
   for (const p of players) {
     if (!p.alive) continue;
     const humanNames = ["Player","Player 1","Player 2","YOU"];
-    if (!humanNames.includes(p.name) && !(mode === "online" && onlineVsBot && p.name !== myPlayName())) updateAI(p, balls);
+    if (!humanNames.includes(p.name)) updateAI(p, balls);
   }
 
   for (const p of players) {
@@ -6437,7 +6397,7 @@ function checkOnlineCollision(me) {
       if (onlineVsBot) {
         const oppElo = onlineOpponent?.elo || 1500;
         const diff = onlineAccount.elo - oppElo;
-        const change = onlineVsBot ? 30 : Math.max(1, Math.min(15, Math.round(5 + diff / 100)));
+        const change = Math.max(1, Math.min(15, Math.round(5 + diff / 100)));
 
         handleOnlineResult(opp ? opp.name : "", change);
       } else {
@@ -6460,7 +6420,7 @@ function checkBotCollision(bot) {
         const oppElo = onlineOpponent?.elo || 1500;
 
         const diff = oppElo - onlineAccount.elo;
-        const change = onlineVsBot ? 30 : Math.max(1, Math.min(15, Math.round(5 + diff / 100)));
+        const change = Math.max(1, Math.min(15, Math.round(5 + diff / 100)));
 
         handleOnlineResult(me.name, change);
       }
@@ -6565,7 +6525,7 @@ async function reportOnlineResult(winnerName, loserName) {
   const loserElo = (loserName === me.name) ? me.elo : opp.elo;
 
   const diff = loserElo - winnerElo;
-  const change = onlineVsBot ? 30 : clampEloChange(Math.max(1, Math.min(15, Math.round(5 + diff / 100))));
+  const change = clampEloChange(Math.max(1, Math.min(15, Math.round(5 + diff / 100))));
 
   // Bot-Match: es gibt keinen Match-Listener, der "done" verarbeitet -> Ergebnis direkt behandeln
   // (sonst laeuft das Spiel nach der Camping-Disqualifikation einfach weiter)