Tick fake players in network updating phase

src/main/java/carpet/CarpetServer.java

@@ -15,6 +15,7 @@
 import carpet.commands.PerimeterInfoCommand;
 import carpet.commands.PlayerCommand;
 import carpet.commands.ProfileCommand;
+import carpet.patches.FakePlayerManager;
 import carpet.script.ScriptCommand;
 import carpet.commands.SpawnCommand;
 import carpet.commands.TestCommand;
@@ -190,6 +191,7 @@ public static void onServerClosed(MinecraftServer server)
             LoggerRegistry.stopLoggers();
             HUDController.resetScarpetHUDs();
             ParticleParser.resetCache();
+            FakePlayerManager.reset();
             extensions.forEach(e -> e.onServerClosed(server));
             minecraft_server = null;
         }

src/main/java/carpet/CarpetSettings.java

@@ -1062,4 +1062,11 @@ public enum FungusGrowthMode {
             category = {SURVIVAL, FEATURE}
     )
     public static FungusGrowthMode thickFungusGrowth = FungusGrowthMode.FALSE;
+
+
+    @Rule(
+            desc = "make fake players ticked in entity update phase, like legacy carpet.",
+            category = CREATIVE
+    )
+    public static boolean fakePlayerTicksInEU = false;
 }

src/main/java/carpet/mixins/PlayerList_fakePlayersMixin.java

@@ -1,5 +1,6 @@
 package carpet.mixins;
 
+import carpet.patches.FakePlayerManager;
 import com.mojang.authlib.GameProfile;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.network.Connection;
@@ -41,7 +42,9 @@ private ServerGamePacketListenerImpl replaceNetworkHandler(MinecraftServer serve
     {
         if (playerIn instanceof EntityPlayerMPFake fake)
         {
-            return new NetHandlerPlayServerFake(this.server, clientConnection, fake, cookie);
+            var handler = new NetHandlerPlayServerFake(this.server, clientConnection, fake, cookie);
+            FakePlayerManager.connections.add(handler);
+            return handler;
         }
         else
         {

src/main/java/carpet/mixins/ServerConnectionListener_fakePlayersMixin.java

@@ -0,0 +1,38 @@
+package carpet.mixins;
+
+import carpet.CarpetSettings;
+import carpet.fakes.ServerPlayerInterface;
+import carpet.patches.FakePlayerManager;
+import net.minecraft.server.network.ServerConnectionListener;
+import net.minecraft.server.network.ServerGamePacketListenerImpl;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
+
+@Mixin(ServerConnectionListener.class)
+public class ServerConnectionListener_fakePlayersMixin {
+    @Inject(
+            method = "tick",
+            at = @At("RETURN")
+    )
+    private void tickFakePlayers(CallbackInfo ci) {
+        if (!CarpetSettings.fakePlayerTicksInEU) {
+            for (ServerGamePacketListenerImpl connection : FakePlayerManager.connections) {
+                // from ServerGamePacketListenerImpl#tick
+
+                connection.player.xo = connection.player.getX();
+                connection.player.yo = connection.player.getY();
+                connection.player.zo = connection.player.getZ();
+
+                // action packet update
+                ((ServerPlayerInterface) connection.player).getActionPack().onUpdate();
+
+                connection.player.doTick();
+                //  connection.player.absMoveTo(connection.firstGoodX, connection.firstGoodY, connection.firstGoodZ, connection.player.getYRot(), connection.player.getXRot());
+
+                // todo: vehicle?
+            }
+        }
+    }
+}

src/main/java/carpet/mixins/ServerLevel_tickMixin.java

@@ -143,4 +143,4 @@ private void endRandomTicks(CallbackInfo ci) {
         }
     }
 
-}
+}

src/main/java/carpet/mixins/ServerPlayer_actionPackMixin.java

@@ -1,7 +1,9 @@
 package carpet.mixins;
 
+import carpet.CarpetSettings;
 import carpet.fakes.ServerPlayerInterface;
 import carpet.helpers.EntityPlayerActionPack;
+import carpet.patches.EntityPlayerMPFake;
 import com.mojang.authlib.GameProfile;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.level.ClientInformation;
@@ -18,6 +20,7 @@ public abstract class ServerPlayer_actionPackMixin implements ServerPlayerInterf
 {
     @Unique
     public EntityPlayerActionPack actionPack;
+
     @Override
     public EntityPlayerActionPack getActionPack()
     {
@@ -33,6 +36,16 @@ private void onServerPlayerEntityContructor(MinecraftServer minecraftServer, Ser
     @Inject(method = "tick", at = @At(value = "HEAD"))
     private void onTick(CallbackInfo ci)
     {
-        actionPack.onUpdate();
+        if (CarpetSettings.fakePlayerTicksInEU) {
+            actionPack.onUpdate();
+        }
+    }
+
+    @Inject(method = "doTick", at = @At(value = "HEAD"))
+    private void tickActionPack(CallbackInfo ci)
+    {
+        if (!((Object) this instanceof EntityPlayerMPFake)) {
+            actionPack.onUpdate();
+        }
     }
 }

src/main/java/carpet/patches/EntityPlayerMPFake.java

@@ -1,6 +1,8 @@
 package carpet.patches;
 
 import carpet.CarpetSettings;
+import carpet.fakes.ServerPlayerInterface;
+import carpet.utils.Messenger;
 import com.mojang.authlib.GameProfile;
 import net.minecraft.core.BlockPos;
 import net.minecraft.core.UUIDUtil;
@@ -30,8 +32,6 @@
 import net.minecraft.world.level.block.entity.SkullBlockEntity;
 import net.minecraft.world.level.block.state.BlockState;
 import net.minecraft.world.phys.Vec3;
-import carpet.fakes.ServerPlayerInterface;
-import carpet.utils.Messenger;
 
 import java.util.Optional;
 import java.util.concurrent.CompletableFuture;
@@ -164,7 +164,9 @@ public void tick()
         try
         {
             super.tick();
-            this.doTick();
+            if (CarpetSettings.fakePlayerTicksInEU) {
+                this.doTick();
+            }
         }
         catch (NullPointerException ignored)
         {

src/main/java/carpet/patches/FakePlayerManager.java

@@ -0,0 +1,14 @@
+package carpet.patches;
+
+import net.minecraft.server.network.ServerGamePacketListenerImpl;
+
+import java.util.ArrayList;
+import java.util.List;
+
+public class FakePlayerManager {
+    public static List<ServerGamePacketListenerImpl> connections = new ArrayList<>();
+
+    public static void reset() {
+        connections.clear();
+    }
+}

src/main/java/carpet/patches/NetHandlerPlayServerFake.java

@@ -1,9 +1,11 @@
 package carpet.patches;
 
+import net.minecraft.Util;
 import net.minecraft.network.Connection;
 import net.minecraft.network.chat.Component;
 import net.minecraft.network.chat.contents.TranslatableContents;
 import net.minecraft.network.protocol.Packet;
+import net.minecraft.network.protocol.game.ClientboundBlockChangedAckPacket;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.level.ServerPlayer;
 import net.minecraft.server.network.CommonListenerCookie;
@@ -23,6 +25,11 @@ public void send(final Packet<?> packetIn)
     {
     }
 
+    @Override
+    protected void keepConnectionAlive()
+    {
+    }
+
     @Override
     public void disconnect(Component message)
     {
@@ -42,6 +49,11 @@ public void teleport(double d, double e, double f, float g, float h, Set<Relativ
         }
     }
 
+    @Override
+    public void tick()
+    {
+        System.out.println("NetHandlerPlayServerFake tick -- not implemented yet");
+    }
 }
 
 

src/main/resources/carpet.mixins.json

@@ -57,6 +57,7 @@
     "MinecraftServer_scarpetMixin",
     "ExperienceOrb_xpNoCooldownMixin",
     "Player_xpNoCooldownMixin",
+    "ServerConnectionListener_fakePlayersMixin",
     "ServerPlayer_actionPackMixin",
     "PlayerList_coreMixin",
     "PlayerList_fakePlayersMixin",