fix: update argument types from UUID to Player in history commands

[TheMeinerLP]

Overview

Fixes #3303

Description

Submitter Checklist

• Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
• Ensure that the pull request title represents the desired changelog entry.
• New public fields and methods are annotated with @since TODO.
• I read and followed the contribution guidelines.

[PierreSchwang]

java.lang.IllegalStateException: No converter for type Key{type=com.sk89q.worldedit.entity.Player, annotationWrapper=NullAnnotationWrapper}

[TheMeinerLP]

java.lang.IllegalStateException: No converter for type Key{type=com.sk89q.worldedit.entity.Player, annotationWrapper=NullAnnotationWrapper}

@MattBDev told me already it

[TheMeinerLP]

@PierreSchwang its more complicated because something has changed in paper since 2 versions.

[PierreSchwang]

I mean, simply keeping the UUID and not looking up the UUID for suggestions would be a quick and working fix:

Index: worldedit-core/src/main/java/com/fastasyncworldedit/core/extension/platform/binding/ConsumeBindings.java
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/worldedit-core/src/main/java/com/fastasyncworldedit/core/extension/platform/binding/ConsumeBindings.java b/worldedit-core/src/main/java/com/fastasyncworldedit/core/extension/platform/binding/ConsumeBindings.java
--- a/worldedit-core/src/main/java/com/fastasyncworldedit/core/extension/platform/binding/ConsumeBindings.java	(revision d83f24a772ab5346a826a9bf8f418cd23ce339b2)
+++ b/worldedit-core/src/main/java/com/fastasyncworldedit/core/extension/platform/binding/ConsumeBindings.java	(date 1758062137429)
@@ -29,6 +29,7 @@
 import com.sk89q.worldedit.world.block.BlockType;
 import org.enginehub.piston.exception.StopExecutionException;
 import org.enginehub.piston.inject.InjectedValueAccess;
+import org.enginehub.piston.inject.Key;
 
 import java.util.UUID;
 
@@ -153,7 +154,7 @@
     }
 
     @Binding
-    public UUID playerUUID(Actor actor, String argument) {
+    public UUID playerUUID(Actor actor, InjectedValueAccess context, String argument) {
         if (argument.equals("me")) {
             return actor.getUniqueId();
         }
@@ -167,6 +168,10 @@
         if (argument.length() > 16) {
             uuid = UUID.fromString(argument);
         } else {
+            // if the command is parsed for suggestions, we just return the actors UUID
+            if (context.injectedValue(Key.of(boolean.class)).orElse(false)) {
+                return actor.getUniqueId();
+            }
             uuid = Fawe.platform().getUUID(argument);
         }
         if (uuid == null) {

There is no reason to get the actual UUID for suggestions either way imo

[TheMeinerLP]

Thanks @PierreSchwang