Fix issue with packets not being registered because protocolversion mapping end is higher than proxy-server version

Author: KotlinFactory

protocolize-api/src/main/java/dev/simplix/protocolize/api/mapping/AbstractProtocolMapping.java

@@ -15,7 +15,12 @@
 public abstract class AbstractProtocolMapping implements ProtocolMapping {
 
     private final int protocolRangeStart;
-    private final int protocolRangeEnd;
+    private int protocolRangeEnd;
+
+    @Override
+    public final void protocolRangeEnd(int protocolRangeEnd) {
+        this.protocolRangeEnd = protocolRangeEnd;
+    }
 
     public static ProtocolIdMapping rangedIdMapping(int start, int end, int id) {
         return new RangeProtocolIdMapping(start, end, id);

protocolize-api/src/main/java/dev/simplix/protocolize/api/mapping/ProtocolMapping.java

@@ -11,6 +11,8 @@ public interface ProtocolMapping {
 
     int protocolRangeEnd();
 
+    void protocolRangeEnd(int protocolRangeStart);
+
     default boolean inRange(int version) {
         return version >= protocolRangeStart() && version <= protocolRangeEnd();
     }

protocolize-velocity/src/main/java/dev/simplix/protocolize/velocity/providers/VelocityProtocolRegistrationProvider.java

@@ -81,14 +81,18 @@ public void registerPacket(List<ProtocolIdMapping> mappings, Protocol protocol,
             Class<? extends MinecraftPacket> velocityPacket = generateVelocityPacket(packetClass);
             List<StateRegistry.PacketMapping> velocityMappings = new ArrayList<>();
 
+            final int maximumProtocolVersion = ProtocolVersion.MAXIMUM_VERSION.getProtocol();
             var sortedMappings = mappings.stream()
-                .filter(mapping -> !(ProtocolVersion.MAXIMUM_VERSION.getProtocol() < mapping.protocolRangeStart() || ProtocolVersion.MAXIMUM_VERSION.getProtocol() < mapping.protocolRangeEnd()))
+                .filter(mapping -> maximumProtocolVersion >= mapping.protocolRangeStart())
                 .sorted(Comparator.comparingInt(ProtocolMapping::protocolRangeEnd))
                 .toArray(ProtocolIdMapping[]::new);
 
             for (int i = 0; i < sortedMappings.length; i++) {
                 var mapping = sortedMappings[i];
                 mappingProvider.registerMapping(new RegisteredPacket(direction, packetClass), mapping);
+                if (mapping.protocolRangeEnd() > maximumProtocolVersion) {
+                    mapping.protocolRangeEnd(maximumProtocolVersion);
+                }
                 velocityMappings.add(createVelocityMapping(mapping.protocolRangeStart(), mapping.protocolRangeEnd(), mapping.id(), i == sortedMappings.length - 1));
             }
             try {