feat: Add logging to backup command sub-commands

Introduced Logger.info calls to all backup command sub-commands for improved traceability of user actions. Fixed the 'info' command to execute the correct handler and corrected the restore command's success message. These changes enhance debugging and provide better feedback in server logs.

Author: Mqxx

src/main/java/dev/loat/command/CommandManager.java

@@ -32,11 +32,9 @@ public static void register() {
                 .then(Commands.literal(Abort.COMMAND).executes(Abort::execute))
                 .then(Commands.literal(List.COMMAND).executes(List::execute))
                 .then(Commands.literal(Info.COMMAND)
-                    .then(Commands.argument(Info.ARGUMENT, StringArgumentType.string()).suggests(new BackupFiles()).executes(Restore::execute))
+                    .then(Commands.argument(Info.ARGUMENT, StringArgumentType.string()).suggests(new BackupFiles()).executes(Info::execute))
                 )
                 .then(Commands.literal(Cleanup.COMMAND).executes(Cleanup::execute))
         ));
     }
 }
-
-

src/main/java/dev/loat/command/sub_command/Abort.java

@@ -1,14 +1,27 @@
 package dev.loat.command.sub_command;
 
 import com.mojang.brigadier.context.CommandContext;
+
+import dev.loat.logging.Logger;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.network.chat.Component;
+import net.minecraft.world.entity.player.Player;
 
 
 public final class Abort {
     public static final String COMMAND = "abort";
 
     public static int execute(CommandContext<CommandSourceStack> context) {
+        String playerName;
+
+        if (context.getSource().getEntity() instanceof Player) {
+            playerName = context.getSource().getPlayer().getName().getString();
+        } else {
+            playerName = "Console";
+        }
+        
+        Logger.info("[%s] Aborting current action".formatted(playerName));
+
         context.getSource().sendSuccess(() -> Component.literal("/backup abort"), false);
         return 1;
     }

src/main/java/dev/loat/command/sub_command/Cleanup.java

@@ -1,6 +1,8 @@
 package dev.loat.command.sub_command;
 
 import com.mojang.brigadier.context.CommandContext;
+
+import dev.loat.logging.Logger;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.network.chat.Component;
 
@@ -9,6 +11,8 @@ public final class Cleanup {
     public static final String COMMAND = "cleanup";
 
     public static int execute(CommandContext<CommandSourceStack> context) {
+        Logger.info("Running cleanup process");
+
         context.getSource().sendSuccess(() -> Component.literal("/backup cleanup"), false);
         return 1;
     }

src/main/java/dev/loat/command/sub_command/Create.java

@@ -16,13 +16,14 @@ public final class Create extends Command {
     @SuppressWarnings("null")
     public static int execute(CommandContext<CommandSourceStack> context) {
         String comment;
+        String playerName = context.getSource().getPlayer().getName().getString();
 
         if(Create.hasArgument(context, Create.ARGUMENT)) {
             comment = StringArgumentType.getString(context, Create.ARGUMENT);
-            Logger.info("Creating new backup with comment \"%s\"".formatted(comment));
+            Logger.info("[%s] Creating new backup with comment \"%s\"".formatted(playerName, comment));
         } else {
             comment = "";
-            Logger.info("Creating new backup");
+            Logger.info("[%s] Creating new backup".formatted(playerName));
         }
 
         context.getSource().sendSuccess(() -> Component.literal("/backup create \"%s\"".formatted(comment)), false);

src/main/java/dev/loat/command/sub_command/Help.java

@@ -1,6 +1,8 @@
 package dev.loat.command.sub_command;
 
 import com.mojang.brigadier.context.CommandContext;
+
+import dev.loat.logging.Logger;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.network.chat.Component;
 
@@ -9,6 +11,8 @@ public final class Help {
     public static final String COMMAND = "help";
 
     public static int execute(CommandContext<CommandSourceStack> context) {
+        Logger.info("Getting help");
+        
         context.getSource().sendSuccess(() -> Component.literal("/backup help"), false);
         return 1;
     }

src/main/java/dev/loat/command/sub_command/Info.java

@@ -1,6 +1,9 @@
 package dev.loat.command.sub_command;
 
+import com.mojang.brigadier.arguments.StringArgumentType;
 import com.mojang.brigadier.context.CommandContext;
+
+import dev.loat.logging.Logger;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.network.chat.Component;
 
@@ -9,8 +12,13 @@ public final class Info {
     public static final String COMMAND = "info";
     public static final String ARGUMENT = "name";
 
+    @SuppressWarnings("null")
     public static int execute(CommandContext<CommandSourceStack> context) {
-        context.getSource().sendSuccess(() -> Component.literal("/backup info"), false);
+        String name = StringArgumentType.getString(context, Info.ARGUMENT);
+
+        Logger.info("Getting info about existing backup \"%s\"".formatted(name));
+        
+        context.getSource().sendSuccess(() -> Component.literal("/backup info \"%s\"".formatted(name)), false);
         return 1;
     }
 }

src/main/java/dev/loat/command/sub_command/List.java

@@ -1,6 +1,8 @@
 package dev.loat.command.sub_command;
 
 import com.mojang.brigadier.context.CommandContext;
+
+import dev.loat.logging.Logger;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.network.chat.Component;
 
@@ -9,6 +11,8 @@ public final class List {
     public static final String COMMAND = "list";
 
     public static int execute(CommandContext<CommandSourceStack> context) {
+        Logger.info("Listing all backups");
+        
         context.getSource().sendSuccess(() -> Component.literal("/backup list"), false);
         return 1;
     }

src/main/java/dev/loat/command/sub_command/Reload.java

@@ -1,6 +1,8 @@
 package dev.loat.command.sub_command;
 
 import com.mojang.brigadier.context.CommandContext;
+
+import dev.loat.logging.Logger;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.network.chat.Component;
 
@@ -9,6 +11,8 @@ public final class Reload {
     public static final String COMMAND = "reload";
 
     public static int execute(CommandContext<CommandSourceStack> context) {
+        Logger.info("Reloading config");
+       
         context.getSource().sendSuccess(() -> Component.literal("/backup reload"), false);
         return 1;
     }

src/main/java/dev/loat/command/sub_command/Restore.java

@@ -29,7 +29,7 @@ public static int execute(CommandContext<CommandSourceStack> context) {
 
         Logger.info("Restoring existing backup \"%s\"".formatted(name));
 
-        context.getSource().sendSuccess(() -> Component.literal("/backup create \"%s\"".formatted(name)), false);
+        context.getSource().sendSuccess(() -> Component.literal("/backup restore \"%s\"".formatted(name)), false);
         return 1;
     }
 }