refactor: Replace JetBrains annotations with JSpecify annotations for nullability

Author: Jakubk15

buildSrc/src/main/kotlin/Versions.kt

@@ -7,6 +7,7 @@ object Versions {
     const val MULTIFICATION = "1.2.4"
 
     const val JETBRAINS_ANNOTATIONS = "26.0.2-1"
+    const val JSPECIFY = "1.0.0"
     const val PLACEHOLDER_API = "2.11.7"
     const val DYNMAP_API = "3.7-beta-6"
     const val LOMBOK = "1.18.42"

eternalcore-api/build.gradle.kts

@@ -7,4 +7,5 @@ plugins {
 dependencies {
     compileOnly("org.spigotmc:spigot-api:${Versions.SPIGOT_API}")
     api("org.jetbrains:annotations:${Versions.JETBRAINS_ANNOTATIONS}")
+    api("org.jspecify:jspecify:${Versions.JSPECIFY}")
 }

eternalcore-api/src/main/java/com/eternalcode/core/feature/adminchat/AdminChatService.java

@@ -3,8 +3,8 @@
 import java.util.Collection;
 import java.util.UUID;
 import org.bukkit.command.CommandSender;
-import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Unmodifiable;
+import org.jspecify.annotations.NonNull;
 
 /**
  * Service responsible for managing admin chat functionality.
@@ -20,15 +20,15 @@ public interface AdminChatService {
      * @param playerUuid the UUID of the player to toggle admin chat for
      * @return {@code true} if persistent chat was enabled, {@code false} if it was disabled
      */
-    boolean toggleChat(@NotNull UUID playerUuid);
+    boolean toggleChat(@NonNull UUID playerUuid);
 
     /**
      * Checks if the persistent admin chat mode is enabled for the specified player.
      *
      * @param playerUuid the UUID of the player to check
      * @return {@code true} if persistent chat is enabled, {@code false} otherwise
      */
-    boolean hasEnabledChat(@NotNull UUID playerUuid);
+    boolean hasEnabledChat(@NonNull UUID playerUuid);
 
     /**
      * Retrieves all players who currently have persistent admin chat enabled.
@@ -39,7 +39,7 @@ public interface AdminChatService {
      *
      * @return an unmodifiable collection of player UUIDs with enabled admin chat
      */
-    @NotNull
+    @NonNull
     @Unmodifiable
     Collection<UUID> getPlayersWithEnabledChat();
 
@@ -52,27 +52,27 @@ public interface AdminChatService {
      * @param message the message content to send
      * @param sender  the command sender who is sending the message
      */
-    void sendAdminChatMessage(@NotNull String message, @NotNull CommandSender sender);
+    void sendAdminChatMessage(@NonNull String message, @NonNull CommandSender sender);
 
     /**
      * Enables persistent admin chat for the specified player.
      *
      * @param playerUuid the UUID of the player
      */
-    void enableChat(@NotNull UUID playerUuid);
+    void enableChat(@NonNull UUID playerUuid);
 
     /**
      * Disables persistent admin chat for the specified player.
      *
      * @param playerUuid the UUID of the player
      */
-    void disableChat(@NotNull UUID playerUuid);
+    void disableChat(@NonNull UUID playerUuid);
 
     /**
      * Checks if the specified command sender has permission to see admin chat messages.
      *
      * @param sender the command sender to check
      * @return {@code true} if the sender can see admin chat messages, {@code false} otherwise
      */
-    boolean canSeeAdminChat(@NotNull CommandSender sender);
+    boolean canSeeAdminChat(@NonNull CommandSender sender);
 }

eternalcore-api/src/main/java/com/eternalcode/core/feature/adminchat/event/AdminChatEvent.java

@@ -4,7 +4,7 @@
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
+import org.jspecify.annotations.NonNull;
 
 /**
  * Event called when an admin chat message is being sent.
@@ -21,7 +21,7 @@ public class AdminChatEvent extends Event implements Cancellable {
     private String content;
     private boolean cancelled;
 
-    public AdminChatEvent(@NotNull CommandSender sender, @NotNull String content) {
+    public AdminChatEvent(@NonNull CommandSender sender, @NonNull String content) {
         super(false);
 
         if (sender == null) {
@@ -36,22 +36,22 @@ public AdminChatEvent(@NotNull CommandSender sender, @NotNull String content) {
         this.cancelled = false;
     }
 
-    @NotNull
+    @NonNull
     public static HandlerList getHandlerList() {
         return HANDLER_LIST;
     }
 
-    @NotNull
+    @NonNull
     public CommandSender getSender() {
         return this.sender;
     }
 
-    @NotNull
+    @NonNull
     public String getContent() {
         return this.content;
     }
 
-    public void setContent(@NotNull String content) {
+    public void setContent(@NonNull String content) {
         if (content == null) {
             throw new IllegalArgumentException("Content cannot be null");
         }
@@ -69,7 +69,7 @@ public void setCancelled(boolean cancelled) {
     }
 
     @Override
-    @NotNull
+    @NonNull
     public HandlerList getHandlers() {
         return HANDLER_LIST;
     }

eternalcore-api/src/main/java/com/eternalcode/core/feature/afk/event/AfkSwitchEvent.java

@@ -4,6 +4,7 @@
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
+import org.jspecify.annotations.NonNull;
 
 /**
  * Called when a player switches their afk status.
@@ -41,7 +42,7 @@ public void setCancelled(boolean cancel) {
     }
 
     @Override
-    public HandlerList getHandlers() {
+    public @NonNull HandlerList getHandlers() {
         return HANDLER_LIST;
     }
 

eternalcore-api/src/main/java/com/eternalcode/core/feature/catboy/CatboyService.java

@@ -1,12 +1,11 @@
 package com.eternalcode.core.feature.catboy;
 
-import org.bukkit.entity.Cat;
-import org.bukkit.entity.Player;
-import org.jetbrains.annotations.Unmodifiable;
-
 import java.util.Collection;
 import java.util.Optional;
 import java.util.UUID;
+import org.bukkit.entity.Cat;
+import org.bukkit.entity.Player;
+import org.jetbrains.annotations.Unmodifiable;
 
 /**
  * Service for managing catboys.

eternalcore-api/src/main/java/com/eternalcode/core/feature/home/HomeService.java

@@ -1,12 +1,11 @@
 package com.eternalcode.core.feature.home;
 
 import java.util.Collection;
-import java.util.Map;
 import java.util.Optional;
 import java.util.UUID;
 import org.bukkit.Location;
 import org.bukkit.entity.Player;
-import org.jetbrains.annotations.Nullable;
+import org.jspecify.annotations.Nullable;
 
 public interface HomeService {
 

eternalcore-api/src/main/java/com/eternalcode/core/feature/ignore/event/IgnoreAllEvent.java

@@ -4,7 +4,7 @@
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
+import org.jspecify.annotations.NonNull;
 
 /**
  * This event is called when a player wants to ignore all players.
@@ -38,7 +38,7 @@ public void setCancelled(boolean cancel) {
         this.cancelled = cancel;
     }
 
-    @NotNull
+    @NonNull
     @Override
     public HandlerList getHandlers() {
         return HANDLER_LIST;

eternalcore-api/src/main/java/com/eternalcode/core/feature/ignore/event/IgnoreEvent.java

@@ -4,7 +4,7 @@
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
+import org.jspecify.annotations.NonNull;
 
 /**
  * This event is called when a player wants to ignore another player.
@@ -48,7 +48,7 @@ public void setCancelled(boolean cancel) {
     }
 
     @Override
-    public @NotNull HandlerList getHandlers() {
+    public @NonNull HandlerList getHandlers() {
         return HANDLER_LIST;
     }
 

eternalcore-api/src/main/java/com/eternalcode/core/feature/ignore/event/UnIgnoreAllEvent.java

@@ -4,7 +4,7 @@
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
+import org.jspecify.annotations.NonNull;
 
 /**
  * This event is called when a player ignores all other players.
@@ -38,7 +38,7 @@ public void setCancelled(boolean cancel) {
         this.cancelled = cancel;
     }
 
-    @NotNull
+    @NonNull
     @Override
     public HandlerList getHandlers() {
         return HANDLER_LIST;