refactor: make more resilient against invalid effect types

Spigot has a potion effect API lookup mismatch bug (SPIGOT-7674) due to the deprecated methods we use to support 1.17

bukkit/src/main/java/net/william278/husksync/data/BukkitData.java

@@ -237,21 +237,20 @@ public abstract class BukkitData implements Data {
 
         @NotNull
         public static BukkitData.PotionEffects adapt(@NotNull Collection<Effect> effects) {
-            return from(
+            return from(effects.stream()
-                    effects.stream()
+                    .map(effect -> {
-                            .map(effect -> new PotionEffect(
+                        final PotionEffectType type = matchEffectType(effect.type());
-                                    Objects.requireNonNull(
+                        return type != null ? new PotionEffect(
-                                            PotionEffectType.getByName(effect.type()),
+                                type,
-                                            "Invalid potion effect type"
+                                effect.duration(),
-                                    ),
+                                effect.amplifier(),
-                                    effect.duration(),
+                                effect.isAmbient(),
-                                    effect.amplifier(),
+                                effect.showParticles(),
-                                    effect.isAmbient(),
+                                effect.hasIcon()
-                                    effect.showParticles(),
+                        ) : null;
-                                    effect.hasIcon()
+                    })
-                            ))
+                    .filter(Objects::nonNull)
-                            .toList()
+                    .toList());
-            );
         }
 
         @NotNull

bukkit/src/main/java/net/william278/husksync/util/BukkitKeyedAdapter.java

@@ -22,6 +22,7 @@ package net.william278.husksync.util;
 import org.bukkit.*;
 import org.bukkit.attribute.Attribute;
 import org.bukkit.entity.EntityType;
+import org.bukkit.potion.PotionEffectType;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -48,6 +49,11 @@ public final class BukkitKeyedAdapter {
         return getRegistryValue(Registry.ATTRIBUTE, key);
     }
 
+    @Nullable
+    public static PotionEffectType matchEffectType(@NotNull String key) {
+        return PotionEffectType.getByName(key); // No registry for this in 1.17 API
+    }
+
     private static <T extends Keyed> T getRegistryValue(@NotNull Registry<T> registry, @NotNull String keyString) {
         final NamespacedKey key = NamespacedKey.fromString(keyString);
         return key != null ? registry.get(key) : null;