Try to revert for testing

Author: eltoder

src/game.js

@@ -401,7 +401,7 @@ function processEvent(internalGameState, event) {
   updateBoard(internalGameState, event, cards);
 }
 
-export function computeState(gameData, gameMode, newEvents = null) {
+export function computeState(gameData, gameMode) {
   if (!modes.hasOwnProperty(gameMode)) {
     throw new Error(`invalid gameMode: ${gameMode}`);
   }
@@ -440,21 +440,10 @@ export function computeState(gameData, gameMode, newEvents = null) {
   };
 
   if (gameData.events) {
-    let events;
-    // Array.sort() is guaranteed to be stable in since around 2018.
-    if (newEvents?.size) {
-      // Always sort new events (with approximate times) after old events.
-      events = Object.entries(gameData.events)
-        .sort(
-          ([k1, e1], [k2, e2]) =>
-            newEvents.has(k1) - newEvents.has(k2) || e1.time - e2.time
-        )
-        .map(([k, e]) => e);
-    } else {
-      events = Object.values(gameData.events).sort(
-        (e1, e2) => e1.time - e2.time
-      );
-    }
+    // Array.sort() is guaranteed to be stable in since around 2018
+    const events = Object.values(gameData.events).sort(
+      (e1, e2) => e1.time - e2.time
+    );
     for (const event of events) {
       processEvent(internalGameState, event);
     }

src/pages/GamePage.js

@@ -124,7 +124,6 @@ function GamePage({ match }) {
 
   const [game, loadingGame] = useFirebaseRef(`games/${gameId}`);
   const [gameData, loadingGameData] = useFirebaseRef(`gameData/${gameId}`);
-  const newEvents = useRef(new Set());
   const [hasNextGame] = useFirebaseRef(
     game?.status === "done" && (!game.users || !(user.id in game.users))
       ? `games/${nextGameId}/status`
@@ -202,11 +201,7 @@ function GamePage({ match }) {
     lastKeptSet,
   } = useMemo(() => {
     if (!gameData) return {};
-    const state = computeState(
-      { ...gameData, random, deck },
-      gameMode,
-      newEvents.current
-    );
+    const state = computeState({ ...gameData, random, deck }, gameMode);
     const { current, boardSize, findState, history } = state;
     const board = current.slice(0, boardSize);
     const answer = findSet(board, gameMode, findState);
@@ -256,29 +251,14 @@ function GamePage({ match }) {
     if (numHints) {
       firebase.database().ref(`gameData/${gameId}/hints`).remove();
     }
-    const eventRef = firebase
+    firebase
       .database()
       .ref(`gameData/${gameId}/events`)
       .push({
         ...event,
         user: user.id,
         time: firebase.database.ServerValue.TIMESTAMP,
       });
-    // Track "new" events that have approximate times. An event is new since
-    // the time it was created until its time is updated from the server.
-    // This happens when our callback is called the second time (the first
-    // time it is called with the approximate time).
-    const eventKey = eventRef.key;
-    const timeRef = eventRef.child("time");
-    newEvents.current.add(eventKey);
-    let updateCount = 0;
-    const timeUpdated = () => {
-      if (++updateCount === 2) {
-        newEvents.current.delete(eventKey);
-        timeRef.off("value", timeUpdated);
-      }
-    };
-    timeRef.on("value", timeUpdated);
   }
 
   const hint = game.enableHint && answer ? answer.slice(0, numHints) : null;