Make stationapi more compatible with mods adding custom packet stuff

Author: calmilamsy

station-networking-v0/src/main/java/net/modificationstation/stationapi/mixin/network/ConnectionMixin.java

@@ -1,5 +1,7 @@
 package net.modificationstation.stationapi.mixin.network;
 
+import com.llamalad7.mixinextras.injector.wrapoperation.Operation;
+import com.llamalad7.mixinextras.injector.wrapoperation.WrapOperation;
 import net.minecraft.network.Connection;
 import net.minecraft.network.NetworkHandler;
 import net.minecraft.network.packet.Packet;
@@ -11,14 +13,14 @@
 
 @Mixin(Connection.class)
 class ConnectionMixin {
-    @Redirect(
+    @WrapOperation(
             method = "method_1129",
             at = @At(
                     value = "INVOKE",
                     target = "Lnet/minecraft/network/packet/Packet;apply(Lnet/minecraft/network/NetworkHandler;)V"
             )
     )
-    private void stationapi_ifIdentifiable(Packet instance, NetworkHandler packetHandler) {
+    private void stationapi_ifIdentifiable(Packet instance, NetworkHandler packetHandler, Operation<Void> original) {
         instance.apply(
                 instance instanceof IdentifiablePacket identifiablePacket ?
                         IdentifiablePacketImpl.HANDLERS.getOrDefault(identifiablePacket.getId(), packetHandler) :

station-networking-v0/src/main/java/net/modificationstation/stationapi/mixin/network/PacketMixin.java

@@ -1,5 +1,8 @@
 package net.modificationstation.stationapi.mixin.network;
 
+import com.llamalad7.mixinextras.injector.wrapoperation.Operation;
+import com.llamalad7.mixinextras.injector.wrapoperation.WrapOperation;
+import com.llamalad7.mixinextras.sugar.Local;
 import net.minecraft.network.packet.Packet;
 import net.modificationstation.stationapi.api.StationAPI;
 import net.modificationstation.stationapi.api.event.network.packet.PacketRegisterEvent;
@@ -64,14 +67,14 @@ private static void stationapi_ifIdentifiable(Packet packet, DataOutputStream ou
             writeString(idPacket.getId().toString(), out);
     }
 
-    @Redirect(
+    @WrapOperation(
             method = "read(Ljava/io/DataInputStream;Z)Lnet/minecraft/network/packet/Packet;",
             at = @At(
                     value = "INVOKE",
                     target = "Lnet/minecraft/network/packet/Packet;create(I)Lnet/minecraft/network/packet/Packet;"
             )
     )
-    private static Packet stationapi_ifIdentifiable(int id, DataInputStream in, boolean server) throws IOException {
+    private static Packet stationapi_ifIdentifiable(int id, Operation<Packet> original, @Local(argsOnly = true) DataInputStream in, @Local(argsOnly = true) boolean server) throws IOException {
         if (id == IdentifiablePacket.PACKET_ID) {
             Identifier identifier = Identifier.of(readString(in, Short.MAX_VALUE));
             if (

station-vanilla-checker-v0/src/main/java/net/modificationstation/stationapi/mixin/network/LoginHelloPacketMixin.java

@@ -52,7 +52,8 @@ private void stationapi_injectStAPIFlag(String username, int protocolVersion, Ca
             constant = @Constant(
                     ordinal = 0,
                     intValue = 16
-            )
+            ),
+            require = 0
     )
     private int stationapi_injectHugeStringLimit(int constant) {
         return Short.MAX_VALUE;