Fix optional positional args for admin claim and pvp commands

The Hytale command system treats withOptionalArg as flag-style arguments
(--name=value) rather than positional arguments. This caused errors like
"Expected: 0, actual: 1" when users typed "/easyclaims admin claim Spawn".

Fix by using setAllowsExtraArguments(true) and parsing the raw input string
to extract the optional positional argument after the command name.

Commands now work as expected:
- /easyclaims admin claim           (creates claim with default "Server" name)
- /easyclaims admin claim Spawn     (creates claim named "Spawn")
- /easyclaims admin pvp             (toggles PvP)
- /easyclaims admin pvp on          (enables PvP)
- /easyclaims admin pvp off         (disables PvP)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: cryptobench

src/main/java/com/easyclaims/commands/subcommands/admin/AdminClaimSubcommand.java

@@ -4,8 +4,6 @@
 import com.hypixel.hytale.component.Store;
 import com.hypixel.hytale.math.vector.Vector3d;
 import com.hypixel.hytale.server.core.command.system.CommandContext;
-import com.hypixel.hytale.server.core.command.system.arguments.system.OptionalArg;
-import com.hypixel.hytale.server.core.command.system.arguments.types.ArgTypes;
 import com.hypixel.hytale.server.core.command.system.basecommands.AbstractPlayerCommand;
 import com.hypixel.hytale.server.core.modules.entity.component.TransformComponent;
 import com.hypixel.hytale.server.core.universe.PlayerRef;
@@ -23,15 +21,15 @@
 /**
  * Admin command to create server-owned admin claims.
  * Admin claims have PvP disabled by default and bypass buffer zones.
+ * Usage: /easyclaims admin claim [name]
  */
 public class AdminClaimSubcommand extends AbstractPlayerCommand {
     private final EasyClaims plugin;
-    private final OptionalArg<String> displayNameArg;
 
     public AdminClaimSubcommand(EasyClaims plugin) {
         super("claim", "Create an admin claim at your location");
         this.plugin = plugin;
-        this.displayNameArg = withOptionalArg("name", "Display name for the claim (e.g., Spawn)", ArgTypes.STRING);
+        setAllowsExtraArguments(true); // Allow optional positional name argument
         requirePermission("easyclaims.admin");
     }
 
@@ -60,8 +58,18 @@ protected void execute(@Nonnull CommandContext ctx,
             return;
         }
 
-        // Get optional display name
-        String displayName = displayNameArg.get(ctx);
+        // Get optional display name from raw input (everything after "claim")
+        String displayName = null;
+        String input = ctx.getInputString();
+        // Input format: "easyclaims admin claim [name]" or just "claim [name]" depending on context
+        // Look for any extra arguments after command parsing
+        int claimIndex = input.toLowerCase().lastIndexOf("claim");
+        if (claimIndex >= 0) {
+            String afterClaim = input.substring(claimIndex + 5).trim();
+            if (!afterClaim.isEmpty()) {
+                displayName = afterClaim;
+            }
+        }
 
         // Register admin name for map display
         plugin.getClaimStorage().setPlayerName(AdminClaims.ADMIN_UUID,

src/main/java/com/easyclaims/commands/subcommands/admin/AdminPvpSubcommand.java

@@ -4,8 +4,6 @@
 import com.hypixel.hytale.component.Store;
 import com.hypixel.hytale.math.vector.Vector3d;
 import com.hypixel.hytale.server.core.command.system.CommandContext;
-import com.hypixel.hytale.server.core.command.system.arguments.system.OptionalArg;
-import com.hypixel.hytale.server.core.command.system.arguments.types.ArgTypes;
 import com.hypixel.hytale.server.core.command.system.basecommands.AbstractPlayerCommand;
 import com.hypixel.hytale.server.core.modules.entity.component.TransformComponent;
 import com.hypixel.hytale.server.core.universe.PlayerRef;
@@ -21,15 +19,15 @@
 /**
  * Admin command to toggle PvP in any claim.
  * Admins can override PvP settings regardless of server mode.
+ * Usage: /easyclaims admin pvp [on/off]
  */
 public class AdminPvpSubcommand extends AbstractPlayerCommand {
     private final EasyClaims plugin;
-    private final OptionalArg<String> stateArg;
 
     public AdminPvpSubcommand(EasyClaims plugin) {
         super("pvp", "Toggle PvP in the current claim");
         this.plugin = plugin;
-        this.stateArg = withOptionalArg("state", "on/off (omit to toggle)", ArgTypes.STRING);
+        setAllowsExtraArguments(true); // Allow optional positional on/off argument
         requirePermission("easyclaims.admin");
     }
 
@@ -50,8 +48,18 @@ protected void execute(@Nonnull CommandContext ctx,
             return;
         }
 
+        // Get optional state from raw input (everything after "pvp")
+        String state = null;
+        String input = ctx.getInputString();
+        int pvpIndex = input.toLowerCase().lastIndexOf("pvp");
+        if (pvpIndex >= 0) {
+            String afterPvp = input.substring(pvpIndex + 3).trim();
+            if (!afterPvp.isEmpty()) {
+                state = afterPvp.split("\\s+")[0]; // Take first word only
+            }
+        }
+
         // Determine new state
-        String state = stateArg.get(ctx);
         boolean newPvpEnabled;
 
         if (state == null || state.isEmpty()) {