From 804749ea53c572b66efd334f0ad52b711204bf01 Mon Sep 17 00:00:00 2001
From: Andre Padez <andre.padez@gmail.com>
Date: Thu, 30 Oct 2025 23:10:13 +0000
Subject: [PATCH] fix(phase1.4): move level-up logic OUTSIDE setScore to
 prevent multiple triggers

- Call setLevel() BEFORE setScore(), not inside the callback
- This prevents setScore from being called multiple times per food
- Level up happens once, then score is updated with correct points
- Fixes: 1st food +1pt (lvl1), 2nd food +2pts (lvl2), 3rd food +3pts (lvl3)
- No more duplicate level ups on same food
---
 src/useGameStatus/useGameStatus.ts | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/useGameStatus/useGameStatus.ts b/src/useGameStatus/useGameStatus.ts
index e728bed..13e7721 100755
--- a/src/useGameStatus/useGameStatus.ts
+++ b/src/useGameStatus/useGameStatus.ts
@@ -94,22 +94,22 @@ export function useGameStatus(
 
              console.log(`[FOOD] Eaten: ${foodsEatenRef.current}, Will Level Up: ${willLevelUp}, Current Level: ${levelRef.current}`);
 
+             // Save the current level BEFORE any changes
+             const pointsToAdd = levelRef.current;
+             
+             // Update level OUTSIDE of setScore to avoid multiple triggers
+             if (willLevelUp) {
+               const newLevel = levelRef.current + 1;
+               levelRef.current = newLevel;
+               setLevel(newLevel);
+               currentFpsRef.current = INITIAL_FPS + (newLevel - 1);
+               foodsEatenRef.current = 0;
+               console.log(`[LEVELUP] New Level: ${newLevel}`);
+             }
+
              setScore((prev) => {
-               // Save the current level BEFORE any changes
-               const pointsToAdd = levelRef.current;
-               
                // Add points at CURRENT level
                let newScore = prev + pointsToAdd;
-               
-               // Level up AFTER this food if threshold was reached
-               if (willLevelUp) {
-                 const newLevel = levelRef.current + 1;
-                 levelRef.current = newLevel;
-                 setLevel(newLevel);
-                 currentFpsRef.current = INITIAL_FPS + (newLevel - 1);
-                 foodsEatenRef.current = 0;
-                 console.log(`[LEVELUP] New Level: ${newLevel}`);
-               }
 
                console.log(`[SCORE] Adding ${pointsToAdd} points, Total: ${newScore}`);