fix: strictly enforce exact roll to finish in Royal Game of Ur logic, preventing stuck turns

Author: rgilks

package.json

@@ -15,7 +15,7 @@
     "pretty": "prettier --write '**/*.*'",
     "deploy:cf": "npm run build && wrangler deploy",
     "deploy:worker": "cd worker && wrangler deploy",
-    "build:wasm": "cd worker/rust_ai_core && wasm-pack build --target web --out-name rgou_ai_worker",
+    "build:wasm": "cd worker/rust_ai_core && wasm-pack build --target web --out-name rgou_ai_worker --quiet",
     "build:wasm-assets": "npm run build:wasm && rm -rf src/lib/wasm && mv worker/rust_ai_core/pkg src/lib/wasm && mkdir -p public/wasm && cp src/lib/wasm/rgou_ai_worker_bg.wasm public/wasm/",
     "clean": "rm -rf .next .wrangler node_modules package-lock.json",
     "reset": "npm run clean && npm install && npm run dev",

src/lib/game-logic.ts

@@ -77,12 +77,11 @@ export function getValidMoves(gameState: GameState): number[] {
         ? -1
         : getPlayerTrack(gameState.currentPlayer).indexOf(piece.square);
     const newTrackPos = currentTrackPos + gameState.diceRoll!;
+    const trackLength = getPlayerTrack(gameState.currentPlayer).length;
 
-    if (newTrackPos >= getPlayerTrack(gameState.currentPlayer).length) {
-      if (newTrackPos === getPlayerTrack(gameState.currentPlayer).length) {
-        validMoves.push(index);
-      }
-    } else {
+    if (newTrackPos === trackLength) {
+      validMoves.push(index);
+    } else if (newTrackPos < trackLength) {
       const newActualPos = getActualPosition(
         gameState.currentPlayer,
         newTrackPos,
@@ -124,15 +123,16 @@ export function makeMove(
       ? -1
       : getPlayerTrack(gameState.currentPlayer).indexOf(piece.square);
   const newTrackPos = currentTrackPos + gameState.diceRoll;
+  const trackLength = getPlayerTrack(gameState.currentPlayer).length;
 
   if (piece.square >= 0) {
     newState.board[piece.square] = null;
   }
 
-  if (newTrackPos >= getPlayerTrack(gameState.currentPlayer).length) {
+  if (newTrackPos === trackLength) {
     currentPieces[pieceIndex] = { ...piece, square: 20 };
     moveType = "finish";
-  } else {
+  } else if (newTrackPos < trackLength) {
     const newActualPos = getActualPosition(
       gameState.currentPlayer,
       newTrackPos,
@@ -180,7 +180,7 @@ export function makeMove(
   }
 
   const landedOnRosette =
-    newTrackPos < getPlayerTrack(gameState.currentPlayer).length &&
+    newTrackPos < trackLength &&
     isRosette(getActualPosition(gameState.currentPlayer, newTrackPos));
 
   if (!landedOnRosette && newState.gameStatus !== "finished") {

worker/src/lib.rs

@@ -101,6 +101,9 @@ pub async fn main(req: Request, _env: Env, _ctx: Context) -> Result<Response> {
     console_error_panic_hook::set_once();
 
     let url = req.url()?;
+    
+    // Only log in development or when explicitly enabled
+    #[cfg(debug_assertions)]
     console_log!("[Rust Worker] {} {}", req.method(), url.path());
 
     // Handle preflight CORS requests