Prevent interactions with vanished players

resolves #50

CHANGELOG.md

@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Containers remaining open by vanished players in certain conditions
 - Pressure plates remaining pressed by vanished players in certain conditions
 - Vanished players blocking merchant (villager / wandering trader) trading
+- Players can attack vanished players
 
 ## [1.5.6] - 2024-06-20
 ### Added

src/main/java/me/drex/vanish/mixin/ServerGamePacketListenerImplMixin.java

@@ -9,9 +9,11 @@
 import net.minecraft.network.protocol.game.ServerboundPlayerActionPacket;
 import net.minecraft.network.protocol.game.ServerboundUseItemOnPacket;
 import net.minecraft.network.protocol.game.ServerboundUseItemPacket;
+import net.minecraft.server.level.ServerLevel;
 import net.minecraft.server.level.ServerPlayer;
 import net.minecraft.server.network.ServerGamePacketListenerImpl;
 import net.minecraft.server.players.PlayerList;
+import net.minecraft.world.entity.Entity;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.asm.mixin.injection.At;
@@ -39,6 +41,21 @@ public void vanish_hideLeaveMessage(PlayerList playerList, Component component,
         }
     }
 
+    @WrapOperation(
+        method = "handleInteract",
+        at = @At(
+            value = "INVOKE",
+            target = "Lnet/minecraft/network/protocol/game/ServerboundInteractPacket;getTarget(Lnet/minecraft/server/level/ServerLevel;)Lnet/minecraft/world/entity/Entity;"
+        )
+    )
+    public Entity vanish_preventInteraction(ServerboundInteractPacket instance, ServerLevel serverLevel, Operation<Entity> original) {
+        Entity entity = original.call(instance, serverLevel);
+        if (entity instanceof ServerPlayer actor && !VanishAPI.canSeePlayer(actor, this.player)) {
+            return null;
+        }
+        return entity;
+    }
+
     @Inject(
         method = "handlePlayerAction",
         at = @At(