Fix xp, health, and food level desyncing when setting cape

Author: eclipseisoffline

src/main/java/xyz/eclipseisoffline/capecommand/CapeCommand.java

@@ -13,7 +13,6 @@
 import net.fabricmc.fabric.api.networking.v1.ServerConfigurationNetworking;
 import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
 import net.minecraft.entity.Entity;
-import net.minecraft.entity.player.PlayerPosition;
 import net.minecraft.network.PacketByteBuf;
 import net.minecraft.network.codec.PacketCodec;
 import net.minecraft.network.packet.CustomPayload;
@@ -28,12 +27,12 @@
 import net.minecraft.server.world.ServerChunkLoadingManager;
 import net.minecraft.text.Text;
 import net.minecraft.util.Identifier;
-import net.minecraft.util.math.Vec3d;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import xyz.eclipseisoffline.capecommand.mixin.EntityAccessor;
 import xyz.eclipseisoffline.capecommand.mixin.ServerChunkLoadingManagerAccessor;
 import xyz.eclipseisoffline.capecommand.mixin.ServerConfigurationNetworkHandlerAccessor;
+import xyz.eclipseisoffline.capecommand.mixin.ServerPlayerEntityAccessor;
 
 import java.util.List;
 
@@ -68,13 +67,19 @@ public void onInitialize() {
 
                                             ServerPlayerEntity player = context.getSource().getPlayerOrThrow();
                                             if (cape.requiresClient() && !CONFIG.hasCapeCommand(player)) {
-                                                throw new SimpleCommandExceptionType(Text.of("This cape requires you to install the Cape Command mod locally.")).create();
+                                                throw new SimpleCommandExceptionType(Text.of("This cape requires you to install the Cape Command mod locally")).create();
                                             }
 
                                             CONFIG.setPlayerCape(context.getSource().getPlayerOrThrow().getGameProfile(), cape);
 
                                             reloadPlayerSkin(context.getSource());
-                                            context.getSource().sendFeedback(() -> Text.of("Cape saved. Note that this cape is only visible to you and other players that have Cape Command installed."), true);
+                                            context.getSource().sendFeedback(() -> Text.of("Now wearing cape \"" + capeString.toLowerCase() + "\""), true);
+
+                                            if (CONFIG.isGeyserAvailable()) {
+                                                context.getSource().sendFeedback(() -> Text.of("Note that this cape is only visible to you, bedrock players, and other Java players that have Cape Command installed"), false);
+                                            } else {
+                                                context.getSource().sendFeedback(() -> Text.of("Note that this cape is only visible to you and other players that have Cape Command installed"), false);
+                                            }
 
                                             return 0;
                                         }))
@@ -158,6 +163,9 @@ private void reloadPlayerSkin(ServerCommandSource source) throws CommandSyntaxEx
                 // Client clears these when respawning
                 source.getServer().getPlayerManager().sendPlayerStatus(player);
                 source.getServer().getPlayerManager().sendStatusEffects(player);
+                ((ServerPlayerEntityAccessor) player).setSyncedExperience(-1);
+                ((ServerPlayerEntityAccessor) player).setSyncedHealth(-1.0F);
+                ((ServerPlayerEntityAccessor) player).setSyncedFoodLevel(-1);
 
                 continue;
             }

src/main/java/xyz/eclipseisoffline/capecommand/CapeConfig.java

@@ -62,6 +62,10 @@ public void unregisterCapeCommandPlayer(ServerPlayerEntity serverPlayerEntity) {
         capeCommandPlayers.remove(serverPlayerEntity.getGameProfile());
     }
 
+    public boolean isGeyserAvailable() {
+        return geyserAvailable;
+    }
+
     private void writeToConfig() {
         JsonObject capesJson = new JsonObject();
         for (Entry<UUID, Cape> playerCape : playerCapes.entrySet()) {

src/main/java/xyz/eclipseisoffline/capecommand/mixin/ServerPlayerEntityAccessor.java

@@ -0,0 +1,18 @@
+package xyz.eclipseisoffline.capecommand.mixin;
+
+import net.minecraft.server.network.ServerPlayerEntity;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.gen.Accessor;
+
+@Mixin(ServerPlayerEntity.class)
+public interface ServerPlayerEntityAccessor {
+
+    @Accessor("syncedExperience")
+    void setSyncedExperience(int syncedExperience);
+
+    @Accessor("syncedHealth")
+    void setSyncedHealth(float syncedHealth);
+
+    @Accessor("syncedFoodLevel")
+    void setSyncedFoodLevel(int syncedFoodLevel);
+}

src/main/resources/capecommand.mixins.json

@@ -9,6 +9,7 @@
     "ServerChunkLoadingManagerAccessor",
     "ServerConfigurationNetworkHandlerAccessor",
     "ServerPlayNetworkHandlerMixin",
+    "ServerPlayerEntityAccessor",
     "TextureUrlCheckerMixin"
   ],
   "client": [