refactor: Kotlin official codestyles

Author: ammodev

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/SurfNpcApi.kt

@@ -4,16 +4,16 @@ import dev.slne.surf.npc.api.npc.Npc
 import dev.slne.surf.npc.api.npc.location.NpcLocation
 import dev.slne.surf.npc.api.npc.property.NpcProperty
 import dev.slne.surf.npc.api.npc.property.NpcPropertyType
-import dev.slne.surf.npc.api.result.NpcCreationResult
-import dev.slne.surf.npc.api.result.NpcDeletionResult
 import dev.slne.surf.npc.api.npc.rotation.NpcRotation
 import dev.slne.surf.npc.api.npc.rotation.NpcRotationType
 import dev.slne.surf.npc.api.npc.skin.NpcSkin
+import dev.slne.surf.npc.api.result.NpcCreationResult
+import dev.slne.surf.npc.api.result.NpcDeletionResult
 import dev.slne.surf.surfapi.core.api.util.requiredService
 import it.unimi.dsi.fastutil.objects.ObjectList
 import it.unimi.dsi.fastutil.objects.ObjectSet
 import net.kyori.adventure.text.Component
-import java.util.UUID
+import java.util.*
 
 /**
  * API for managing NPCs (Non-Player Characters) in the Surf framework.

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/dsl/NpcDslBuilder.kt

@@ -2,10 +2,10 @@ package dev.slne.surf.npc.api.dsl
 
 import dev.slne.surf.npc.api.npc.location.NpcLocation
 import dev.slne.surf.npc.api.npc.property.NpcProperty
-import dev.slne.surf.npc.api.result.NpcCreationResult
 import dev.slne.surf.npc.api.npc.rotation.NpcRotation
 import dev.slne.surf.npc.api.npc.rotation.NpcRotationType
 import dev.slne.surf.npc.api.npc.skin.NpcSkin
+import dev.slne.surf.npc.api.result.NpcCreationResult
 import dev.slne.surf.npc.api.surfNpcApi
 import net.kyori.adventure.text.Component
 

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/dsl/NpcDslSubBuilders.kt

@@ -117,7 +117,8 @@ class NpcPropertyBuilder {
     /**
      * The type of the property. Defaults to the "string" type.
      */
-    var type: NpcPropertyType = surfNpcApi.getPropertyType("string") ?: error("Default property type 'string' not found")
+    var type: NpcPropertyType =
+        surfNpcApi.getPropertyType("string") ?: error("Default property type 'string' not found")
 
     /**
      * Builds the NPC property.

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/event/NpcCollisionEvent.kt

@@ -11,10 +11,10 @@ import org.bukkit.event.HandlerList
  * @property npc The NPC involved in the collision.
  * @property player The player who collided with the NPC.
  */
-class NpcCollisionEvent (
+class NpcCollisionEvent(
     val npc: Npc,
     val player: Player
-): Event() {
+) : Event() {
 
     /**
      * Returns the handler list for this event.

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/event/NpcCreateEvent.kt

@@ -9,7 +9,7 @@ import org.bukkit.event.HandlerList
  *
  * @property npc The NPC that was created.
  */
-class NpcCreateEvent (
+class NpcCreateEvent(
     val npc: Npc
 ) : Event() {
 

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/event/NpcDeleteEvent.kt

@@ -9,7 +9,7 @@ import org.bukkit.event.HandlerList
  *
  * @property npc The NPC that was deleted.
  */
-class NpcDeleteEvent (
+class NpcDeleteEvent(
     val npc: Npc
 ) : Event() {
 

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/event/NpcDespawnEvent.kt

@@ -11,7 +11,7 @@ import org.bukkit.event.HandlerList
  * @property npc The NPC that despawned.
  * @property player The player associated with the despawn event, if applicable.
  */
-class NpcDespawnEvent (
+class NpcDespawnEvent(
     val npc: Npc,
     val player: Player
 ) : Event() {

surf-npc-api/src/main/kotlin/dev/slne/surf/npc/api/event/NpcSpawnEvent.kt

@@ -11,7 +11,7 @@ import org.bukkit.event.HandlerList
  * @property npc The NPC that was spawned.
  * @property player The player associated with the spawn event, if applicable.
  */
-class NpcSpawnEvent (
+class NpcSpawnEvent(
     val npc: Npc,
     val player: Player
 ) : Event() {

surf-npc-bukkit/src/main/kotlin/dev/slne/surf/npc/bukkit/BukkitMain.kt

@@ -4,30 +4,21 @@ import com.github.retrooper.packetevents.PacketEvents
 import com.github.retrooper.packetevents.event.PacketListenerPriority
 import com.github.shynixn.mccoroutine.folia.SuspendingJavaPlugin
 import dev.slne.surf.npc.api.npc.property.NpcPropertyType
-
 import dev.slne.surf.npc.bukkit.command.NpcCommand
 import dev.slne.surf.npc.bukkit.listener.ConnectionListener
 import dev.slne.surf.npc.bukkit.listener.NpcListener
-import dev.slne.surf.npc.bukkit.npc.property.impl.BooleanPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.ComponentPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.DoublePropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.FloatPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.IntPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.LongPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.NamedTextColorPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.NpcLocationPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.NpcRotationPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.StringPropertyType
-import dev.slne.surf.npc.bukkit.npc.property.impl.UuidPropertyType
+import dev.slne.surf.npc.bukkit.npc.property.impl.*
 import dev.slne.surf.npc.core.property.propertyTypeRegistry
 import dev.slne.surf.npc.core.service.storageService
-
 import org.bukkit.Bukkit
 import org.bukkit.plugin.java.JavaPlugin
 
 class BukkitMain : SuspendingJavaPlugin() {
     override fun onEnable() {
-        PacketEvents.getAPI().eventManager.registerListener(NpcListener(), PacketListenerPriority.NORMAL)
+        PacketEvents.getAPI().eventManager.registerListener(
+            NpcListener(),
+            PacketListenerPriority.NORMAL
+        )
         Bukkit.getPluginManager().registerEvents(ConnectionListener(), this)
 
         propertyTypeRegistry.register(BooleanPropertyType(NpcPropertyType.Types.BOOLEAN))

surf-npc-bukkit/src/main/kotlin/dev/slne/surf/npc/bukkit/BukkitPackets.kt

@@ -7,31 +7,33 @@ import com.github.retrooper.packetevents.protocol.player.GameMode
 import com.github.retrooper.packetevents.protocol.player.UserProfile
 import com.github.retrooper.packetevents.util.Vector3d
 import com.github.retrooper.packetevents.wrapper.play.server.*
-import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerTeams
 import io.github.retrooper.packetevents.util.SpigotConversionUtil
 import net.kyori.adventure.text.Component
 import net.kyori.adventure.text.format.NamedTextColor
 import org.bukkit.Location
-import java.util.UUID
+import java.util.*
 
-fun createPlayerInfoPacket(profile: UserProfile, displayName: Component, listed: Boolean = false) = WrapperPlayServerPlayerInfoUpdate(
-    WrapperPlayServerPlayerInfoUpdate.Action.ADD_PLAYER,
-    WrapperPlayServerPlayerInfoUpdate.PlayerInfo(
-        profile,
-        listed,
-        0,
-        GameMode.SURVIVAL,
-        displayName,
-        null
+fun createPlayerInfoPacket(profile: UserProfile, displayName: Component, listed: Boolean = false) =
+    WrapperPlayServerPlayerInfoUpdate(
+        WrapperPlayServerPlayerInfoUpdate.Action.ADD_PLAYER,
+        WrapperPlayServerPlayerInfoUpdate.PlayerInfo(
+            profile,
+            listed,
+            0,
+            GameMode.SURVIVAL,
+            displayName,
+            null
+        )
     )
-)
+
 fun createEntityMetadataPacket(npcEntityId: Int) = WrapperPlayServerEntityMetadata(
     npcEntityId,
     listOf(
         EntityData(17, EntityDataTypes.BYTE, 0x7F.toByte()),
         EntityData(0, EntityDataTypes.BYTE, 0x02.toByte()),
     )
 )
+
 fun createPlayerSpawnPacket(
     entityId: Int,
     uuid: UUID,
@@ -42,11 +44,18 @@ fun createPlayerSpawnPacket(
     entityId,
     uuid,
     EntityTypes.PLAYER,
-    com.github.retrooper.packetevents.protocol.world.Location(Vector3d(location.x, location.y, location.z), yaw, pitch),
+    com.github.retrooper.packetevents.protocol.world.Location(
+        Vector3d(
+            location.x,
+            location.y,
+            location.z
+        ), yaw, pitch
+    ),
     yaw,
     0,
     null
 )
+
 fun createNametagSpawnPacket(
     entityId: Int,
     uuid: UUID,
@@ -55,11 +64,18 @@ fun createNametagSpawnPacket(
     entityId,
     uuid,
     EntityTypes.TEXT_DISPLAY,
-    com.github.retrooper.packetevents.protocol.world.Location(Vector3d(location.x, location.y + 2, location.z), 0f, 0f),
+    com.github.retrooper.packetevents.protocol.world.Location(
+        Vector3d(
+            location.x,
+            location.y + 2,
+            location.z
+        ), 0f, 0f
+    ),
     0f,
     0,
     null
 )
+
 fun createNametagMetadataPacket(
     entityId: Int,
     displayName: Component
@@ -71,6 +87,7 @@ fun createNametagMetadataPacket(
         EntityData(27, EntityDataTypes.BYTE, 0x02.toByte())
     )
 )
+
 fun createTeamCreatePacket(
     teamName: String,
     displayName: Component
@@ -96,14 +113,17 @@ fun createTeamAddEntityPacket(teamName: String, entityUuid: String) = WrapperPla
     nullInfo,
     entityUuid
 )
+
 fun createDestroyPacket(vararg entityIds: Int) = WrapperPlayServerDestroyEntities(*entityIds)
 fun createPlayerInfoRemovePacket(npcUuid: UUID) = WrapperPlayServerPlayerInfoRemove(npcUuid)
 fun createRotationPackets(entityId: Int, yaw: Float, pitch: Float) = Pair(
     WrapperPlayServerEntityRotation(entityId, yaw, pitch, true),
     WrapperPlayServerEntityHeadLook(entityId, yaw)
 )
-fun createTeleportPacket(entityId: Int, location: Location, onGround: Boolean = false) = WrapperPlayServerEntityTeleport(
-    entityId,
-    SpigotConversionUtil.fromBukkitLocation(location),
-    onGround
-)
+
+fun createTeleportPacket(entityId: Int, location: Location, onGround: Boolean = false) =
+    WrapperPlayServerEntityTeleport(
+        entityId,
+        SpigotConversionUtil.fromBukkitLocation(location),
+        onGround
+    )