Command Rework

src/main/java/ch/njol/skript/Skript.java

@@ -4,7 +4,6 @@
 import ch.njol.skript.bukkitutil.BurgerHelper;
 import ch.njol.skript.classes.ClassInfo;
 import ch.njol.skript.classes.data.*;
-import ch.njol.skript.command.Commands;
 import ch.njol.skript.doc.Documentation;
 import ch.njol.skript.events.EvtSkript;
 import ch.njol.skript.expressions.arithmetic.ExprArithmetic;
@@ -44,15 +43,14 @@
 import org.bukkit.event.Event;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
-import org.bukkit.event.player.PlayerCommandPreprocessEvent;
 import org.bukkit.event.player.PlayerJoinEvent;
 import org.bukkit.event.server.PluginDisableEvent;
-import org.bukkit.event.server.ServerCommandEvent;
 import org.bukkit.event.server.ServerLoadEvent;
 import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.java.JavaPlugin;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 import org.jetbrains.annotations.UnknownNullability;
 import org.jetbrains.annotations.Unmodifiable;
@@ -63,6 +61,7 @@
 import org.skriptlang.skript.addon.AddonModule;
 import org.skriptlang.skript.bukkit.BukkitModule;
 import org.skriptlang.skript.bukkit.SkriptMetrics;
+import org.skriptlang.skript.bukkit.command.elements.effects.EffCommand;
 import org.skriptlang.skript.bukkit.lang.eventvalue.EventValueRegistry;
 import org.skriptlang.skript.bukkit.log.runtime.BukkitRuntimeErrorConsumer;
 import org.skriptlang.skript.bukkit.registration.BukkitSyntaxInfos;
@@ -80,7 +79,9 @@
 import org.skriptlang.skript.lang.script.Script;
 import org.skriptlang.skript.lang.structure.Structure;
 import org.skriptlang.skript.lang.structure.StructureInfo;
+import org.skriptlang.skript.log.runtime.ErrorSource;
 import org.skriptlang.skript.log.runtime.RuntimeErrorManager;
+import org.skriptlang.skript.log.runtime.RuntimeErrorProducer;
 import org.skriptlang.skript.registration.DefaultSyntaxInfos;
 import org.skriptlang.skript.registration.SyntaxInfo;
 import org.skriptlang.skript.registration.SyntaxRegistry;
@@ -576,8 +577,6 @@ public String name() {
 		// todo: remove completely 2.11 or 2.12
 		CompletableFuture<Boolean> aliases = Aliases.loadAsync();
 
-		Commands.registerListeners();
-
 		if (logNormal())
 			info(" " + Language.get("skript.copyright"));
 
@@ -1811,26 +1810,24 @@ public static <E extends Structure> void registerStructure(
 	 * @param sender
 	 * @param command
 	 * @return Whether the command was run
+	 * @deprecated There is no replacement for this method.
 	 */
-	public static boolean dispatchCommand(final CommandSender sender, final String command) {
-		try {
-			if (sender instanceof Player) {
-				final PlayerCommandPreprocessEvent e = new PlayerCommandPreprocessEvent((Player) sender, "/" + command);
-				Bukkit.getPluginManager().callEvent(e);
-				if (e.isCancelled() || !e.getMessage().startsWith("/"))
-					return false;
-				return Bukkit.dispatchCommand(e.getPlayer(), e.getMessage().substring(1));
-			} else {
-				final ServerCommandEvent e = new ServerCommandEvent(sender, command);
-				Bukkit.getPluginManager().callEvent(e);
-				if (e.getCommand().isEmpty() || e.isCancelled())
-					return false;
-				return Bukkit.dispatchCommand(e.getSender(), e.getCommand());
+	@Deprecated(since = "INSERT VERSION", forRemoval = true)
+	public static boolean dispatchCommand(CommandSender sender, String command) {
+		return EffCommand.dispatchCommand(sender, command, new RuntimeErrorProducer() {
+			@Override
+			public @NotNull ErrorSource getErrorSource() {
+				throw new UnsupportedOperationException();
 			}
-		} catch (final Exception ex) {
-			ex.printStackTrace(); // just like Bukkit
-			return false;
-		}
+			@Override
+			public void error(String message) { }
+			@Override
+			public void error(String message, String highlight) { }
+			@Override
+			public void warning(String message) { }
+			@Override
+			public void warning(String message, String highlight) { }
+		});
 	}
 
 	// ================ LOGGING ================

src/main/java/ch/njol/skript/bukkitutil/CommandReloader.java

@@ -10,7 +10,9 @@
 
 /**
  * Utilizes CraftServer with reflection to re-send commands to clients.
+ * @deprecated This behavior should not be relied upon. There is no replacement.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class CommandReloader {
 
 	@Nullable

src/main/java/ch/njol/skript/command/Argument.java

@@ -22,7 +22,10 @@
  * Represents an argument of a command
  * 
  * @author Peter Güttinger
+ * @deprecated There is no direct replacement for this class.
+ * The closest alternative is {@link org.skriptlang.skript.bukkit.command.custom.ArgumentData}.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class Argument<T> {
 
 	@Nullable

src/main/java/ch/njol/skript/command/CommandUsage.java

@@ -10,7 +10,9 @@
 /**
  * Holds info about the usage of a command.
  * TODO: replace with record when java 17
+ * @deprecated There is no direct replacement for this class.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class CommandUsage {
 
 	/**

src/main/java/ch/njol/skript/command/Commands.java

@@ -22,8 +22,6 @@
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
 import org.bukkit.event.player.AsyncPlayerChatEvent;
-import org.bukkit.event.player.PlayerCommandPreprocessEvent;
-import org.bukkit.event.server.ServerCommandEvent;
 import org.bukkit.help.HelpMap;
 import org.bukkit.help.HelpTopic;
 import org.bukkit.plugin.SimplePluginManager;
@@ -125,46 +123,6 @@ public static String unescape(String string) {
 		return "" + unescape.matcher(string).replaceAll("$0");
 	}
 
-	private final static Listener commandListener = new Listener() {
-
-		@EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
-		public void onPlayerCommand(PlayerCommandPreprocessEvent event) {
-			// Spigot will simply report that the command doesn't exist if a player does not have permission to use it.
-			// This is good security but, well, it's a breaking change for Skript. So we need to check for permissions
-			// ourselves and handle those messages, for every command.
-
-			// parse command, see if it's a skript command
-			String[] cmd = event.getMessage().substring(1).split("\\s+", 2);
-			String label = cmd[0].toLowerCase(Locale.ENGLISH);
-			String arguments = cmd.length == 1 ? "" : "" + cmd[1];
-			ScriptCommand command = commands.get(label);
-
-			// is it a skript command?
-			if (command != null) {
-				// if so, check permissions to handle ourselves
-				if (!command.checkPermissions(event.getPlayer(), label, arguments))
-					event.setCancelled(true);
-
-				// we can also handle case sensitivity here:
-				if (SkriptConfig.caseInsensitiveCommands.value()) {
-					cmd[0] = event.getMessage().charAt(0) + label;
-					event.setMessage(String.join(" ", cmd));
-				}
-			}
-		}
-
-		@SuppressWarnings("null")
-		@EventHandler(priority = EventPriority.HIGHEST)
-		public void onServerCommand(ServerCommandEvent event) {
-			if (event.getCommand().isEmpty() || event.isCancelled())
-				return;
-			if ((Skript.testing() || SkriptConfig.enableEffectCommands.value()) && event.getCommand().startsWith(SkriptConfig.effectCommandToken.value())) {
-				if (handleEffectCommand(event.getSender(), event.getCommand()))
-					event.setCancelled(true);
-			}
-		}
-	};
-
 	static boolean handleEffectCommand(CommandSender sender, String command) {
 		if (!(Skript.testing() || sender instanceof ConsoleCommandSender || sender.hasPermission("skript.effectcommands") || SkriptConfig.allowOpsToUseEffectCommands.value() && sender.isOp()))
 			return false;
@@ -279,8 +237,6 @@ public static void unregisterCommand(ScriptCommand scriptCommand) {
 
 	public static void registerListeners() {
 		if (!registeredListeners) {
-			Bukkit.getPluginManager().registerEvents(commandListener, Skript.getInstance());
-
 			Bukkit.getPluginManager().registerEvents(new Listener() {
 				@EventHandler(priority = EventPriority.LOWEST, ignoreCancelled = true)
 				public void onPlayerChat(AsyncPlayerChatEvent event) {

src/main/java/ch/njol/skript/command/ScriptCommand.java

@@ -64,7 +64,10 @@
 
 /**
  * This class is used for user-defined commands.
+ * @deprecated There is no direct replacement for this command.
+ * The closest alternative is {@link org.skriptlang.skript.bukkit.command.custom.ScriptBrigadierCommand}.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class ScriptCommand implements TabExecutor {
 
 	public final static Message m_executable_by_players = new Message("commands.executable by players");
@@ -332,7 +335,7 @@ public boolean execute(final CommandSender sender, final String commandLabel, fi
 	boolean execute2(final ScriptCommandEvent event, final CommandSender sender, final String commandLabel, final String rest) {
 		final ParseLogHandler log = SkriptLogger.startParseLogHandler();
 		try {
-			final boolean ok = SkriptParser.parseArguments(rest, ScriptCommand.this, event);
+			final boolean ok = false;
 			if (!ok) {
 				final LogEntry e = log.getError();
 				if (e != null)

src/main/java/ch/njol/skript/command/ScriptCommandEvent.java

@@ -6,6 +6,11 @@
 import org.bukkit.entity.Player;
 import org.bukkit.event.HandlerList;
 
+/**
+ * @deprecated There is no direct replacement for this class.
+ * The closest alternative is {@link org.skriptlang.skript.bukkit.command.custom.ScriptCommandEvent}.
+ */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class ScriptCommandEvent extends CommandEvent {
 
 	private final ScriptCommand scriptCommand;

src/main/java/ch/njol/skript/command/package-info.java

@@ -1,25 +1 @@
-/**
- *   This file is part of Skript.
- *
- *  Skript is free software: you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation, either version 3 of the License, or
- *  (at your option) any later version.
- *
- *  Skript is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with Skript.  If not, see <http://www.gnu.org/licenses/>.
- *
- * Copyright Peter Güttinger, SkriptLang team and contributors
- */
-/**
- * Code related to handling commands, either Skript commands or custom script commands.
- * 
- * @author Peter Güttinger
- */
 package ch.njol.skript.command;
-