feat: add logger for internal plugin, and format api code (#696)

leaves-api/src/main/java/org/leavesmc/leaves/config/LeavesConfigValue.java

@@ -1,5 +1,7 @@
 package org.leavesmc.leaves.config;
 
+import org.jetbrains.annotations.NotNull;
+
 public record LeavesConfigValue(Object value) {
 
     public int getInt() {
@@ -30,7 +32,7 @@ public record LeavesConfigValue(Object value) {
         throw new ClassCastException("Value is not a string");
     }
 
-    public String toString() {
+    public @NotNull String toString() {
         return value.toString();
     }
 }

leaves-api/src/main/java/org/leavesmc/leaves/entity/bot/action/SwapAction.java

@@ -1,6 +1,7 @@
 package org.leavesmc.leaves.entity.bot.action;
 
 import org.bukkit.Bukkit;
+
 /**
  * Represents an action for a bot to swap items between the main hand and off-hand.
  */

leaves-server/src/main/java/org/leavesmc/leaves/plugin/MinecraftInternalPlugin.java

@@ -1,11 +1,15 @@
 package org.leavesmc.leaves.plugin;
 
+import io.papermc.paper.plugin.configuration.PluginMeta;
+import io.papermc.paper.plugin.lifecycle.event.LifecycleEventManager;
+import org.bukkit.Bukkit;
 import org.bukkit.Server;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.generator.BiomeProvider;
 import org.bukkit.generator.ChunkGenerator;
+import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginBase;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.PluginLoader;
@@ -24,10 +28,12 @@ public class MinecraftInternalPlugin extends PluginBase {
     private boolean enabled = true;
 
     private final PluginDescriptionFile pdf;
+    private final PluginLogger logger;
 
     public MinecraftInternalPlugin() {
         String pluginName = "Minecraft";
         pdf = new PluginDescriptionFile(pluginName, "1.0", "nms");
+        logger = new PluginLogger(this);
     }
 
     public void setEnabled(boolean enabled) {
@@ -35,27 +41,47 @@ public class MinecraftInternalPlugin extends PluginBase {
     }
 
     @Override
-    public File getDataFolder() {
+    public @NotNull PluginDescriptionFile getDescription() {
-        throw new UnsupportedOperationException("Not supported.");
-    }
-
-    @Override
-    public PluginDescriptionFile getDescription() {
         return pdf;
     }
 
     @Override
-    public io.papermc.paper.plugin.configuration.PluginMeta getPluginMeta() {
+    public @NotNull PluginMeta getPluginMeta() {
         return pdf;
     }
 
     @Override
-    public FileConfiguration getConfig() {
+    public @NotNull PluginLogger getLogger() {
+        return logger;
+    }
+
+    @Override
+    public @NotNull Server getServer() {
+        return Bukkit.getServer();
+    }
+
+    @Override
+    public boolean isEnabled() {
+        return enabled;
+    }
+
+    @Override
+    public @NotNull PluginLoader getPluginLoader() {
         throw new UnsupportedOperationException("Not supported.");
     }
 
     @Override
-    public InputStream getResource(String filename) {
+    public @NotNull File getDataFolder() {
+        throw new UnsupportedOperationException("Not supported.");
+    }
+
+    @Override
+    public @NotNull FileConfiguration getConfig() {
+        throw new UnsupportedOperationException("Not supported.");
+    }
+
+    @Override
+    public InputStream getResource(@NotNull String filename) {
         throw new UnsupportedOperationException("Not supported.");
     }
 
@@ -70,7 +96,7 @@ public class MinecraftInternalPlugin extends PluginBase {
     }
 
     @Override
-    public void saveResource(String resourcePath, boolean replace) {
+    public void saveResource(@NotNull String resourcePath, boolean replace) {
         throw new UnsupportedOperationException("Not supported.");
     }
 
@@ -79,26 +105,6 @@ public class MinecraftInternalPlugin extends PluginBase {
         throw new UnsupportedOperationException("Not supported.");
     }
 
-    @Override
-    public PluginLogger getLogger() {
-        throw new UnsupportedOperationException("Not supported.");
-    }
-
-    @Override
-    public PluginLoader getPluginLoader() {
-        throw new UnsupportedOperationException("Not supported.");
-    }
-
-    @Override
-    public Server getServer() {
-        throw new UnsupportedOperationException("Not supported.");
-    }
-
-    @Override
-    public boolean isEnabled() {
-        return enabled;
-    }
-
     @Override
     public void onDisable() {
         throw new UnsupportedOperationException("Not supported.");
@@ -125,7 +131,7 @@ public class MinecraftInternalPlugin extends PluginBase {
     }
 
     @Override
-    public ChunkGenerator getDefaultWorldGenerator(String worldName, String id) {
+    public ChunkGenerator getDefaultWorldGenerator(@NotNull String worldName, String id) {
         throw new UnsupportedOperationException("Not supported.");
     }
 
@@ -135,17 +141,17 @@ public class MinecraftInternalPlugin extends PluginBase {
     }
 
     @Override
-    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+    public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, String @NotNull [] args) {
         throw new UnsupportedOperationException("Not supported.");
     }
 
     @Override
-    public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
+    public List<String> onTabComplete(@NotNull CommandSender sender, @NotNull Command command, @NotNull String alias, String @NotNull [] args) {
         throw new UnsupportedOperationException("Not supported.");
     }
 
     @Override
-    public @NotNull io.papermc.paper.plugin.lifecycle.event.LifecycleEventManager<org.bukkit.plugin.Plugin> getLifecycleManager() {
+    public @NotNull LifecycleEventManager<@NotNull Plugin> getLifecycleManager() {
         throw new UnsupportedOperationException("Not supported.");
     }
 }