Improve NPC logging and entity validation

Added detailed logging for NPC interactions in NPCEventListener and improved entity validation and logging in NPCImpl's nearby follow task. Removed redundant debug logs and ensured checks use npcEntity directly for consistency.

Author: CoasterFreakDE

src/main/kotlin/cc/modlabs/kpaper/npc/NPCEventListener.kt

@@ -2,6 +2,7 @@ package cc.modlabs.kpaper.npc
 
 import cc.modlabs.kpaper.event.listen
 import cc.modlabs.kpaper.extensions.timer
+import cc.modlabs.kpaper.util.getLogger
 import org.bukkit.entity.Player
 import org.bukkit.event.entity.EntityDamageByEntityEvent
 import org.bukkit.event.player.PlayerAnimationEvent
@@ -41,6 +42,7 @@ object NPCEventListener {
             val npc = npcMap[entity] ?: return@listen
             val player = event.player
             val isSneaking = player.isSneaking
+            getLogger().info("Interacted with NPC $npc for player $player ($isSneaking) - ${event.hand.name}")
 
             // Check if NPC has a conversation and start it (only on normal right-click, not shift-click)
             if (!isSneaking && npc.getConversation() != null) {

src/main/kotlin/cc/modlabs/kpaper/npc/NPCImpl.kt

@@ -640,9 +640,6 @@ class NPCImpl(
         // Start monitoring task - this will run continuously every nearbyFollowCheckInterval ticks
         logDebug("[NPC] followNearbyPlayers: Starting monitoring task with interval=$nearbyFollowCheckInterval ticks (${nearbyFollowCheckInterval * 50}ms)")
         nearbyFollowTask = timer(nearbyFollowCheckInterval, "NPCNearbyFollow") {
-            // Log every tick to confirm task is running
-            logDebug("[NPC] NearbyFollow task: [TICK START] isFollowingNearbyPlayers=$isFollowingNearbyPlayers")
-
             // Wrap everything in try-catch to ensure task never stops on error
             try {
                 // Only cancel if explicitly stopped
@@ -653,14 +650,13 @@ class NPCImpl(
                     return@timer
                 }
 
-                val currentEntity = getMannequin() as? LivingEntity
-                if (currentEntity == null || !currentEntity.isValid) {
-                    logDebug("[NPC] NearbyFollow task: Entity invalid, but continuing to check")
+                if (!npcEntity.isValid) {
+                    logDebug("[NPC] NearbyFollow task: Entity for ${npcEntity.name} invalid, but continuing to check")
                     // Don't stop the task, just skip this tick
                     return@timer
                 }
 
-                val npcLocation = currentEntity.location
+                val npcLocation = npcEntity.location
                 val npcWorld = npcLocation.world
                 if (npcWorld == null) {
                     logDebug("[NPC] NearbyFollow task: World is null, but continuing to check")