Merge branch 'UI-Updates' into 276-refactor-the-game-to-use-bitboards-instead

.github/workflows/checks.yaml

@@ -1,42 +1,42 @@
-#name: Checks
+name: Checks
 
-#on:
-#  push:
-#    branches:
-#      - 'main'
-#  pull_request:
-#    branches:
-#      - 'main'
-#
-#jobs:
-#  formatting-check:
-#    name: Follow Google Formatting Guidelines
-#    runs-on: ubuntu-latest
-#    steps:
-#      - uses: actions/checkout@v5
-#        with:
-#          fetch-depth: 0 # Fix for incremental formatting
-#      - uses: actions/setup-java@v5
-#        with:
-#          java-version: '25'
-#          distribution: 'temurin'
-#          cache: maven
-#      - name: Run Format Check
-#        run: mvn spotless:check
+on:
+  push:
+    branches:
+      - 'main'
+  pull_request:
+    branches:
+      - 'main'
 
-#  tests:
-#    name: Unittests
-#    runs-on: ${{ matrix.os }}
-#    needs: formatting-check
-#    strategy:
-#      matrix:
-#        os: [ubuntu-latest] #windows-latest, macos-latest
-#    steps:
-#      - uses: actions/checkout@v5
-#      - uses: actions/setup-java@v5
-#        with:
-#          java-version: '25'
-#          distribution: 'temurin'
-#          cache: maven
-#      - name: Run Unittests
-#        run: mvn -B test
+jobs:
+  formatting-check:
+    name: Follow Google Formatting Guidelines
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+        with:
+          fetch-depth: 0 # Fix for incremental formatting
+      - uses: actions/setup-java@v5
+        with:
+          java-version: '25'
+          distribution: 'temurin'
+          cache: maven
+      - name: Run Format Check
+        run: mvn spotless:check
+
+  tests:
+    name: Unittests
+    runs-on: ${{ matrix.os }}
+    needs: formatting-check
+    strategy:
+      matrix:
+        os: [ubuntu-latest] #windows-latest, macos-latest
+    steps:
+      - uses: actions/checkout@v5
+      - uses: actions/setup-java@v5
+        with:
+          java-version: '25'
+          distribution: 'temurin'
+          cache: maven
+      - name: Run Unittests
+        run: mvn -B test

.idea/inspectionProfiles/Project_Default.xml

@@ -2,7 +2,7 @@
   <profile version="1.0">
     <option name="myName" value="Project Default" />
     <inspection_tool class="AutoCloseableResource" enabled="true" level="WARNING" enabled_by_default="true">
-      <option name="METHOD_MATCHER_CONFIG" value="java.util.Formatter,format,java.io.Writer,append,com.google.common.base.Preconditions,checkNotNull,org.hibernate.Session,close,java.io.PrintWriter,printf,java.io.PrintStream,printf,java.lang.foreign.Arena,ofAuto,java.lang.foreign.Arena,global,org.toop.framework.audio.AudioPlayer,play,java.util.Map,remove,java.util.concurrent.Executors,newSingleThreadScheduledExecutor|newFixedThreadPool|newSingleThreadExecutor" />
+      <option name="METHOD_MATCHER_CONFIG" value="java.util.Formatter,format,java.io.Writer,append,com.google.common.base.Preconditions,checkNotNull,org.hibernate.Session,close,java.io.PrintWriter,printf,java.io.PrintStream,printf,java.lang.foreign.Arena,ofAuto,java.lang.foreign.Arena,global,org.toop.framework.audio.AudioPlayer,play,java.util.Map,remove,java.util.concurrent.Executors,newSingleThreadScheduledExecutor" />
     </inspection_tool>
     <inspection_tool class="WriteOnlyObject" enabled="false" level="WARNING" enabled_by_default="false" />
   </profile>

app/src/main/java/org/toop/Main.java

@@ -1,68 +1,16 @@
 package org.toop;
 
 import org.toop.app.App;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.players.ArtificialPlayer;
-import org.toop.game.players.ai.MCTSAI;
-import org.toop.game.players.ai.RandomAI;
-import org.toop.game.players.ai.mcts.MCTSAI1;
-import org.toop.game.players.ai.mcts.MCTSAI2;
-import org.toop.game.players.ai.mcts.MCTSAI3;
-import org.toop.game.players.ai.mcts.MCTSAI4;
-
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
+import org.toop.framework.audio.*;
+import org.toop.framework.networking.NetworkingClientEventListener;
+import org.toop.framework.networking.NetworkingClientManager;
+import org.toop.framework.resource.ResourceLoader;
+import org.toop.framework.resource.ResourceManager;
+import org.toop.framework.resource.resources.MusicAsset;
+import org.toop.framework.resource.resources.SoundEffectAsset;
 
 public final class Main {
     static void main(String[] args) {
-		App.run(args);
-
-		// final ExecutorService executor = Executors.newFixedThreadPool(1);
-		// executor.execute(() -> testAIs(25));
+        App.run(args);
     }
-
-	private static void testAIs(int games) {
-		var versions = new ArtificialPlayer[5];
-		versions[0] = new ArtificialPlayer(new RandomAI(), "Random AI");
-		versions[1] = new ArtificialPlayer(new MCTSAI1(1000), "MCTS V1 AI");
-		versions[2] = new ArtificialPlayer(new MCTSAI2(1000), "MCTS V2 AI");
-		versions[3] = new ArtificialPlayer(new MCTSAI3(10, 10), "MCTS V3 AI");
-		versions[4] = new ArtificialPlayer(new MCTSAI4(10, 10), "MCTS V4 AI");
-
-		for (int i = 0; i < versions.length; i++) {
-			for (int j = i + 1; j < versions.length; j++) {
-				final int playerIndex1 = i % versions.length;
-				final int playerIndex2 = j % versions.length;
-
-				testAIVSAI(games, new ArtificialPlayer[] { versions[playerIndex1], versions[playerIndex2]});
-			}
-		}
-	}
-
-	private static void testAIVSAI(int games, ArtificialPlayer[] ais) {
-		int wins = 0;
-		int ties = 0;
-
-		for (int i = 0; i < games; i++) {
-			final BitboardReversi match = new BitboardReversi();
-			match.init(ais);
-
-			while (!match.isTerminal()) {
-				final int currentAI = match.getCurrentTurn();
-				final long move = ais[currentAI].getMove(match);
-
-				match.play(move);
-			}
-
-			if (match.getWinner() < 0) {
-				ties++;
-				continue;
-			}
-
-			wins += match.getWinner() == 0? 1 : 0;
-		}
-
-		System.out.printf("Out of %d games, %s won %d -- tied %d -- lost %d, games against %s\n", games, ais[0].getName(), wins, ties, games - wins - ties, ais[1].getName());
-		System.out.printf("Average win rate was: %.2f\n\n", wins / (float)games);
-	}
 }

app/src/main/java/org/toop/app/App.java

@@ -1,14 +1,9 @@
 package org.toop.app;
 
-import javafx.application.Application;
 import javafx.application.Platform;
-import javafx.geometry.Pos;
-import javafx.scene.Scene;
 import javafx.scene.input.KeyCode;
-import javafx.scene.input.KeyCodeCombination;
 import javafx.scene.input.KeyEvent;
-import javafx.scene.layout.StackPane;
-import javafx.stage.Stage;
+
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.WidgetContainer;
 import org.toop.app.widget.complex.LoadingWidget;
@@ -19,9 +14,8 @@ import org.toop.app.widget.view.MainView;
 import org.toop.framework.audio.*;
 import org.toop.framework.audio.events.AudioEvents;
 import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.eventbus.GlobalEventBus;
-import org.toop.framework.networking.connection.NetworkingClientEventListener;
-import org.toop.framework.networking.connection.NetworkingClientManager;
+import org.toop.framework.networking.NetworkingClientEventListener;
+import org.toop.framework.networking.NetworkingClientManager;
 import org.toop.framework.resource.ResourceLoader;
 import org.toop.framework.resource.ResourceManager;
 import org.toop.framework.resource.events.AssetLoaderEvents;
@@ -31,10 +25,13 @@ import org.toop.framework.resource.resources.SoundEffectAsset;
 import org.toop.local.AppContext;
 import org.toop.local.AppSettings;
 
+import javafx.application.Application;
+import javafx.geometry.Pos;
+import javafx.scene.Scene;
+import javafx.scene.layout.StackPane;
+import javafx.stage.Stage;
+
 import java.util.Objects;
-import java.util.concurrent.CountDownLatch;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
 
 public final class App extends Application {
 	private static Stage stage;
@@ -48,7 +45,7 @@ public final class App extends Application {
 	}
 
 	@Override
-	public void start(Stage stage) {
+	public void start(Stage stage) throws Exception {
 		// Start loading localization
 		ResourceManager.loadAssets(new ResourceLoader("app/src/main/resources/localization"));
 		ResourceManager.loadAssets(new ResourceLoader("app/src/main/resources/style"));
@@ -66,8 +63,8 @@ public final class App extends Application {
 
 		scene.getRoot();
 
-        stage.setMinWidth(1200);
-        stage.setMinHeight(800);
+        stage.setMinWidth(1080);
+        stage.setMinHeight(720);
 		stage.setOnCloseRequest(event -> {
 			event.consume();
 			quit();
@@ -84,7 +81,7 @@ public final class App extends Application {
 
 		AppSettings.applySettings();
 
-        setKeybinds(root);
+		setKeybinds(root);
 
         LoadingWidget loading = new LoadingWidget(Primitive.text(
                 "Loading...", false), 0, 0, Integer.MAX_VALUE, false, false // Just set a high default
@@ -92,7 +89,7 @@ public final class App extends Application {
 
 		WidgetContainer.setCurrentView(loading);
 
-        setOnLoadingSuccess(loading);
+		setOnLoadingSuccess(loading);
 
         EventFlow loadingFlow = new EventFlow();
 
@@ -109,33 +106,30 @@ public final class App extends Application {
 						Platform.runLater(() -> stage.setOpacity(1.0));
 					}
 
+					Platform.runLater(() -> loading.setMaxAmount(e.isLoadingAmount()));
+
 					Platform.runLater(() -> {
-                        loading.setMaxAmount(e.isLoadingAmount());
 						try {
 							loading.setAmount(e.hasLoadedAmount());
 						} catch (Exception ex) {
 							throw new RuntimeException(ex);
 						}
-                        if (e.hasLoadedAmount() >= e.isLoadingAmount()-1) {
-                            Platform.runLater(loading::triggerSuccess);
-                            loadingFlow.unsubscribe("init_loading");
-                        }
 					});
 
+                    if (e.hasLoadedAmount() >= e.isLoadingAmount()) {
+                        Platform.runLater(loading::triggerSuccess);
+                        loadingFlow.unsubscribe("init_loading");
+                    }
 
                 }, false, "init_loading");
 
-		ExecutorService executor = Executors.newSingleThreadExecutor();
-		try {
-			executor.submit(
-				() -> ResourceManager.loadAssets(new ResourceLoader("app/src/main/resources/assets")
-			));
-		} finally {
-			executor.shutdown();
-		}
+		// Start loading assets
+		new Thread(() -> ResourceManager.loadAssets(new ResourceLoader("app/src/main/resources/assets")))
+				.start();
 
 		stage.show();
-    }
+
+	}
 
 	private void setKeybinds(StackPane root) {
 		root.addEventHandler(KeyEvent.KEY_PRESSED,event -> {
@@ -143,20 +137,9 @@ public final class App extends Application {
 				escapePopup();
 			}
 		});
-        stage.setFullScreenExitKeyCombination(
-                new KeyCodeCombination(
-                        KeyCode.F11
-                )
-        );
 	}
 
 	public void escapePopup() {
-
-		if (   WidgetContainer.getCurrentView() == null
-			|| WidgetContainer.getCurrentView() instanceof MainView) {
-			return;
-		}
-
 		if (!Objects.requireNonNull(
 				WidgetContainer.find(widget -> widget instanceof QuitPopup || widget instanceof EscapePopup)
 		).isEmpty()) {
@@ -171,14 +154,8 @@ public final class App extends Application {
 
 	private void setOnLoadingSuccess(LoadingWidget loading) {
 		loading.setOnSuccess(() -> {
-
-            try {
-                initSystems();
-            } catch (InterruptedException e) {
-                throw new RuntimeException(e);
-            }
-
-            AppSettings.applyMusicVolumeSettings();
+			initSystems();
+			AppSettings.applyMusicVolumeSettings();
 			new EventFlow().addPostEvent(new AudioEvents.StartBackgroundMusic()).postEvent();
             loading.hide();
 			WidgetContainer.add(Pos.CENTER, new MainView());
@@ -195,55 +172,36 @@ public final class App extends Application {
 		});
 	}
 
-	private void initSystems() throws InterruptedException { // TODO Move to better place
+	private void initSystems() { // TODO Move to better place
+		new Thread(() -> new NetworkingClientEventListener(new NetworkingClientManager())).start();
 
-		final int THREAD_COUNT = 2;
-		CountDownLatch latch = new CountDownLatch(THREAD_COUNT);
+		new Thread(() -> {
+			MusicManager<MusicAsset> musicManager =
+					new MusicManager<>(ResourceManager.getAllOfTypeAndRemoveWrapper(MusicAsset.class), true);
 
-		@SuppressWarnings("resource")
-		ExecutorService threads = Executors.newFixedThreadPool(THREAD_COUNT);
+			SoundEffectManager<SoundEffectAsset> soundEffectManager =
+					new SoundEffectManager<>(ResourceManager.getAllOfType(SoundEffectAsset.class));
 
-		try {
+			AudioVolumeManager audioVolumeManager = new AudioVolumeManager()
+					.registerManager(VolumeControl.MASTERVOLUME, musicManager)
+					.registerManager(VolumeControl.MASTERVOLUME, soundEffectManager)
+					.registerManager(VolumeControl.FX, soundEffectManager)
+					.registerManager(VolumeControl.MUSIC, musicManager);
 
-			threads.submit(() -> {
-				new NetworkingClientEventListener(
-						GlobalEventBus.get(),
-						new NetworkingClientManager(GlobalEventBus.get()));
+			new AudioEventListener<>(
+					musicManager,
+					soundEffectManager,
+					audioVolumeManager
+			).initListeners("medium-button-click.wav");
 
-				latch.countDown();
-			});
+		}).start();
 
-			threads.submit(() -> {
-				MusicManager<MusicAsset> musicManager =
-						new MusicManager<>(
-								GlobalEventBus.get(),
-								ResourceManager.getAllOfTypeAndRemoveWrapper(MusicAsset.class),
-								true
-						);
-
-				SoundEffectManager<SoundEffectAsset> soundEffectManager =
-						new SoundEffectManager<>(ResourceManager.getAllOfType(SoundEffectAsset.class));
-
-				AudioVolumeManager audioVolumeManager = new AudioVolumeManager()
-						.registerManager(VolumeControl.MASTERVOLUME, musicManager)
-						.registerManager(VolumeControl.MASTERVOLUME, soundEffectManager)
-						.registerManager(VolumeControl.FX, soundEffectManager)
-						.registerManager(VolumeControl.MUSIC, musicManager);
-
-				new AudioEventListener<>(
-						GlobalEventBus.get(),
-						musicManager,
-						soundEffectManager,
-						audioVolumeManager
-				).initListeners("medium-button-click.wav");
-
-				latch.countDown();
-			});
-
-		} finally {
-			latch.await();
-			threads.shutdown();
-		}
+		// Threads must be ready, before continue, TODO use latch instead
+        try {
+            Thread.sleep(200);
+        } catch (InterruptedException e) {
+            throw new RuntimeException(e);
+        }
     }
 
 	public static void quit() {

app/src/main/java/org/toop/app/Server.java

@@ -11,19 +11,18 @@ import org.toop.app.widget.popup.ErrorPopup;
 import org.toop.app.widget.popup.SendChallengePopup;
 import org.toop.app.widget.view.ServerView;
 import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.game.players.ArtificialPlayer;
-import org.toop.framework.game.players.OnlinePlayer;
 import org.toop.framework.gameFramework.controller.GameController;
-import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.networking.connection.clients.TournamentNetworkingClient;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
-import org.toop.framework.networking.server.gateway.NettyGatewayServer;
-import org.toop.game.players.ai.mcts.MCTSAI3;
+import org.toop.framework.networking.clients.TournamentNetworkingClient;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.framework.networking.types.NetworkingConnector;
+import org.toop.game.games.reversi.BitboardReversi;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
+import org.toop.game.players.ArtificialPlayer;
+import org.toop.game.players.OnlinePlayer;
+import org.toop.game.players.RandomAI;
 import org.toop.local.AppContext;
 
-import java.util.Arrays;
 import java.util.List;
 import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.concurrent.Executors;
@@ -32,8 +31,6 @@ import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 
 public final class Server {
-	private NettyGatewayServer nettyGatewayServer;
-
 	private String user = "";
 	private long clientId = -1;
 
@@ -61,13 +58,10 @@ public final class Server {
 		return null;
 	}
 
-	public Server(String ip, String port, String user) {
-		this(ip, port, user, null);
-	}
 
     // Server has to deal with ALL network related listen events. This "server" can then interact with the manager to make stuff happen.
     // This prevents data races where events get sent to the game manager but the manager isn't ready yet.
-	public Server(String ip, String port, String user, NettyGatewayServer nettyGatewayServer) {
+	public Server(String ip, String port, String user) {
 		if (ip.split("\\.").length < 4) {
 			new ErrorPopup("\"" + ip + "\" " + AppContext.getString("is-not-a-valid-ip-address"));
 			return;
@@ -87,19 +81,17 @@ public final class Server {
 			return;
 		}
 
-		this.nettyGatewayServer = nettyGatewayServer;
-
 		final int reconnectAttempts = 10;
 
 		LoadingWidget loading = new LoadingWidget(
                 Primitive.text("connecting"), 0, 0, reconnectAttempts, true, true
         );
 
-		WidgetContainer.getCurrentView().transitionNextCustom(loading, "disconnect", this::disconnect);
+		WidgetContainer.getCurrentView().transitionNext(loading);
 
 		var a = new EventFlow()
 			.addPostEvent(NetworkEvents.StartClient.class,
-				new TournamentNetworkingClient(GlobalEventBus.get()),
+				new TournamentNetworkingClient(),
 				new NetworkingConnector(ip, parsedPort, reconnectAttempts, 1, TimeUnit.SECONDS)
 			);
 
@@ -112,21 +104,20 @@ public final class Server {
             );
         });
 
-		a.onResponse(NetworkEvents.CreatedIdForClient.class, e -> clientId = e.clientId(), true);
-
 		a.onResponse(NetworkEvents.StartClientResponse.class, e -> {
+
 			if (!e.successful()) {
 				return;
 			}
 
-			primary = new ServerView(user, this::sendChallenge, user, clientId);
-
+			primary = new ServerView(user, this::sendChallenge);
 			WidgetContainer.getCurrentView().transitionNextCustom(primary, "disconnect", this::disconnect);
 
 			a.unsubscribe("connecting");
 			a.unsubscribe("startclient");
 
 			this.user = user;
+			clientId = e.clientId();
 
 			new EventFlow().addPostEvent(new NetworkEvents.SendLogin(clientId, user)).postEvent();
 
@@ -137,33 +128,28 @@ public final class Server {
 
 		}, false, "startclient")
 				.listen(
-						NetworkEvents.ConnectTry.class,
-						e -> {
-							if (clientId != e.clientId()) return;
-							Platform.runLater(
-									() -> {
-										try {
-											loading.setAmount(e.amount());
-											if (e.amount() >= loading.getMaxAmount()) {
-												loading.triggerFailure();
-											}
-										} catch (Exception ex) {
-											throw new RuntimeException(ex);
-										}
-									}
-							);
-						},
-						false, "connecting"
-				)
+                        NetworkEvents.ConnectTry.class,
+                        e -> Platform.runLater(
+                                () -> {
+                                    try {
+                                        loading.setAmount(e.amount());
+                                        if (e.amount() >= loading.getMaxAmount()) {
+                                            loading.triggerFailure();
+                                        }
+                                    } catch (Exception ex) {
+                                        throw new RuntimeException(ex);
+                                    }
+                                }
+                        ),
+                        false, "connecting"
+                )
 				.postEvent();
 
 		a.listen(NetworkEvents.ChallengeResponse.class, this::handleReceivedChallenge, false, "challenge")
                 .listen(NetworkEvents.GameMatchResponse.class, this::handleMatchResponse, false, "match-response")
                 .listen(NetworkEvents.GameResultResponse.class, this::handleGameResult, false, "game-result")
                 .listen(NetworkEvents.GameMoveResponse.class, this::handleReceivedMove, false, "game-move")
-                .listen(NetworkEvents.YourTurnResponse.class, this::handleYourTurn, false, "your-turn")
-				.listen(NetworkEvents.ClosedConnection.class, this::closedConnection, false, "closed-connection")
-                .listen(NetworkEvents.TournamentResultResponse.class, this::handleTournamentResult, false, "tournament-result");
+                .listen(NetworkEvents.YourTurnResponse.class, this::handleYourTurn, false, "your-turn");
 
 		connectFlow = a;
 	}
@@ -187,7 +173,7 @@ public final class Server {
 
         gameController = null;
 
-//        if (!isPolling) return;
+        //if (!isPolling) return;
 
         String gameType = extractQuotedValue(response.gameType());
         if (response.clientId() == clientId) {
@@ -200,30 +186,45 @@ public final class Server {
                 return;
             }
 
-            final String startingPlayer = response.playerToMove();
-            final int userStartingTurn = startingPlayer.equalsIgnoreCase(user) ? 0 : 1;
-            final int opponentStartingTurn = 1 - userStartingTurn;
+            final int myTurn = response.playerToMove().equalsIgnoreCase(response.opponent()) ? 1 : 0;
 
             final GameInformation information = new GameInformation(type);
-            information.players[userStartingTurn].name = user;
-            information.players[opponentStartingTurn].name = response.opponent();
+            //information.players[0] = playerInformation;
+            information.players[0].name = user;
+            information.players[0].isHuman = false;
+            information.players[0].computerDifficulty = 5;
+            information.players[0].computerThinkTime = 1;
+            information.players[1].name = response.opponent();
+
+            /*switch (type){
+                case TICTACTOE ->{
+                    players[myTurn] = new ArtificialPlayer<>(new TicTacToeAIR(9), user);
+                }
+                case REVERSI ->{
+                    players[myTurn] = new ArtificialPlayer<>(new ReversiAIR(), user);
+                }
+            }*/
 
-            Player[] players = new Player[2];
 
-            players[userStartingTurn] = new ArtificialPlayer(new MCTSAI3(1000, Runtime.getRuntime().availableProcessors()), user);
-            players[opponentStartingTurn] = new OnlinePlayer(response.opponent());
 
             switch (type) {
-                case TICTACTOE -> gameController = new TicTacToeBitController(players);
-                case REVERSI -> gameController = new ReversiBitController(players);
+                case TICTACTOE ->{
+                        Player<BitboardTicTacToe>[] players = new Player[2];
+                        players[(myTurn + 1) % 2] = new OnlinePlayer<>(response.opponent());
+                        players[myTurn] = new ArtificialPlayer<>(new RandomAI<BitboardTicTacToe>(), user);
+                        gameController = new TicTacToeBitController(players);
+                }
+                case REVERSI -> {
+                    Player<BitboardReversi>[] players = new Player[2];
+                    players[(myTurn + 1) % 2] = new OnlinePlayer<>(response.opponent());
+                    players[myTurn] = new ArtificialPlayer<>(new RandomAI<BitboardReversi>(), user);
+                    gameController = new ReversiBitController(players);}
                 default -> new ErrorPopup("Unsupported game type.");
 
             }
 
-            if (gameController != null) {
-				primary.reEnableButton(); // Re enable subscribe button
+            if (gameController != null){
                 gameController.start();
-				isPolling = true; // Fixes server getting stuck
             }
         }
     }
@@ -243,13 +244,6 @@ public final class Server {
         gameController.gameFinished(response);
     }
 
-    private void handleTournamentResult(NetworkEvents.TournamentResultResponse response) {
-        IO.println(response.gameType());
-        IO.println(Arrays.toString(response.names()));
-		IO.println(Arrays.toString(response.scoreTypes()));
-        IO.println(Arrays.toString(response.scores().toArray()));
-    }
-
     private void handleReceivedMove(NetworkEvents.GameMoveResponse response) {
         if (gameController == null) {
             return;
@@ -264,7 +258,7 @@ public final class Server {
 		String gameType = extractQuotedValue(response.gameType());
 		final String finalGameType = gameType;
 		var a = new ChallengePopup(challengerName, gameType, (playerInformation) -> {
-			final long challengeId = Long.parseLong(response.challengeId().replaceAll("\\D", ""));
+			final int challengeId = Integer.parseInt(response.challengeId().replaceAll("\\D", ""));
 			new EventFlow().addPostEvent(new NetworkEvents.SendAcceptChallenge(clientId, challengeId)).postEvent();
             isSingleGame.set(true);
 		});
@@ -282,27 +276,9 @@ public final class Server {
 		stopScheduler();
 		connectFlow.unsubscribeAll();
 
-		if (nettyGatewayServer != null) {
-			nettyGatewayServer.stop();
-		}
-
 		WidgetContainer.getCurrentView().transitionPrevious();
 	}
 
-	private void closedConnection(NetworkEvents.ClosedConnection e) {
-		new EventFlow().addPostEvent(new NetworkEvents.CloseClient(clientId)).postEvent();
-		isPolling = false;
-		stopScheduler();
-		connectFlow.unsubscribeAll();
-
-		if (nettyGatewayServer != null) {
-			nettyGatewayServer.stop();
-		}
-
-		WidgetContainer.getCurrentView().transitionPrevious();
-		WidgetContainer.add(Pos.CENTER, new ErrorPopup("Server closed connection."));
-	}
-
 	private void forfeitGame() {
 		new EventFlow().addPostEvent(new NetworkEvents.SendForfeit(clientId)).postEvent();
 	}
@@ -349,10 +325,7 @@ public final class Server {
 
 	private void gamesListFromServerHandler(NetworkEvents.GamelistResponse event) {
 		gameList.clear();
-		var gl = new java.util.ArrayList<>(List.of(event.gamelist()));
-        gl.sort(String::compareTo);
-		gameList.addAll(gl);
-		primary.updateGameList(gl);
+		gameList.addAll(List.of(event.gamelist()));
 	}
 
 	public void populateGameList() {

app/src/main/java/org/toop/app/canvas/BitGameCanvas.java

@@ -14,7 +14,7 @@ import org.toop.framework.gameFramework.view.GUIEvents;
 
 import java.util.function.Consumer;
 
-public abstract class BitGameCanvas implements GameCanvas {
+public abstract class BitGameCanvas<T extends TurnBasedGame<T>> implements GameCanvas<T> {
 	protected record Cell(float x, float y, float width, float height) {
 		public boolean isInside(double x, double y) {
 			return x >= this.x && x <= this.x + width &&
@@ -78,7 +78,6 @@ public abstract class BitGameCanvas implements GameCanvas {
 		}
 
 		canvas.setOnMouseClicked(event -> {
-
 			if (event.getButton() != MouseButton.PRIMARY) {
 				return;
 			}
@@ -94,6 +93,9 @@ public abstract class BitGameCanvas implements GameCanvas {
 			}
 		});
 
+
+
+
 		render();
 	}
 

app/src/main/java/org/toop/app/canvas/GameCanvas.java

@@ -3,7 +3,6 @@ package org.toop.app.canvas;
 import javafx.scene.canvas.Canvas;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface GameCanvas {
+public interface GameCanvas<T extends TurnBasedGame<T>> extends GameDrawer<T>{
     Canvas getCanvas();
-    void redraw(TurnBasedGame gameCopy);
 }

app/src/main/java/org/toop/app/canvas/GameDrawer.java

@@ -0,0 +1,7 @@
+package org.toop.app.canvas;
+
+import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+
+public interface GameDrawer<T extends TurnBasedGame<T>> {
+    void redraw(T gameCopy);
+}

app/src/main/java/org/toop/app/canvas/ReversiBitCanvas.java

@@ -2,13 +2,12 @@ package org.toop.app.canvas;
 
 import javafx.scene.paint.Color;
 import org.toop.app.App;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.players.LocalPlayer;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.game.games.reversi.BitboardReversi;
 
-public class ReversiBitCanvas extends BitGameCanvas {
-    private TurnBasedGame gameCopy;
-    private int previousCell;
+import java.util.Arrays;
+import java.util.function.Consumer;
+
+public class ReversiBitCanvas extends BitGameCanvas<BitboardReversi> {
     public ReversiBitCanvas() {
         super(Color.GRAY, new Color(0f, 0.4f, 0.2f, 1f), (App.getHeight() / 4) * 3, (App.getHeight() / 4) * 3, 8, 8, 5, true);
         canvas.setOnMouseMoved(event -> {
@@ -24,9 +23,6 @@ public class ReversiBitCanvas extends BitGameCanvas {
                     break;
                 }
             }
-            if (hovered != null) {
-                checkHoverDots(hovered, cellId);
-            }
         });
     }
 
@@ -37,32 +33,10 @@ public class ReversiBitCanvas extends BitGameCanvas {
     }
 
     @Override
-    public void redraw(TurnBasedGame gameCopy) {
-        this.gameCopy = gameCopy;
+    public void redraw(BitboardReversi gameCopy) {
         clearAll();
         long[] board = gameCopy.getBoard();
         loopOverBoard(board[0], (i) -> drawDot(Color.WHITE, i));
         loopOverBoard(board[1], (i) -> drawDot(Color.BLACK, i));
     }
-
-    public void drawLegalDots(TurnBasedGame gameCopy){
-        long legal = gameCopy.getLegalMoves();
-        loopOverBoard(legal, (i) -> drawInnerDot(gameCopy.getCurrentTurn()==0?new Color(1f,1f,1f,0.65f) :new Color(0f,0f,0f,0.65f), i,false));
-    }
-
-    private void checkHoverDots(BitGameCanvas.Cell hovered, int cellId){
-        if (previousCell == cellId){
-            return;
-        }
-        long backflips = ((BitboardReversi)gameCopy).getFlips(1L << previousCell);
-        loopOverBoard(backflips, (i) -> drawInnerDot(gameCopy.getCurrentTurn()==1?Color.WHITE:Color.BLACK, i,true));
-        previousCell = cellId;
-        if (gameCopy.getPlayer(gameCopy.getCurrentTurn()) instanceof LocalPlayer) {
-            long legal = gameCopy.getLegalMoves();
-            if ((legal & (1L << cellId)) != 0) {
-                long flips = ((BitboardReversi) gameCopy).getFlips(1L << cellId);
-                loopOverBoard(flips, (i) -> drawInnerDot(gameCopy.getCurrentTurn() == 0 ? Color.WHITE : Color.BLACK, i, false));
-            }
-        }
-    }
 }

app/src/main/java/org/toop/app/canvas/TicTacToeBitCanvas.java

@@ -2,9 +2,12 @@ package org.toop.app.canvas;
 
 import javafx.scene.paint.Color;
 import org.toop.app.App;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
 
-public class TicTacToeBitCanvas extends BitGameCanvas{
+import java.util.Arrays;
+import java.util.function.Consumer;
+
+public class TicTacToeBitCanvas extends BitGameCanvas<BitboardTicTacToe>{
     public TicTacToeBitCanvas() {
         super(
                 Color.GRAY,
@@ -19,7 +22,7 @@ public class TicTacToeBitCanvas extends BitGameCanvas{
     }
 
     @Override
-    public void redraw(TurnBasedGame gameCopy) {
+    public void redraw(BitboardTicTacToe gameCopy) {
         clearAll();
         drawMoves(gameCopy.getBoard());
     }

app/src/main/java/org/toop/app/gameControllers/GenericGameController.java

@@ -5,21 +5,24 @@ import javafx.geometry.Pos;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.app.canvas.GameCanvas;
-import org.toop.app.canvas.ReversiBitCanvas;
 import org.toop.app.widget.WidgetContainer;
 import org.toop.app.widget.view.GameView;
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.gameFramework.controller.GameController;
-import org.toop.framework.gameFramework.model.game.threadBehaviour.SupportsOnlinePlay;
+import org.toop.framework.gameFramework.model.game.SupportsOnlinePlay;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.ThreadBehaviour;
 import org.toop.framework.gameFramework.model.player.Player;
 import org.toop.framework.gameFramework.view.GUIEvents;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.game.players.LocalPlayer;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.game.players.LocalPlayer;
 
-public class GenericGameController implements GameController {
+import java.util.ArrayList;
+import java.util.List;
+import java.util.function.Consumer;
+
+public class GenericGameController<T extends TurnBasedGame<T>> implements GameController {
     protected final EventFlow eventFlow = new EventFlow();
 
     // Logger for logging
@@ -29,23 +32,21 @@ public class GenericGameController implements GameController {
     protected final GameView gameView;
 
     // Reference to game canvas
-    protected final GameCanvas canvas;
+    protected final GameCanvas<T> canvas;
 
-    protected final TurnBasedGame game;       // Reference to game instance
+    protected final TurnBasedGame<T> game;       // Reference to game instance
     private final ThreadBehaviour gameThreadBehaviour;
 
     // TODO: Change gameType to automatically happen with either dependency injection or something else.
-    public GenericGameController(GameCanvas canvas, TurnBasedGame game, ThreadBehaviour gameThreadBehaviour, String gameType) {
-        logger.info("Creating: {}", this.getClass());
+    public GenericGameController(GameCanvas<T> canvas, T game, ThreadBehaviour gameThreadBehaviour, String gameType) {
+        logger.info("Creating: " + this.getClass());
 
         this.canvas = canvas;
         this.game = game;
         this.gameThreadBehaviour = gameThreadBehaviour;
 
         // Tell thread how to send moves
-        this.gameThreadBehaviour.setOnSendMove(
-                (id, m) -> GlobalEventBus.get().post(new NetworkEvents.SendMove(id, (short)translateMove(m)))
-        );
+        this.gameThreadBehaviour.setOnSendMove((id, m) -> GlobalEventBus.postAsync(new NetworkEvents.SendMove(id, (short)translateMove(m))));
 
         // Tell thread how to update UI
         this.gameThreadBehaviour.setOnUpdateUI(() -> Platform.runLater(this::updateUI));
@@ -56,42 +57,26 @@ public class GenericGameController implements GameController {
         WidgetContainer.getCurrentView().transitionNext(gameView, true);
 
         // Listen to updates
-        logger.info("Game controller started listening");
         eventFlow
                 .listen(GUIEvents.GameEnded.class, this::onGameFinish, false)
-                .listen(GUIEvents.PlayerAttemptedMove.class, event -> {
-                    logger.info("User attempting move {}", event.move());
-                    logger.info("Current player's turn {}", getCurrentPlayer().getName());
-                    logger.info("First player {}", game.getPlayer(0).getName());
-                    logger.info("Username {}", getCurrentPlayer().getName());
-                    logger.info("User is class {}, {}", getCurrentPlayer().getClass(), getCurrentPlayer() instanceof LocalPlayer);
-                    if (getCurrentPlayer() instanceof LocalPlayer lp) {
-                        try {
-                            lp.setLastMove(event.move());
-                        } catch (Exception e) {
-                            IO.println(e);
-                        }
-                    }
-                }, false);
+                .listen(GUIEvents.PlayerAttemptedMove.class, event -> {if (getCurrentPlayer() instanceof LocalPlayer<T> lp){lp.setMove(event.move());}}, false);
     }
 
     public void start(){
         logger.info("Starting GameManager");
         updateUI();
         gameThreadBehaviour.start();
-        logger.info("GameManager started");
     }
 
     public void stop(){
         logger.info("Stopping GameManager");
         removeListeners();
         gameThreadBehaviour.stop();
-        logger.info("GameManager stopped");
     }
 
-    public Player getCurrentPlayer(){
+    public Player<T> getCurrentPlayer(){
         return game.getPlayer(getCurrentPlayerIndex());
-    }
+    };
 
     public int getCurrentPlayerIndex(){
         return game.getCurrentTurn();
@@ -110,14 +95,14 @@ public class GenericGameController implements GameController {
     }
 
     private void onGameFinish(GUIEvents.GameEnded event){
-        logger.info("OnlineTurnBasedGame Finished");
+        logger.info("Game Finished");
         String name = event.winner() == -1 ? null : getPlayer(event.winner()).getName();
         gameView.gameOver(event.winOrTie(), name);
         stop();
     }
 
-    public Player getPlayer(int player){
-        if (player < 0 || player > game.getPlayerCount()-1){ // TODO: Make game turn player count
+    public Player<T> getPlayer(int player){
+        if (player < 0 || player >= 2){ // TODO: Make game turn player count
             logger.error("Invalid player index");
             throw new IllegalArgumentException("player out of range");
         }
@@ -154,12 +139,6 @@ public class GenericGameController implements GameController {
 
     @Override
     public void updateUI() {
-        TurnBasedGame gameCopy = game.deepCopy();
-        canvas.redraw(gameCopy);
-        String gameType = game.getClass().getSimpleName().replace("Bitboard","");
-        gameView.nextPlayer(true, getCurrentPlayer().getName(), game.getPlayer(1-getCurrentPlayerIndex()).getName(),gameType);
-        if (getCurrentPlayer() instanceof LocalPlayer && gameType.equals("Reversi")){
-            ((ReversiBitCanvas)canvas).drawLegalDots(gameCopy);
-        }
+        canvas.redraw(game.deepCopy());
     }
 }

app/src/main/java/org/toop/app/gameControllers/ReversiBitController.java

@@ -3,21 +3,20 @@ package org.toop.app.gameControllers;
 import org.toop.app.canvas.ReversiBitCanvas;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.ThreadBehaviour;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.gameThreads.LocalThreadBehaviour;
-import org.toop.framework.game.gameThreads.OnlineThreadBehaviour;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.players.OnlinePlayer;
-
-import java.util.Arrays;
-
-public class ReversiBitController extends GenericGameController {
-    public ReversiBitController(Player[] players) {
-        BitboardReversi game = new BitboardReversi();
-        game.init(players);
-
-        ThreadBehaviour thread = Arrays.stream(players).anyMatch(e -> e instanceof OnlinePlayer) ?
-                new OnlineThreadBehaviour(game) : new LocalThreadBehaviour(game);
+import org.toop.game.gameThreads.LocalThreadBehaviour;
+import org.toop.game.gameThreads.OnlineThreadBehaviour;
+import org.toop.game.games.reversi.BitboardReversi;
+import org.toop.game.players.OnlinePlayer;
 
+public class ReversiBitController extends GenericGameController<BitboardReversi> {
+    public ReversiBitController(Player<BitboardReversi>[] players) {
+        BitboardReversi game = new BitboardReversi(players);
+        ThreadBehaviour thread = new LocalThreadBehaviour<>(game);
+        for (Player<BitboardReversi> player : players) {
+            if (player instanceof OnlinePlayer<BitboardReversi>){
+                thread = new OnlineThreadBehaviour<>(game);
+            }
+        }
         super(new ReversiBitCanvas(), game, thread, "Reversi");
     }
 }

app/src/main/java/org/toop/app/gameControllers/TicTacToeBitController.java

@@ -3,21 +3,21 @@ package org.toop.app.gameControllers;
 import org.toop.app.canvas.TicTacToeBitCanvas;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.ThreadBehaviour;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.gameThreads.LocalThreadBehaviour;
-import org.toop.framework.game.gameThreads.OnlineThreadBehaviour;
-import org.toop.framework.game.games.tictactoe.BitboardTicTacToe;
-import org.toop.framework.game.players.OnlinePlayer;
+import org.toop.game.gameThreads.LocalFixedRateThreadBehaviour;
+import org.toop.game.gameThreads.LocalThreadBehaviour;
+import org.toop.game.gameThreads.OnlineThreadBehaviour;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
+import org.toop.game.players.OnlinePlayer;
 
-import java.util.Arrays;
-
-public class TicTacToeBitController extends GenericGameController {
-    public TicTacToeBitController(Player[] players) {
-        BitboardTicTacToe game = new BitboardTicTacToe();
-        game.init(players);
-
-        ThreadBehaviour thread = Arrays.stream(players).anyMatch(e -> e instanceof OnlinePlayer) ?
-                new OnlineThreadBehaviour(game) : new LocalThreadBehaviour(game);
-
-        super(new TicTacToeBitCanvas(), game, thread, "TicTacToe");
+public class TicTacToeBitController extends GenericGameController<BitboardTicTacToe> {
+    public TicTacToeBitController(Player<BitboardTicTacToe>[] players) {
+        BitboardTicTacToe game = new BitboardTicTacToe(players);
+        ThreadBehaviour thread = new LocalThreadBehaviour<>(game);
+        for (Player<BitboardTicTacToe> player : players) {
+            if (player instanceof OnlinePlayer<BitboardTicTacToe>){
+                thread = new OnlineThreadBehaviour<>(game);
+            }
+        }
+        super(new TicTacToeBitCanvas(), game, thread , "TicTacToe");
     }
 }

app/src/main/java/org/toop/app/widget/Primitive.java

@@ -6,6 +6,7 @@ import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.resource.resources.ImageAsset;
 import org.toop.local.AppContext;
 
+import java.io.File;
 import java.util.function.Consumer;
 
 import javafx.collections.FXCollections;
@@ -64,7 +65,7 @@ public final class Primitive {
         return imageView;
     }
 
-	public static Button button(String key, Runnable onAction, boolean localize, boolean disableOnClick) {
+	public static Button button(String key, Runnable onAction, boolean localize) {
 		var button = new Button();
 		button.getStyleClass().add("button");
 
@@ -75,7 +76,6 @@ public final class Primitive {
 
 		if (onAction != null) {
 			button.setOnAction(_ -> {
-				if (disableOnClick) button.setDisable(true);
                 onAction.run();
                 playButtonSound();
             });
@@ -84,8 +84,8 @@ public final class Primitive {
 		return button;
 	}
 
-	public static Button button(String key, Runnable onAction, boolean disableOnClick) {
-		return button(key, onAction, true, disableOnClick);
+	public static Button button(String key, Runnable onAction) {
+		return button(key, onAction, true);
 	}
 
 	public static TextField input(String promptKey, String text, Consumer<String> onValueChanged, boolean localize) {

app/src/main/java/org/toop/app/widget/WidgetContainer.java

@@ -1,5 +1,6 @@
 package org.toop.app.widget;
 
+import javafx.collections.ObservableList;
 import javafx.scene.Node;
 import org.toop.app.widget.complex.PopupWidget;
 import org.toop.app.widget.complex.ViewWidget;

app/src/main/java/org/toop/app/widget/complex/ConfirmWidget.java

@@ -26,7 +26,7 @@ public class ConfirmWidget implements Widget {
 
 	public void addButton(String key, Runnable onClick) {
 		Platform.runLater(() -> {
-			var button = Primitive.button(key, onClick, false);
+			var button = Primitive.button(key, onClick);
 			buttonsContainer.getChildren().add(button);
 		});
 	}

app/src/main/java/org/toop/app/widget/complex/LoadingWidget.java

@@ -3,6 +3,7 @@ package org.toop.app.widget.complex;
 import javafx.application.Platform;
 import javafx.geometry.Pos;
 import javafx.scene.Node;
+import javafx.scene.control.Control;
 import javafx.scene.control.ProgressBar;
 import javafx.scene.control.ProgressIndicator;
 import javafx.scene.layout.VBox;

app/src/main/java/org/toop/app/widget/complex/PlayerInfoWidget.java

@@ -23,7 +23,7 @@ public class PlayerInfoWidget {
 	}
 
 	private ToggleWidget buildToggle() {
-        return new ToggleWidget(
+		return new ToggleWidget(
 			"computer", "player",
 			information.isHuman,
 			isHuman -> {
@@ -37,14 +37,7 @@ public class PlayerInfoWidget {
 	}
 
 	private Node buildContent() {
-
         if (information.isHuman) {
-            var spacer = Primitive.vbox(
-                    makeAIButton(0, 0, "zwartepiet"),
-                    makeAIButton(0, 0, "sinterklaas"),
-                    makeAIButton(0, 0, "santa")
-            );                                                                  //todo make a better solution
-            spacer.setVisible(false);
             var nameInput = new LabeledInputWidget(
                     "name",
                     "enter-your-name",
@@ -52,7 +45,7 @@ public class PlayerInfoWidget {
                     newName -> information.name = newName
             );
 
-            return Primitive.vbox(spacer,nameInput.getNode());
+            return nameInput.getNode();
         } else {
             var AIBox = Primitive.vbox(
                     makeAIButton(0, 1, "zwartepiet"),
@@ -91,7 +84,7 @@ public class PlayerInfoWidget {
             information.computerDifficulty = depth;
             information.computerThinkTime = thinktime;
             this.playerName.setText(getName(name));
-        }, false);
+        });
     }
 
     private String getName(String name) {

app/src/main/java/org/toop/app/widget/complex/TableWidget.java

@@ -1,61 +0,0 @@
-package org.toop.app.widget.complex;
-
-import javafx.collections.FXCollections;
-import javafx.collections.ObservableList;
-import javafx.geometry.Pos;
-import javafx.scene.control.TableColumn;
-import javafx.scene.control.TableView;
-import javafx.scene.control.cell.PropertyValueFactory;
-
-import java.util.ArrayList;
-import java.util.List;
-
-public class TableWidget<DATATYPE> extends PopupWidget {
-    private ObservableList<DATATYPE> serverList = FXCollections.observableArrayList();
-    private TableView<DATATYPE> table = new TableView<>();
-
-
-    public TableWidget(String... columns) {
-        var cols = new ArrayList<TableColumn<DATATYPE, String>>();
-
-        for (String column : columns) {
-            TableColumn<DATATYPE, String> col = new TableColumn<>(column.toUpperCase());
-            col.setCellValueFactory(new PropertyValueFactory<>(column));
-            cols.add(col);
-        }
-
-        table.getColumns().addAll(cols);
-        update();
-        onColumnClicked();
-
-        add(Pos.CENTER, table);
-    }
-
-    public void add(DATATYPE serverFound) {
-        serverList.add(serverFound);
-        update();
-    }
-
-    public void add(List<DATATYPE> serverFound) {
-        serverList.addAll(serverFound);
-    }
-
-    public void remove(DATATYPE serverFound) {
-        serverList.remove(serverFound);
-        update();
-    }
-
-    public void onColumnClicked() {
-        table.setOnMouseClicked(event -> {
-            DATATYPE selected = table.getSelectionModel().getSelectedItem();
-            if (selected == null) return;
-            IO.println(selected.toString());
-        });
-    }
-
-    private void update() {
-        table.setItems(serverList);
-    }
-
-
-}

app/src/main/java/org/toop/app/widget/complex/ViewWidget.java

@@ -32,7 +32,7 @@ public abstract class ViewWidget extends StackWidget {
 
 		var backButton = Primitive.button("back", () -> {
 			view.transitionPrevious();
-		}, false);
+		});
 
 		view.add(Pos.BOTTOM_LEFT, Primitive.vbox(backButton));
 	}
@@ -45,7 +45,7 @@ public abstract class ViewWidget extends StackWidget {
 		var customButton = Primitive.button(key, () -> {
 			runnable.run();
 			view.transitionPrevious();
-		}, false);
+		});
 
 		view.add(Pos.BOTTOM_LEFT, Primitive.vbox(customButton));
 	}
@@ -97,7 +97,7 @@ public abstract class ViewWidget extends StackWidget {
 
 		var backButton = Primitive.button("back", () -> {
 			view.transitionPrevious();
-		}, false);
+		});
 
 		view.add(Pos.BOTTOM_LEFT, Primitive.vbox(backButton));
 	}

app/src/main/java/org/toop/app/widget/display/SongDisplay.java

@@ -49,11 +49,11 @@ public class SongDisplay extends VBox implements Widget {
 		previousButton.getStyleClass().setAll("previous-button");
 
 		skipButton.setOnAction( event -> {
-			GlobalEventBus.get().post(new AudioEvents.SkipMusic());
+			GlobalEventBus.post(new AudioEvents.SkipMusic());
 		});
 
 		pauseButton.setOnAction(event -> {
-			GlobalEventBus.get().post(new AudioEvents.PauseMusic());
+			GlobalEventBus.post(new AudioEvents.PauseMusic());
 			if (pauseButton.getText().equals("⏸")) {
 				pauseButton.setText("▶");
 			}
@@ -63,7 +63,7 @@ public class SongDisplay extends VBox implements Widget {
 		});
 
 		previousButton.setOnAction( event -> {
-			GlobalEventBus.get().post(new AudioEvents.PreviousMusic());
+			GlobalEventBus.post(new AudioEvents.PreviousMusic());
 		});
 
 		HBox control = new HBox(10, previousButton, pauseButton, skipButton);

app/src/main/java/org/toop/app/widget/popup/ChallengePopup.java

@@ -37,8 +37,8 @@ public final class ChallengePopup extends PopupWidget {
 		var acceptButton = Primitive.button("accept", () -> {
 			onAccept.accept(playerInformation);
 			this.hide();
-		}, false);
-		var denyButton = Primitive.button("deny", () -> hide(), false);
+		});
+		var denyButton = Primitive.button("deny", () -> hide());
 
 		var leftSection = Primitive.vbox(
 			challengeText,

app/src/main/java/org/toop/app/widget/popup/EscapePopup.java

@@ -1,48 +1,31 @@
 package org.toop.app.widget.popup;
 
 import javafx.geometry.Pos;
-import javafx.scene.Node;
+import org.toop.app.App;
 import org.toop.app.widget.Primitive;
-import org.toop.app.widget.Widget;
 import org.toop.app.widget.WidgetContainer;
+import org.toop.app.widget.complex.ConfirmWidget;
 import org.toop.app.widget.complex.PopupWidget;
-import org.toop.app.widget.complex.ViewWidget;
-import org.toop.app.widget.view.GameView;
 import org.toop.app.widget.view.OptionsView;
-import org.toop.local.AppContext;
-
-import java.util.ArrayList;
 
 public class EscapePopup extends PopupWidget {
     public EscapePopup() {
-        ViewWidget currentView = WidgetContainer.getCurrentView();
-        ArrayList<Node> nodes = new ArrayList<>();
+        var con = Primitive.button("Continue", this::hide, false); // TODO, localize
 
-        nodes.add(Primitive.button("Continue", this::hide, false, false)); // TODO, localize
+        var qui = Primitive.button("quit", () -> {
+            hide();
+            WidgetContainer.add(Pos.CENTER, new QuitPopup());
+        });
 
-        if (!(currentView.getClass().isAssignableFrom(OptionsView.class))) {
+        if (!(WidgetContainer.getCurrentView().getClass().isAssignableFrom(OptionsView.class))) {
             var opt = Primitive.button("options", () -> {
                 hide();
                 WidgetContainer.getCurrentView().transitionNext(new OptionsView());
-            }, false);
-            nodes.add(opt);
+            });
+            add(Pos.CENTER, Primitive.vbox(con, opt, qui));
+        } else {
+            add(Pos.CENTER, Primitive.vbox(con, qui));
         }
 
-        if (currentView.getClass().isAssignableFrom(GameView.class)) {
-            Widget tut = AppContext.currentTutorial();
-            if (tut != null) {
-                nodes.add(Primitive.button("tutorialstring", () -> {
-                    WidgetContainer.getCurrentView().add(Pos.CENTER, tut);
-                }, false));
-            }
-        }
-
-        nodes.add(Primitive.button("quit", () -> {
-            hide();
-            WidgetContainer.add(Pos.CENTER, new QuitPopup());
-        }, false));
-
-        add(Pos.CENTER, Primitive.vbox(nodes.toArray(new Node[0])));
-
     }
 }

app/src/main/java/org/toop/app/widget/popup/GameOverPopup.java

@@ -2,6 +2,7 @@ package org.toop.app.widget.popup;
 
 import org.toop.app.widget.complex.ConfirmWidget;
 import org.toop.app.widget.complex.PopupWidget;
+import org.toop.local.AppContext;
 
 import javafx.geometry.Pos;
 

app/src/main/java/org/toop/app/widget/popup/SendChallengePopup.java

@@ -62,9 +62,9 @@ public final class SendChallengePopup extends PopupWidget {
 		var sendButton = Primitive.button(
 			"send",
 			() -> { onSend.accept(playerInformation, gameChoice.getValue()); this.hide(); }
-		, false);
+		);
 
-		var cancelButton = Primitive.button("cancel", () -> hide(), false);
+		var cancelButton = Primitive.button("cancel", () -> hide());
 
 		var leftSection = Primitive.vbox(
 			challengeText,

app/src/main/java/org/toop/app/widget/tutorial/BaseTutorialWidget.java

@@ -7,6 +7,7 @@ import javafx.scene.text.Text;
 import org.apache.maven.surefire.shared.lang3.tuple.ImmutablePair;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.Updatable;
+import org.toop.app.widget.WidgetContainer;
 import org.toop.app.widget.complex.PopupWidget;
 
 import org.toop.framework.resource.resources.ImageAsset;
@@ -47,8 +48,8 @@ public class BaseTutorialWidget extends PopupWidget implements Updatable {
         this.pages = pages;
         this.nextScreen = nextScreen;
 
-        previousButton = Primitive.button("goback", () -> { update(false); this.hide(); }, false);
-        nextButton = Primitive.button(">", () -> update(true), false);
+        previousButton = Primitive.button("goback", () -> { update(false); this.hide(); });
+        nextButton = Primitive.button(">", () -> update(true));
 
         var w = Primitive.hbox(
                 previousButton,
@@ -58,6 +59,8 @@ public class BaseTutorialWidget extends PopupWidget implements Updatable {
         var x = Primitive.vbox(imagery, tutorialText);
 
         add(Pos.CENTER, Primitive.vbox(x, w));
+
+        WidgetContainer.add(Pos.CENTER, this);
     }
 
     @Override

app/src/main/java/org/toop/app/widget/tutorial/ShowEnableTutorialWidget.java

@@ -10,9 +10,9 @@ public class ShowEnableTutorialWidget extends PopupWidget {
 
     public ShowEnableTutorialWidget(Runnable tutorial, Runnable nextScreen, Runnable appSettingsSetter) {
         var a = Primitive.hbox(
-                Primitive.button("ok", () -> { appSettingsSetter.run(); tutorial.run(); this.hide(); }, false),
-                Primitive.button("no", () -> { appSettingsSetter.run(); nextScreen.run(); this.hide(); }, false),
-                Primitive.button("never", () -> { AppSettings.getSettings().setTutorialFlag(false); nextScreen.run(); this.hide(); }, false)
+                Primitive.button("ok", () -> { appSettingsSetter.run(); tutorial.run(); this.hide(); }),
+                Primitive.button("no", () -> { appSettingsSetter.run(); nextScreen.run(); this.hide(); }),
+                Primitive.button("never", () -> { AppSettings.getSettings().setTutorialFlag(false); nextScreen.run(); this.hide(); })
         );
 
         var txt = Primitive.text("tutorial");

app/src/main/java/org/toop/app/widget/view/GameView.java

@@ -6,8 +6,6 @@ import javafx.scene.text.Font;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.complex.ViewWidget;
 import org.toop.app.widget.popup.GameOverPopup;
-
-import java.util.Objects;
 import java.util.function.Consumer;
 import javafx.application.Platform;
 import javafx.geometry.Pos;
@@ -28,6 +26,7 @@ public final class GameView extends ViewWidget {
     private Circle player2Icon;
 	private final Button forfeitButton;
 	private final Button exitButton;
+    private final Button tutorialButton;
     private final TextField chatInput;
     private final Text keyThingy;
     private boolean hasSet = false;
@@ -42,7 +41,7 @@ public final class GameView extends ViewWidget {
         player2Icon = new Circle();
 
 		if (onForfeit != null) {
-			forfeitButton = Primitive.button("forfeit", () -> onForfeit.run(), false);
+			forfeitButton = Primitive.button("forfeit", () -> onForfeit.run());
 		} else {
 			forfeitButton = null;
 		}
@@ -50,7 +49,7 @@ public final class GameView extends ViewWidget {
 		exitButton = Primitive.button("exit", () -> {
 			onExit.run();
 			transitionPrevious();
-		}, false);
+		});
 
 		if (onMessage != null) {
 			chatInput = Primitive.input("enter-your-message", "", null);
@@ -62,16 +61,15 @@ public final class GameView extends ViewWidget {
 			chatInput = null;
 		}
 
-        switch (gameType) {
+        switch(gameType) {
             case "TicTacToe":
-                AppContext.setCurrentTutorial(new TicTacToeTutorialWidget(() -> {}));
-                break;
+                this.tutorialButton = Primitive.button("tutorialstring", () -> new TicTacToeTutorialWidget(() -> {})); break;
             case "Reversi":
-                AppContext.setCurrentTutorial(new ReversiTutorialWidget(() -> {}));
-                break;
+                this.tutorialButton = Primitive.button("tutorialstring", () -> new ReversiTutorialWidget(() -> {})); break;
             case "Connect4":
-                AppContext.setCurrentTutorial(new Connect4TutorialWidget(() -> {}));
-                break;
+                this.tutorialButton = Primitive.button("tutorialstring", () -> new Connect4TutorialWidget(() -> {})); break;
+            default:
+                this.tutorialButton = null; break;
         }
 
 		setupLayout();
@@ -94,9 +92,13 @@ public final class GameView extends ViewWidget {
 		if (chatInput != null) {
 			add(Pos.BOTTOM_RIGHT, Primitive.vbox(chatInput));
 		}
+
+        if (tutorialButton != null) {
+            add(Pos.TOP_LEFT, tutorialButton);
+        }
 	}
 
-	public void nextPlayer(boolean isMe, String currentPlayer, String nextPlayer, String GameType) {
+	public void nextPlayer(boolean isMe, String currentPlayer, String currentMove, String nextPlayer, char GameType) {
 		Platform.runLater(() -> {
             if (!(hasSet)) {
                 playerHeader.setText(currentPlayer + " vs. " + nextPlayer);
@@ -114,8 +116,8 @@ public final class GameView extends ViewWidget {
 		new GameOverPopup(iWon, winner).show(Pos.CENTER);
 	}
 
-    private void setPlayerHeaders(boolean isMe, String currentPlayer, String nextPlayer, String GameType) {
-        if (Objects.equals(GameType, "TicTacToe")) {
+    private void setPlayerHeaders(boolean isMe, String currentPlayer, String nextPlayer, char GameType) {
+        if (GameType == 'T') {
             if (isMe) {
                 player1Header.setText("X: " + currentPlayer);
                 player2Header.setText("O: " + nextPlayer);
@@ -126,7 +128,7 @@ public final class GameView extends ViewWidget {
             }
             setPlayerInfoTTT();
         }
-        else if (Objects.equals(GameType, "Reversi")) {
+        else if (GameType == 'R') {
             if (isMe) {
                 player1Header.setText(currentPlayer);
                 player2Header.setText(nextPlayer);
@@ -174,8 +176,8 @@ public final class GameView extends ViewWidget {
 
         player1Icon.setRadius(player1Header.fontProperty().map(Font::getSize).getValue());
         player2Icon.setRadius(player2Header.fontProperty().map(Font::getSize).getValue());
-        player1Icon.setFill(Color.WHITE);
-        player2Icon.setFill(Color.BLACK);
+        player1Icon.setFill(Color.BLACK);
+        player2Icon.setFill(Color.WHITE);
         add(Pos.TOP_RIGHT, playerInfo);
     }
 }

app/src/main/java/org/toop/app/widget/view/LocalMultiplayerView.java

@@ -2,21 +2,24 @@ package org.toop.app.widget.view;
 
 import javafx.application.Platform;
 import org.toop.app.GameInformation;
+import org.toop.app.canvas.ReversiBitCanvas;
+import org.toop.app.canvas.TicTacToeBitCanvas;
+import org.toop.app.gameControllers.GenericGameController;
 import org.toop.app.gameControllers.ReversiBitController;
 import org.toop.app.gameControllers.TicTacToeBitController;
-import org.toop.framework.game.players.LocalPlayer;
 import org.toop.framework.gameFramework.controller.GameController;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.players.ArtificialPlayer;
+import org.toop.game.games.reversi.BitboardReversi;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
+import org.toop.game.players.ArtificialPlayer;
+import org.toop.game.players.LocalPlayer;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.complex.PlayerInfoWidget;
 import org.toop.app.widget.complex.ViewWidget;
 import org.toop.app.widget.popup.ErrorPopup;
 import org.toop.app.widget.tutorial.*;
-import org.toop.game.players.ai.MiniMaxAI;
-import org.toop.game.players.ai.mcts.MCTSAI1;
-import org.toop.game.players.ai.mcts.MCTSAI3;
-import org.toop.game.players.ai.mcts.MCTSAI4;
+import org.toop.game.players.MiniMaxAI;
+import org.toop.game.players.RandomAI;
 import org.toop.local.AppContext;
 
 import javafx.geometry.Pos;
@@ -24,6 +27,9 @@ import javafx.scene.control.ScrollPane;
 import javafx.scene.layout.VBox;
 import org.toop.local.AppSettings;
 
+import java.util.Arrays;
+import java.util.Random;
+
 public class LocalMultiplayerView extends ViewWidget {
 	private final GameInformation information;
 
@@ -52,14 +58,14 @@ public class LocalMultiplayerView extends ViewWidget {
 			switch (information.type) {
                 case TICTACTOE:
                     if (information.players[0].isHuman) {
-                        players[0] = new LocalPlayer(information.players[0].name);
+                        players[0] = new LocalPlayer<>(information.players[0].name);
                     } else {
-                        players[0] = new ArtificialPlayer(new MCTSAI1(100), "MCTS AI");
+                        players[0] = new ArtificialPlayer<>(new RandomAI<BitboardTicTacToe>(), "Random AI");
                     }
                     if (information.players[1].isHuman) {
-                        players[1] = new LocalPlayer(information.players[1].name);
+                        players[1] = new LocalPlayer<>(information.players[1].name);
                     } else {
-                        players[1] = new ArtificialPlayer(new MiniMaxAI(9), "MiniMax AI");
+                        players[1] = new ArtificialPlayer<>(new MiniMaxAI<BitboardTicTacToe>(9), "MiniMax AI");
                     }
                     if (AppSettings.getSettings().getTutorialFlag() && AppSettings.getSettings().getFirstTTT()) {
                         new ShowEnableTutorialWidget(
@@ -80,15 +86,14 @@ public class LocalMultiplayerView extends ViewWidget {
                     break;
                 case REVERSI:
                     if (information.players[0].isHuman) {
-                        players[0] = new LocalPlayer(information.players[0].name);
+                        players[0] = new LocalPlayer<>(information.players[0].name);
                     } else {
-						// players[0] = new ArtificialPlayer(new RandomAI<BitboardReversi>(), "Random AI");
-						players[0] = new ArtificialPlayer(new MCTSAI4(500, 4), "MCTS V4 AI");
+                        players[0] = new ArtificialPlayer<>(new RandomAI<BitboardReversi>(), "Random AI");
                     }
                     if (information.players[1].isHuman) {
-                        players[1] = new LocalPlayer(information.players[1].name);
+                        players[1] = new LocalPlayer<>(information.players[1].name);
                     } else {
-						players[1] = new ArtificialPlayer(new MCTSAI1(500), "MCTS V1 AI");
+                        players[1] = new ArtificialPlayer<>(new MiniMaxAI<BitboardReversi>(6), "MiniMax");
                     }
                     if (AppSettings.getSettings().getTutorialFlag() && AppSettings.getSettings().getFirstReversi()) {
                         new ShowEnableTutorialWidget(
@@ -108,7 +113,7 @@ public class LocalMultiplayerView extends ViewWidget {
                     }
                     break;
             }
-        }, false);
+        });
 
 		var playerSection = setupPlayerSections();
 

app/src/main/java/org/toop/app/widget/view/LocalView.java

@@ -10,11 +10,11 @@ public class LocalView extends ViewWidget {
 	public LocalView() {
 		var ticTacToeButton = Primitive.button("tic-tac-toe", () -> {
 			transitionNext(new LocalMultiplayerView(GameInformation.Type.TICTACTOE));
-		}, false);
+		});
 
 		var reversiButton = Primitive.button("reversi", () -> {
 			transitionNext(new LocalMultiplayerView(GameInformation.Type.REVERSI));
-		}, false);
+		});
 
 		add(Pos.CENTER, Primitive.vbox(
 			ticTacToeButton,

app/src/main/java/org/toop/app/widget/view/MainView.java

@@ -1,5 +1,6 @@
 package org.toop.app.widget.view;
 
+import org.toop.app.App;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.complex.ViewWidget;
 import javafx.geometry.Pos;
@@ -9,24 +10,24 @@ public class MainView extends ViewWidget {
 	public MainView() {
 		var localButton = Primitive.button("local", () -> {
 			transitionNext(new LocalView());
-		}, false);
+		});
 
 		var onlineButton = Primitive.button("online", () -> {
 			transitionNext(new OnlineView());
-		}, false);
+		});
 
 		var creditsButton = Primitive.button("credits", () -> {
 			transitionNext(new CreditsView());
-		}, false);
+		});
 
 		var optionsButton = Primitive.button("options", () -> {
 			transitionNext(new OptionsView());
-		}, false);
+		});
 
 		var quitButton = Primitive.button("quit", () -> {
 			var a = new QuitPopup();
 			a.show(Pos.CENTER);
-		}, false);
+		});
 
 		add(Pos.CENTER, Primitive.vbox(
 			localButton,

app/src/main/java/org/toop/app/widget/view/OnlineView.java

@@ -3,17 +3,10 @@ package org.toop.app.widget.view;
 import org.toop.app.Server;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.complex.LabeledInputWidget;
+import org.toop.app.widget.complex.LoadingWidget;
 import org.toop.app.widget.complex.ViewWidget;
 
 import javafx.geometry.Pos;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.games.tictactoe.BitboardTicTacToe;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.networking.server.gateway.NettyGatewayServer;
-import org.toop.framework.networking.server.stores.TurnBasedGameTypeStore;
-
-import java.time.Duration;
-import java.util.concurrent.ConcurrentHashMap;
 
 public class OnlineView extends ViewWidget {
 	public OnlineView() {
@@ -29,31 +22,7 @@ public class OnlineView extends ViewWidget {
 				serverPortInput.getValue(),
 				playerNameInput.getValue()
 			);
-		}, false);
-
-		var localHostButton = Primitive.button("host!", () -> {
-
-			var tps = new TurnBasedGameTypeStore();
-			tps.register("tic-tac-toe", BitboardTicTacToe::new);
-			tps.register("reversi", BitboardReversi::new);
-
-			var a = new NettyGatewayServer(6666, tps, Duration.ofSeconds(10));
-
-			new Thread(() -> {
-				try {
-					a.start();
-				} catch (InterruptedException e) {
-					throw new RuntimeException(e);
-				}
-			}).start();
-
-			new Server(
-					"127.0.0.1",
-					"6666",
-					"host",
-					a
-			);
-		}, false, false);
+		});
 
 		add(Pos.CENTER, Primitive.vbox(
 			serverInformationHeader,
@@ -64,9 +33,7 @@ public class OnlineView extends ViewWidget {
 			playerNameInput.getNode(),
 			Primitive.separator(),
 
-			connectButton,
-			Primitive.separator(),
-			localHostButton
+			connectButton
 		));
 	}
 }

app/src/main/java/org/toop/app/widget/view/OptionsView.java

@@ -74,7 +74,7 @@ public class OptionsView extends ViewWidget {
 				AppSettings.getSettings().setVolume(val);
 				new EventFlow()
 					.addPostEvent(new AudioEvents.ChangeVolume(val, VolumeControl.MASTERVOLUME))
-					.postEvent();
+					.asyncPostEvent();
 			}
 		);
 
@@ -86,7 +86,7 @@ public class OptionsView extends ViewWidget {
 				AppSettings.getSettings().setFxVolume(val);
 				new EventFlow()
 					.addPostEvent(new AudioEvents.ChangeVolume(val, VolumeControl.FX))
-					.postEvent();
+					.asyncPostEvent();
 			}
 		);
 
@@ -98,7 +98,7 @@ public class OptionsView extends ViewWidget {
 				AppSettings.getSettings().setMusicVolume(val);
 				new EventFlow()
 					.addPostEvent(new AudioEvents.ChangeVolume(val, VolumeControl.MUSIC))
-					.postEvent();
+					.asyncPostEvent();
 			}
 		);
 

app/src/main/java/org/toop/app/widget/view/ServerView.java

@@ -1,89 +1,47 @@
 package org.toop.app.widget.view;
 
-import javafx.collections.FXCollections;
-import javafx.css.converter.StringConverter;
-import javafx.scene.control.ComboBox;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.complex.ViewWidget;
 
-import java.io.Reader;
-import java.util.ArrayList;
 import java.util.List;
-import java.util.Locale;
 import java.util.function.Consumer;
 
 import javafx.application.Platform;
 import javafx.geometry.Pos;
 import javafx.scene.control.Button;
 import javafx.scene.control.ListView;
-import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.eventbus.GlobalEventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
 
 public final class ServerView extends ViewWidget {
 	private final String user;
 	private final Consumer<String> onPlayerClicked;
-	private final long clientId;
 
-	private final ComboBox<String> gameListSub;
-    private final ComboBox<String> gameListTour;
 	private final ListView<Button> listView;
-	private Button subscribeButton;
 
-	public ServerView(String user, Consumer<String> onPlayerClicked, String userName, long clientId) {
+	public ServerView(String user, Consumer<String> onPlayerClicked) {
 		this.user = user;
 		this.onPlayerClicked = onPlayerClicked;
-		this.clientId = clientId;
 
-		this.gameListSub = new ComboBox<>();
-        this.gameListTour = new ComboBox<>();
 		this.listView = new ListView<>();
 
-		setupLayout(userName);
+		setupLayout();
 	}
 
-	private void setupLayout(String userName) {
+	private void setupLayout() {
 		var playerHeader = Primitive.header(user, false);
 
-        if (userName.equals("host")) { // TODO is fragile
-            var tournamentButton = Primitive.hbox(
-                    gameListTour,
-                    Primitive.button(
-                            "tournament",
-                            () -> GlobalEventBus.get().post(new NetworkEvents.SendCommand(clientId, "tournament", "start", gameListTour.getValue())),
-                            false,
-                            false
-                    )
-            );
+		var playerListSection = Primitive.vbox(
+			playerHeader,
+			Primitive.separator(),
+			listView
+		);
 
-            add(Pos.BOTTOM_CENTER, tournamentButton);
-        } else {
-            subscribeButton = Primitive.button(
-                    "subscribe",
-                    () -> new EventFlow().addPostEvent(new NetworkEvents.SendSubscribe(clientId, gameListSub.getValue())).postEvent(),
-                    false,
-                    true
-            ); // TODO localize
+		add(Pos.CENTER, playerListSection);
 
-            var subscribe = Primitive.hbox(gameListSub, subscribeButton);
+		var disconnectButton = Primitive.button("disconnect", () -> {
+			transitionPrevious();
+		});
 
-            var playerListSection = Primitive.vbox(
-                    playerHeader,
-                    Primitive.separator(),
-                    subscribe,
-                    listView
-            );
-
-            add(Pos.CENTER, playerListSection);
-
-            var disconnectButton = Primitive.button(
-                    "disconnect",
-					this::transitionPrevious,
-					false
-			);
-
-            add(Pos.BOTTOM_LEFT, Primitive.vbox(disconnectButton));
-        }
+		add(Pos.BOTTOM_LEFT, Primitive.vbox(disconnectButton));
 	}
 
 	public void update(List<String> players) {
@@ -91,25 +49,9 @@ public final class ServerView extends ViewWidget {
 			listView.getItems().clear();
 
 			for (String player : players) {
-				var playerButton = Primitive.button(player, () -> onPlayerClicked.accept(player), false, false);
+				var playerButton = Primitive.button(player, () -> onPlayerClicked.accept(player), false);
 				listView.getItems().add(playerButton);
 			}
 		});
 	}
-
-	public void updateGameList(List<String> games) {
-		Platform.runLater(() -> {
-			gameListSub.getItems().clear();
-			gameListSub.setItems(FXCollections.observableArrayList(games));
-			gameListSub.getSelectionModel().select(0);
-
-            gameListTour.getItems().clear();
-            gameListTour.setItems(FXCollections.observableArrayList(games));
-            gameListTour.getSelectionModel().select(0);
-		});
-	}
-
-	public void reEnableButton() {
-		subscribeButton.setDisable(false);
-	}
 }

app/src/main/java/org/toop/local/AppContext.java

@@ -1,10 +1,10 @@
 package org.toop.local;
 
+import java.util.Locale;
 import java.util.MissingResourceException;
 
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.toop.app.widget.tutorial.BaseTutorialWidget;
 import org.toop.framework.resource.ResourceManager;
 import org.toop.framework.resource.resources.LocalizationAsset;
 
@@ -16,13 +16,11 @@ import javafx.beans.property.ObjectProperty;
 import javafx.beans.property.SimpleObjectProperty;
 
 public class AppContext {
-    private static final Logger logger = LogManager.getLogger(AppContext.class);
-
     private static final LocalizationAsset localization = ResourceManager.get("localization");
     private static Locale locale = Locale.forLanguageTag("en");
 
 	private static final ObjectProperty<Locale> localeProperty = new SimpleObjectProperty<>(locale);
-    private static BaseTutorialWidget tutorialWidget;
+    private static final Logger logger = LogManager.getLogger(AppContext.class);
 
     public static LocalizationAsset getLocalization() {
         return localization;
@@ -75,12 +73,4 @@ public class AppContext {
     public static StringBinding bindToKey(String key) {
         return bindToKey(key, true);
     }
-
-    public static void setCurrentTutorial(BaseTutorialWidget tutorial) {
-        AppContext.tutorialWidget = tutorial;
-    }
-
-    public static BaseTutorialWidget currentTutorial() {
-        return AppContext.tutorialWidget;
-    }
 }

app/src/main/java/org/toop/local/AppSettings.java

@@ -36,13 +36,13 @@ public class AppSettings {
 
 		new EventFlow()
 				.addPostEvent(new AudioEvents.ChangeVolume(settingsData.volume, VolumeControl.MASTERVOLUME))
-				.postEvent();
+				.asyncPostEvent();
 		new EventFlow()
 				.addPostEvent(new AudioEvents.ChangeVolume(settingsData.fxVolume, VolumeControl.FX))
-				.postEvent();
+				.asyncPostEvent();
 		new EventFlow()
 				.addPostEvent(new AudioEvents.ChangeVolume(settingsData.musicVolume, VolumeControl.MUSIC))
-				.postEvent();
+				.asyncPostEvent();
 	}
 
 	public static SettingsAsset getPath() {

framework/pom.xml

@@ -147,6 +147,7 @@
             <version>2.42.0</version>
         </dependency>
 	</dependencies>
+
 	<build>
 		<plugins>
 			<plugin>

framework/src/main/java/org/toop/framework/audio/AudioEventListener.java

@@ -5,29 +5,26 @@ import org.toop.framework.audio.interfaces.MusicManager;
 import org.toop.framework.audio.interfaces.SoundEffectManager;
 import org.toop.framework.audio.interfaces.VolumeManager;
 import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.eventbus.bus.EventBus;
+import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.resource.types.AudioResource;
 
 public class AudioEventListener<T extends AudioResource, K extends AudioResource> {
-    private final EventBus eventBus;
     private final MusicManager<T> musicManager;
     private final SoundEffectManager<K> soundEffectManager;
     private final VolumeManager audioVolumeManager;
 
     public AudioEventListener(
-            EventBus eventBus,
             MusicManager<T> musicManager,
             SoundEffectManager<K> soundEffectManager,
             VolumeManager audioVolumeManager
     ) {
-        this.eventBus = eventBus;
         this.musicManager = musicManager;
         this.soundEffectManager = soundEffectManager;
         this.audioVolumeManager = audioVolumeManager;
     }
 
     public AudioEventListener<?, ?> initListeners(String buttonSoundToPlay) {
-        new EventFlow(eventBus)
+        new EventFlow()
                 .listen(AudioEvents.StopAudioManager.class, this::handleStopMusicManager, false)
                 .listen(AudioEvents.PlayEffect.class, this::handlePlaySound, false)
                 .listen(AudioEvents.SkipMusic.class, this::handleSkipSong, false)
@@ -76,7 +73,7 @@ public class AudioEventListener<T extends AudioResource, K extends AudioResource
     }
 
     private void handleGetVolume(AudioEvents.GetVolume event) {
-        eventBus.post(new AudioEvents.GetVolumeResponse(
+        GlobalEventBus.postAsync(new AudioEvents.GetVolumeResponse(
                 audioVolumeManager.getVolume(event.controlType()),
                 event.identifier()));
     }

framework/src/main/java/org/toop/framework/audio/MusicManager.java

@@ -6,7 +6,8 @@ import org.toop.framework.audio.events.AudioEvents;
 import org.toop.framework.dispatch.interfaces.Dispatcher;
 import org.toop.framework.dispatch.JavaFXDispatcher;
 import org.toop.annotations.TestsOnly;
-import org.toop.framework.eventbus.bus.EventBus;
+import org.toop.framework.eventbus.EventFlow;
+import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.resource.types.AudioResource;
 
 import java.util.*;
@@ -17,7 +18,6 @@ import java.util.concurrent.TimeUnit;
 public class MusicManager<T extends AudioResource> implements org.toop.framework.audio.interfaces.MusicManager<T> {
     private static final Logger logger = LogManager.getLogger(MusicManager.class);
 
-    private final EventBus eventBus;
     private final List<T> backgroundMusic = new ArrayList<>();
     private final Dispatcher dispatcher;
     private final List<T> resources;
@@ -27,8 +27,7 @@ public class MusicManager<T extends AudioResource> implements org.toop.framework
     private ScheduledExecutorService scheduler;
 
 
-    public MusicManager(EventBus eventbus, List<T> resources, boolean shuffleMusic) {
-        this.eventBus = eventbus;
+    public MusicManager(List<T> resources, boolean shuffleMusic) {
         this.dispatcher = new JavaFXDispatcher();
         this.resources = resources;
         // Shuffle if wanting to shuffle
@@ -41,8 +40,7 @@ public class MusicManager<T extends AudioResource> implements org.toop.framework
      * {@code @TestsOnly} DO NOT USE
      */
     @TestsOnly
-    public MusicManager(EventBus eventBus, List<T> resources, Dispatcher dispatcher) {
-        this.eventBus = eventBus;
+    public MusicManager(List<T> resources, Dispatcher dispatcher) {
         this.dispatcher = dispatcher;
         this.resources = new ArrayList<>(resources);
         backgroundMusic.addAll(resources);
@@ -126,7 +124,7 @@ public class MusicManager<T extends AudioResource> implements org.toop.framework
         Runnable currentMusicTask = new Runnable() {
             @Override
             public void run() {
-                eventBus.post(new AudioEvents.PlayingMusic(track.getName(), track.currentPosition(), track.duration()));
+                GlobalEventBus.post(new AudioEvents.PlayingMusic(track.getName(), track.currentPosition(), track.duration()));
                 scheduler.schedule(this, 1, TimeUnit.SECONDS);
             }
         };

framework/src/main/java/org/toop/framework/audio/SoundEffectManager.java

@@ -2,10 +2,18 @@ package org.toop.framework.audio;
 
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
+import org.toop.framework.resource.ResourceManager;
 import org.toop.framework.resource.ResourceMeta;
 import org.toop.framework.resource.resources.BaseResource;
+import org.toop.framework.resource.resources.MusicAsset;
+import org.toop.framework.resource.resources.SoundEffectAsset;
 import org.toop.framework.resource.types.AudioResource;
 
+import javax.sound.sampled.Clip;
+import javax.sound.sampled.LineEvent;
+import javax.sound.sampled.LineUnavailableException;
+import javax.sound.sampled.UnsupportedAudioFileException;
+import java.io.IOException;
 import java.util.*;
 import java.util.stream.Collectors;
 

framework/src/main/java/org/toop/framework/audio/events/AudioEvents.java

@@ -2,6 +2,8 @@ package org.toop.framework.audio.events;
 
 import org.toop.framework.audio.VolumeControl;
 import org.toop.framework.eventbus.events.*;
+import org.toop.framework.eventbus.events.ResponseToUniqueEvent;
+import org.toop.framework.eventbus.events.UniqueEvent;
 
 public class AudioEvents extends EventsBase {
     /** Stops the audio manager. */

framework/src/main/java/org/toop/framework/audio/interfaces/SoundEffectManager.java

@@ -1,7 +1,12 @@
 package org.toop.framework.audio.interfaces;
 
+import org.toop.framework.resource.resources.SoundEffectAsset;
 import org.toop.framework.resource.types.AudioResource;
 
+import javax.sound.sampled.LineUnavailableException;
+import javax.sound.sampled.UnsupportedAudioFileException;
+import java.io.IOException;
+
 public interface SoundEffectManager<T extends AudioResource> extends AudioManager<T> {
     void play(String name, boolean loop);
     void stop(String name);

framework/src/main/java/org/toop/framework/eventbus/EventFlow.java

@@ -13,14 +13,10 @@ import org.toop.framework.SnowflakeGenerator;
 import org.toop.framework.eventbus.events.EventType;
 import org.toop.framework.eventbus.events.ResponseToUniqueEvent;
 import org.toop.framework.eventbus.events.UniqueEvent;
-import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.eventbus.subscriber.DefaultNamedSubscriber;
-import org.toop.framework.eventbus.subscriber.NamedSubscriber;
-import org.toop.framework.eventbus.subscriber.Subscriber;
 
 /**
  * EventFlow is a utility class for creating, posting, and optionally subscribing to events in a
- * type-safe and chainable manner. It is designed to work with the {@link EventBus}.
+ * type-safe and chainable manner. It is designed to work with the {@link GlobalEventBus}.
  *
  * <p>This class supports automatic UUID assignment for {@link UniqueEvent} events, and
  * allows filtering subscribers so they only respond to events with a specific UUID. All
@@ -35,8 +31,6 @@ public class EventFlow {
     /** Cache of constructor handles for event classes to avoid repeated reflection lookups. */
     private static final Map<Class<?>, MethodHandle> CONSTRUCTOR_CACHE = new ConcurrentHashMap<>();
 
-    private final EventBus eventBus;
-
     /** Automatically assigned UUID for {@link UniqueEvent} events. */
     private long eventSnowflake = -1;
 
@@ -44,19 +38,13 @@ public class EventFlow {
     private EventType event = null;
 
     /** The listener returned by GlobalEventBus subscription. Used for unsubscription. */
-    private final List<NamedSubscriber<?>> listeners = new ArrayList<>();
+    private final List<ListenerHandler<?>> listeners = new ArrayList<>();
 
     /** Holds the results returned from the subscribed event, if any. */
     private Map<String, ?> result = null;
 
     /** Empty constructor (event must be added via {@link #addPostEvent(Class, Object...)}). */
-    public EventFlow(EventBus eventBus) {
-        this.eventBus = eventBus;
-    }
-
-    public EventFlow() {
-        this.eventBus = GlobalEventBus.get();
-    }
+    public EventFlow() {}
 
     /**
      *
@@ -157,19 +145,21 @@ public class EventFlow {
 
             action.accept(eventClass);
 
-            if (unsubscribeAfterSuccess) unsubscribe(String.valueOf(id));
+            if (unsubscribeAfterSuccess) unsubscribe(id);
 
             this.result = eventClass.result();
         };
 
-        var subscriber = new DefaultNamedSubscriber<>(
+        // TODO Remove casts
+        var listener = new ListenerHandler<>(
+                id,
                 name,
-                event,
-                newAction
+                (Class<ResponseToUniqueEvent>) event,
+                (Consumer<ResponseToUniqueEvent>) newAction
         );
 
-        eventBus.subscribe(subscriber);
-        this.listeners.add(subscriber);
+        GlobalEventBus.subscribe(listener);
+        this.listeners.add(listener);
         return this;
     }
 
@@ -237,7 +227,7 @@ public class EventFlow {
                     TT typedEvent = (TT) uuidEvent;
                     action.accept(typedEvent);
 
-                    if (unsubscribeAfterSuccess) unsubscribe(String.valueOf(id));
+                    if (unsubscribeAfterSuccess) unsubscribe(id);
 
                     this.result = typedEvent.result();
                 } catch (ClassCastException _) {
@@ -249,13 +239,14 @@ public class EventFlow {
             }
         };
 
-        var listener = new DefaultNamedSubscriber<>(
+        var listener = new ListenerHandler<>(
+                id,
                 name,
                 (Class<TT>) action.getClass().getDeclaredMethods()[0].getParameterTypes()[0],
                 newAction
         );
 
-        eventBus.subscribe(listener);
+        GlobalEventBus.subscribe(listener);
         this.listeners.add(listener);
         return this;
     }
@@ -293,16 +284,17 @@ public class EventFlow {
         Consumer<TT> newAction = eventc -> {
             action.accept(eventc);
 
-            if (unsubscribeAfterSuccess) unsubscribe(String.valueOf(id));
+            if (unsubscribeAfterSuccess) unsubscribe(id);
         };
 
-        var listener = new DefaultNamedSubscriber<>(
+        var listener = new ListenerHandler<>(
+                        id,
                         name,
                         event,
                         newAction
                 );
 
-        eventBus.subscribe(listener);
+        GlobalEventBus.subscribe(listener);
         this.listeners.add(listener);
         return this;
     }
@@ -370,7 +362,7 @@ public class EventFlow {
             try {
                 TT typedEvent = (TT) nonUuidEvent;
                 action.accept(typedEvent);
-                if (unsubscribeAfterSuccess) unsubscribe(String.valueOf(id));
+                if (unsubscribeAfterSuccess) unsubscribe(id);
             } catch (ClassCastException _) {
                 throw new ClassCastException(
                         "Cannot cast "
@@ -379,13 +371,14 @@ public class EventFlow {
             }
         };
 
-        var listener = new DefaultNamedSubscriber<>(
+        var listener = new ListenerHandler<>(
+                id,
                 name,
                 eventClass,
                 newAction
         );
 
-        eventBus.subscribe(listener);
+        GlobalEventBus.subscribe(listener);
         this.listeners.add(listener);
         return this;
     }
@@ -408,16 +401,15 @@ public class EventFlow {
      * Posts the event added through {@link #addPostEvent}.
      */
     public EventFlow postEvent() {
-        eventBus.post(this.event);
+        GlobalEventBus.post(this.event);
         return this;
     }
 
     /**
      * Posts the event added through {@link #addPostEvent} asynchronously.
-     *
      */
     public EventFlow asyncPostEvent() {
-        GlobalEventBus.get().post(this.event);
+        GlobalEventBus.postAsync(this.event);
         return this;
     }
 
@@ -425,12 +417,28 @@ public class EventFlow {
      *
      * Unsubscribe from an event.
      *
-     * @param action The listener object to remove and unsubscribe.
+     * @param listenerObject The listener object to remove and unsubscribe.
      */
-    public void unsubscribe(Consumer<?> action) {
+    public void unsubscribe(Object listenerObject) {
         this.listeners.removeIf(handler -> {
-            if (handler.handler().equals(action)) {
-                eventBus.unsubscribe(handler);
+            if (handler.getListener() == listenerObject) {
+                GlobalEventBus.unsubscribe(handler);
+                return true;
+            }
+            return false;
+        });
+    }
+
+    /**
+     *
+     * Unsubscribe from an event.
+     *
+     * @param listenerId The id given to the {@link ListenerHandler}.
+     */
+    public void unsubscribe(long listenerId) {
+        this.listeners.removeIf(handler -> {
+            if (handler.getId() == listenerId) {
+                GlobalEventBus.unsubscribe(handler);
                 return true;
             }
             return false;
@@ -444,8 +452,8 @@ public class EventFlow {
      */
     public void unsubscribe(String name) {
         this.listeners.removeIf(handler -> {
-            if (handler.id().equals(name)) {
-                eventBus.unsubscribe(handler);
+            if (handler.getName().equals(name)) {
+                GlobalEventBus.unsubscribe(handler);
                 return true;
             }
             return false;
@@ -457,7 +465,7 @@ public class EventFlow {
      */
     public void unsubscribeAll() {
         listeners.removeIf(handler -> {
-            eventBus.unsubscribe(handler);
+            GlobalEventBus.unsubscribe(handler);
             return true;
         });
     }
@@ -495,8 +503,8 @@ public class EventFlow {
      *
      * @return Copy of the list of listeners.
      */
-    public Subscriber<?>[] getListeners() {
-        return listeners.toArray(new Subscriber[0]);
+    public ListenerHandler[] getListeners() {
+        return listeners.toArray(new ListenerHandler[0]);
     }
 
     /**

framework/src/main/java/org/toop/framework/eventbus/GlobalEventBus.java

@@ -1,57 +1,193 @@
 package org.toop.framework.eventbus;
 
+import com.lmax.disruptor.*;
+import com.lmax.disruptor.dsl.Disruptor;
+import com.lmax.disruptor.dsl.ProducerType;
+import java.util.Map;
+import java.util.concurrent.*;
+import java.util.function.Consumer;
+
 import org.apache.logging.log4j.LogManager;
-import org.toop.framework.eventbus.bus.AsyncEventBus;
-import org.toop.framework.eventbus.bus.DefaultEventBus;
-import org.toop.framework.eventbus.bus.DisruptorEventBus;
-import org.toop.framework.eventbus.bus.EventBus;
+import org.apache.logging.log4j.Logger;
 import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.store.DefaultSubscriberStore;
-import org.toop.framework.eventbus.subscriber.Subscriber;
+import org.toop.framework.eventbus.events.UniqueEvent;
 
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
+/**
+ * GlobalEventBus backed by the LMAX Disruptor for ultra-low latency, high-throughput event
+ * publishing.
+ */
+public final class GlobalEventBus {
+    private static final Logger logger = LogManager.getLogger(GlobalEventBus.class);
 
-public class GlobalEventBus implements AsyncEventBus {
-    private static final AsyncEventBus INSTANCE = new DefaultEventBus(
-            LogManager.getLogger(DefaultEventBus.class),
-            new DefaultSubscriberStore()
-    );
+    /** Map of event class to type-specific listeners. */
+    private static final Map<Class<?>, CopyOnWriteArrayList<ListenerHandler<?>>>
+            LISTENERS = new ConcurrentHashMap<>();
 
+    /** Map of event class to Snowflake-ID-specific listeners. */
+    private static final Map<
+                    Class<?>, ConcurrentHashMap<Long, Consumer<? extends UniqueEvent>>>
+            UUID_LISTENERS = new ConcurrentHashMap<>();
+
+    /** Disruptor ring buffer size (must be power of two). */
+    private static final int RING_BUFFER_SIZE = 1024 * 64;
+
+    /** Disruptor instance. */
+    private static final Disruptor<EventHolder> DISRUPTOR;
+
+    /** Ring buffer used for publishing events. */
+    private static final RingBuffer<EventHolder> RING_BUFFER;
+
+    static {
+        ThreadFactory threadFactory =
+                r -> {
+                    Thread t = new Thread(r, "EventBus-Disruptor");
+                    t.setDaemon(true);
+                    return t;
+                };
+
+        DISRUPTOR =
+                new Disruptor<>(
+                        EventHolder::new,
+                        RING_BUFFER_SIZE,
+                        threadFactory,
+                        ProducerType.MULTI,
+                        new BusySpinWaitStrategy());
+
+        DISRUPTOR.handleEventsWith(
+                (holder, seq, endOfBatch) -> {
+                    if (holder.event != null) {
+                        dispatchEvent(holder.event);
+                        holder.event = null;
+                    }
+                });
+
+        DISRUPTOR.start();
+        RING_BUFFER = DISRUPTOR.getRingBuffer();
+    }
+
+    /** Prevent instantiation. */
     private GlobalEventBus() {}
 
-    public static EventBus get() {
-        return INSTANCE;
+    /** Wrapper used inside the ring buffer. */
+    private static class EventHolder {
+        EventType event;
     }
 
-    @Override
-    public void subscribe(Subscriber<? extends EventType> listener) {
-        INSTANCE.subscribe(listener);
+    // ------------------------------------------------------------------------
+    // Subscription
+    // ------------------------------------------------------------------------
+    public static <T extends EventType> void subscribe(ListenerHandler<T> listener) {
+        logger.debug("Subscribing to {}: {}", listener.getListenerClass().getSimpleName(), listener.getListener().getClass().getSimpleName());
+        LISTENERS.computeIfAbsent(listener.getListenerClass(), _ -> new CopyOnWriteArrayList<>()).add(listener);
     }
 
-    @Override
-    public void unsubscribe(Subscriber<? extends EventType> listener) {
-        INSTANCE.unsubscribe(listener);
+    // TODO
+    public static <T extends UniqueEvent> void subscribeById(
+            Class<T> eventClass, long eventId, Consumer<T> listener) {
+        UUID_LISTENERS
+                .computeIfAbsent(eventClass, _ -> new ConcurrentHashMap<>())
+                .put(eventId, listener);
     }
 
-    @Override
-    public <T extends EventType> void post(T event) {
-        INSTANCE.post(event);
+    public static void unsubscribe(ListenerHandler<?> listener) {
+        logger.debug("Unsubscribing from {}: {}", listener.getListenerClass().getSimpleName(), listener.getListener().getClass().getSimpleName());
+        LISTENERS.getOrDefault(listener.getListenerClass(), new CopyOnWriteArrayList<>())
+                .remove(listener);
+        LISTENERS.entrySet().removeIf(entry -> entry.getValue().isEmpty());
     }
 
-    @Override
-    public <T extends EventType> void asyncPost(T event) {
-        INSTANCE.asyncPost(event);
+    // TODO
+    public static <T extends UniqueEvent> void unsubscribeById(
+            Class<T> eventClass, long eventId) {
+        Map<Long, Consumer<? extends UniqueEvent>> map = UUID_LISTENERS.get(eventClass);
+        if (map != null) map.remove(eventId);
     }
 
-    @Override
-    public void shutdown() {
-        INSTANCE.shutdown();
+    // ------------------------------------------------------------------------
+    // Posting
+    // ------------------------------------------------------------------------
+    public static <T extends EventType> void post(T event) {
+        dispatchEvent(event); // synchronous
     }
 
-    @Override
-    public void reset() {
-        INSTANCE.reset();
+    public static <T extends EventType> void postAsync(T event) {
+        long seq = RING_BUFFER.next();
+        try {
+            EventHolder holder = RING_BUFFER.get(seq);
+            holder.event = event;
+        } finally {
+            RING_BUFFER.publish(seq);
+        }
     }
 
+    @SuppressWarnings("unchecked")
+    private static <T extends EventType> void callListener(ListenerHandler<?> raw, EventType event) {
+        ListenerHandler<T> handler = (ListenerHandler<T>) raw;
+        Consumer<T> listener = handler.getListener();
+
+        T casted = (T) event;
+
+        listener.accept(casted);
+    }
+
+    @SuppressWarnings("unchecked")
+    private static void dispatchEvent(EventType event) {
+        Class<?> clazz = event.getClass();
+
+        logger.debug("Triggered event: {}", event.getClass().getSimpleName());
+
+        CopyOnWriteArrayList<ListenerHandler<?>> classListeners = LISTENERS.get(clazz);
+        if (classListeners != null) {
+            for (ListenerHandler<?> listener : classListeners) {
+                try {
+                    callListener(listener, event);
+                } catch (Throwable e) {
+                    e.printStackTrace();
+                }
+            }
+        }
+
+        CopyOnWriteArrayList<ListenerHandler<?>> genericListeners = LISTENERS.get(Object.class);
+        if (genericListeners != null) {
+            for (ListenerHandler<?> listener : genericListeners) {
+                try {
+                    callListener(listener, event);
+                } catch (Throwable e) {
+                     e.printStackTrace();
+                }
+            }
+        }
+
+        if (event instanceof UniqueEvent snowflakeEvent) {
+            Map<Long, Consumer<? extends UniqueEvent>> map = UUID_LISTENERS.get(clazz);
+            if (map != null) {
+                Consumer<UniqueEvent> listener =
+                        (Consumer<UniqueEvent>) map.remove(snowflakeEvent.getIdentifier());
+                if (listener != null) {
+                    try {
+                        listener.accept(snowflakeEvent);
+                    } catch (Throwable ignored) {
+                    }
+                }
+            }
+        }
+    }
+
+    // ------------------------------------------------------------------------
+    // Lifecycle
+    // ------------------------------------------------------------------------
+    public static void shutdown() {
+        DISRUPTOR.shutdown();
+        LISTENERS.clear();
+        UUID_LISTENERS.clear();
+    }
+
+    public static void reset() {
+        LISTENERS.clear();
+        UUID_LISTENERS.clear();
+    }
+
+    public static Map<Class<?>, CopyOnWriteArrayList<ListenerHandler<?>>> getAllListeners() {
+        return LISTENERS;
+    }
 }

framework/src/main/java/org/toop/framework/eventbus/ListenerHandler.java

@@ -0,0 +1,48 @@
+package org.toop.framework.eventbus;
+
+import org.toop.framework.SnowflakeGenerator;
+import org.toop.framework.eventbus.events.EventType;
+
+import java.util.function.Consumer;
+
+public class ListenerHandler<T extends EventType> {
+    private final long id;
+    private final String name;
+    private final Class<T> clazz;
+    private final Consumer<T> listener;
+
+    public ListenerHandler(long id, String name, Class<T> clazz, Consumer<T> listener) {
+        this.id = id;
+        this.name = name;
+        this.clazz = clazz;
+        this.listener = listener;
+    }
+
+    public ListenerHandler(String name, Class<T> clazz, Consumer<T> listener) {
+        this(SnowflakeGenerator.nextId(), name, clazz, listener);
+    }
+
+    public ListenerHandler(long id, Class<T> clazz, Consumer<T> listener) {
+        this(id, String.valueOf(id), clazz, listener);
+    }
+
+    public ListenerHandler(Class<T> clazz, Consumer<T> listener) {
+        this(SnowflakeGenerator.nextId(), clazz, listener);
+    }
+
+    public long getId() {
+        return id;
+    }
+
+    public Consumer<T> getListener() {
+        return listener;
+    }
+
+    public Class<T> getListenerClass() {
+        return clazz;
+    }
+
+    public String getName() {
+        return name;
+    }
+}

framework/src/main/java/org/toop/framework/eventbus/bus/AsyncEventBus.java

@@ -1,7 +0,0 @@
-package org.toop.framework.eventbus.bus;
-
-import org.toop.framework.eventbus.events.EventType;
-
-public interface AsyncEventBus extends EventBus {
-    <T extends EventType> void asyncPost(T event);
-}

framework/src/main/java/org/toop/framework/eventbus/bus/DefaultEventBus.java

@@ -1,63 +0,0 @@
-package org.toop.framework.eventbus.bus;
-
-import org.apache.logging.log4j.Logger;
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.store.SubscriberStore;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
-import java.util.function.Consumer;
-
-public class DefaultEventBus implements AsyncEventBus {
-    private final Logger logger;
-    private final SubscriberStore eventsHolder;
-
-    private final ExecutorService asyncExecutor = Executors.newCachedThreadPool();
-
-    public DefaultEventBus(Logger logger, SubscriberStore eventsHolder) {
-        this.logger = logger;
-        this.eventsHolder = eventsHolder;
-    }
-
-    @Override
-    public void subscribe(Subscriber<? extends EventType> subscriber) {
-        eventsHolder.add(subscriber);
-    }
-
-    @Override
-    public void unsubscribe(Subscriber<? extends EventType> subscriber) {
-        eventsHolder.remove(subscriber);
-    }
-
-    @Override
-    @SuppressWarnings("unchecked")
-    public <T extends EventType> void post(T event) {
-        Class<T> eventType = (Class<T>) event.getClass();
-        var subs = eventsHolder.get(eventType);
-        if (subs != null) {
-            for (Subscriber<?> subscriber : subs) {
-                Class<T> eventClass = (Class<T>) subscriber.event();
-                Consumer<EventType> action = (Consumer<EventType>) subscriber.handler();
-
-                action.accept(eventClass.cast(event));
-            }
-        }
-    }
-
-    @Override
-    public <T extends EventType> void asyncPost(T event) {
-        asyncExecutor.submit(() -> post(event));
-    }
-
-    @Override
-    public void shutdown() {
-        eventsHolder.reset();
-    }
-
-    @Override
-    public void reset() {
-        eventsHolder.reset();
-    }
-
-}

framework/src/main/java/org/toop/framework/eventbus/bus/DisruptorEventBus.java

@@ -1,117 +0,0 @@
-package org.toop.framework.eventbus.bus;
-
-import com.lmax.disruptor.BusySpinWaitStrategy;
-import com.lmax.disruptor.RingBuffer;
-import com.lmax.disruptor.dsl.Disruptor;
-import com.lmax.disruptor.dsl.ProducerType;
-import org.apache.logging.log4j.Logger;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.store.SubscriberStore;
-
-import java.util.concurrent.ThreadFactory;
-import java.util.function.Consumer;
-
-public class DisruptorEventBus implements EventBus {
-    /** Wrapper used inside the ring buffer. */
-    private static class EventHolder<T> {
-        T event;
-    }
-
-    private final Logger logger;
-    private final SubscriberStore eventsHolder;
-
-    private final Disruptor<EventHolder<? extends EventType>> disruptor;
-    private final RingBuffer<EventHolder<? extends EventType>> ringBuffer;
-
-    public DisruptorEventBus(Logger logger, SubscriberStore eventsHolder) {
-        this.logger = logger;
-        this.eventsHolder = eventsHolder;
-
-        ThreadFactory threadFactory =
-                r -> {
-                    Thread t = new Thread(r, "EventBus-Disruptor");
-                    t.setDaemon(true);
-                    return t;
-                };
-
-        disruptor = getEventHolderDisruptor(threadFactory);
-
-        disruptor.start();
-        this.ringBuffer = disruptor.getRingBuffer();
-    }
-
-    private Disruptor<EventHolder<? extends EventType>> getEventHolderDisruptor(ThreadFactory threadFactory) {
-        int RING_BUFFER_SIZE = 1024 * 64;
-        Disruptor<EventHolder<? extends EventType>> disruptor = new Disruptor<>(
-                EventHolder::new,
-                RING_BUFFER_SIZE,
-                threadFactory,
-                ProducerType.MULTI,
-                new BusySpinWaitStrategy());
-
-        disruptor.handleEventsWith(
-                (holder, _, _) -> {
-                    if (holder.event != null) {
-                        dispatchEvent(holder.event);
-                        holder.event = null;
-                    }
-                });
-        return disruptor;
-    }
-
-    @Override
-    public void subscribe(Subscriber<? extends EventType> listener) {
-        eventsHolder.add(listener);
-    }
-
-    @Override
-    public void unsubscribe(Subscriber<? extends EventType> listener) {
-        eventsHolder.remove(listener);
-    }
-
-    @Override
-    public <T extends EventType> void post(T event) {
-        long seq = ringBuffer.next();
-        try {
-            @SuppressWarnings("unchecked")
-            EventHolder<T> holder = (EventHolder<T>) ringBuffer.get(seq);
-            holder.event = event;
-        } finally {
-            ringBuffer.publish(seq);
-        }
-    }
-
-    @Override
-    public void shutdown() {
-        disruptor.shutdown();
-        eventsHolder.reset();
-    }
-
-    @Override
-    public void reset() {
-        eventsHolder.reset();
-    }
-
-    private <T extends EventType> void dispatchEvent(T event) {
-        var classListeners = eventsHolder.get(event.getClass());
-        if (classListeners != null) {
-            for (Subscriber<?> listener : classListeners) {
-                try {
-                    callListener(listener, event);
-                } catch (Throwable e) {
-                    logger.warn("Exception while handling event: {}", event, e);
-                }
-            }
-        }
-    }
-
-
-    @SuppressWarnings("unchecked")
-    private <T> void callListener(Subscriber<?> subscriber, T event) {
-        Class<T> eventClass = (Class<T>) subscriber.event();
-        Consumer<EventType> action = (Consumer<EventType>) subscriber.handler();
-
-        action.accept((EventType) eventClass.cast(event));
-    }
-}

framework/src/main/java/org/toop/framework/eventbus/bus/EventBus.java

@@ -1,12 +0,0 @@
-package org.toop.framework.eventbus.bus;
-
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-
-public interface EventBus {
-    void subscribe(Subscriber<? extends EventType> subscriber);
-    void unsubscribe(Subscriber<? extends EventType> subscriber);
-    <T extends EventType> void post(T event);
-    void shutdown();
-    void reset();
-}

framework/src/main/java/org/toop/framework/eventbus/store/AsyncSubscriberStore.java

@@ -1,47 +0,0 @@
-package org.toop.framework.eventbus.store;
-
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-
-import java.util.concurrent.ConcurrentHashMap;
-import java.util.concurrent.ConcurrentLinkedQueue;
-
-public class AsyncSubscriberStore implements SubscriberStore {
-    private final ConcurrentHashMap<Class<? extends EventType>, ConcurrentLinkedQueue<Subscriber<? extends EventType>>> queues = new ConcurrentHashMap<>();
-    private final ConcurrentHashMap<Class<? extends EventType>, Subscriber<? extends EventType>[]> snapshots = new ConcurrentHashMap<>();
-
-    @Override
-    public void add(Subscriber<? extends EventType> sub) {
-        queues.computeIfAbsent(sub.event(), _ -> new ConcurrentLinkedQueue<>()).add(sub);
-        rebuildSnapshot(sub.event());
-    }
-
-    @Override
-    public void remove(Subscriber<? extends EventType> sub) {
-        ConcurrentLinkedQueue<Subscriber<?>> queue = queues.get(sub.event());
-        if (queue != null) {
-            queue.remove(sub);
-            rebuildSnapshot(sub.event());
-        }
-    }
-
-    @Override
-    public Subscriber<? extends EventType>[] get(Class<? extends EventType> event) {
-        return snapshots.getOrDefault(event, new Subscriber<?>[0]);
-    }
-
-    @Override
-    public void reset() {
-        queues.clear();
-        snapshots.clear();
-    }
-
-    private void rebuildSnapshot(Class<? extends EventType> event) {
-        ConcurrentLinkedQueue<Subscriber<?>> queue = queues.get(event);
-        if (queue != null) {
-            snapshots.put(event, queue.toArray(new Subscriber<?>[0]));
-        } else {
-            snapshots.put(event, new Subscriber<?>[0]);
-        }
-    }
-}

framework/src/main/java/org/toop/framework/eventbus/store/DefaultSubscriberStore.java

@@ -1,73 +0,0 @@
-package org.toop.framework.eventbus.store;
-
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.subscriber.NamedSubscriber;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-
-import java.util.concurrent.ConcurrentHashMap;
-
-public class DefaultSubscriberStore implements SubscriberStore {
-
-    private static final Subscriber<? extends EventType>[] EMPTY = new Subscriber<?>[0];
-
-    private final ConcurrentHashMap<Class<? extends EventType>, Subscriber<? extends EventType>[]>
-            listeners = new ConcurrentHashMap<>();
-
-    @Override
-    public void add(Subscriber<? extends EventType> sub) {
-        listeners.compute(sub.event(), (_, arr) -> {
-            if (arr == null || arr.length == 0) {
-                return new Subscriber<?>[]{sub};
-            }
-
-            int len = arr.length;
-            Subscriber<?>[] newArr = new Subscriber[len + 1];
-            System.arraycopy(arr, 0, newArr, 0, len);
-            newArr[len] = sub;
-            return newArr;
-        });
-    }
-
-    @Override
-    public void remove(Subscriber<? extends EventType> sub) {
-        listeners.computeIfPresent(sub.event(), (_, arr) -> {
-            int len = arr.length;
-
-            if (len == 1) {
-                return arr[0].equals(sub) ? null : arr;
-            }
-
-            int keep = 0;
-            for (Subscriber<?> s : arr) {
-                if (!s.equals(sub)) keep++;
-            }
-
-            if (keep == len) {
-                return arr;
-            }
-            if (keep == 0) {
-                return null;
-            }
-
-            Subscriber<?>[] newArr = new Subscriber[keep];
-            int i = 0;
-            for (Subscriber<?> s : arr) {
-                if (!s.equals(sub)) {
-                    newArr[i++] = s;
-                }
-            }
-
-            return newArr;
-        });
-    }
-
-    @Override
-    public Subscriber<? extends EventType>[] get(Class<? extends EventType> event) {
-        return listeners.getOrDefault(event, EMPTY);
-    }
-
-    @Override
-    public void reset() {
-        listeners.clear();
-    }
-}

framework/src/main/java/org/toop/framework/eventbus/store/SubscriberStore.java

@@ -1,11 +0,0 @@
-package org.toop.framework.eventbus.store;
-
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-
-public interface SubscriberStore {
-    void add(Subscriber<? extends EventType> subscriber);
-    void remove(Subscriber<? extends EventType> subscriber);
-    Subscriber<? extends EventType>[] get(Class<? extends EventType> event);
-    void reset();
-}

framework/src/main/java/org/toop/framework/eventbus/store/SyncSubscriberStore.java

@@ -1,37 +0,0 @@
-package org.toop.framework.eventbus.store;
-
-import org.toop.framework.eventbus.events.EventType;
-import org.toop.framework.eventbus.subscriber.Subscriber;
-
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
-
-public class SyncSubscriberStore implements SubscriberStore {
-    private final Map<Class<? extends EventType>, List<Subscriber<? extends EventType>>> LISTENERS = new ConcurrentHashMap<>();
-    private static final Subscriber<? extends EventType>[] EMPTY = new Subscriber<?>[0];
-
-    @Override
-    public void add(Subscriber<? extends EventType> sub) {
-        LISTENERS.computeIfAbsent(sub.event(), _ -> new ArrayList<>()).add(sub);
-    }
-
-    @Override
-    public void remove(Subscriber<? extends EventType> sub) {
-        LISTENERS.getOrDefault(sub.event(), new ArrayList<>()).remove(sub);
-        LISTENERS.entrySet().removeIf(entry -> entry.getValue().isEmpty());
-    }
-
-    @Override
-    public Subscriber<? extends EventType>[] get(Class<? extends EventType> event) {
-        List<Subscriber<? extends EventType>> list = LISTENERS.get(event);
-        if (list == null || list.isEmpty()) return EMPTY;
-        return list.toArray(EMPTY);
-    }
-
-    @Override
-    public void reset() {
-        LISTENERS.clear();
-    }
-}

framework/src/main/java/org/toop/framework/eventbus/subscriber/DefaultNamedSubscriber.java

@@ -1,8 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-import org.toop.framework.eventbus.events.EventType;
-
-import java.util.function.Consumer;
-
-public record DefaultNamedSubscriber<K extends EventType>(String id, Class<K> event, Consumer<K> handler)
-        implements NamedSubscriber<K> {}

framework/src/main/java/org/toop/framework/eventbus/subscriber/DefaultSubscriber.java

@@ -1,8 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-import org.toop.framework.eventbus.events.EventType;
-
-import java.util.function.Consumer;
-
-public record DefaultSubscriber<K extends EventType>(Class<K> event, Consumer<K> handler) implements Subscriber<K> {}
-

framework/src/main/java/org/toop/framework/eventbus/subscriber/HasId.java

@@ -1,5 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-public interface HasId<ID> {
-    ID id();
-}

framework/src/main/java/org/toop/framework/eventbus/subscriber/IdSubscriber.java

@@ -1,5 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-import org.toop.framework.eventbus.events.EventType;
-
-public interface IdSubscriber<K extends EventType> extends Subscriber<K>, HasId<Long> {}

framework/src/main/java/org/toop/framework/eventbus/subscriber/LongIdSubscriber.java

@@ -1,8 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-import org.toop.framework.eventbus.events.EventType;
-
-import java.util.function.Consumer;
-
-public record LongIdSubscriber<K extends EventType>(Long id, Class<K> event, Consumer<K> handler)
-        implements IdSubscriber<K> {}

framework/src/main/java/org/toop/framework/eventbus/subscriber/NamedSubscriber.java

@@ -1,5 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-import org.toop.framework.eventbus.events.EventType;
-
-public interface NamedSubscriber<K extends EventType> extends Subscriber<K>, HasId<String> {}

framework/src/main/java/org/toop/framework/eventbus/subscriber/Subscriber.java

@@ -1,10 +0,0 @@
-package org.toop.framework.eventbus.subscriber;
-
-import org.toop.framework.eventbus.events.EventType;
-
-import java.util.function.Consumer;
-
-public interface Subscriber<K extends EventType> {
-    Class<K> event();
-    Consumer<K> handler();
-}

framework/src/main/java/org/toop/framework/game/gameThreads/ServerThreadBehaviour.java

@@ -1,109 +0,0 @@
-package org.toop.framework.game.gameThreads;
-
-import org.toop.framework.gameFramework.GameState;
-import org.toop.framework.gameFramework.model.game.PlayResult;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.game.threadBehaviour.AbstractThreadBehaviour;
-import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.utils.ImmutablePair;
-import org.toop.framework.utils.Pair;
-
-import java.time.Duration;
-import java.util.concurrent.*;
-import java.util.function.Consumer;
-
-public class ServerThreadBehaviour extends AbstractThreadBehaviour implements Runnable {
-    private final Consumer<ImmutablePair<String, Integer>> onPlayerMove;
-    private final Consumer<Pair<GameState, Integer>> onGameEnd;
-
-    private final ExecutorService moveExecutor = Executors.newSingleThreadExecutor();
-
-    private final Duration timeOut;
-    /**
-     * Creates a new base behaviour for the specified game.
-     *
-     * @param game the turn-based game to control
-     */
-    public ServerThreadBehaviour(
-            TurnBasedGame game,
-            Consumer<ImmutablePair<String,
-                    Integer>> onPlayerMove,
-            Consumer<Pair<GameState, Integer>> onGameEnd,
-            Duration timeOut
-    ) {
-        this.onPlayerMove = onPlayerMove;
-        this.onGameEnd = onGameEnd;
-        this.timeOut = timeOut;
-        super(game);
-    }
-
-    private void notifyPlayerMove(ImmutablePair<String, Integer> pair) {
-        if (onPlayerMove != null) {
-            onPlayerMove.accept(pair);
-        }
-    }
-
-    private void notifyGameEnd(ImmutablePair<GameState, Integer> pair) {
-        if (onGameEnd != null) {
-            onGameEnd.accept(pair);
-        }
-    }
-
-    /** Starts the game loop in a new thread. */
-    @Override
-    public void start() {
-        if (isRunning.compareAndSet(false, true)) {
-            new Thread(this).start();
-        }
-    }
-
-    /** Stops the game loop after the current iteration. */
-    @Override
-    public void stop() {
-        isRunning.set(false);
-    }
-
-    @Override
-    public void run() {
-        while (isRunning.get()) {
-            Player currentPlayer = game.getPlayer(game.getCurrentTurn());
-
-            Future<Long> move = moveExecutor.submit(() -> currentPlayer.getMove(game.deepCopy()));
-
-            PlayResult result;
-            try {
-                long moveResult = move.get(timeOut.toMillis(), TimeUnit.MILLISECONDS);
-                result = game.play(moveResult);
-
-                GameState state = result.state();
-                notifyPlayerMove(new ImmutablePair<>(currentPlayer.getName(), Long.numberOfTrailingZeros(moveResult)));
-
-                switch (state) {
-                    case WIN, DRAW -> {
-                        isRunning.set(false);
-                        moveExecutor.shutdown();
-                        notifyGameEnd(new ImmutablePair<>(state, game.getWinner()));
-                    }
-                    case NORMAL, TURN_SKIPPED -> { /* continue normally */ }
-                    default -> {
-                        logger.error("Unexpected state {}", state);
-                        isRunning.set(false);
-                        moveExecutor.shutdown();
-                        throw new RuntimeException("Unknown state: " + state);
-                    }
-                }
-
-            } catch (InterruptedException | ExecutionException e) {
-                isRunning.set(false);
-                notifyGameEnd(new ImmutablePair<>(GameState.DRAW, 0));
-                moveExecutor.shutdown();
-                return;
-            } catch (TimeoutException e) {
-                isRunning.set(false);
-                notifyGameEnd(new ImmutablePair<>(GameState.WIN, 1+game.getWinner()%2));
-                moveExecutor.shutdown();
-                return;
-            }
-        }
-    }
-}

framework/src/main/java/org/toop/framework/game/games/reversi/BitboardReversi.java

@@ -1,371 +0,0 @@
-package org.toop.framework.game.games.reversi;
-
-import org.toop.framework.game.BitboardGame;
-import org.toop.framework.gameFramework.GameState;
-import org.toop.framework.gameFramework.model.game.PlayResult;
-import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.BitboardGame;
-
-public class BitboardReversi extends BitboardGame {
-
-    public record Score(int black, int white) {}
-
-	private final long notAFile = 0xfefefefefefefefeL;
-	private final long notHFile = 0x7f7f7f7f7f7f7f7fL;
-
-	public BitboardReversi() {
-		super(8, 8, 2);
-	}
-
-	@Override
-	public void init(Player[] players) {
-		super.init(players);
-        // Black (player 0)
-        setPlayerBitboard(0, (1L << (3 + 4 * 8)) | (1L << (4 + 3 * 8)));
-
-        // White (player 1)
-        setPlayerBitboard(1, (1L << (3 + 3 * 8)) | (1L << (4 + 4 * 8)));
-	}
-
-    public BitboardReversi(BitboardReversi other) {
-        super(other);
-    }
-
-	public long getLegalMoves() {
-		long legalMoves = 0L;
-
-		final long player = getPlayerBitboard(getCurrentPlayerIndex());
-		final long opponent = getPlayerBitboard(getNextPlayer());
-
-		final long empty = ~(player | opponent);
-
-		long mask;
-		long direction;
-
-		// north
-		mask = opponent;
-		direction = (player << 8) & mask;
-
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		legalMoves |= (direction << 8) & empty;
-
-		// south
-		mask = opponent;
-		direction = (player >>> 8) & mask;
-
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		legalMoves |= (direction >>> 8) & empty;
-
-		// east
-		mask = opponent & notAFile;
-		direction = (player << 1) & mask;
-
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		legalMoves |= (direction << 1) & empty & notAFile;
-
-		// west
-		mask = opponent & notHFile;
-		direction = (player >>> 1) & mask;
-
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		legalMoves |= (direction >>> 1) & empty & notHFile;
-
-		// north-east
-		mask = opponent & notAFile;
-		direction = (player << 9) & mask;
-
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		legalMoves |= (direction << 9) & empty & notAFile;
-
-		// north-west
-		mask = opponent & notHFile;
-		direction = (player << 7) & mask;
-
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		legalMoves |= (direction << 7) & empty & notHFile;
-
-		// south-east
-		mask = opponent & notAFile;
-		direction = (player >>> 7) & mask;
-
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		legalMoves |= (direction >>> 7) & empty & notAFile;
-
-		// south-west
-		mask = opponent & notHFile;
-		direction = (player >>> 9) & mask;
-
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		legalMoves |= (direction >>> 9) & empty & notHFile;
-
-		return legalMoves;
-	}
-
-	public long getFlips(long move) {
-		long flips = 0L;
-
-		final long player = getPlayerBitboard(getCurrentPlayerIndex());
-		final long opponent = getPlayerBitboard(getNextPlayer());
-
-		long mask;
-		long direction;
-
-		// north
-		mask = opponent;
-		direction = (move << 8) & mask;
-
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-		direction |= (direction << 8) & mask;
-
-		if (((direction << 8) & player) != 0) {
-			flips |= direction;
-		}
-
-		// south
-		mask = opponent;
-		direction = (move >>> 8) & mask;
-
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-		direction |= (direction >>> 8) & mask;
-
-		if (((direction >>> 8) & player) != 0) {
-			flips |= direction;
-		}
-
-		// east
-		mask = opponent & notAFile;
-		direction = (move << 1) & mask;
-
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-		direction |= (direction << 1) & mask;
-
-		if (((direction << 1) & player & notAFile) != 0) {
-			flips |= direction;
-		}
-
-		// west
-		mask = opponent & notHFile;
-		direction = (move >>> 1) & mask;
-
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-		direction |= (direction >>> 1) & mask;
-
-		if (((direction >>> 1) & player & notHFile) != 0) {
-			flips |= direction;
-		}
-
-		// north-east
-		mask = opponent & notAFile;
-		direction = (move << 9) & mask;
-
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-		direction |= (direction << 9) & mask;
-
-		if (((direction << 9) & player & notAFile) != 0) {
-			flips |= direction;
-		}
-
-		// north-west
-		mask = opponent & notHFile;
-		direction = (move << 7) & mask;
-
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-		direction |= (direction << 7) & mask;
-
-		if (((direction << 7) & player & notHFile) != 0) {
-			flips |= direction;
-		}
-
-		// south-east
-		mask = opponent & notAFile;
-		direction = (move >>> 7) & mask;
-
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-		direction |= (direction >>> 7) & mask;
-
-		if (((direction >>> 7) & player & notAFile) != 0) {
-			flips |= direction;
-		}
-
-		// south-west
-		mask = opponent & notHFile;
-		direction = (move >>> 9) & mask;
-
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-		direction |= (direction >>> 9) & mask;
-
-		if (((direction >>> 9) & player & notHFile) != 0) {
-			flips |= direction;
-		}
-
-		return flips;
-	}
-
-    @Override
-    public BitboardReversi deepCopy() {return new BitboardReversi(this);}
-
-	public PlayResult play(long move) {
-		final long flips = getFlips(move);
-
-		long player = getPlayerBitboard(getCurrentPlayerIndex());
-		long opponent = getPlayerBitboard(getNextPlayer());
-
-		player |= move | flips;
-		opponent &= ~flips;
-
-		setPlayerBitboard(getCurrentPlayerIndex(), player);
-		setPlayerBitboard(getNextPlayer(), opponent);
-
-		nextTurn();
-
-		final long nextLegalMoves = getLegalMoves();
-
-		if (nextLegalMoves == 0) {
-			nextTurn();
-
-			final long skippedLegalMoves = getLegalMoves();
-
-			if (skippedLegalMoves == 0) {
-				int winner = getWinner();
-
-				if (winner == -1) {
-					state = new PlayResult(GameState.DRAW, -1);
-					return state;
-				}
-
-				state = new PlayResult(GameState.WIN, winner);
-				return state;
-			}
-
-			state = new PlayResult(GameState.TURN_SKIPPED, getCurrentPlayerIndex());
-			return state;
-		}
-
-		state = new PlayResult(GameState.NORMAL, getCurrentPlayerIndex());
-		return state;
-	}
-
-	public Score getScore() {
-		return new Score(
-			Long.bitCount(getPlayerBitboard(0)),
-			Long.bitCount(getPlayerBitboard(1))
-		);
-	}
-
-    public int getWinner(){
-        final long black = getPlayerBitboard(0);
-        final long white = getPlayerBitboard(1);
-
-        final int blackCount = Long.bitCount(black);
-        final int whiteCount = Long.bitCount(white);
-
-        if (blackCount == whiteCount){
-            return -1;
-        }
-        else if (blackCount > whiteCount){
-            return 0;
-        }
-        else {
-            return 1;
-        }
-    }
-
-	@Override
-	public float rateMove(long move) {
-		final long corners = 0x8100000000000081L;
-
-		if ((move & corners) != 0L) {
-			return 0.4f;
-		}
-
-		final long xSquares = 0x0042000000004200L;
-
-		if ((move & xSquares) != 0) {
-			return -0.4f;
-		}
-
-		final long cSquares = 0x4281000000008142L;
-
-		if ((move & cSquares) != 0) {
-			return -0.1f;
-		}
-
-		return 0.0f;
-	}
-
-	@Override
-	public long heuristicMove(long legalMoves) {
-		long bestMove = 0L;
-		float bestMoveRate = Float.NEGATIVE_INFINITY;
-
-		while (legalMoves != 0L) {
-			final long move = legalMoves & -legalMoves;
-			final float moveRate = rateMove(move);
-
-			if (moveRate > bestMoveRate) {
-				bestMove = move;
-				bestMoveRate = moveRate;
-			}
-
-			legalMoves &= ~move;
-		}
-
-		return bestMove;
-	}
-}

framework/src/main/java/org/toop/framework/game/players/LocalPlayer.java

@@ -1,108 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.player.AbstractPlayer;
-
-import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.ExecutionException;
-
-/**
- * Represents a local player who provides moves manually.
- *
- * @param <T> the type of turn-based game
- */
-public class LocalPlayer extends AbstractPlayer {
-
-    private CompletableFuture<Long> LastMove = new CompletableFuture<>();
-
-    /**
-     * Creates a new local player with the given name.
-     *
-     * @param name the player's name
-     */
-    public LocalPlayer(String name) {
-        super(name);
-    }
-
-    /**
-     * Creates a copy of another local player.
-     *
-     * @param other the player to copy
-     */
-    public LocalPlayer(LocalPlayer other) {
-        super(other);
-        this.LastMove = other.LastMove;
-    }
-
-    /**
-     * Waits for and returns the player's next legal move.
-     *
-     * @param gameCopy a copy of the current game
-     * @return the chosen move
-     */
-    @Override
-    protected long determineMove(TurnBasedGame gameCopy) {
-        long legalMoves = gameCopy.getLegalMoves();
-        long move;
-
-        try {
-            do {
-                move = getLastMove();
-                IO.println("GETTING MOVE");
-            } while ((legalMoves & move) == 0);
-            return move;
-        } catch (Exception e) {
-            IO.println(e);
-        }
-        return -1;
-    }
-
-    /**
-     * Sets the player's last move.
-     *
-     * @param move the move to set
-     */
-    public void setLastMove(long move) {
-        LastMove.complete(move);
-    }
-
-    /**
-     * Waits for the next move from the player.
-     *
-     * @return the chosen move or 0 if interrupted
-     */
-    private long getLastMove() {
-        LastMove = new CompletableFuture<>(); // Reset the future
-        try {
-            return LastMove.get();
-        } catch (ExecutionException | InterruptedException e) {
-            return 0;
-        }
-    }
-
-    /**
-     * Creates a deep copy of this local player.
-     *
-     * @return a copy of this player
-     */
-    @Override
-    public LocalPlayer deepCopy() {
-        return new LocalPlayer(this);
-    }
-
-    /*public void register() {
-        // Listening to PlayerAttemptedMove
-        new EventFlow().listen(GUIEvents.PlayerAttemptedMove.class, event -> {
-            if (!LastMove.isDone()) {
-                LastMove.complete(event.move()); // complete the future
-            }
-        }, true); // auto-unsubscribe
-    }
-
-    // This blocks until the next move arrives
-    public int take() throws ExecutionException, InterruptedException {
-        int move = LastMove.get(); // blocking
-        LastMove = new CompletableFuture<>(); // reset for next move
-        return move;
-    }*/
-}

framework/src/main/java/org/toop/framework/game/players/OnlinePlayer.java

@@ -1,51 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.player.AbstractPlayer;
-import org.toop.framework.gameFramework.model.player.Player;
-
-/**
- * Represents a player that participates online.
- *
- * @param <T> the type of turn-based game
- */
-public class OnlinePlayer extends AbstractPlayer {
-
-    /**
-     * Creates a new online player with the given name.
-     *
-     * @param name the name of the player
-     */
-    public OnlinePlayer(String name) {
-        super(name);
-    }
-
-    /**
-     * Creates a copy of another online player.
-     *
-     * @param other the player to copy
-     */
-    public OnlinePlayer(OnlinePlayer other) {
-        super(other);
-    }
-
-    /**
-     * {@inheritDoc}
-     * <p>
-     * This method is not supported for online players.
-     *
-     * @throws UnsupportedOperationException always
-     */
-    @Override
-    protected long determineMove(TurnBasedGame gameCopy) {
-        throw new UnsupportedOperationException("An online player does not support determining move");
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public Player deepCopy() {
-        return new OnlinePlayer(this);
-    }
-}

framework/src/main/java/org/toop/framework/game/players/ServerPlayer.java

@@ -1,36 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.player.AbstractPlayer;
-import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.networking.server.client.NettyClient;
-
-import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.ExecutionException;
-
-public class ServerPlayer extends AbstractPlayer {
-    private NettyClient client;
-    private CompletableFuture<Long> lastMove;
-
-    public ServerPlayer(NettyClient client) {
-        super(client.name());
-        this.client = client;
-    }
-
-    public void setMove(long move) {
-        lastMove.complete(move);
-    }
-
-    @Override
-    public Player deepCopy() {
-        return null;
-    }
-
-    @Override
-    public long determineMove(TurnBasedGame game) {
-        lastMove = new CompletableFuture<>();
-
-        client.send("SVR GAME YOURTURN {TURNMESSAGE: \"<bericht voor deze beurt>\"}");
-        return lastMove.join();
-    }
-}

framework/src/main/java/org/toop/framework/gameFramework/GameState.java

@@ -4,13 +4,13 @@ package org.toop.framework.gameFramework;
  * Represents the current state of a turn-based game.
  */
 public enum GameState {
-    /** OnlineTurnBasedGame is ongoing and no special condition applies. */
+    /** Game is ongoing and no special condition applies. */
     NORMAL,
 
-    /** OnlineTurnBasedGame ended in a draw. */
+    /** Game ended in a draw. */
     DRAW,
 
-    /** OnlineTurnBasedGame ended with a win for a player. */
+    /** Game ended with a win for a player. */
     WIN,
 
     /** Next player's turn was skipped. */

framework/src/main/java/org/toop/framework/gameFramework/controller/GameController.java

@@ -1,8 +1,10 @@
 package org.toop.framework.gameFramework.controller;
 
-import org.toop.framework.networking.connection.events.NetworkEvents;
+import org.toop.framework.gameFramework.model.game.SupportsOnlinePlay;
+import org.toop.framework.gameFramework.model.game.threadBehaviour.Controllable;
+import org.toop.framework.networking.events.NetworkEvents;
 
-public interface GameController {
+public interface GameController extends Controllable, UpdatesGameUI {
     /** Called when it is this player's turn to make a move. */
     void onYourTurn(NetworkEvents.YourTurnResponse event);
 
@@ -12,12 +14,5 @@ public interface GameController {
     /** Called when the game has finished, with the final result. */
     void gameFinished(NetworkEvents.GameResultResponse event);
 
-    void start();
-
-    void stop();
-
-    /** Called to refresh or update the game UI. */
-    void updateUI();
-
     void sendMove(long clientId, long move);
 }

framework/src/main/java/org/toop/framework/gameFramework/controller/UpdatesGameUI.java

@@ -0,0 +1,10 @@
+package org.toop.framework.gameFramework.controller;
+
+/**
+ * Interface for classes that can trigger a UI update.
+ */
+public interface UpdatesGameUI {
+
+    /** Called to refresh or update the game UI. */
+    void updateUI();
+}

framework/src/main/java/org/toop/framework/gameFramework/model/game/BoardProvider.java

@@ -0,0 +1,5 @@
+package org.toop.framework.gameFramework.model.game;
+
+public interface BoardProvider {
+    long[] getBoard();
+}

framework/src/main/java/org/toop/framework/gameFramework/model/game/Playable.java

@@ -0,0 +1,24 @@
+package org.toop.framework.gameFramework.model.game;
+
+import org.toop.framework.gameFramework.GameState;
+
+/**
+ * Interface for turn-based games that can be played and queried for legal moves.
+ */
+public interface Playable {
+
+    /**
+     * Returns the moves that are currently valid in the game.
+     *
+     * @return an array of integers representing legal moves
+     */
+    long getLegalMoves();
+
+    /**
+     * Plays the given move and returns the resulting game state.
+     *
+     * @param move the move to apply
+     * @return the {@link GameState} and additional info after the move
+     */
+    PlayResult play(long move);
+}

framework/src/main/java/org/toop/framework/gameFramework/model/game/PlayerProvider.java

@@ -0,0 +1,7 @@
+package org.toop.framework.gameFramework.model.game;
+
+import org.toop.framework.gameFramework.model.player.Player;
+
+public interface PlayerProvider<T extends TurnBasedGame<T>> {
+    Player<T> getPlayer(int index);
+}

framework/src/main/java/org/toop/framework/gameFramework/model/game/SupportsOnlinePlay.java

@@ -1,4 +1,6 @@
-package org.toop.framework.gameFramework.model.game.threadBehaviour;
+package org.toop.framework.gameFramework.model.game;
+
+import org.toop.framework.networking.events.NetworkEvents;
 
 /**
  * Interface for games that support online multiplayer play.

framework/src/main/java/org/toop/framework/gameFramework/model/game/TurnBasedGame.java

@@ -1,19 +1,7 @@
 package org.toop.framework.gameFramework.model.game;
 
-import org.toop.framework.gameFramework.model.player.Player;
-
-public interface TurnBasedGame extends DeepCopyable<TurnBasedGame> {
-    void init(Player[] players);
-    long[] getBoard();
+public interface TurnBasedGame<T extends TurnBasedGame<T>> extends Playable, DeepCopyable<T>, PlayerProvider<T>, BoardProvider {
     int getCurrentTurn();
     int getPlayerCount();
-    Player getPlayer(int index);
     int getWinner();
-    long getLegalMoves();
-    PlayResult play(long move);
-	PlayResult getState();
-	boolean isTerminal();
-
-	float rateMove(long move);
-	long heuristicMove(long legalMoves);
 }

framework/src/main/java/org/toop/framework/gameFramework/model/game/threadBehaviour/AbstractThreadBehaviour.java

@@ -3,7 +3,9 @@ package org.toop.framework.gameFramework.model.game.threadBehaviour;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.framework.gameFramework.LongPairConsumer;
+import org.toop.framework.gameFramework.controller.GameController;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.framework.gameFramework.model.player.Player;
 
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.function.Consumer;
@@ -15,14 +17,14 @@ import java.util.function.Consumer;
  * a running flag, a game reference, and a logger.
  * Subclasses implement the actual game-loop logic.
  */
-public abstract class AbstractThreadBehaviour implements ThreadBehaviour {
+public abstract class AbstractThreadBehaviour<T extends TurnBasedGame<T>> implements ThreadBehaviour {
     private LongPairConsumer onSendMove;
     private Runnable onUpdateUI;
     /** Indicates whether the game loop or event processing is active. */
     protected final AtomicBoolean isRunning = new AtomicBoolean();
 
     /** The game instance controlled by this behaviour. */
-    protected final TurnBasedGame game;
+    protected final T game;
 
     /** Logger for the subclass to report errors or debug info. */
     protected final Logger logger = LogManager.getLogger(this.getClass());
@@ -32,7 +34,7 @@ public abstract class AbstractThreadBehaviour implements ThreadBehaviour {
      *
      * @param game the turn-based game to control
      */
-    public AbstractThreadBehaviour(TurnBasedGame game) {
+    public AbstractThreadBehaviour(T game) {
         this.game = game;
     }
 

framework/src/main/java/org/toop/framework/gameFramework/model/game/threadBehaviour/Controllable.java

@@ -0,0 +1,7 @@
+package org.toop.framework.gameFramework.model.game.threadBehaviour;
+
+public interface Controllable {
+    void start();
+
+    void stop();
+}

framework/src/main/java/org/toop/framework/gameFramework/model/game/threadBehaviour/ThreadBehaviour.java

@@ -3,6 +3,8 @@ package org.toop.framework.gameFramework.model.game.threadBehaviour;
 import org.toop.framework.gameFramework.LongPairConsumer;
 import org.toop.framework.gameFramework.controller.GameController;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.framework.gameFramework.model.player.AbstractPlayer;
+import org.toop.framework.gameFramework.model.player.Player;
 
 import java.util.function.Consumer;
 
@@ -11,11 +13,8 @@ import java.util.function.Consumer;
  * <p>
  * Defines how a game's execution is started, stopped, and which player is active.
  */
-public interface ThreadBehaviour {
+public interface ThreadBehaviour extends Controllable {
     void setOnUpdateUI(Runnable onUpdateUI);
     void setOnSendMove(LongPairConsumer onSendMove);
-    void start();
-    void stop();
-
 }
 

framework/src/main/java/org/toop/framework/gameFramework/model/player/AI.java

@@ -3,6 +3,5 @@ package org.toop.framework.gameFramework.model.player;
 import org.toop.framework.gameFramework.model.game.DeepCopyable;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface AI extends DeepCopyable<AI> {
-    long getMove(TurnBasedGame game);
+public interface AI<T extends TurnBasedGame<T>> extends MoveProvider<T>, DeepCopyable<AI<T>> {
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/AbstractAI.java

@@ -12,6 +12,6 @@ import org.toop.framework.gameFramework.model.game.TurnBasedGame;
  *
  * @param <T> the specific type of game this AI can play, extending {@link GameR}
  */
-public abstract class AbstractAI implements AI {
+public abstract class AbstractAI<T extends TurnBasedGame<T>> implements AI<T> {
     // Concrete AI implementations should override findBestMove(T game, int depth)
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/AbstractPlayer.java

@@ -15,17 +15,16 @@ import org.toop.framework.gameFramework.model.game.TurnBasedGame;
  * specific move logic.
  * </p>
  */
-public abstract class AbstractPlayer implements Player {
+public abstract class AbstractPlayer<T extends TurnBasedGame<T>> implements Player<T> {
     private final Logger logger = LogManager.getLogger(this.getClass());
 
     private final String name;
 
     protected AbstractPlayer(String name) {
-        System.out.println("Player " + name + " has been created");
         this.name = name;
     }
 
-    protected AbstractPlayer(AbstractPlayer other) {
+    protected AbstractPlayer(AbstractPlayer<T> other) {
         this.name = other.name;
     }
     /**
@@ -40,29 +39,12 @@ public abstract class AbstractPlayer implements Player {
      * @return an integer representing the chosen move
      * @throws UnsupportedOperationException if the method is not overridden
      */
-    public final long getMove(TurnBasedGame game) {
-        return determineMove(game.deepCopy());
+    public long getMove(T gameCopy) {
+        logger.error("Method getMove not implemented.");
+        throw new UnsupportedOperationException("Not supported yet.");
     }
 
-
-    /**
-     * Determines the player's move using a safe copy of the game.
-     * <p>
-     * This method is called by {@link #getMove(T)} and should contain
-     * the player's strategy for choosing a move.
-     *
-     * @param gameCopy a deep copy of the game
-     * @return the chosen move
-     */
-    protected abstract long determineMove(TurnBasedGame gameCopy);
-
-
-    /**
-     * Returns the player's name.
-     *
-     * @return the name
-     */
-    public String getName() {
+    public String getName(){
         return this.name;
     }
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/MoveProvider.java

@@ -0,0 +1,7 @@
+package org.toop.framework.gameFramework.model.player;
+
+import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+
+public interface MoveProvider<T extends TurnBasedGame<T>> {
+    long getMove(T game);
+}

framework/src/main/java/org/toop/framework/gameFramework/model/player/NameProvider.java

@@ -0,0 +1,5 @@
+package org.toop.framework.gameFramework.model.player;
+
+public interface NameProvider {
+    String getName();
+}

framework/src/main/java/org/toop/framework/gameFramework/model/player/Player.java

@@ -3,7 +3,5 @@ package org.toop.framework.gameFramework.model.player;
 import org.toop.framework.gameFramework.model.game.DeepCopyable;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface Player extends DeepCopyable<Player> {
-    String getName();
-    long getMove(TurnBasedGame game);
+public interface Player<T extends TurnBasedGame<T>> extends NameProvider, MoveProvider<T>, DeepCopyable<Player<T>> {
 }

framework/src/main/java/org/toop/framework/networking/NetworkingClientEventListener.java

@@ -1,23 +1,22 @@
-package org.toop.framework.networking.connection;
+package org.toop.framework.networking;
 
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.framework.SnowflakeGenerator;
 import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.exceptions.ClientNotFoundException;
-import org.toop.framework.networking.connection.interfaces.NetworkingClientManager;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.framework.networking.exceptions.ClientNotFoundException;
+import org.toop.framework.networking.interfaces.NetworkingClientManager;
 
 public class NetworkingClientEventListener {
-    private static final Logger logger = LogManager.getLogger(NetworkingClientEventListener.class);
 
+    private static final Logger logger = LogManager.getLogger(NetworkingClientEventListener.class);
     private final NetworkingClientManager clientManager;
 
     /** Starts a connection manager, to manage, connections. */
-    public NetworkingClientEventListener(EventBus eventBus, NetworkingClientManager clientManager) {
+    public NetworkingClientEventListener(NetworkingClientManager clientManager) {
         this.clientManager = clientManager;
-        new EventFlow(eventBus)
+        new EventFlow()
                 .listen(NetworkEvents.StartClient.class, this::handleStartClient, false)
                 .listen(NetworkEvents.SendCommand.class, this::handleCommand, false)
                 .listen(NetworkEvents.SendLogin.class, this::handleSendLogin, false)
@@ -41,7 +40,6 @@ public class NetworkingClientEventListener {
 
     void handleStartClient(NetworkEvents.StartClient event) {
         long clientId = SnowflakeGenerator.nextId();
-        new EventFlow().addPostEvent(new NetworkEvents.CreatedIdForClient(clientId, event.identifier())).postEvent();
         clientManager.startClient(
                 clientId,
                 event.networkingClient(),

framework/src/main/java/org/toop/framework/networking/NetworkingClientManager.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection;
+package org.toop.framework.networking;
 
 import java.util.*;
 import java.util.concurrent.ConcurrentHashMap;
@@ -8,23 +8,18 @@ import java.util.concurrent.TimeUnit;
 
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.exceptions.ClientNotFoundException;
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
-import org.toop.framework.networking.connection.interfaces.NetworkingClient;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
+import org.toop.framework.eventbus.GlobalEventBus;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.framework.networking.exceptions.ClientNotFoundException;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.interfaces.NetworkingClient;
+import org.toop.framework.networking.types.NetworkingConnector;
 
-public class NetworkingClientManager implements org.toop.framework.networking.connection.interfaces.NetworkingClientManager {
+public class NetworkingClientManager implements org.toop.framework.networking.interfaces.NetworkingClientManager {
     private static final Logger logger = LogManager.getLogger(NetworkingClientManager.class);
-
-    private final EventBus eventBus;
     private final Map<Long, NetworkingClient> networkClients = new ConcurrentHashMap<>();
 
-    public NetworkingClientManager(EventBus eventBus) {
-        this.eventBus = eventBus;
-    }
+    public NetworkingClientManager() {}
 
     private void connectHelper(
             long id,
@@ -33,16 +28,8 @@ public class NetworkingClientManager implements org.toop.framework.networking.co
             Runnable onSuccess,
             Runnable onFailure
     ) {
-
         ScheduledExecutorService scheduler = Executors.newSingleThreadScheduledExecutor();
 
-        EventFlow closeEvent = new EventFlow()
-                .listen(
-                    NetworkEvents.CloseClient.class,
-                    e -> {
-                            if (e.clientId() == id) scheduler.shutdownNow();
-                        }, "close");
-
         Runnable connectTask = new Runnable() {
             int attempts = 0;
 
@@ -59,7 +46,7 @@ public class NetworkingClientManager implements org.toop.framework.networking.co
                     nClient.connect(id, nConnector.host(), nConnector.port());
                     networkClients.put(id, nClient);
                     logger.info("New client started successfully for {}:{}", nConnector.host(), nConnector.port());
-                    eventBus.post(new NetworkEvents.ConnectTry(id, attempts, nConnector.reconnectAttempts(), true));
+                    GlobalEventBus.post(new NetworkEvents.ConnectTry(id, attempts, nConnector.reconnectAttempts(), true));
                     onSuccess.run();
                     scheduler.shutdown();
                 } catch (CouldNotConnectException e) {
@@ -67,17 +54,17 @@ public class NetworkingClientManager implements org.toop.framework.networking.co
                     if (attempts < nConnector.reconnectAttempts()) {
                         logger.warn("Could not connect to {}:{}. Retrying in {} {}",
                                 nConnector.host(), nConnector.port(), nConnector.timeout(), nConnector.timeUnit());
-                        eventBus.post(new NetworkEvents.ConnectTry(id, attempts, nConnector.reconnectAttempts(), false));
+                        GlobalEventBus.post(new NetworkEvents.ConnectTry(id, attempts, nConnector.reconnectAttempts(), false));
                         scheduler.schedule(this, nConnector.timeout(), nConnector.timeUnit());
                     } else {
                         logger.error("Failed to start client for {}:{} after {} attempts", nConnector.host(), nConnector.port(), attempts);
-                        eventBus.post(new NetworkEvents.ConnectTry(id, -1, nConnector.reconnectAttempts(), false));
+                        GlobalEventBus.post(new NetworkEvents.ConnectTry(id, -1, nConnector.reconnectAttempts(), false));
                         onFailure.run();
                         scheduler.shutdown();
                     }
                 } catch (Exception e) {
                     logger.error("Unexpected exception during startClient", e);
-                    eventBus.post(new NetworkEvents.ConnectTry(id, -1, nConnector.reconnectAttempts(), false));
+                    GlobalEventBus.post(new NetworkEvents.ConnectTry(id, -1, nConnector.reconnectAttempts(), false));
                     onFailure.run();
                     scheduler.shutdown();
                 }
@@ -85,8 +72,6 @@ public class NetworkingClientManager implements org.toop.framework.networking.co
         };
 
         scheduler.schedule(connectTask, 0, TimeUnit.MILLISECONDS);
-//
-//        closeEvent.unsubscribe("close");
     }
 
     @Override

framework/src/main/java/org/toop/framework/networking/clients/TournamentNetworkingClient.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.clients;
+package org.toop.framework.networking.clients;
 
 import io.netty.bootstrap.Bootstrap;
 import io.netty.channel.*;
@@ -11,24 +11,17 @@ import io.netty.handler.codec.string.StringEncoder;
 import io.netty.util.CharsetUtil;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
-import org.toop.framework.networking.connection.handlers.NetworkingGameClientHandler;
-import org.toop.framework.networking.connection.interfaces.NetworkingClient;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.handlers.NetworkingGameClientHandler;
+import org.toop.framework.networking.interfaces.NetworkingClient;
 
 import java.net.InetSocketAddress;
 
 public class TournamentNetworkingClient implements NetworkingClient {
     private static final Logger logger = LogManager.getLogger(TournamentNetworkingClient.class);
-
-    private final EventBus eventBus;
     private Channel channel;
-    private long clientId;
 
-    public TournamentNetworkingClient(EventBus eventBus) {
-        this.eventBus = eventBus;
-    }
+    public TournamentNetworkingClient() {}
 
     @Override
     public InetSocketAddress getAddress() {
@@ -37,7 +30,6 @@ public class TournamentNetworkingClient implements NetworkingClient {
 
     @Override
     public void connect(long clientId, String host, int port) throws CouldNotConnectException {
-        this.clientId = clientId;
         try {
             Bootstrap bootstrap = new Bootstrap();
             EventLoopGroup workerGroup = new MultiThreadIoEventLoopGroup(NioIoHandler.newFactory());
@@ -48,7 +40,7 @@ public class TournamentNetworkingClient implements NetworkingClient {
                     new ChannelInitializer<SocketChannel>() {
                         @Override
                         public void initChannel(SocketChannel ch) {
-                            NetworkingGameClientHandler handler = new NetworkingGameClientHandler(eventBus, clientId);
+                            NetworkingGameClientHandler handler = new NetworkingGameClientHandler(clientId);
 
                             ChannelPipeline pipeline = ch.pipeline();
                             pipeline.addLast(new LineBasedFrameDecoder(1024)); // split at \n
@@ -78,7 +70,6 @@ public class TournamentNetworkingClient implements NetworkingClient {
             logger.info("Connection {} sent message: '{}' ", this.channel.remoteAddress(), literalMsg);
         } else {
             logger.warn("Cannot send message: '{}', connection inactive. ", literalMsg);
-            eventBus.post(new NetworkEvents.ClosedConnection(clientId));
         }
     }
 

framework/src/main/java/org/toop/framework/networking/connection/types/ServerMessage.java

@@ -1,3 +0,0 @@
-package org.toop.framework.networking.connection.types;
-
-public record ServerMessage(String message) {}

framework/src/main/java/org/toop/framework/networking/events/NetworkEvents.java

@@ -1,18 +1,17 @@
-package org.toop.framework.networking.connection.events;
+package org.toop.framework.networking.events;
 
 import java.util.*;
 import java.util.concurrent.CompletableFuture;
 
 import org.toop.annotations.AutoResponseResult;
-import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.eventbus.events.*;
-import org.toop.framework.networking.connection.interfaces.NetworkingClient;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
+import org.toop.framework.networking.interfaces.NetworkingClient;
+import org.toop.framework.networking.types.NetworkingConnector;
 
 /**
  * Defines all event types related to the networking subsystem.
  * <p>
- * These events are used in conjunction with the {@link GlobalEventBus}
+ * These events are used in conjunction with the {@link org.toop.framework.eventbus.GlobalEventBus}
  * and {@link org.toop.framework.eventbus.EventFlow} to communicate between components
  * such as networking clients, managers, and listeners.
  * </p>
@@ -65,9 +64,6 @@ public class NetworkEvents extends EventsBase {
     public record GameResultResponse(long clientId, String condition)
             implements GenericEvent {}
 
-    public record TournamentResultResponse(long clientId, String gameType, String[] names, String[] scoreTypes, List<Integer[]> scores)
-            implements GenericEvent {}
-
     /** Indicates that a game move has been processed or received. */
     public record GameMoveResponse(long clientId, String player, String move, String details)
             implements GenericEvent {}
@@ -105,7 +101,7 @@ public class NetworkEvents extends EventsBase {
             implements GenericEvent {}
 
     /** Requests to accept an existing challenge. */
-    public record SendAcceptChallenge(long clientId, long challengeId)
+    public record SendAcceptChallenge(long clientId, int challengeId)
             implements GenericEvent {}
 
     /** Requests to forfeit the current game. */
@@ -170,10 +166,6 @@ public class NetworkEvents extends EventsBase {
             long identifier)
             implements UniqueEvent {}
 
-    public record CreatedIdForClient(long clientId, long identifier) implements ResponseToUniqueEvent {}
-
-    public record ConnectTry(long clientId, int amount, int maxAmount, boolean success) implements GenericEvent {}
-
     /**
      * Response confirming that a client has been successfully started.
      * <p>
@@ -189,6 +181,8 @@ public class NetworkEvents extends EventsBase {
     public record StartClientResponse(long clientId, boolean successful, long identifier)
             implements ResponseToUniqueEvent {}
 
+    public record ConnectTry(long clientId, int amount, int maxAmount, boolean success) implements GenericEvent {}
+
     /**
      * Requests reconnection of an existing client using its previous configuration.
      * <p>
@@ -222,5 +216,4 @@ public class NetworkEvents extends EventsBase {
     /** Response to a {@link ChangeAddress} event, carrying the success result. */
     public record ChangeAddressResponse(boolean successful, long identifier)
             implements ResponseToUniqueEvent {}
-
 }

framework/src/main/java/org/toop/framework/networking/exceptions/ClientNotFoundException.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.exceptions;
+package org.toop.framework.networking.exceptions;
 
 /**
  * Thrown when an operation is attempted on a networking client

framework/src/main/java/org/toop/framework/networking/exceptions/CouldNotConnectException.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.exceptions;
+package org.toop.framework.networking.exceptions;
 
 public class CouldNotConnectException extends RuntimeException {
 

framework/src/main/java/org/toop/framework/networking/exceptions/NetworkingInitializationException.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.exceptions;
+package org.toop.framework.networking.exceptions;
 
 public class NetworkingInitializationException extends RuntimeException {
     public NetworkingInitializationException(String message, Throwable cause) {

framework/src/main/java/org/toop/framework/networking/handlers/NetworkingGameClientHandler.java

@@ -1,27 +1,22 @@
-package org.toop.framework.networking.connection.handlers;
+package org.toop.framework.networking.handlers;
 
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.channel.ChannelInboundHandlerAdapter;
 
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
 import java.util.regex.MatchResult;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
+import org.toop.framework.eventbus.EventFlow;
+import org.toop.framework.networking.events.NetworkEvents;
 
 public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
     private static final Logger logger = LogManager.getLogger(NetworkingGameClientHandler.class);
 
-    private final EventBus eventBus;
     private final long connectionId;
 
-    public NetworkingGameClientHandler(EventBus eventBus, long connectionId) {
-        this.eventBus = eventBus;
+    public NetworkingGameClientHandler(long connectionId) {
         this.connectionId = connectionId;
     }
 
@@ -45,7 +40,9 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                     "Received SVR message from server-{}, data: {}",
                     ctx.channel().remoteAddress(),
                     msg);
-            eventBus.post(new NetworkEvents.ServerResponse(this.connectionId));
+            new EventFlow()
+                    .addPostEvent(new NetworkEvents.ServerResponse(this.connectionId))
+                    .asyncPostEvent();
             parseServerReturn(rec);
             return;
         }
@@ -97,9 +94,6 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                     case "HELP":
                         helpHandler(recSrvRemoved);
                         return;
-                    case "RESULTS":
-                        resultsHandler(recSrvRemoved);
-                        return;
                     default:
                         // return
                 }
@@ -109,66 +103,6 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
         }
     }
 
-    private static String extract(String input, String key) {
-        Pattern p = Pattern.compile(
-                key + "\\s*:\\s*(\\[[^]]*]|\"[^\"]*\")",
-                Pattern.CASE_INSENSITIVE
-        );
-        Matcher m = p.matcher(input);
-        return m.find() ? m.group(1) : null;
-    }
-
-    private void resultsHandler(String rec) {
-        // TODO all of this
-
-        IO.println(rec);
-
-        String gameTypeRaw = extract(rec, "GAMETYPE");
-        String usersRaw = extract(rec, "USERS");
-        String scoreTypesRaw = extract(rec, "SCORETYPES");
-        String scoresRaw = extract(rec, "SCORES");
-
-        if (usersRaw == null) return;
-
-        String[] users;
-        if (usersRaw.length() > 2) {
-            users = Arrays.stream(usersRaw.substring(1, usersRaw.length() - 1).split(","))
-                .map(s -> s.trim().replace("\"", ""))
-                .toArray(String[]::new);
-        } else {
-            users = new String[]{};
-        }
-
-        String[] scoreTypes;
-        if (scoreTypesRaw.length() > 2) {
-            scoreTypes = Arrays.stream(scoreTypesRaw.substring(1, usersRaw.length() - 1).split(","))
-                    .map(s -> s.trim().replace("\"", ""))
-                    .toArray(String[]::new);
-        } else {
-            scoreTypes = new String[]{};
-        }
-
-        if (scoresRaw == null) return;
-        if (scoresRaw.length() > 2) {
-            List<Integer[]> scores = Arrays.stream(
-                            scoresRaw.substring(1, scoresRaw.length() - 1) // remove outer []
-                                    .split("\\],\\[")
-                    )
-                    .map(part -> part.replace("[", "").replace("]", ""))
-                    .map(part -> Arrays.stream(part.split(","))
-                            .map(String::trim)
-                            .map(Integer::parseInt)
-                            .toArray(Integer[]::new)
-                    )
-                    .toList();
-
-            eventBus.post(new NetworkEvents.TournamentResultResponse(this.connectionId, gameTypeRaw, users, scoreTypes, scores));
-        } else {
-            eventBus.post(new NetworkEvents.TournamentResultResponse(this.connectionId, gameTypeRaw, users, scoreTypes, new ArrayList<>()));
-        }
-
-    }
-
     private void gameMoveHandler(String rec) {
         String[] msg =
                 Pattern.compile(
@@ -179,7 +113,11 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                         .map(m -> m.group(1).trim())
                         .toArray(String[]::new);
 
-        eventBus.post(new NetworkEvents.GameMoveResponse(this.connectionId, msg[0], msg[1], msg[2]));
+        new EventFlow()
+                .addPostEvent(
+                        new NetworkEvents.GameMoveResponse(
+                                this.connectionId, msg[0], msg[1], msg[2]))
+                .asyncPostEvent();
     }
 
     private void gameWinConditionHandler(String rec) {
@@ -190,7 +128,9 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                 .findFirst()
                 .orElse("");
 
-        eventBus.post(new NetworkEvents.GameResultResponse(this.connectionId, condition));
+        new EventFlow()
+                .addPostEvent(new NetworkEvents.GameResultResponse(this.connectionId, condition))
+                .asyncPostEvent();
     }
 
     private void gameChallengeHandler(String rec) {
@@ -205,9 +145,17 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                             .toArray(String[]::new);
 
             if (isCancelled)
-                eventBus.post(new NetworkEvents.GameResultResponse(this.connectionId, msg[0]));
+                new EventFlow()
+                        .addPostEvent(
+                                new NetworkEvents.ChallengeCancelledResponse(
+                                        this.connectionId, msg[0]))
+                        .asyncPostEvent();
             else
-                eventBus.post(new NetworkEvents.ChallengeResponse(this.connectionId, msg[0], msg[1], msg[2]));
+                new EventFlow()
+                        .addPostEvent(
+                                new NetworkEvents.ChallengeResponse(
+                                        this.connectionId, msg[0], msg[1], msg[2]))
+                        .asyncPostEvent();
         } catch (ArrayIndexOutOfBoundsException e) {
             logger.error("Array out of bounds for: {}", rec, e);
         }
@@ -223,7 +171,11 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                             .toArray(String[]::new);
 
             // [0] playerToMove, [1] gameType, [2] opponent
-            eventBus.post(new NetworkEvents.GameMatchResponse(this.connectionId, msg[0], msg[1], msg[2]));
+            new EventFlow()
+                    .addPostEvent(
+                            new NetworkEvents.GameMatchResponse(
+                                    this.connectionId, msg[0], msg[1], msg[2]))
+                    .asyncPostEvent();
         } catch (ArrayIndexOutOfBoundsException e) {
             logger.error("Array out of bounds for: {}", rec, e);
         }
@@ -238,7 +190,9 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                         .toString()
                         .trim();
 
-        eventBus.post(new NetworkEvents.YourTurnResponse(this.connectionId, msg));
+        new EventFlow()
+                .addPostEvent(new NetworkEvents.YourTurnResponse(this.connectionId, msg))
+                .asyncPostEvent();
     }
 
     private void playerlistHandler(String rec) {
@@ -249,7 +203,9 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                         .map(m -> m.group(1).trim())
                         .toArray(String[]::new);
 
-        eventBus.post(new NetworkEvents.PlayerlistResponse(this.connectionId, players));
+        new EventFlow()
+                .addPostEvent(new NetworkEvents.PlayerlistResponse(this.connectionId, players))
+                .asyncPostEvent();
     }
 
     private void gamelistHandler(String rec) {
@@ -260,7 +216,9 @@ public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
                         .map(m -> m.group(1).trim())
                         .toArray(String[]::new);
 
-        eventBus.post(new NetworkEvents.GamelistResponse(this.connectionId, gameTypes));
+        new EventFlow()
+                .addPostEvent(new NetworkEvents.GamelistResponse(this.connectionId, gameTypes))
+                .asyncPostEvent();
     }
 
     private void helpHandler(String rec) {

framework/src/main/java/org/toop/framework/networking/interfaces/NetworkingClient.java

@@ -1,6 +1,6 @@
-package org.toop.framework.networking.connection.interfaces;
+package org.toop.framework.networking.interfaces;
 
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
 
 import java.net.InetSocketAddress;
 

framework/src/main/java/org/toop/framework/networking/interfaces/NetworkingClientManager.java

@@ -1,8 +1,8 @@
-package org.toop.framework.networking.connection.interfaces;
+package org.toop.framework.networking.interfaces;
 
-import org.toop.framework.networking.connection.exceptions.ClientNotFoundException;
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
+import org.toop.framework.networking.exceptions.ClientNotFoundException;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.types.NetworkingConnector;
 
 public interface NetworkingClientManager {
     void startClient(

framework/src/main/java/org/toop/framework/networking/server/GameResultFuture.java

@@ -1,7 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-
-import java.util.concurrent.CompletableFuture;
-
-public record GameResultFuture(OnlineGame<TurnBasedGame> game, CompletableFuture<Integer> result) {}

framework/src/main/java/org/toop/framework/networking/server/GameServer.java

@@ -1,26 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-
-import java.time.Duration;
-import java.util.List;
-import java.util.concurrent.CompletableFuture;
-
-public interface GameServer<GAMETYPE, CLIENT, CHALLENGEIDTYPE> {
-    GameResultFuture startGame(String gameType, Duration turnTime, CLIENT... clients);
-
-    void addClient(CLIENT client);
-    void removeClient(CLIENT client);
-
-    void challengeClient(String fromClientName, String toClientName, String gameTypeKey);
-    void acceptChallenge(CHALLENGEIDTYPE challengeId);
-
-    void subscribeClient(String clientName, String gameTypeKey);
-    void unsubscribeClient(String clientName);
-
-    List<String> gameTypes();
-    List<OnlineGame<GAMETYPE>> ongoingGames();
-
-    List<CLIENT> onlineUsers();
-    void shutdown();
-}

framework/src/main/java/org/toop/framework/networking/server/MatchExecutor.java

@@ -1,10 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.networking.server.client.NettyClient;
-
-import java.time.Duration;
-
-@FunctionalInterface
-public interface MatchExecutor {
-    GameResultFuture submit(String gameType, Duration turnTime, NettyClient... clients);
-}

framework/src/main/java/org/toop/framework/networking/server/OnlineGame.java

@@ -1,10 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.networking.server.client.NettyClient;
-
-public interface OnlineGame<T> {
-    long id();
-    T game();
-    NettyClient[] users();
-    void start();
-}

framework/src/main/java/org/toop/framework/networking/server/OnlineTurnBasedGame.java

@@ -1,88 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.game.gameThreads.ServerThreadBehaviour;
-import org.toop.framework.gameFramework.GameState;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.networking.server.client.NettyClient;
-
-import java.time.Duration;
-import java.util.Arrays;
-import java.util.concurrent.CompletableFuture;
-
-public class OnlineTurnBasedGame implements OnlineGame<TurnBasedGame> {
-
-    private long id;
-    private NettyClient[] clients;
-    private NettyClient[] admins;
-    private TurnBasedGame game;
-    private ServerThreadBehaviour gameThread;
-
-    private final CompletableFuture<Integer> resultFuture;
-
-    public OnlineTurnBasedGame(NettyClient[] admins, TurnBasedGame game, CompletableFuture<Integer> resultFuture, Duration timeOut, NettyClient... clients) {
-        this.game = game;
-        this.gameThread = new ServerThreadBehaviour(
-                game,
-                (pair) -> notifyMoveMade(pair.getLeft(), pair.getRight()),
-                (pair) -> notifyGameEnd(pair.getLeft(), pair.getRight()),
-                timeOut
-        );
-        this.resultFuture = resultFuture;
-        this.clients = clients;
-        this.admins = admins;
-    }
-
-    private void notifyMoveMade(String speler, int move){
-        for (NettyClient admin : admins) {
-            admin.send(String.format("SVR GAME MOVE {PLAYER: \"%s\", MOVE: \"%s\", DETAILS: \"<reactie spel op zet>\"}", speler, move));
-        }
-        for (NettyClient client : clients) {
-            client.send(String.format("SVR GAME MOVE {PLAYER: \"%s\", MOVE: \"%s\", DETAILS: \"<reactie spel op zet>\"}", speler, move));
-        }
-    }
-
-    private void notifyGameEnd(GameState state, int winner) {
-        if (state == GameState.DRAW) {
-            Arrays.stream(admins).forEach(a -> a.send("SVR GAME END"));
-
-            for (NettyClient client : clients) {
-                client.send("SVR GAME DRAW {PLAYERONESCORE: \"<score speler1>\", PLAYERTWOSCORE: \"<score speler2>\", COMMENT: \"<comment>\"}");
-            }
-        } else {
-            Arrays.stream(admins).forEach(a -> a.send("SVR GAME END"));
-            clients[winner].send("SVR GAME WIN {PLAYERONESCORE: \"<score speler1>\", PLAYERTWOSCORE: \"<score speler2>\", COMMENT: \"<comment>\"}");
-            clients[(winner+1)%2].send("SVR GAME LOSS {PLAYERONESCORE: \"<score speler1>\", PLAYERTWOSCORE: \"<score speler2>\", COMMENT: \"<comment>\"}");
-        }
-
-        // Remove game from clients
-        for (NettyClient client : clients) {
-            admins = null;
-            client.clearGame();
-        }
-
-        if (resultFuture != null) {
-            if (state.equals(GameState.DRAW)) resultFuture.complete(-1); // Return -1 if draw
-            else resultFuture.complete(winner); // Return number for winner's index
-        }
-    }
-
-    @Override
-    public long id() {
-        return id;
-    }
-
-    @Override
-    public org.toop.framework.gameFramework.model.game.TurnBasedGame game() {
-        return game;
-    }
-
-    @Override
-    public NettyClient[] users() {
-        return clients;
-    }
-
-    @Override
-    public void start() {
-        this.gameThread.start();
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/Server.java

@@ -1,358 +0,0 @@
-package org.toop.framework.networking.server;
-
-import com.google.gson.Gson;
-import org.toop.framework.game.players.ServerPlayer;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.networking.server.challenges.gamechallenge.GameChallenge;
-import org.toop.framework.networking.server.challenges.gamechallenge.GameChallengeTimer;
-import org.toop.framework.networking.server.client.NettyClient;
-import org.toop.framework.networking.server.stores.ClientStore;
-import org.toop.framework.networking.server.stores.SubscriptionStore;
-import org.toop.framework.networking.server.stores.TurnBasedGameStore;
-import org.toop.framework.networking.server.stores.TurnBasedGameTypeStore;
-import org.toop.framework.networking.server.tournaments.*;
-import org.toop.framework.networking.server.tournaments.matchmakers.DoubleRoundRobinMatchMaker;
-import org.toop.framework.networking.server.tournaments.scoresystems.*;
-import org.toop.framework.utils.ImmutablePair;
-
-import java.util.*;
-import java.util.concurrent.*;
-import java.time.Duration;
-
-public class Server implements GameServer<TurnBasedGame, NettyClient, Long> {
-
-    final private TurnBasedGameTypeStore gameTypesStore;
-    final private ClientStore<Long, NettyClient> clientStore;
-    final private List<GameChallenge> gameChallenges = new CopyOnWriteArrayList<>();
-    final private TurnBasedGameStore gameStore;
-
-    final private SubscriptionStore subscriptionStore;
-
-    final private Duration challengeDuration;
-    final private ScheduledExecutorService scheduler = Executors.newSingleThreadScheduledExecutor();
-
-    private final List<NettyClient> admins = new ArrayList<>();
-
-    public Server(
-            Duration challengeDuration,
-            TurnBasedGameTypeStore turnBasedGameTypeStore,
-            ClientStore<Long, NettyClient> clientStore,
-            TurnBasedGameStore gameStore,
-            SubscriptionStore subStore
-    ) {
-        this.gameTypesStore = turnBasedGameTypeStore;
-        this.challengeDuration = challengeDuration;
-        this.clientStore = clientStore;
-        this.gameStore = gameStore;
-        this.subscriptionStore = subStore;
-
-        scheduler.schedule(this::serverTask, 0, TimeUnit.MILLISECONDS);
-    }
-
-    @Override
-    public void addClient(NettyClient client) {
-        if (admins.isEmpty()) admins.addLast(client);
-        clientStore.add(client);
-    }
-
-    @Override
-    public void removeClient(NettyClient client) {
-        admins.remove(client);
-        clientStore.remove(client.id());
-    }
-
-    @Override
-    public List<String> gameTypes() {
-        return new ArrayList<>(gameTypesStore.all().keySet());
-    }
-
-    @Override
-    public List<OnlineGame<TurnBasedGame>> ongoingGames() {
-        return gameStore.all().stream().toList();
-    }
-
-    @Override
-    public void challengeClient(String fromUser, String toUser, String gameType) {
-
-        NettyClient from = getUser(fromUser);
-        if (from == null) {
-            return;
-        }
-
-        if (!gameTypesStore.all().containsKey(gameType)) {
-            from.send("ERR gametype not found \n");
-            return;
-        }
-
-        NettyClient to = getUser(toUser);
-        if (to == null) {
-            from.send("ERR user not found \n");
-            return;
-        }
-
-        var ch = new GameChallenge(from, to, gameType, new GameChallengeTimer(challengeDuration));
-
-        to.send(
-                "SVR GAME CHALLENGE {CHALLENGER: \"%s\", CHALLENGENUMBER: \"%s\", GAMETYPE: \"%s\"} \n"
-                        .formatted(from.name(), ch.id(), gameType)
-        );
-
-        if (!isValidChallenge(ch)) {
-            warnUserExpiredChallenge(from, ch.id());
-            ch.forceExpire();
-            return;
-        }
-
-        gameChallenges.addLast(ch);
-    }
-
-    @Override
-    public void acceptChallenge(Long challengeId) {
-        for (var challenge : gameChallenges) {
-            if (challenge.id() == challengeId) {
-                startGame(challenge.acceptChallenge(), Duration.ofSeconds(10), challenge.getUsers());
-                break;
-            }
-        }
-    }
-
-    @Override
-    public void subscribeClient(String clientName, String gameTypeKey) {
-
-        if (!gameTypesStore.all().containsKey(gameTypeKey)) {
-            return;
-        }
-
-        subscriptionStore.add(new ImmutablePair<>(gameTypeKey, clientName));
-    }
-
-    @Override
-    public void unsubscribeClient(String clientName) {
-        subscriptionStore.remove(clientName);
-    }
-
-    @Override
-    public GameResultFuture startGame(String gameType, Duration turnTime, NettyClient... clients) {
-        if (!gameTypesStore.all().containsKey(gameType)) return null;
-
-        try {
-
-            ServerPlayer[] players = new ServerPlayer[clients.length];
-
-            var gameResult = new CompletableFuture<Integer>();
-
-            var game = new OnlineTurnBasedGame(
-                    getAdmins().toArray(NettyClient[]::new),
-                    gameTypesStore.create(gameType),
-                    gameResult,
-                    turnTime,
-                    clients
-            );
-
-            var grfReturn = new GameResultFuture(game, gameResult);
-
-            for (int i = 0; i < clients.length; i++) {
-                players[i] = new ServerPlayer(clients[i]);
-                clients[i].setGame(new ImmutablePair<>(game, players[i]));
-            }
-
-            game.game().init(players);
-            gameStore.add(game);
-
-            clients[0].send(String.format("SVR GAME MATCH {PLAYERTOMOVE: \"%s\", GAMETYPE: \"%s\", OPPONENT: \"%s\"}\n",
-                    clients[0].name(),
-                    gameType,
-                    clients[1].name()));
-            clients[1].send(String.format("SVR GAME MATCH {PLAYERTOMOVE: \"%s\", GAMETYPE: \"%s\", OPPONENT: \"%s\"}\n",
-                    clients[0].name(),
-                    gameType,
-                    clients[0].name()));
-
-            game.start();
-            return grfReturn;
-        } catch (Exception e) {
-            IO.println("ERROR: Failed to start OnlineTurnBasedGame");
-            e.printStackTrace();
-        }
-        return null;
-    }
-
-    @Override
-    public List<NettyClient> onlineUsers() {
-        return clientStore.all().stream().toList();
-    }
-
-    public List<NettyClient> getAdmins() {
-        return new ArrayList<>(admins); // Clone so the list can't be edited.
-    }
-
-    @Override
-    public void shutdown() {
-        scheduler.shutdown();
-        gameChallenges.clear();
-    }
-
-    private void serverTask() {
-        checkChallenges();
-        checkSubscriptions();
-        scheduler.schedule(this::serverTask, 500, TimeUnit.MILLISECONDS);
-    }
-
-    private void checkChallenges() {
-        for (int i = gameChallenges.size() - 1; i >= 0; i--) {
-            var challenge = gameChallenges.get(i);
-
-            if (isValidChallenge(challenge)) continue;
-
-            if (challenge.isExpired()) {
-                if (!challenge.isChallengeAccepted()) Arrays.stream(challenge.getUsers())
-                        .forEach(user -> warnUserExpiredChallenge(user, challenge.id()));
-
-                gameChallenges.remove(i);
-            }
-        }
-    }
-
-    private void checkSubscriptions() {
-        if (subscriptionStore.allKeys().isEmpty()) return;
-
-        Random ran = new Random();
-
-        for (String key : subscriptionStore.allKeys()) {
-            List<String> userNames = (List<String>) subscriptionStore.allValues(key);
-            if (userNames.size() < 2) continue;
-
-            while (userNames.size() > 1) {
-                int left = ran.nextInt(userNames.size());
-                int right;
-                do {
-                    right = ran.nextInt(userNames.size());
-                } while (left == right);
-
-                String userLeft = userNames.get(left);
-                String userRight = userNames.get(right);
-
-                // Remove user if he is in a game
-                boolean userInGame = false;
-
-                if (getUser(userLeft).game() != null) {
-                    userNames.remove(userLeft);
-                    userInGame = true;
-                } else if (getUser(userRight).game() != null) {
-                    userNames.remove(userRight);
-                    userInGame = true;
-                }
-
-                if  (userInGame) { continue; }
-
-                int first = Math.max(left, right);
-                int second = Math.min(left, right);
-
-                userNames.remove(first);
-                userNames.remove(second);
-
-                startGame(key, Duration.ofSeconds(10), getUser(userLeft), getUser(userRight));
-            }
-        }
-    }
-
-    private NettyClient getUser(String username) {
-        return clientStore.all().stream().filter(e -> e.name().equalsIgnoreCase(username)).findFirst().orElse(null);
-    }
-
-    private NettyClient getUser(long id) {
-        return clientStore.get(id);
-    }
-
-    private void warnUserExpiredChallenge(NettyClient client, long challengeId) {
-        client.send("SVR GAME CHALLENGE CANCELLED {CHALLENGENUMBER: \"" + challengeId + "\"}" + "\n");
-    }
-
-    private boolean isValidChallenge(GameChallenge gameChallenge) { // TODO move to challenge class
-        for (var user : gameChallenge.getUsers()) {
-            if (clientStore.get(user.id()) == null) {
-                return false;
-            }
-
-            if (user.game() != null) {
-                return false;
-            }
-
-            if (gameChallenge.isExpired()) {
-                return false;
-            }
-        }
-
-        return true;
-    }
-
-    public void startTournament(String gameType, NettyClient requestor, boolean shuffle) {
-        if (!admins.contains(requestor)) {
-            requestor.send("ERR you do not have the privileges to start a tournament");
-            return;
-        }
-
-        var tournamentUsers = new ArrayList<>(onlineUsers());
-        tournamentUsers.removeIf(admins::contains);
-
-        Tournament tournament = new Tournament.Builder()
-                .matchExecutor(this::startGame)
-                .tournamentRunner(new AsyncTournamentRunner())
-                .matchMaker(new DoubleRoundRobinMatchMaker())
-                .addScoreSystem(new MatchCountScoreSystem())
-                .addScoreSystem(new WinCountScoreSystem())
-                .addScoreSystem(new DrawCountScoreSystem())
-                .addScoreSystem(new LoseCountScoreSystem())
-                .resultBroadcaster(this::endTournament)
-                .turnTimeout(Duration.ofSeconds(10))
-                .addPlayers(tournamentUsers.toArray(NettyClient[]::new))
-                .addAdmins(admins.toArray(NettyClient[]::new))
-                .build();
-
-        new Thread(() -> tournament.run(gameType)).start();
-    }
-
-    public void endTournament(List<IntegerScoreSystem> systems) {
-        if (systems.isEmpty()) return;
-
-        Map<String, List<ImmutablePair<String, Integer>>> combined = new HashMap<>();
-
-        for (var system : systems) {
-            for (var player : system.getScore().keySet()) {
-                combined.putIfAbsent(player.name(), new ArrayList<>());
-                combined.get(player.name()).addLast(new ImmutablePair<>(system.scoreName(), system.getScore().get(player)));
-            }
-        }
-
-        List<String> names = new ArrayList<>();
-        List<String> systemNames = new ArrayList<>();
-        List<List<Integer>> scores = new ArrayList<>();
-
-        for (var player : combined.entrySet()) {
-            names.addLast(player.getKey());
-            scores.addLast(new ArrayList<>());
-            for (var system : player.getValue()) {
-                if (!systemNames.contains(system.getLeft())) systemNames.addLast(system.getLeft());
-                scores.getLast().addLast(system.getRight());
-            }
-        }
-
-        Gson gson = new Gson();
-
-        String namesJson = gson.toJson(names);
-        String systemNamesJson = gson.toJson(systemNames);
-        String scoresJson = gson.toJson(scores);
-
-        String msg = String.format(
-                "SVR RESULTS {GAMETYPE: \"%s\", USERS: %s, SCORETYPES: %s, SCORES: %s, TOURNAMENT: 1}",
-                "none", // TODO gametype
-                namesJson,
-                systemNamesJson,
-                scoresJson
-        );
-
-        for (var user : onlineUsers()) {
-            user.send(msg);
-        }
-    }
-}