stop transmitting move color

Author: h908714124

README.md

@@ -1,7 +1,20 @@
-https://www.jessym.com/articles/bundling-react-vite-with-spring-boot
+This is a go / baduk server.
+
+### Hacking
+
+First, start the backend server.
 
 ```
 make run
 ```
 
-http://localhost:8080/app/
+Now, in a new terminal window or tab, start vite:
+
+```
+make dev
+```
+
+The app is running at [http://localhost:3006](http://localhost:3006/app).
+
+Try editing a file in `src/main/client/src`.
+The change should immediately be visible in the browser.

src/main/client/src/store.js

@@ -9,7 +9,7 @@ import {
 } from "zustand/middleware"
 import {
   BLACK,
-  WHITE,
+  COLORS,
 } from "./util.js"
 import {
   rehydrate,
@@ -98,7 +98,7 @@ export const useGameStore = create((set, get) => ({
     if (!moves.length) {
       return BLACK
     }
-    return moves[moves.length - 1].color ^ (BLACK | WHITE)
+    return moves[moves.length - 1].color ^ COLORS
   },
   counting: false,
   gameState: {
@@ -127,7 +127,7 @@ export const useGameStore = create((set, get) => ({
       if (!counting) {
         state.queueLength = get().queueLength + 1
       }
-      let [storedMove, updated, forbidden] = createMoveData(baseBoard, moves, move, counting)
+      let [storedMove, updated, forbidden] = createMoveData(baseBoard, moves, move, counting, get().handicap)
       state.moves.push(storedMove)
       state.lastMove = move.pass ? undefined : move
       state.baseBoard = updated
@@ -167,7 +167,7 @@ export const useGameStore = create((set, get) => ({
         } else {
           passes = 0
         }
-        let [storedMove, updated, newForbidden] = createMoveData(baseBoard, moves, move, counting)
+        let [storedMove, updated, newForbidden] = createMoveData(baseBoard, moves, move, counting, game.handicap)
         moves.push(storedMove)
         forbidden = newForbidden
         baseBoard = updated
@@ -189,7 +189,8 @@ export const useGameStore = create((set, get) => ({
   },
 }))
 
-function createMoveData(baseBoard, moves, move, counting) {
+function createMoveData(baseBoard, moves, colorlessMove, counting, handicap) {
+  let move = {...colorlessMove, color: nextMoveColor(moves, handicap)}
   if (move.pass && moves.length && moves[moves.length - 1].pass) {
     return [move, count(baseBoard), [-1, -1]]
   }
@@ -204,3 +205,14 @@ function createMoveData(baseBoard, moves, move, counting) {
   let forbidden = getForbidden(baseBoard, updated, storedMove)
   return [storedMove, updated, forbidden]
 }
+
+function nextMoveColor(moves, handicap) {
+    let remainingHandicap = Math.max(0, handicap - moves.length)
+    if (remainingHandicap) {
+      return BLACK
+    }
+    if (!moves.length) {
+      return BLACK
+    }
+    return moves[moves.length - 1].color ^ COLORS
+  }

src/main/client/src/util.js

@@ -21,7 +21,7 @@ export const TOGGLE_STUFF = TOGGLE_B | TOGGLE_W
 export const TERRITORY = TERRITORY_W | TERRITORY_B
 export const ANY_REMOVED = REMOVED_W | REMOVED_B
 
-export const COLORS = BLACK | WHITE;
+export const COLORS = BLACK | WHITE
 
 export async function tfetch(url, options) {
   let response

src/main/java/com/bernd/GameController.java

@@ -10,6 +10,7 @@
 import com.bernd.model.OpenGame;
 import com.bernd.model.ViewGame;
 import com.bernd.util.RandomString;
+import java.security.Principal;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.messaging.core.MessageSendingOperations;
@@ -22,8 +23,6 @@
 import org.springframework.web.bind.annotation.ResponseBody;
 import org.springframework.web.server.ResponseStatusException;
 
-import java.security.Principal;
-
 import static com.bernd.util.Auth.getPrincipal;
 import static com.bernd.util.Util.COLORS;
 
@@ -82,7 +81,7 @@ public void action(Move move, Principal p) {
     Game updated = game.update(updatedMove);
     games.put(updated);
     GameMove lastMove = game.getLastMove();
-    operations.convertAndSend("/topic/move/" + game.id(), lastMove);
+    operations.convertAndSend("/topic/move/" + game.id(), lastMove.removeColor());
   }
 
   private int getColorFromGameState(Game game) {

src/main/java/com/bernd/game/MoveList.java

@@ -2,9 +2,9 @@
 
 import com.bernd.model.GameMove;
 import com.bernd.model.Move;
-
 import java.util.ArrayList;
 import java.util.List;
+import java.util.stream.Stream;
 
 public final class MoveList {
 
@@ -42,6 +42,10 @@ public List<GameMove> asList() {
     return List.copyOf(moves);
   }
 
+  public Stream<GameMove> asStream() {
+    return moves.stream();
+  }
+
   @Override
   public String toString() {
     return moves.toString();

src/main/java/com/bernd/model/ColorlessMove.java

@@ -0,0 +1,11 @@
+package com.bernd.model;
+
+public record ColorlessMove(
+    int n,
+    boolean pass,
+    int x,
+    int y,
+    boolean resetCounting,
+    boolean agreeCounting,
+    boolean end) {
+}

src/main/java/com/bernd/model/GameMove.java

@@ -9,4 +9,9 @@ public record GameMove(
     boolean resetCounting,
     boolean agreeCounting,
     boolean end) {
+
+  public ColorlessMove removeColor() {
+    return new ColorlessMove(
+        n, pass, x, y, resetCounting, agreeCounting, end);
+  }
 }

src/main/java/com/bernd/model/Move.java

@@ -13,10 +13,6 @@ public GameMove toGameMove() {
     return new GameMove(n, color, pass, x, y, resetCounting, agreeCounting, false);
   }
 
-  public GameMove gameEnd() {
-    return new GameMove(n, color, pass, x, y, resetCounting, agreeCounting, true);
-  }
-
   public Move withColor(int color) {
     return new Move(color, n, pass, resetCounting, agreeCounting, x, y);
   }

src/main/java/com/bernd/model/ViewGame.java

@@ -9,7 +9,7 @@ public record ViewGame(
     int dim,
     int handicap,
     int remainingHandicap,
-    List<GameMove> moves
+    List<ColorlessMove> moves
 ) {
 
   static ViewGame fromGame(Game game) {
@@ -20,6 +20,6 @@ static ViewGame fromGame(Game game) {
         game.dim(),
         game.handicap(),
         game.remainingHandicap(),
-        game.moves().asList());
+        game.moves().asStream().map(GameMove::removeColor).toList());
   }
 }