✨ feat(nms): enhance NMS support with new internal bridge and refactor packet handling

- add InternalNmsBridge interface for improved packet handling
- update NmsProvider to use getPacketBridgeHandler method
- apply NmsUseWithCaution annotation to relevant classes
- suppress ClassName warnings for consistency in naming conventions

Author: twisti-dev

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-common/src/main/kotlin/dev/slne/surf/api/paper/nms/common/NmsProvider.kt

@@ -54,7 +54,7 @@ interface NmsProvider {
     /**
      * Creates the packet bridge handler for wrapping/unwrapping NMS packets.
      */
-    fun createPacketBridgeHandler(): NmsPacketBridgeHandler
+    fun getPacketBridgeHandler(): NmsPacketBridgeHandler
 
     /**
      * Creates the packet lore registry for managing version-specific lore handlers.

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v1-21-11/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v1_21_11/V1_21_11NmsPacketBridgeHandler.kt

@@ -10,7 +10,7 @@ import dev.slne.surf.api.paper.server.nms.v1_21_11.listener.packets.V1_21_11Pack
 import net.minecraft.network.protocol.Packet
 
 @NmsUseWithCaution
-class V1_21_11NmsPacketBridgeHandler : NmsPacketBridgeHandler {
+object V1_21_11NmsPacketBridgeHandler : NmsPacketBridgeHandler {
 
     @Suppress("UNCHECKED_CAST")
     override fun wrapServerboundPacket(nmsPacket: Any): NmsServerboundPacket? {

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v1-21-11/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v1_21_11/V1_21_11NmsProvider.kt

@@ -73,8 +73,7 @@ class V1_21_11NmsProvider : NmsProvider {
         V1_21_11SurfPaperNmsPlayerToastPacketsImpl()
 
     override fun createTickThreadGuard(): TickThreadGuard = V1_21_11TickThreadGuard()
-    override fun createPacketBridgeHandler(): NmsPacketBridgeHandler =
-        V1_21_11NmsPacketBridgeHandler()
+    override fun getPacketBridgeHandler(): NmsPacketBridgeHandler = V1_21_11NmsPacketBridgeHandler
 
     override fun createPacketLoreRegistry(): PacketLoreRegistry = V1_21_11PacketLoreRegistry()
     override fun createGlowingLifecycleHandler(): GlowingLifecycleHandler =

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/V26_1NmsPacketBridgeHandler.kt

@@ -10,7 +10,8 @@ import dev.slne.surf.api.paper.server.nms.v26_1.listener.packets.V26_1PacketRegi
 import net.minecraft.network.protocol.Packet
 
 @NmsUseWithCaution
-class V26_1NmsPacketBridgeHandler : NmsPacketBridgeHandler {
+@Suppress("ClassName")
+object V26_1NmsPacketBridgeHandler : NmsPacketBridgeHandler {
 
     @Suppress("UNCHECKED_CAST")
     override fun wrapServerboundPacket(nmsPacket: Any): NmsServerboundPacket? {

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/V26_1NmsProvider.kt

@@ -29,6 +29,7 @@ import dev.slne.surf.api.paper.server.nms.v26_1.packet.lore.V26_1PacketLoreRegis
 import dev.slne.surf.api.paper.server.nms.v26_1.reflection.V26_1Reflection
 import dev.slne.surf.api.paper.server.nms.v26_1.region.V26_1TickThreadGuard
 
+@Suppress("ClassName")
 @OptIn(NmsUseWithCaution::class)
 @AutoService(NmsProvider::class)
 class V26_1NmsProvider : NmsProvider {
@@ -73,8 +74,7 @@ class V26_1NmsProvider : NmsProvider {
         V26_1SurfPaperNmsPlayerToastPacketsImpl()
 
     override fun createTickThreadGuard(): TickThreadGuard = V26_1TickThreadGuard()
-    override fun createPacketBridgeHandler(): NmsPacketBridgeHandler =
-        V26_1NmsPacketBridgeHandler()
+    override fun getPacketBridgeHandler(): NmsPacketBridgeHandler = V26_1NmsPacketBridgeHandler
 
     override fun createPacketLoreRegistry(): PacketLoreRegistry = V26_1PacketLoreRegistry()
     override fun createGlowingLifecycleHandler(): GlowingLifecycleHandler =

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/bridges/V26_1SurfPaperNmsBridgeImpl.kt

@@ -14,6 +14,7 @@ import java.util.concurrent.ConcurrentHashMap
 import java.util.concurrent.CopyOnWriteArraySet
 
 @NmsUseWithCaution
+@Suppress("ClassName")
 class V26_1SurfPaperNmsBridgeImpl : SurfPaperNmsBridge {
     private typealias PacketListenerMap<T> = ConcurrentHashMap<Class<*>, CopyOnWriteArraySet<T>>
 

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/bridges/V26_1SurfPaperNmsCommandArgumentTypesBridgeImpl.kt

@@ -19,6 +19,7 @@ import java.lang.reflect.InvocationHandler
 import java.util.concurrent.ConcurrentHashMap
 
 @NmsUseWithCaution
+@Suppress("ClassName")
 class V26_1SurfPaperNmsCommandArgumentTypesBridgeImpl : SurfPaperNmsCommandArgumentTypesBridge {
 
     override fun compoundTag(): ArgumentType<*> {

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/bridges/V26_1SurfPaperNmsCommonBridgeImpl.kt

@@ -20,6 +20,7 @@ import org.bukkit.entity.Player
 import java.net.InetSocketAddress
 
 @NmsUseWithCaution
+@Suppress("ClassName")
 class V26_1SurfPaperNmsCommonBridgeImpl : SurfPaperNmsCommonBridge {
 
     @Suppress("DEPRECATION")

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/bridges/V26_1SurfPaperNmsEntityBridgeImpl.kt

@@ -18,6 +18,7 @@ import org.bukkit.World
 import org.bukkit.entity.EntityType
 
 @NmsUseWithCaution
+@Suppress("ClassName")
 class V26_1SurfPaperNmsEntityBridgeImpl : SurfPaperNmsEntityBridge {
 
     @Suppress("UnstableApiUsage")

surf-api-paper/surf-api-paper-nms/surf-api-paper-nms-v26-1/src/main/kotlin/dev/slne/surf/api/paper/server/nms/v26_1/bridges/V26_1SurfPaperNmsGlowingBridgeImpl.kt

@@ -14,6 +14,7 @@ import net.minecraft.network.syncher.SynchedEntityData.DataValue
 import org.bukkit.entity.Entity
 
 @NmsUseWithCaution
+@Suppress("ClassName")
 object V26_1SurfPaperNmsGlowingBridgeImpl : SurfPaperNmsGlowingBridge {
     fun createTeam(data: V26_1TeamData): PacketOperation =
         V26_1PacketOperationImpl.simple {