fix: (combat) five adversarially-verified defects in Velocity and KillAura

Velocity Simple mode zeroed explosion knockback exactly when the user asked to keep it (inverted guard) and Click mode discarded its computed target whenever WhenFacingEnemyOnly was off, making the mode inert. KillAura no longer retargets through the raycast to entities that fail isSelected (teams/anti-bot/invisibility/focus), its per-tick attack loop stops driving the click budget negative when distance/double-click bonus clicks fire, and prevTargetEntities only accumulates in Switch mode instead of leaking one id per hit forever in Single/Multi.

Author: opZywl

src/main/java/net/ccbluex/liquidbounce/features/module/modules/combat/KillAura.kt

@@ -650,7 +650,7 @@ object KillAura : Module("KillAura", Category.COMBAT, Category.SubCategory.COMBA
                 val wasBlocking = blockStatus
 
                 runAttack(it == 0, it + 1 == maxClicks)
-                clicks--
+                if (clicks > 0) clicks--
 
                 if (wasBlocking && !blockStatus && (releaseAutoBlock && !ignoreTickRule || autoBlock == "Off")) {
                     return@handler
@@ -882,12 +882,9 @@ object KillAura : Module("KillAura", Category.COMBAT, Category.SubCategory.COMBA
 
         val switchMode = targetMode == "Switch"
 
-        if (!switchMode || switchTimer.hasTimePassed(switchDelay)) {
+        if (switchMode && switchTimer.hasTimePassed(switchDelay)) {
             prevTargetEntities += currentTarget.entityId
-
-            if (switchMode) {
-                switchTimer.reset()
-            }
+            switchTimer.reset()
         }
 
         // Open inventory
@@ -1142,7 +1139,7 @@ private fun updateHittable() {
         ) { entity -> !livingRaycast || entity is EntityLivingBase && entity !is EntityArmorStand }
 
         if (raycastEntity != null && raycastEntity is EntityLivingBase && (!(raycastEntity is EntityPlayer && raycastEntity.isClientFriend()))) {
-            if (raycastIgnored && target != raycastEntity) {
+            if (raycastIgnored && target != raycastEntity && isSelected(raycastEntity, true)) {
                 this.target = raycastEntity
             }
 

src/main/java/net/ccbluex/liquidbounce/features/module/modules/combat/Velocity.kt

@@ -883,7 +883,9 @@ object Velocity : Module("Velocity", Category.COMBAT, Category.SubCategory.COMBA
                 }
 
                 entity = result
-            } else getNearestEntityInRange(clickRange)?.takeIf { isSelected(it, true) }
+            } else {
+                entity = getNearestEntityInRange(clickRange)?.takeIf { isSelected(it, true) }
+            }
         }
 
         entity ?: return@handler
@@ -1907,7 +1909,7 @@ object Velocity : Module("Velocity", Category.COMBAT, Category.SubCategory.COMBA
             }
         } else if (packet is S27PacketExplosion) {
             // Don't cancel explosions, modify them, they could change blocks in the world
-            if (horizontal != 0f && vertical != 0f) {
+            if (horizontal == 0f && vertical == 0f) {
                 packet.field_149152_f = 0f
                 packet.field_149153_g = 0f
                 packet.field_149159_h = 0f