refactor: simplify Netty listener registration and update NettyPacket…

[twisti-dev]

This pull request includes several changes to the surf-cloud-api and surf-cloud-core projects, focusing on simplifying annotations, improving data classes, and cleaning up imports and coroutine usage. Below are the most important changes:

Simplification of Annotations:

• Removed unnecessary methods from the SurfNettyPacketHandler annotation class in surf-cloud-api-common.api.
• Simplified the SurfNettyPacketHandler annotation by removing AliasFor and KClass imports and their associated properties in SurfNettyPacketHandler.kt. [1] [2]

Data Class Improvements:

• Changed NettyPacketInfo from a value class to a data class to improve data handling and provide additional functionality in NettyPacketInfo.kt.

Import Cleanup:

• Removed unused imports in NettyListenerRegistry.kt to streamline the file.
• Cleaned up imports in RegisteredListener.kt to remove unnecessary dependencies.

Coroutine Usage:

• Simplified the coroutine handling in the handle method of RegisteredListener by removing withContext(Dispatchers.IO) and directly handling the invoker type. [1] [2]

[Copilot]

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (1)

• surf-cloud-api/surf-cloud-api-common/api/surf-cloud-api-common.api: Language not supported

Comments suppressed due to low confidence (1)

surf-cloud-core/surf-cloud-core-common/src/main/kotlin/dev/slne/surf/cloud/core/common/netty/registry/listener/RegisteredListener.kt:50

• Removing withContext(Dispatchers.IO) changes the execution context of handle; please ensure that invoker implementations avoid blocking operations on the caller's thread.

suspend fun handle(packet: NettyPacket, info: NettyPacketInfo) = when (invokerType) {