Most of the cat command

Reviewed-on: #8

.env

@@ -0,0 +1,63 @@
+# In all environments, the following files are loaded if they exist,
+# the latter taking precedence over the former:
+#
+#  * .env                contains default values for the environment variables needed by the app
+#  * .env.local          uncommitted file with local overrides
+#  * .env.$APP_ENV       committed environment-specific defaults
+#  * .env.$APP_ENV.local uncommitted environment-specific overrides
+#
+# Real environment variables win over .env files.
+#
+# DO NOT DEFINE PRODUCTION SECRETS IN THIS FILE NOR IN ANY OTHER COMMITTED FILES.
+# https://symfony.com/doc/current/configuration/secrets.html
+#
+# Run "composer dump-env prod" to compile .env files for production use (requires symfony/flex >=1.2).
+# https://symfony.com/doc/current/best_practices.html#use-environment-variables-for-infrastructure-configuration
+
+###> symfony/framework-bundle ###
+APP_ENV=dev
+APP_SECRET=
+###< symfony/framework-bundle ###
+
+###> doctrine/doctrine-bundle ###
+# Format described at https://www.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/configuration.html#connecting-using-a-url
+# IMPORTANT: You MUST configure your server version, either here or in config/packages/doctrine.yaml
+#
+# DATABASE_URL="sqlite:///%kernel.project_dir%/var/data_%kernel.environment%.db"
+# DATABASE_URL="mysql://app:!ChangeMe!@127.0.0.1:3306/app?serverVersion=8.0.32&charset=utf8mb4"
+DATABASE_URL="mysql://escapepage:b.0nqrxJ%%2FD%%2ALuf9N@localhost:3306/escapepage?serverVersion=8.0.32&charset=utf8mb4"
+###< doctrine/doctrine-bundle ###
+
+###> symfony/messenger ###
+# Choose one of the transports below
+# MESSENGER_TRANSPORT_DSN=amqp://guest:guest@localhost:5672/%2f/messages
+# MESSENGER_TRANSPORT_DSN=redis://localhost:6379/messages
+MESSENGER_TRANSPORT_DSN=doctrine://default?auto_setup=0
+###< symfony/messenger ###
+
+###> symfony/mailer ###
+# Development: use Mailpit (docker compose override provides service `mailer` on port 1025)
+MAILER_DSN=smtp://mailer:1025
+# Production/Stage (uncomment and set SENDGRID_API_KEY in real env or secrets):
+# MAILER_DSN=sendgrid+api://%env(SENDGRID_API_KEY)%
+# Alternatively, via SMTP (no extra package needed):
+# MAILER_DSN="smtp://apikey:%env(SENDGRID_API_KEY)%@smtp.sendgrid.net:587?encryption=tls"
+# Optional default sender (used by test command if --from not passed):
+# MAILER_FROM=no-reply@your-domain.tld
+# SENDGRID_API_KEY=your_real_key_goes_here   # Do NOT commit this; set in .env.local or deployment env
+###< symfony/mailer ###
+
+###> symfony/sendgrid-mailer ###
+# MAILER_DSN=sendgrid://KEY@default
+###< symfony/sendgrid-mailer ###
+
+###> mercure ###
+# Internal hub URL used by the PHP app (reachable from the php container)
+MERCURE_URL=http://mercure/.well-known/mercure
+# Public hub URL used by browsers
+MERCURE_PUBLIC_URL=http://localhost:8090/.well-known/mercure
+# Shared secret for signing JWTs (dev only). In prod, set via real env/secrets.
+MERCURE_JWT_SECRET=!ChangeThisMercureJWTSignedBySymfonySecretKey!
+# Base URL for Mercure topics. Use .dev in development; override to .com in prod via .env.prod or real env.
+MERCURE_TOPIC_BASE=https://escapepage.dev
+###< mercure ###

.env.dev

@@ -0,0 +1,4 @@
+
+###> symfony/framework-bundle ###
+APP_SECRET=620e9ce5f88a714b636179eb39d5be4f
+###< symfony/framework-bundle ###

.env.prod

@@ -0,0 +1,9 @@
+### Compiled or real environment variables should be used in production.
+### Configure MAILER_DSN to use SendGrid API transport.
+### Prefer storing SENDGRID_API_KEY using Symfony Secrets or real env vars.
+
+###> symfony/mailer ###
+# Example using SendGrid API key (replace with real secret via vault/secrets):
+# SENDGRID_API_KEY=SG.xxxxx
+MAILER_DSN=sendgrid+api://%env(resolve:SENDGRID_API_KEY)%@default
+###< symfony/mailer ###

.env.test

@@ -0,0 +1,3 @@
+# define your env variables for the test env here
+KERNEL_CLASS='App\Kernel'
+APP_SECRET='$ecretf0rt3st'

config/services.yaml

@@ -16,9 +16,5 @@ services:
     App\:
         resource: '../src/'
 
-    App\Game\Service\GameResponseService:
-        arguments:
-            $projectDir: '%kernel.project_dir%'
-
     # add more service definitions when explicit configuration is needed
     # please note that last definitions always *replace* previous ones

src/Game/Controller/GameAdminController.php

@@ -1,63 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace App\Game\Controller;
-
-use App\Game\Entity\Session;
-use App\Game\Repository\SessionRepository;
-use App\Tech\Repository\UserRepository;
-use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
-use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\Routing\Annotation\Route;
-use Symfony\Component\Security\Http\Attribute\IsGranted;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
-
-#[Route('/admin')]
-#[IsGranted('ROLE_ADMIN')]
-final class GameAdminController extends AbstractController
-{
-    public function __construct(
-        #[Autowire('%kernel.project_dir%')]
-        private string $projectDir
-    ) {
-    }
-
-    #[Route('', name: 'game_admin_dashboard', methods: ['GET'])]
-    public function index(
-        UserRepository $userRepository,
-        SessionRepository $sessionRepository
-    ): Response {
-        $players = $userRepository->findByRole('ROLE_PLAYER');
-        $sessions = $sessionRepository->findAll();
-
-        return $this->render('game/admin/index.html.twig', [
-            'players' => $players,
-            'sessions' => $sessions,
-        ]);
-    }
-
-    #[Route('/session/{session}', name: 'game_admin_view_session', methods: ['GET'])]
-    public function viewSession(Session $session): Response
-    {
-        $playersLogs = [];
-        foreach ($session->getPlayers() as $player) {
-            $username = $player->getUser()->getUsername();
-            $logFile = $this->projectDir . '/var/log/sessions/' . $session->getId() . '/' . $username . '.txt';
-
-            $logs = '';
-            if (file_exists($logFile)) {
-                $logs = file_get_contents($logFile);
-            }
-
-            $playersLogs[] = [
-                'username' => $username,
-                'logs' => $logs,
-            ];
-        }
-
-        return $this->render('game/admin/session.html.twig', [
-            'session' => $session,
-            'playersLogs' => $playersLogs,
-        ]);
-    }
-}

src/Game/Controller/GameController.php

@@ -4,14 +4,9 @@ declare(strict_types=1);
 namespace App\Game\Controller;
 
 use App\Game\Entity\Session;
-use App\Game\Entity\SessionSetting;
-use App\Game\Enum\SessionSettingType;
-use App\Game\Enum\SessionStatus;
 use App\Game\Repository\GameRepository;
-use App\Game\Repository\PlayerRepository;
 use App\Game\Repository\SessionRepository;
 use App\Game\Service\GameDashboardService;
-use App\Tech\Entity\User;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Bundle\SecurityBundle\Security;
 use Symfony\Component\HttpFoundation\Request;
@@ -19,16 +14,9 @@ use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Security\Http\Attribute\IsGranted;
 use Symfony\Component\ExpressionLanguage\Expression;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
 
 final class GameController extends AbstractController
 {
-    public function __construct(
-        #[Autowire('%env(MERCURE_PUBLIC_URL)%')]
-        private string $mercurePublicUrl
-    ) {
-    }
-
     #[Route(path: '', name: 'game_dashboard', methods: ['GET', 'POST'])]
     #[IsGranted(new Expression("is_granted('ROLE_PLAYER') or is_granted('ROLE_ADMIN')"))]
     public function dashboard(
@@ -104,54 +92,17 @@ final class GameController extends AbstractController
         ]);
     }
 
-    #[Route(path: '/{session}', name: 'game', methods: ['GET', 'POST'])]
+    #[Route(path: '/{session}', name: 'game')]
     #[IsGranted(new Expression("is_granted('ROLE_PLAYER') or is_granted('ROLE_ADMIN')"))]
     #[IsGranted('SESSION_VIEW', subject: 'session')]
     public function index(
         Session $session,
-        Request $request,
         Security $security,
-        PlayerRepository $playerRepository,
+        \App\Game\Repository\PlayerRepository $playerRepository
-        GameDashboardService $dashboardService
     ): Response
     {
         $user = $security->getUser();
-        if (!$user instanceof User) {
-            throw $this->createAccessDeniedException();
-        }
-
         $player = $playerRepository->findOneBy(['session' => $session, 'user' => $user]);
-
-        if ($request->isMethod('POST') && $request->request->has('toggle_ready')) {
-            $dashboardService->toggleReady($session, $user);
-            return $this->redirectToRoute('game', ['session' => $session->getId()]);
-        }
-
-        // Periodically check readiness timeout
-        $dashboardService->checkAllPlayersReady($session);
-
-        if ($session->getStatus() === SessionStatus::READY) {
-            $isReady = false;
-            $readyAt = null;
-            if ($player) {
-                $settingName = SessionSettingType::tryFrom('ReadyAtForPlayer' . $player->getScreen());
-                if ($settingName) {
-                    $setting = $session->getSettings()->filter(fn(SessionSetting $s) => $s->getName() === $settingName && $s->getPlayer() === $player)->first();
-                    if ($setting) {
-                        $isReady = true;
-                        $readyAt = (int)$setting->getValue();
-                    }
-                }
-            }
-
-            return $this->render('game/waiting.html.twig', [
-                'session' => $session,
-                'isReady' => $isReady,
-                'readyAt' => $readyAt,
-                'mercure_public_url' => $this->mercurePublicUrl,
-            ]);
-        }
-
         $screen = $player ? $player->getScreen() : 0;
 
         return $this->render('game/index.html.twig', [

src/Game/Entity/Player.php

@@ -60,7 +60,7 @@ class Player
         return $this->screen;
     }
 
-    public function setScreen(?int $screen): static
+    public function setScreen(int $screen): static
     {
         $this->screen = $screen;
 

src/Game/Enum/SessionSettingType.php

@@ -60,14 +60,4 @@ enum SessionSettingType: string
     case SPECIAL_REPORT_CODE_DOYLE = 'SpecialReportCodeDoyle';
     case SPECIAL_REPORT_CODE_VEGA = 'SpecialReportCodeVega';
     case SPECIAL_REPORT_CODE_LENNOX = 'SpecialReportCodeLennox';
-    case READY_AT_FOR_PLAYER1 = 'ReadyAtForPlayer1';
-    case READY_AT_FOR_PLAYER2 = 'ReadyAtForPlayer2';
-    case READY_AT_FOR_PLAYER3 = 'ReadyAtForPlayer3';
-    case READY_AT_FOR_PLAYER4 = 'ReadyAtForPlayer4';
-    case READY_AT_FOR_PLAYER5 = 'ReadyAtForPlayer5';
-    case READY_AT_FOR_PLAYER6 = 'ReadyAtForPlayer6';
-    case READY_AT_FOR_PLAYER7 = 'ReadyAtForPlayer7';
-    case READY_AT_FOR_PLAYER8 = 'ReadyAtForPlayer8';
-    case READY_AT_FOR_PLAYER9 = 'ReadyAtForPlayer9';
-    case READY_AT_FOR_PLAYER10 = 'ReadyAtForPlayer10';
 }

src/Game/Service/GameDashboardService.php

@@ -15,9 +15,6 @@ use App\Game\Repository\SessionRepository;
 use App\Tech\Entity\User;
 use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Component\Security\Core\User\UserInterface;
-use Symfony\Component\Mercure\HubInterface;
-use Symfony\Component\Mercure\Update;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
 
 final class GameDashboardService
 {
@@ -25,9 +22,6 @@ final class GameDashboardService
         private readonly GameRepository $gameRepository,
         private readonly SessionRepository $sessionRepository,
         private readonly EntityManagerInterface $entityManager,
-        private readonly HubInterface $hub,
-        #[Autowire('%env(MERCURE_TOPIC_BASE)%')]
-        private readonly string $mercureTopicBase,
     ) {
     }
 
@@ -72,17 +66,13 @@ final class GameDashboardService
 
         $player = new Player();
         $player->setUser($user);
-        $session->addPlayer($player);
+        $player->setSession($session);
 
         $this->entityManager->persist($session);
         $this->entityManager->persist($player);
 
         $this->entityManager->flush();
 
-        if (count($session->getPlayers()) === $session->getGame()->getNumberOfPlayers()) {
-            $this->startSession($session);
-        }
-
         return $session;
     }
 
@@ -117,16 +107,12 @@ final class GameDashboardService
 
         $player = new Player();
         $player->setUser($user);
-        $session->addPlayer($player);
+        $player->setSession($session);
 
         $this->entityManager->persist($player);
 
         $this->entityManager->flush();
 
-        if (count($session->getPlayers()) === $session->getGame()->getNumberOfPlayers()) {
-            $this->startSession($session);
-        }
-
         return true;
     }
 
@@ -136,7 +122,7 @@ final class GameDashboardService
             return false;
         }
 
-        if (!in_array($session->getStatus(), [SessionStatus::CREATED, SessionStatus::READY]) || $session->getTimer() > 0) {
+        if ($session->getStatus() !== SessionStatus::CREATED || $session->getTimer() > 0) {
             return false;
         }
 
@@ -152,14 +138,6 @@ final class GameDashboardService
             return false;
         }
 
-        if ($session->getStatus() === SessionStatus::READY) {
-            $session->setStatus(SessionStatus::CREATED);
-            // Clear assignments for all remaining players since the game needs to be "started" again
-            foreach ($session->getPlayers() as $player) {
-                $player->setScreen(null);
-            }
-        }
-
         // Remove player specific settings (like rights)
         foreach ($session->getSettings() as $setting) {
             if ($setting->getPlayer() === $playerToDelete) {
@@ -262,14 +240,6 @@ final class GameDashboardService
             return false;
         }
 
-        // Clean up any existing assignments (e.g. if we reverted from READY to CREATED)
-        foreach ($session->getSettings() as $setting) {
-            if ($setting->getPlayer() !== null) {
-                $session->removeSetting($setting);
-                $this->entityManager->remove($setting);
-            }
-        }
-
         // Shuffle players to assign random screens
         shuffle($players);
 
@@ -287,133 +257,6 @@ final class GameDashboardService
         return true;
     }
 
-    public function toggleReady(Session $session, User $user): bool
-    {
-        if ($session->getStatus() !== SessionStatus::READY) {
-            return false;
-        }
-
-        $player = null;
-        foreach ($session->getPlayers() as $p) {
-            if ($p->getUser() === $user) {
-                $player = $p;
-                break;
-            }
-        }
-
-        if (!$player) {
-            return false;
-        }
-
-        $settingName = SessionSettingType::tryFrom('ReadyAtForPlayer' . $player->getScreen());
-        if (!$settingName) {
-            return false;
-        }
-
-        /** @var \App\Game\Repository\SessionSettingRepository $settingRepo */
-        $settingRepo = $this->entityManager->getRepository(SessionSetting::class);
-        $setting = $settingRepo->getSetting($session, $settingName, $player);
-
-        if ($setting) {
-            $session->removeSetting($setting);
-            $this->entityManager->remove($setting);
-        } else {
-            $setting = new SessionSetting();
-            $setting->setSession($session);
-            $setting->setPlayer($player);
-            $setting->setName($settingName);
-            $setting->setValue((string)(new \DateTime())->getTimestamp());
-            $this->entityManager->persist($setting);
-        }
-
-        $this->checkAllPlayersReady($session);
-        $this->entityManager->flush();
-
-        try {
-            $topic = $this->mercureTopicBase . '/game/hub-' . $session->getId();
-            $this->hub->publish(new Update($topic, json_encode(['type' => 'player_ready', 'player' => $player->getScreen(), 'ready' => !$setting])));
-        } catch (\Exception $e) {
-            // Mercure might be down, but we don't want to crash the game
-        }
-
-        return true;
-    }
-
-    public function checkAllPlayersReady(Session $session): void
-    {
-        if ($session->getStatus() !== SessionStatus::READY) {
-            return;
-        }
-
-        $players = $session->getPlayers();
-        $numPlayers = $session->getGame()->getNumberOfPlayers();
-
-        if (count($players) < $numPlayers) {
-            return;
-        }
-
-        $readyPlayersCount = 0;
-        $now = new \DateTime();
-        $anyReset = false;
-
-        /** @var \App\Game\Repository\SessionSettingRepository $settingRepo */
-        $settingRepo = $this->entityManager->getRepository(SessionSetting::class);
-
-        foreach ($players as $player) {
-            $settingName = SessionSettingType::tryFrom('ReadyAtForPlayer' . $player->getScreen());
-            if (!$settingName) {
-                continue;
-            }
-
-            $setting = $settingRepo->getSetting($session, $settingName, $player);
-            if ($setting) {
-                $readyAtTimestamp = (int)$setting->getValue();
-                // Check timeout: 1 minute = 60 seconds
-                if (($now->getTimestamp() - $readyAtTimestamp) > 60) {
-                    $session->removeSetting($setting);
-                    $this->entityManager->remove($setting);
-                    $anyReset = true;
-                } else {
-                    $readyPlayersCount++;
-                }
-            }
-        }
-
-        if ($anyReset) {
-            $this->entityManager->flush();
-            try {
-                $topic = $this->mercureTopicBase . '/game/hub-' . $session->getId();
-                $this->hub->publish(new Update($topic, json_encode(['type' => 'player_ready'])));
-            } catch (\Exception $e) {
-                // Mercure might be down
-            }
-        }
-
-        if ($readyPlayersCount === $numPlayers) {
-            $session->setStatus(SessionStatus::PLAYING);
-            $this->entityManager->persist($session);
-
-            // Clean up ready settings
-            foreach ($players as $player) {
-                $settingName = SessionSettingType::tryFrom('ReadyAtForPlayer' . $player->getScreen());
-                if ($settingName) {
-                    $setting = $settingRepo->getSetting($session, $settingName, $player);
-                    if ($setting) {
-                        $session->removeSetting($setting);
-                        $this->entityManager->remove($setting);
-                    }
-                }
-            }
-
-            try {
-                $topic = $this->mercureTopicBase . '/game/hub-' . $session->getId();
-                $this->hub->publish(new Update($topic, json_encode(['type' => 'all_ready'])));
-            } catch (\Exception $e) {
-                // Mercure might be down, but we don't want to crash the game
-            }
-        }
-    }
-
     public function generateInviteCode(Session $session, UserInterface $user, bool $isAdmin): ?string
     {
         // Security check: is user part of this session?

src/Game/Service/GameResponseService.php

@@ -12,7 +12,6 @@ use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Bundle\SecurityBundle\Security;
 use Symfony\Component\Mercure\HubInterface;
 use Symfony\Component\Mercure\Update;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
 
 class GameResponseService
 {
@@ -22,9 +21,6 @@ class GameResponseService
         private SessionSettingRepository $sessionSettingRepository,
         private HubInterface $hub,
         private EntityManagerInterface $entityManager,
-        private string $projectDir,
-        #[Autowire('%env(MERCURE_TOPIC_BASE)%')]
-        private string $mercureTopicBase,
     ) {
     }
 
@@ -48,7 +44,7 @@ class GameResponseService
         if(!$player)
             return ['error' => 'You are not in a game.'];
 
-        $this->logSessionActivity($player, 'PLAYER: ' . $message);
+        // TODO: Here i need to add a message handler to save the message in a big log.
 
         $data = [];
 
@@ -58,43 +54,9 @@ class GameResponseService
             $data = $this->checkConsoleCommando($message, $player);
         }
 
-        $responseLog = '';
-        if (isset($data['result']) && is_array($data['result'])) {
-            foreach ($data['result'] as $line) {
-                if (is_array($line)) {
-                    $responseLog .= json_encode($line) . "\n";
-                } elseif (is_string($line) || is_numeric($line)) {
-                    $responseLog .= (string)$line . "\n";
-                }
-            }
-        } elseif (isset($data['error'])) {
-            $responseLog = 'ERROR: ' . $data['error'];
-        }
-
-        if ($responseLog !== '') {
-            $this->logSessionActivity($player, 'SERVER: ' . trim($responseLog));
-        }
-
         return $data;
     }
 
-    private function logSessionActivity(Player $player, string $content): void
-    {
-        $sessionId = $player->getSession()->getId();
-        $username = $player->getUser()->getUsername();
-        $logDir = $this->projectDir . '/var/log/sessions/' . $sessionId;
-
-        if (!is_dir($logDir)) {
-            mkdir($logDir, 0777, true);
-        }
-
-        $logFile = $logDir . '/' . $username . '.txt';
-        $timestamp = date('Y-m-d H:i:s');
-        $logMessage = sprintf("[%s] %s\n", $timestamp, $content);
-
-        file_put_contents($logFile, $logMessage, FILE_APPEND);
-    }
-
     private function getRechten(Player $player): array
     {
         $settingName = SessionSettingType::tryFrom('RightsForPlayer' . $player->getScreen());
@@ -336,12 +298,8 @@ class GameResponseService
         if(is_null($activeGame))
             return false;
 
-        $topic = $this->mercureTopicBase . '/game/hub-' . $activeGame;
+        $topic = $_ENV['MERCURE_TOPIC_BASE'] . '/game/hub-' . $activeGame;
-        try {
         $this->hub->publish(new Update($topic, json_encode([$sendTo, $message])));
-        } catch (\Exception $e) {
-            // Mercure might be down
-        }
 
         $this->updateChatTracking($player, (int)$sendTo);
 
@@ -382,14 +340,10 @@ class GameResponseService
             $this->entityManager->flush();
 
             // Notify the player that their codes have changed
-            $topic = $this->mercureTopicBase . '/game/hub-' . $session->getId();
+            $topic = $_ENV['MERCURE_TOPIC_BASE'] . '/game/hub-' . $session->getId();
             $notification = "Security Alert: One of your verify codes was shared and has been regenerated.";
             // We send it only to this player (screen)
-            try {
             $this->hub->publish(new Update($topic, json_encode([$screen, $notification])));
-            } catch (\Exception $e) {
-                // Mercure might be down
-            }
         }
     }
 
@@ -673,13 +627,9 @@ class GameResponseService
             $this->entityManager->persist($everyoneVerifiedSetting);
             $this->entityManager->flush();
 
-            $topic = $this->mercureTopicBase . '/game/hub-' . $session->getId();
+            $topic = $_ENV['MERCURE_TOPIC_BASE'] . '/game/hub-' . $session->getId();
             $message = "Mainframe Help Modus: Agents Doyle, Vega and Lennox rapports have been updated with coded messages.";
-            try {
             $this->hub->publish(new Update($topic, json_encode([0, $message])));
-            } catch (\Exception $e) {
-                // Mercure might be down
-            }
         }
     }
 

src/Tech/Repository/UserRepository.php

@@ -32,16 +32,4 @@ class UserRepository extends ServiceEntityRepository implements PasswordUpgrader
         $this->getEntityManager()->persist($user);
         $this->getEntityManager()->flush();
     }
-
-    /**
-     * @return User[]
-     */
-    public function findByRole(string $role): array
-    {
-        return $this->createQueryBuilder('u')
-            ->andWhere('u.roles LIKE :role')
-            ->setParameter('role', '%"' . $role . '"%')
-            ->getQuery()
-            ->getResult();
-    }
 }

templates/game/admin/index.html.twig

@@ -1,82 +0,0 @@
-{% extends 'base.html.twig' %}
-
-{% block title %}Game Admin Dashboard{% endblock %}
-
-{% block body %}
-    <h1>Game Admin Dashboard</h1>
-
-    <div style="display: flex; gap: 2rem;">
-        <div style="flex: 1;">
-            <h2>All Players ({{ players|length }})</h2>
-            <table style="width: 100%; border-collapse: collapse;">
-                <thead>
-                    <tr style="background-color: #f2f2f2;">
-                        <th>ID</th>
-                        <th>Username</th>
-                        <th>Email</th>
-                        <th>Roles</th>
-                        <th>Verified</th>
-                    </tr>
-                </thead>
-                <tbody>
-                    {% for player in players %}
-                        <tr>
-                            <td>{{ player.id }}</td>
-                            <td>{{ player.username }}</td>
-                            <td>{{ player.email }}</td>
-                            <td>{{ player.roles|join(', ') }}</td>
-                            <td>{{ player.isVerified ? 'Yes' : 'No' }}</td>
-                        </tr>
-                    {% else %}
-                        <tr>
-                            <td colspan="5">No players found.</td>
-                        </tr>
-                    {% endfor %}
-                </tbody>
-            </table>
-        </div>
-
-        <div style="flex: 2;">
-            <h2>All Sessions ({{ sessions|length }})</h2>
-            <table style="width: 100%; border-collapse: collapse;">
-                <thead>
-                    <tr style="background-color: #f2f2f2;">
-                        <th>ID</th>
-                        <th>Game</th>
-                        <th>Status</th>
-                        <th>Players Joined</th>
-                        <th>Created At</th>
-                        <th>Actions</th>
-                    </tr>
-                </thead>
-                <tbody>
-                    {% for session in sessions %}
-                        <tr>
-                            <td>{{ session.id }}</td>
-                            <td>{{ session.game.name }}</td>
-                            <td>{{ session.status.value }}</td>
-                            <td>
-                                <ul>
-                                    {% for p in session.players %}
-                                        <li>{{ p.user.username }} (Screen: {{ p.screen ?? 'N/A' }})</li>
-                                    {% else %}
-                                        <li>No players</li>
-                                    {% endfor %}
-                                </ul>
-                                ({{ session.players|length }} / {{ session.game.numberOfPlayers }})
-                            </td>
-                            <td>{{ session.created|date('Y-m-d H:i') }}</td>
-                            <td>
-                                <a href="{{ path('game_admin_view_session', {session: session.id}) }}">View Game Logs</a>
-                            </td>
-                        </tr>
-                    {% else %}
-                        <tr>
-                            <td colspan="6">No sessions found.</td>
-                        </tr>
-                    {% endfor %}
-                </tbody>
-            </table>
-        </div>
-    </div>
-{% endblock %}

templates/game/admin/session.html.twig

@@ -1,49 +0,0 @@
-{% extends 'base.html.twig' %}
-
-{% block title %}View Session Logs - {{ session.id }}{% endblock %}
-
-{% block body %}
-    <h1>Session: {{ session.game.name }} (#{{ session.id }})</h1>
-    <p><a href="{{ path('game_admin_dashboard') }}">Back to Dashboard</a></p>
-
-    <div class="tabs">
-        <ul style="display: flex; list-style: none; padding: 0; border-bottom: 1px solid #ccc;">
-            {% for playerLog in playersLogs %}
-                <li style="margin-right: 10px;">
-                    <button
-                        onclick="openTab(event, 'player-{{ loop.index }}')"
-                        class="tablinks {{ loop.first ? 'active' : '' }}"
-                        style="padding: 10px; cursor: pointer; border: 1px solid #ccc; border-bottom: none; background: {{ loop.first ? '#eee' : '#fff' }};"
-                    >
-                        {{ playerLog.username }}
-                    </button>
-                </li>
-            {% endfor %}
-        </ul>
-    </div>
-
-    {% for playerLog in playersLogs %}
-        <div id="player-{{ loop.index }}" class="tabcontent" style="display: {{ loop.first ? 'block' : 'none' }}; border: 1px solid #ccc; border-top: none; padding: 20px;">
-            <h3>Logs for {{ playerLog.username }}</h3>
-            <pre style="background: #f4f4f4; padding: 15px; overflow-x: auto; white-space: pre-wrap; word-wrap: break-word;">{{ playerLog.logs ?: 'No logs found for this player.' }}</pre>
-        </div>
-    {% endfor %}
-
-    <script>
-        function openTab(evt, playerName) {
-            var i, tabcontent, tablinks;
-            tabcontent = document.getElementsByClassName("tabcontent");
-            for (i = 0; i < tabcontent.length; i++) {
-                tabcontent[i].style.display = "none";
-            }
-            tablinks = document.getElementsByClassName("tablinks");
-            for (i = 0; i < tablinks.length; i++) {
-                tablinks[i].className = tablinks[i].className.replace(" active", "");
-                tablinks[i].style.background = "#fff";
-            }
-            document.getElementById(playerName).style.display = "block";
-            evt.currentTarget.className += " active";
-            evt.currentTarget.style.background = "#eee";
-        }
-    </script>
-{% endblock %}

templates/game/dashboard.html.twig

@@ -5,10 +5,6 @@
 {% block body %}
     <h1>Game Dashboard</h1>
 
-    {% if is_granted('ROLE_ADMIN') %}
-        <p><a href="{{ path('game_admin_dashboard') }}">Go to Game Admin Dashboard</a></p>
-    {% endif %}
-
     <h2>Create New Session</h2>
     {% if availableGames is not empty %}
         <form method="post">

templates/game/waiting.html.twig

@@ -1,125 +0,0 @@
-{% extends 'base.html.twig' %}
-
-{% block title %}Waiting for players - {{ session.game.name }}{% endblock %}
-
-{% block body %}
-<div class="container mt-5">
-    <div class="row justify-content-center">
-        <div class="col-md-8">
-            <div class="card shadow-sm">
-                <div class="card-header bg-primary text-white">
-                    <h3 class="card-title mb-0">Waiting for all players to be ready</h3>
-                </div>
-                <div class="card-body">
-                    <h4>{{ session.game.name }}</h4>
-                    <p>Welcome to the game! Please wait for all players to join and signal they are ready to start.</p>
-
-                    <div class="game-info">
-                        Please keep the following things in mind:
-                        <ul>
-                            <li>This game is best played in full screen mode. For windows, press F11. For Mac, press Cmd+Ctrl+F.</li>
-                            <li>There is no need to reload the page. There is even a chance this could break the game.</li>
-                            <li>If your internet connection is lost, you can get back in the game after internet has been fixed.</li>
-                        </ul>
-                    </div>
-
-                    <div class="alert alert-info">
-                        <strong>Game Information:</strong>
-                        <ul class="mb-0">
-                            <li>Number of players: {{ session.game.numberOfPlayers }}</li>
-                            <li>Current players: {{ session.players|length }} / {{ session.game.numberOfPlayers }}</li>
-                        </ul>
-                    </div>
-
-                    <div class="mt-4">
-                        <h5>Players status:</h5>
-                        <ul class="list-group">
-                            {% for player in session.players %}
-                                <li class="list-group-item d-flex justify-content-between align-items-center">
-                                    {{ player.user.username }}
-                                    {% set playerReady = false %}
-                                    {% set settingName = 'ReadyAtForPlayer' ~ player.screen %}
-                                    {% for setting in session.settings %}
-                                        {% if setting.name.value == settingName and setting.player == player %}
-                                            {% set playerReady = true %}
-                                        {% endif %}
-                                    {% endfor %}
-
-                                    {% if playerReady %}
-                                        <span class="badge bg-success">Ready</span>
-                                    {% else %}
-                                        <span class="badge bg-secondary">Not ready</span>
-                                    {% endif %}
-                                </li>
-                            {% endfor %}
-                        </ul>
-                    </div>
-
-                    <hr>
-
-                    <form method="post" class="mt-4">
-                        <div class="form-check form-switch mb-3">
-                            <input class="form-check-input" type="checkbox" id="toggle_ready" name="toggle_ready" onchange="this.form.submit()" {{ isReady ? 'checked' : '' }}>
-                            <label class="form-check-label" for="toggle_ready">
-                                <strong>I am ready to start!</strong>
-                            </label>
-                        </div>
-                        <p class="text-muted small">
-                            Note: If all players are not ready within 1 minute of you checking this, your status will be reset automatically.
-                        </p>
-                    </form>
-
-                    <div class="mt-3">
-                        <a href="{{ path('game_dashboard') }}" class="btn btn-outline-secondary">Back to Dashboard</a>
-                    </div>
-                </div>
-            </div>
-        </div>
-    </div>
-</div>
-
-<div id="mercure-config"
-     data-mercure-public-url="{{ mercure_public_url|e('html_attr') }}"
-     data-topic="{{ (mercure_topic_base ~ '/game/hub-' ~ session.id)|e('html_attr') }}"
-     data-ready-at="{{ readyAt|e('html_attr') }}"
-     style="display:none">
-</div>
-
-<script>
-    const config = document.getElementById('mercure-config');
-    const publicUrl = config.dataset.mercurePublicUrl;
-    const topic = config.dataset.topic;
-    const readyAt = config.dataset.readyAt;
-
-    if (publicUrl && topic) {
-        const url = new URL(publicUrl);
-        url.searchParams.append('topic', topic);
-
-        const eventSource = new EventSource(url);
-        eventSource.onmessage = event => {
-            const data = JSON.parse(event.data);
-            if (data.type === 'all_ready' || data.type === 'player_ready') {
-                window.location.reload();
-            }
-        };
-    }
-
-    // Client-side timeout for ready status
-    if (readyAt) {
-        const timeoutMs = 61000; // 61 seconds (slightly more than server-side 60s)
-        const now = Date.now();
-        const readyAtMs = readyAt * 1000;
-        const timeElapsed = now - readyAtMs;
-        const timeLeft = timeoutMs - timeElapsed;
-
-        if (timeLeft > 0) {
-            setTimeout(() => {
-                window.location.reload();
-            }, timeLeft);
-        } else {
-            // Already timed out, reload to sync with server
-            window.location.reload();
-        }
-    }
-</script>
-{% endblock %}

tests/Game/GameDashboardServiceTest.php

@@ -8,14 +8,12 @@ use App\Game\Entity\Player;
 use App\Game\Entity\Session;
 use App\Game\Entity\SessionSetting;
 use App\Game\Enum\GameStatus;
-use App\Game\Enum\SessionStatus;
 use App\Game\Enum\SessionSettingType;
 use App\Game\Service\GameDashboardService;
 use App\Tech\Entity\User;
 use Doctrine\ORM\EntityManagerInterface;
 use App\Game\Repository\GameRepository;
 use App\Game\Repository\SessionRepository;
-use Symfony\Component\Mercure\HubInterface;
 use PHPUnit\Framework\TestCase;
 
 class GameDashboardServiceTest extends TestCase
@@ -23,7 +21,6 @@ class GameDashboardServiceTest extends TestCase
     private $entityManager;
     private $gameRepository;
     private $sessionRepository;
-    private $hub;
     private $service;
 
     protected function setUp(): void
@@ -31,14 +28,11 @@ class GameDashboardServiceTest extends TestCase
         $this->entityManager = $this->createMock(EntityManagerInterface::class);
         $this->gameRepository = $this->createMock(GameRepository::class);
         $this->sessionRepository = $this->createMock(SessionRepository::class);
-        $this->hub = $this->createMock(HubInterface::class);
 
         $this->service = new GameDashboardService(
             $this->gameRepository,
             $this->sessionRepository,
-            $this->entityManager,
+            $this->entityManager
-            $this->hub,
-            'http://localhost/topic'
         );
     }
 
@@ -199,225 +193,24 @@ class GameDashboardServiceTest extends TestCase
         $this->assertCount(1, $session->getPlayers());
     }
 
-    public function testToggleReady(): void
+    public function testLeaveSessionDeletesSessionIfLastPlayer(): void
     {
         $user = new User();
-        $game = new Game();
-        $game->setNumberOfPlayers(1);
         $session = new Session();
-        $session->setGame($game);
+        $session->setStatus(\App\Game\Enum\SessionStatus::CREATED);
-        $session->setStatus(SessionStatus::READY);
+        $session->setTimer(0);
 
         $player = new Player();
         $player->setUser($user);
-        $player->setScreen(1);
+        $player->setSession($session);
         $session->addPlayer($player);
 
-        $repo = $this->createMock(\App\Game\Repository\SessionSettingRepository::class);
+        $this->entityManager->expects($this->exactly(2))
-        $this->entityManager->method('getRepository')
-            ->willReturn($repo);
-
-        // First call: toggle ON
-        $repo->expects($this->atLeastOnce())
-            ->method('getSetting')
-            ->willReturn(null); // No setting initially
-
-        $this->entityManager->expects($this->atLeastOnce())
-            ->method('persist')
-            ->with($this->callback(function($entity) {
-                return $entity instanceof SessionSetting && $entity->getName() === SessionSettingType::READY_AT_FOR_PLAYER1;
-            }));
-
-        $result = $this->service->toggleReady($session, $user);
-        $this->assertTrue($result);
-    }
-
-    public function testCheckAllPlayersReadyTransitionsStatus(): void
-    {
-        $game = new Game();
-        $game->setNumberOfPlayers(2);
-        $session = new Session();
-        $session->setGame($game);
-        $session->setStatus(SessionStatus::READY);
-
-        $player1 = new Player();
-        $player1->setScreen(1);
-        $session->addPlayer($player1);
-
-        $player2 = new Player();
-        $player2->setScreen(2);
-        $session->addPlayer($player2);
-
-        $repo = $this->createMock(\App\Game\Repository\SessionSettingRepository::class);
-        $this->entityManager->method('getRepository')
-            ->willReturn($repo);
-
-        $setting1 = new SessionSetting();
-        $setting1->setName(SessionSettingType::READY_AT_FOR_PLAYER1);
-        $setting1->setValue((string)time());
-        $setting1->setPlayer($player1);
-
-        $setting2 = new SessionSetting();
-        $setting2->setName(SessionSettingType::READY_AT_FOR_PLAYER2);
-        $setting2->setValue((string)time());
-        $setting2->setPlayer($player2);
-
-        $repo->method('getSetting')
-            ->willReturnCallback(function($s, $name, $p) use ($setting1, $setting2) {
-                if ($name === SessionSettingType::READY_AT_FOR_PLAYER1) return $setting1;
-                if ($name === SessionSettingType::READY_AT_FOR_PLAYER2) return $setting2;
-                return null;
-            });
-
-        $this->service->checkAllPlayersReady($session);
-
-        $this->assertEquals(SessionStatus::PLAYING, $session->getStatus());
-    }
-
-    public function testCheckAllPlayersReadyTimeouts(): void
-    {
-        $game = new Game();
-        $game->setNumberOfPlayers(2);
-        $session = new Session();
-        $session->setGame($game);
-        $session->setStatus(SessionStatus::READY);
-
-        $player1 = new Player();
-        $player1->setScreen(1);
-        $session->addPlayer($player1);
-
-        $player2 = new Player();
-        $player2->setScreen(2);
-        $session->addPlayer($player2);
-
-        $repo = $this->createMock(\App\Game\Repository\SessionSettingRepository::class);
-        $this->entityManager->method('getRepository')
-            ->willReturn($repo);
-
-        $setting1 = new SessionSetting();
-        $setting1->setName(SessionSettingType::READY_AT_FOR_PLAYER1);
-        $setting1->setValue((string)(time() - 70)); // Timed out
-        $setting1->setPlayer($player1);
-        $session->addSetting($setting1);
-
-        $repo->method('getSetting')
-            ->willReturnCallback(function($s, $name, $p) use ($setting1) {
-                if ($name === SessionSettingType::READY_AT_FOR_PLAYER1) return $setting1;
-                return null;
-            });
-
-        $this->entityManager->expects($this->once())
-            ->method('remove')
-            ->with($setting1);
-
-        $this->entityManager->expects($this->atLeastOnce())
-            ->method('flush');
-
-        $this->hub->expects($this->once())
-            ->method('publish');
-
-        $this->service->checkAllPlayersReady($session);
-
-        $this->assertEquals(SessionStatus::READY, $session->getStatus());
-    }
-
-    public function testJoinSessionTriggersStartSessionWhenFull(): void
-    {
-        $user = new User();
-        $user->setUsername('testuser');
-        $game = new Game();
-        $game->setNumberOfPlayers(2);
-        $session = new Session();
-        $session->setGame($game);
-        $session->setStatus(\App\Game\Enum\SessionStatus::CREATED);
-
-        $existingUser = new User();
-        $existingUser->setUsername('existing');
-        $existingPlayer = new Player();
-        $existingPlayer->setUser($existingUser);
-        $session->addPlayer($existingPlayer);
-
-        $setting = new SessionSetting();
-        $setting->setSession($session);
-        $setting->setName(SessionSettingType::INVITE_CODE);
-        $setting->setValue('abc-123');
-
-        $repo = $this->createMock(\Doctrine\ORM\EntityRepository::class);
-        $this->entityManager->method('getRepository')
-            ->willReturn($repo);
-
-        $repo->method('findOneBy')
-            ->willReturn($setting);
-
-        // Expectation:
-        // 1. Persist new Player
-        // 2. Persist Player 1 (existing) screen update
-        // 3. Persist Player 2 (new) screen update
-        // 4. Persist Session status update
-        // 5. 5 Settings for Player 1
-        // 6. 5 Settings for Player 2
-        // Total = 1 + 1 + 1 + 1 + 5 + 5 = 14 persists
-        $this->entityManager->expects($this->exactly(14))
-            ->method('persist');
-
-        $result = $this->service->joinSession('abc-123', $user);
-
-        $this->assertTrue($result);
-        $this->assertEquals(\App\Game\Enum\SessionStatus::READY, $session->getStatus());
-    }
-
-    public function testLeaveSessionRevertsFromReady(): void
-    {
-        $user = new User();
-        $user->setUsername('testuser');
-        $session = new Session();
-        $session->setStatus(\App\Game\Enum\SessionStatus::READY);
-        $session->setTimer(0);
-
-        $player1 = new Player();
-        $player1->setUser($user);
-        $player1->setScreen(1);
-        $player1->setSession($session);
-        $session->addPlayer($player1);
-
-        $player2 = new Player();
-        $player2->setUser(new User());
-        $player2->setScreen(2);
-        $player2->setSession($session);
-        $session->addPlayer($player2);
-
-        $this->entityManager->expects($this->exactly(1))
             ->method('remove');
-            // 1. Player1, any settings (none in this test setup)
+            // 1. Player, 2. Session
 
         $result = $this->service->leaveSession($session, $user);
 
         $this->assertTrue($result);
-        $this->assertEquals(\App\Game\Enum\SessionStatus::CREATED, $session->getStatus());
-        $this->assertNull($player2->getScreen());
-    }
-
-    public function testCreateSessionForOnePlayerGame(): void
-    {
-        $game = new Game();
-        $game->setStatus(GameStatus::OPEN);
-        $game->setNumberOfPlayers(1);
-
-        $user = new User();
-        $user->setUsername('testuser');
-
-        // 1. Session persist
-        // 2. Player persist
-        // 3. Player screen persist (during startSession)
-        // 4. Session status persist (during startSession)
-        // 5. 5 Settings for Player (during startSession)
-        // Total = 1 + 1 + 1 + 1 + 5 = 9 persists
-        $this->entityManager->expects($this->exactly(9))
-            ->method('persist');
-
-        $session = $this->service->createSession($game, $user, false);
-
-        $this->assertInstanceOf(Session::class, $session);
-        $this->assertEquals(\App\Game\Enum\SessionStatus::READY, $session->getStatus());
     }
 }

tests/Game/GameResponseServiceChatVerifyCodeTest.php

@@ -3,7 +3,6 @@ declare(strict_types=1);
 
 namespace App\Tests\Game;
 
-use App\Game\Entity\Game;
 use App\Game\Entity\Player;
 use App\Game\Entity\Session;
 use App\Game\Entity\SessionSetting;
@@ -40,8 +39,7 @@ class GameResponseServiceChatVerifyCodeTest extends TestCase
             $this->playerService,
             $this->sessionSettingRepository,
             $this->hub,
-            $this->entityManager,
+            $this->entityManager
-            'H:\escapepage'
         );
 
         $_ENV['MERCURE_TOPIC_BASE'] = 'http://test';
@@ -52,12 +50,8 @@ class GameResponseServiceChatVerifyCodeTest extends TestCase
         $user = new User();
         $user->setUsername('testuser');
 
-        $game = $this->createMock(Game::class);
-        $game->method('getNumberOfPlayers')->willReturn(4);
-
         $session = $this->createMock(Session::class);
         $session->method('getId')->willReturn(123);
-        $session->method('getGame')->willReturn($game);
 
         $player = $this->createMock(Player::class);
         $player->method('getUser')->willReturn($user);

tests/Game/SessionLoggingTest.php

@@ -1,102 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace App\Tests\Game;
-
-use App\Game\Entity\Player;
-use App\Game\Entity\Session;
-use App\Game\Entity\SessionSetting;
-use App\Game\Enum\SessionSettingType;
-use App\Game\Repository\SessionSettingRepository;
-use App\Game\Service\GameResponseService;
-use App\Game\Service\PlayerService;
-use App\Tech\Entity\User;
-use Doctrine\ORM\EntityManagerInterface;
-use PHPUnit\Framework\TestCase;
-use Symfony\Bundle\SecurityBundle\Security;
-use Symfony\Component\Mercure\HubInterface;
-
-class SessionLoggingTest extends TestCase
-{
-    private string $tempDir;
-    private $security;
-    private $playerService;
-    private $sessionSettingRepository;
-    private $hub;
-    private $entityManager;
-    private $service;
-
-    protected function setUp(): void
-    {
-        $this->tempDir = sys_get_temp_dir() . '/escapepage_test_' . uniqid();
-        mkdir($this->tempDir, 0777, true);
-
-        $this->security = $this->createMock(Security::class);
-        $this->playerService = $this->createMock(PlayerService::class);
-        $this->sessionSettingRepository = $this->createMock(SessionSettingRepository::class);
-        $this->hub = $this->createMock(HubInterface::class);
-        $this->entityManager = $this->createMock(EntityManagerInterface::class);
-
-        $this->service = new GameResponseService(
-            $this->security,
-            $this->playerService,
-            $this->sessionSettingRepository,
-            $this->hub,
-            $this->entityManager,
-            $this->tempDir
-        );
-    }
-
-    protected function tearDown(): void
-    {
-        $this->removeDir($this->tempDir);
-    }
-
-    private function removeDir(string $dir): void
-    {
-        if (!is_dir($dir)) return;
-        $files = array_diff(scandir($dir), ['.', '..']);
-        foreach ($files as $file) {
-            (is_dir("$dir/$file")) ? $this->removeDir("$dir/$file") : unlink("$dir/$file");
-        }
-        rmdir($dir);
-    }
-
-    public function testLogging(): void
-    {
-        $user = new User();
-        $user->setUsername('player1');
-
-        $session = $this->createMock(Session::class);
-        $session->method('getId')->willReturn(456);
-
-        $player = $this->createMock(Player::class);
-        $player->method('getUser')->willReturn($user);
-        $player->method('getSession')->willReturn($session);
-        $player->method('getScreen')->willReturn(1);
-
-        $this->security->method('getUser')->willReturn($user);
-        $this->playerService->method('GetCurrentlyActiveAsPlayer')->willReturn($player);
-
-        // Mock rights
-        $rightsSetting = new SessionSetting();
-        $rightsSetting->setValue(json_encode(['chat']));
-        $this->sessionSettingRepository->method('getSetting')
-            ->willReturnMap([
-                [$session, SessionSettingType::RIGHTS_FOR_PLAYER1, $player, $rightsSetting],
-            ]);
-
-        // Simulate 'help' command (always returns something)
-        $raw = json_encode(['message' => 'help', 'ts' => '123']);
-        $result = $this->service->getGameResponse($raw);
-
-        $this->assertNotEmpty($result);
-
-        $logFilePath = $this->tempDir . '/var/log/sessions/456/player1.txt';
-        $this->assertFileExists($logFilePath);
-
-        $logContent = file_get_contents($logFilePath);
-        $this->assertStringContainsString('PLAYER: help', $logContent);
-        $this->assertStringContainsString('SERVER:', $logContent);
-    }
-}