fix(command): /fp say 假人名字补全 + 修复 say stop 崩溃 (#66)

#64 /fp say once|continuous|interval 的 name 参数没有补全:
- 新增 CommandSupports#fakeplayerName(String): 底层保持 StringArgument (保留
  「单参数回退成消息」, /fp say once 1234 行为不变), 但挂上假人名字的 ArgumentSuggestions
- 补全语义与 fakeplayer(...) 一致: op / 控制台列出全部, 普通玩家只列自己的

#65 /fp say stop 完全不可用:
- say stop 的 name 来自假人参数 (Player), SayCommand 无条件强转 String -> ClassCastException
- 不传名字时又被「消息不能为空」拦下 -> 该子命令两种写法都走不到停止逻辑
- 现按类型取名字, 且 stop 不再要求消息

均为本地实机验证 (Purpur 26.3 + CommandAPI 12.1.0)。

Co-authored-by: EndlessPixel <EndlessPixel@users.noreply.github.com>

Author: EndlessPixel

CHANGELOG.md

@@ -20,6 +20,8 @@ Changelog for FakePlayer CE. Version `fp.buildN` matches the git tag / GitHub Re
 - Fixed `follow-quiting-force` removing a creator's fake players even when the creator reconnected within the configured delay (quick reconnect, or coming back from another server) — the fake players are now kept when the creator is online again.
 - `/fp config` (including `config list` and `config set`) now works from the console — previously it failed with CommandAPI's internal error `This command has no implementations for terminalconsolecommandsender`, because the command was player-only. On the console `list` shows the global defaults, and operators / the console can target a specific player with the new optional `[player]` argument (e.g. `/fp config set autofish true <player>`), while regular players can still only configure themselves. `/fp config help` was added as well — it previously printed nothing at all.
 - Fixed `/fp say` (and `say continuous` / `say interval`) rejecting the name of a fake player owned by someone else with "you haven't spawned a fake player yet" — from the console this made it impossible to address any fake player by name at all. An explicitly given name is now resolved with the same rules as every other command: operators and the console may target any fake player, regular players only their own.
+- `/fp say once` / `continuous` / `interval` now TAB-complete fake player names — the name argument previously offered no suggestions at all (unlike every other command), because it is parsed as a plain string so that a single argument still falls back to being the message.
+- Fixed `/fp say stop` being entirely unusable: with a fake-player name it threw a `ClassCastException` ("An unexpected error occurred"), and without one it complained about an empty message instead of stopping. `stop` no longer requires a message; the single-argument message fallback of the other subcommands is unchanged.
 
 ### 中文
 
@@ -31,6 +33,8 @@ Changelog for FakePlayer CE. Version `fp.buildN` matches the git tag / GitHub Re
 - 修复 `follow-quiting-force` 在创建者于延迟内重新上线（快速重连，或从其他服务器返回）时仍会清理其假人的问题；创建者已回到服务器时现在会保留假人。
 - `/fp config`（含 `config list` 与 `config set`）现在可以在控制台使用——此前会报 CommandAPI 的内部错误 `This command has no implementations for terminalconsolecommandsender`，因为该命令只注册了玩家执行器。控制台下 `list` 展示全局默认值；op / 控制台可用新增的可选 `[player]` 参数指定目标玩家（如 `/fp config set autofish true <玩家>`），普通玩家仍然只能配置自己。同时补上了 `/fp config help`——此前执行它完全没有输出。
 - 修复 `/fp say`（及 `say continuous` / `say interval`）指定「他人的假人名称」时报「你还没召唤假人呢」的问题——在控制台里这会导致完全无法按名字指定假人。显式给出的名字现按与其他命令一致的规则解析：op 与控制台可指定任意假人，普通玩家只能指定自己召唤的假人。
+- `/fp say once` / `continuous` / `interval` 的假人名字现在支持 TAB 补全——此前该参数没有任何补全（与其他命令不同），因为为了保留「单参数即消息」的回退，它按字符串解析。
+- 修复 `/fp say stop` 完全不可用的问题：带假人名字时会抛 `ClassCastException`（提示发生意外错误），不带名字时则报「消息不能为空」而不执行停止。`stop` 不再要求消息；其余子命令的「单参数回退成消息」行为不变。
 
 ## fp.build12 - 2026-09-26
 

fakeplayer-core/src/main/java/io/github/hello09x/fakeplayer/core/command/CommandRegistry.java

@@ -5,7 +5,6 @@
 import dev.jorel.commandapi.CommandPermission;
 import dev.jorel.commandapi.arguments.EntitySelectorArgument;
 import dev.jorel.commandapi.arguments.GreedyStringArgument;
-import dev.jorel.commandapi.arguments.StringArgument;
 import io.github.hello09x.devtools.command.HelpCommand;
 import io.github.hello09x.devtools.core.translation.TranslatorUtils;
 import io.github.hello09x.fakeplayer.api.spi.ActionSetting;
@@ -504,20 +503,20 @@ public void register() {
                                         command("once")
                                                 .withShortDescription("fakeplayer.command.say.once")
                                                 .withOptionalArguments(
-                                                        new StringArgument("name"),
+                                                        fakeplayerName("name"),
                                                         new GreedyStringArgument("message"))
                                                 .executes(sayCommand.say(ActionSetting.once())),
                                         command("continuous")
                                                 .withShortDescription("fakeplayer.command.say.continuous")
                                                 .withOptionalArguments(
-                                                        new StringArgument("name"),
+                                                        fakeplayerName("name"),
                                                         new GreedyStringArgument("message"))
                                                 .executes(sayCommand.say(ActionSetting.continuous())),
                                         command("interval")
                                                 .withShortDescription("fakeplayer.command.say.interval")
                                                 .withOptionalArguments(
                                                         int32("ticks", 1),
-                                                        new StringArgument("name"),
+                                                        fakeplayerName("name"),
                                                         new GreedyStringArgument("message"))
                                                 .executes((sender, args) -> {
                                                     int interval = (int) args.getOptional("ticks").orElse(1);

fakeplayer-core/src/main/java/io/github/hello09x/fakeplayer/core/command/CommandSupports.java

@@ -101,6 +101,28 @@ public abstract class CommandSupports {
         return fakeplayer(nodeName, null);
     }
 
+    /**
+     * 与 {@link #fakeplayer(String)} 相同的假人名字补全, 但参数底层类型保持字符串。
+     * <p>供 `/fp say once|continuous|interval` 使用: 这些子命令的「单参数回退成消息」
+     * (如 {@code /fp say once 1234}) 依赖 name 是字符串, 不能换成 {@link CustomArgument}
+     * 那种假人参数 (否则 1234 会因为解析不出假人而直接报错), 所以这里只额外挂上 TAB 补全。</p>
+     */
+    public static @NotNull Argument<String> fakeplayerName(@NotNull String nodeName) {
+        return new StringArgument(nodeName).replaceSuggestions(ArgumentSuggestions.strings(info -> {
+            var sender = info.sender();
+            var arg = info.currentArg().toLowerCase(Locale.ROOT);
+
+            var targets = sender.isOp()
+                    ? manager().getAll()
+                    : manager().getAll(sender);
+
+            return targets.stream()
+                    .map(Player::getName)
+                    .filter(name -> name.toLowerCase(Locale.ROOT).contains(arg))
+                    .toArray(String[]::new);
+        }));
+    }
+
     public static @NotNull Argument<List<Player>> fakeplayers(@NotNull String nodeName) {
         return new CustomArgument<List<Player>, String>(new StringArgument(nodeName), info -> {
             var sender = info.sender();

fakeplayer-core/src/main/java/io/github/hello09x/fakeplayer/core/command/impl/SayCommand.java

@@ -7,6 +7,7 @@
 import io.github.hello09x.fakeplayer.api.spi.ActionType;
 import io.github.hello09x.fakeplayer.core.manager.action.ActionManager;
 import org.bukkit.command.ConsoleCommandSender;
+import org.bukkit.entity.Player;
 import org.jetbrains.annotations.NotNull;
 
 import static net.kyori.adventure.text.Component.translatable;
@@ -29,17 +30,25 @@ public SayCommand(ActionManager actionManager) {
      */
     public @NotNull CommandExecutor say(@NotNull ActionSetting setting) {
         return (sender, args) -> {
-            var name = (String) args.get("name");
+            // name 在 once / continuous / interval 里是字符串 (为了兼容单参数回退成消息),
+            // 在 stop 里则来自假人参数 (Player), 所以这里不能直接强转 String
+            var rawName = args.get("name");
+            var name = rawName instanceof Player player ? player.getName() : (String) rawName;
             var message = (String) args.get("message");
-            // 单参数回退: fp say once 1234 -> 把 1234 当作消息, 作用于默认假人
-            if (message == null || message.isBlank()) {
-                if (name != null && !name.isBlank()) {
-                    message = name;
-                } else {
-                    sender.sendMessage(translatable("fakeplayer.command.say.error.empty"));
-                    return;
+
+            // stop 只是停止动作, 不需要消息
+            if (!setting.equals(ActionSetting.stop())) {
+                // 单参数回退: fp say once 1234 -> 把 1234 当作消息, 作用于默认假人
+                if (message == null || message.isBlank()) {
+                    if (name != null && !name.isBlank()) {
+                        message = name;
+                    } else {
+                        sender.sendMessage(translatable("fakeplayer.command.say.error.empty"));
+                        return;
+                    }
                 }
             }
+
             var fake = super.getFakeplayer(sender, args);
             var copy = setting.clone();
             copy.message = message;