fix(ravengard): dungeon join dedupe and observable relight

Every queued join fired its own teleport when the dungeon became ready
and all but the first crashed on entering the instance the runner was
already in, which is what the repeated scheduler exceptions were. Joins
now dedupe through a pending set, a runner already inside teleports
instead of switching instance, and the relight pass logs its duration
and any failure instead of swallowing them, so the next report shows
what lighting actually did.

Author: Swofty-Developments

type.ravengarddungeon/src/main/java/net/swofty/type/ravengarddungeon/game/DungeonInstanceRegistry.java

@@ -205,9 +205,11 @@ public static DungeonInstance findByIdPrefix(String prefix) {
         return null;
     }
 
+    private static final Set<UUID> PENDING_TELEPORTS = ConcurrentHashMap.newKeySet();
+
     public static void sendPlayerIn(net.minestom.server.entity.Player player, DungeonInstance instance) {
-        if (instance.getPlayers().contains(player.getUuid()) && !instance.whenReady().isDone()) {
-            player.sendMessage("§7Your dungeon is still being stamped, hold on...");
+        if (!PENDING_TELEPORTS.add(player.getUuid())) {
+            player.sendMessage("§7Your dungeon is still being prepared, hold on...");
             return;
         }
         instance.markPlayerJoined(player.getUuid());
@@ -226,10 +228,18 @@ public static void sendPlayerIn(net.minestom.server.entity.Player player, Dungeo
             } else {
                 spawn = instance.getGenerated().spawn().withY(67);
             }
-            player.setInstance(instance.getInstance(), spawn);
-            player.sendMessage("§aEntered dungeon §f" + instance.getGameId().toString().substring(0, 8)
-                    + "§a (" + instance.getGenerated().dungeon().getRoomCount() + " rooms, mode "
-                    + instance.getMode() + ").");
+            try {
+                if (player.getInstance() == instance.getInstance()) {
+                    player.teleport(spawn);
+                } else {
+                    player.setInstance(instance.getInstance(), spawn);
+                }
+                player.sendMessage("§aEntered dungeon §f" + instance.getGameId().toString().substring(0, 8)
+                        + "§a (" + instance.getGenerated().dungeon().getRoomCount() + " rooms, mode "
+                        + instance.getMode() + ").");
+            } finally {
+                PENDING_TELEPORTS.remove(player.getUuid());
+            }
         }));
     }
 

type.ravengarddungeon/src/main/java/net/swofty/type/ravengarddungeon/generator/RavengardDungeonGenerator.java

@@ -127,10 +127,17 @@ public static CompletableFuture<Void> stamp(GeneratedDungeon generated, Instance
                         // executor and the client renders unlit sections as invisible,
                         // so lighting is part of readiness
                         Thread.startVirtualThread(() -> {
+                            long lightingStarted = System.currentTimeMillis();
                             try {
                                 net.minestom.server.instance.LightingChunk.relight(instance,
                                         new ArrayList<>(instance.getChunks()));
-                            } catch (Exception ignored) {
+                                org.tinylog.Logger.info("Dungeon relight of {} chunks took {}ms",
+                                        instance.getChunks().size(),
+                                        System.currentTimeMillis() - lightingStarted);
+                            } catch (Exception exception) {
+                                org.tinylog.Logger.error(exception,
+                                        "Dungeon relight failed after {}ms",
+                                        System.currentTimeMillis() - lightingStarted);
                             }
                             future.complete(null);
                         })))