Refactored to support @Nullable annotations

Updated method signatures and equality checks across multiple providers to include `@Nullable` annotations. Converted `SuperPermsPermissionController` to a record for simplification and improved consistency.

Author: NonSwag

providers/citizens/src/main/java/net/thenextlvl/service/providers/citizens/CitizensCharacter.java

@@ -29,7 +29,7 @@
 
 @NullMarked
 public final class CitizensCharacter<T extends Entity> implements Character<T> {
-    protected final NPC npc;
+    private final NPC npc;
 
     public CitizensCharacter(final NPC npc) {
         this.npc = npc;
@@ -277,7 +277,7 @@ public void setTablistEntryHidden(final boolean hidden) {
     }
 
     @Override
-    public boolean equals(final Object o) {
+    public boolean equals(@Nullable final Object o) {
         if (o == null || getClass() != o.getClass()) return false;
         final CitizensCharacter<?> that = (CitizensCharacter<?>) o;
         return Objects.equals(npc, that.npc);

providers/fancy-holograms-v2/src/main/java/net/thenextlvl/service/providers/fancyholograms/v2/FancyHologramLine.java

@@ -10,6 +10,7 @@
 import org.bukkit.World;
 import org.joml.Vector3f;
 import org.jspecify.annotations.NullMarked;
+import org.jspecify.annotations.Nullable;
 
 import java.util.Objects;
 import java.util.Optional;
@@ -128,7 +129,7 @@ public float getYaw() {
     }
 
     @Override
-    public boolean equals(final Object o) {
+    public boolean equals(@Nullable final Object o) {
         if (o == null || getClass() != o.getClass()) return false;
         final FancyHologramLine<?, ?> that = (FancyHologramLine<?, ?>) o;
         return Objects.equals(data, that.data);

providers/fancy-holograms-v3/src/main/java/net/thenextlvl/service/providers/fancyholograms/v3/FancyHologramLine.java

@@ -10,6 +10,7 @@
 import org.bukkit.World;
 import org.joml.Vector3f;
 import org.jspecify.annotations.NullMarked;
+import org.jspecify.annotations.Nullable;
 
 import java.util.Objects;
 import java.util.Optional;
@@ -128,7 +129,7 @@ public float getYaw() {
     }
 
     @Override
-    public boolean equals(final Object o) {
+    public boolean equals(@Nullable final Object o) {
         if (o == null || getClass() != o.getClass()) return false;
         final FancyHologramLine<?, ?> that = (FancyHologramLine<?, ?>) o;
         return Objects.equals(data, that.data);

providers/luckperms/src/main/java/net/thenextlvl/service/providers/luckperms/LuckPermsChatProfile.java

@@ -116,7 +116,7 @@ private boolean unsetSuffix(final int priority) {
     }
 
     @Override
-    public <T> Optional<T> getInfoNode(final String key, final Function<String, @Nullable T> mapper) {
+    public <T> Optional<T> getInfoNode(final String key, final Function<@Nullable String, @Nullable T> mapper) {
         return user().getCachedData().getMetaData(options()).getMetaValue(key, mapper);
     }
 

providers/luckperms/src/main/java/net/thenextlvl/service/providers/luckperms/LuckPermsGroup.java

@@ -151,7 +151,7 @@ public boolean setPermission(final String permission, final boolean value) {
     }
 
     @Override
-    public <T> Optional<T> getInfoNode(final String key, final Function<String, T> mapper) {
+    public <T> Optional<T> getInfoNode(final String key, final Function<@Nullable String, @Nullable T> mapper) {
         return group().getCachedData().getMetaData(options()).getMetaValue(key, mapper);
     }
 

providers/luckperms/src/main/java/net/thenextlvl/service/providers/luckperms/LuckPermsPermissionHolder.java

@@ -10,6 +10,7 @@
 import net.thenextlvl.service.api.group.GroupHolder;
 import org.jetbrains.annotations.Unmodifiable;
 import org.jspecify.annotations.NullMarked;
+import org.jspecify.annotations.Nullable;
 
 import java.util.Map;
 import java.util.Optional;
@@ -53,7 +54,7 @@ public boolean setPermission(final String permission, final boolean value) {
     }
 
     @Override
-    public <T> Optional<T> getInfoNode(final String key, final Function<String, T> mapper) {
+    public <T> Optional<T> getInfoNode(final String key, final Function<@Nullable String, @Nullable T> mapper) {
         return user().getCachedData().getMetaData(options()).getMetaValue(key, mapper);
     }
 

providers/superperms/src/main/java/net/thenextlvl/service/providers/superperms/SuperPermsPermissionController.java

@@ -16,13 +16,7 @@
 
 @DoNotWrap
 @NullMarked
-public final class SuperPermsPermissionController implements PermissionController {
-    private final Plugin plugin;
-
-    public SuperPermsPermissionController(final Plugin plugin) {
-        this.plugin = plugin;
-    }
-
+public record SuperPermsPermissionController(Plugin plugin) implements PermissionController {
     @Override
     public CompletableFuture<PermissionHolder> loadPermissionHolder(final OfflinePlayer player, @Nullable final World world) {
         return CompletableFuture.completedFuture(getPermissionHolder(player.getPlayer()).orElse(null));

providers/superperms/src/main/java/net/thenextlvl/service/providers/superperms/SuperPermsPermissionHolder.java

@@ -6,6 +6,7 @@
 import org.bukkit.permissions.PermissionAttachmentInfo;
 import org.jetbrains.annotations.Unmodifiable;
 import org.jspecify.annotations.NullMarked;
+import org.jspecify.annotations.Nullable;
 
 import java.util.Map;
 import java.util.Optional;
@@ -44,7 +45,7 @@ public boolean setPermission(final String permission, final boolean value) {
     }
 
     @Override
-    public <T> Optional<T> getInfoNode(final String key, final Function<String, T> mapper) {
+    public <T> Optional<T> getInfoNode(final String key, final Function<@Nullable String, @Nullable T> mapper) {
         return Optional.empty();
     }