feat: Allow overriding stream codec registrations without throwing, to avoid conflicts with older versions of Shogi and because it's counterproductive to be overly protective here

Author: BlayTheNinth

shogi-api/src/main/java/net/blay09/mods/shogi/network/ShogiStreamCodecs.java

@@ -85,14 +85,18 @@ private static Identifier id(String path) {
      * @param type runtime class handled by the codec
      * @param codec codec used for encoding/decoding payload values
      * @param <T> payload type
-     * @throws IllegalArgumentException if the class or identifier was already registered
      */
     public static synchronized <T> void register(Identifier typeId, Class<? super T> type, StreamCodec<RegistryFriendlyByteBuf, T> codec) {
-        if (byClass.containsKey(type)) {
-            throw new IllegalArgumentException("A synced payload codec is already registered for class " + type.getName());
+        final var existingByClass = byClass.remove(type);
+        if (existingByClass != null) {
+            logger.warn("Overriding synced payload codec registration for class {}", type.getName());
+            byIdentifier.remove(existingByClass.id());
         }
-        if (byIdentifier.containsKey(typeId)) {
-            throw new IllegalArgumentException("A synced payload codec is already registered for identifier " + typeId);
+
+        final var existingByIdentifier = byIdentifier.remove(typeId);
+        if (existingByIdentifier != null) {
+            logger.warn("Overriding synced payload codec registration for identifier '{}'", typeId);
+            byClass.remove(existingByIdentifier.type());
         }
 
         @SuppressWarnings("unchecked")