Reapply "[skip ci] add more debug to PlayerChannelInjector.kt and SurfBukkitVisualizerApiImpl.kt [please revert after testing]"

This reverts commit fd23b6c.

Author: twisti-dev

surf-api-bukkit/surf-api-bukkit-server/src/main/kotlin/dev/slne/surf/surfapi/bukkit/server/impl/visualizer/SurfBukkitVisualizerApiImpl.kt

@@ -11,6 +11,7 @@ import dev.slne.surf.surfapi.bukkit.server.impl.visualizer.visualizer.AbstractSu
 import dev.slne.surf.surfapi.bukkit.server.impl.visualizer.visualizer.SurfVisualizerAreaImpl
 import dev.slne.surf.surfapi.bukkit.server.impl.visualizer.visualizer.SurfVisualizerMultipleLocationsImpl
 import dev.slne.surf.surfapi.bukkit.server.impl.visualizer.visualizer.SurfVisualizerSingleLocationImpl
+import dev.slne.surf.surfapi.core.api.util.logger
 import org.bukkit.Chunk
 import org.bukkit.Location
 import org.bukkit.World
@@ -60,16 +61,38 @@ class SurfBukkitVisualizerApiImpl : SurfBukkitVisualizerApi {
         visualizers.asMap().values.filter { it.isVisualizing() && it.visibleTo(player) }
 
 
+    private val log = logger()
     fun processChunkReceiveUpdateForPlayer(player: Player, chunk: Chunk) {
-        getActiveVisualizers(player).forEach { it.onPlayerReceiveChunk(player, chunk) }
+        val active = getActiveVisualizers(player)
+
+        if (active.isNotEmpty()) {
+            log.atInfo()
+                .log("Received update for player ${player.name} for ${active.size} visualizers")
+        }
+
+        active.forEach { it.onPlayerReceiveChunk(player, chunk) }
     }
 
     fun processChunkUnloadForPlayer(player: Player, chunk: Chunk) {
-        getActiveVisualizers(player).forEach { it.onPlayerUnloadChunk(player, chunk) }
+        val active = getActiveVisualizers(player)
+
+        if (active.isNotEmpty()) {
+            log.atInfo()
+                .log("Received unload for player ${player.name} for ${active.size} visualizers")
+        }
+
+        active.forEach { it.onPlayerUnloadChunk(player, chunk) }
     }
 
     fun processPlayerQuit(player: Player) {
-        visualizers.asMap().values.forEach { it.removeViewer(player) }
+        val active = visualizers.asMap().values
+
+        if (active.isNotEmpty()) {
+            log.atInfo()
+                .log("Player ${player.name} quit, removing from ${active.size} visualizers")
+        }
+        
+        active.forEach { it.removeViewer(player) }
     }
 }
 

surf-api-bukkit/surf-api-bukkit-server/src/main/kotlin/dev/slne/surf/surfapi/bukkit/server/packet/listener/PlayerChannelInjector.kt

@@ -49,6 +49,10 @@ object PlayerChannelInjector : Listener {
 
     private val playerInjectorCache = Caffeine.newBuilder()
         .weakValues()
+        .removalListener<UUID, Connection> { key, value, cause ->
+            log.atWarning()
+                .log("Player connection cache for $key was removed due to $cause")
+        }
         .expireAfterAccess(1.minutes)
         .build<UUID, Connection>()