Exclude CODEOWNERS from language statistics

Bumps [gradle-wrapper](https://github.com/gradle/gradle) from 9.4.1 to 9.5.0.
- [Release notes](https://github.com/gradle/gradle/releases)
- [Commits](https://github.com/gradle/gradle/compare/v9.4.1...v9.5.0)

---
updated-dependencies:
- dependency-name: gradle-wrapper
  dependency-version: 9.5.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

.gitattributes

@@ -0,0 +1 @@
+.gitea/CODEOWNERS linguist-detectable=false

.gitea/CODEOWNERS

@@ -0,0 +1,3 @@
+# Pull requests request review from the eligible staff team(s).
+# One approval from either an Admin or Developer satisfies branch protection.
+*   @trueog/Admins @trueog/Developers

.github/workflows/build.yml

@@ -10,13 +10,16 @@ jobs:
     runs-on: ubuntu-24.04
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # 6.0.2
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
         with:
           persist-credentials: false
       - name: Set up Gradle
-        uses: gradle/actions/setup-gradle@50e97c2cd7a37755bbfafc9c5b7cafaece252f6e # 6.1.0
+        uses: gradle/actions/setup-gradle@50e97c2cd7a37755bbfafc9c5b7cafaece252f6e # v6.1.0
+        with:
+          cache-disabled: false
+          cache-provider: basic
       - name: Set up JDK 17
-        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 # 5.2.0
+        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 # v5.2.0
         with:
           distribution: 'temurin'
           java-version: 17
@@ -24,7 +27,7 @@ jobs:
       - name: Build with Gradle
         run: ./gradlew build
       - name: Upload Artifacts to GitHub
-        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # 7.0.0
+        uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
         with:
           name: Artifacts
           path: build/libs/

.github/workflows/publish.yml

@@ -15,13 +15,16 @@ jobs:
     runs-on: ubuntu-24.04
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # 6.0.2
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
         with:
           persist-credentials: false
       - name: Set up Gradle
-        uses: gradle/actions/setup-gradle@50e97c2cd7a37755bbfafc9c5b7cafaece252f6e # 6.1.0
+        uses: gradle/actions/setup-gradle@50e97c2cd7a37755bbfafc9c5b7cafaece252f6e # v6.1.0
+        with:
+          cache-disabled: false
+          cache-provider: basic
       - name: Set up JDK 17
-        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 # 5.2.0
+        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 # v5.2.0
         with:
           distribution: 'temurin'
           java-version: 17

build.gradle.kts

@@ -2,6 +2,7 @@ import de.florianreuth.baseproject.*
 
 plugins {
     `java-library`
+    eclipse
     id("io.papermc.hangar-publish-plugin")
     id("com.modrinth.minotaur")
     id("de.florianreuth.baseproject")

common/src/main/java/com/viaversion/viarewind/ViaRewindConfig.java

@@ -55,8 +55,8 @@ public class ViaRewindConfig extends Config implements com.viaversion.viarewind.
         cooldownIndicator = CooldownIndicator.valueOf(getString("cooldown-indicator", "TITLE").toUpperCase());
         replaceAdventureMode = getBoolean("replace-adventure", false);
         replaceParticles = getBoolean("replace-particles", false);
-        maxBookPages = getInt("end-book-pages", 100);
-        maxBookPageSize = getInt("end-book-page-length", 5000);
+        maxBookPages = getInt("max-book-pages", 100);
+        maxBookPageSize = getInt("max-book-page-length", 5000);
         emulateWorldBorder = getBoolean("emulate-world-border", true);
         alwaysShowOriginalMobName = getBoolean("always-show-original-mob-name", true);
         worldBorderParticle = getString("world-border-particle", "fireworksSpark");

common/src/main/java/com/viaversion/viarewind/api/ViaRewindConfig.java

@@ -54,14 +54,14 @@ public interface ViaRewindConfig extends Config {
     /**
      * Max amount of pages for written books before a client gets kicked
      *
-     * @return the end amount of pages
+     * @return the max amount of pages
      */
     int getMaxBookPages();
 
     /**
      * Max amount of characters in the json (!) string of a book page before a client gets kicked
      *
-     * @return the end amount of characters
+     * @return the max amount of characters
      */
     int getMaxBookPageSize();
 

common/src/main/java/com/viaversion/viarewind/api/data/RewindMappingData.java

@@ -22,7 +22,7 @@ import com.viaversion.viabackwards.api.data.BackwardsMappingData;
 import com.viaversion.viarewind.ViaRewind;
 import java.util.logging.Logger;
 
-public final class RewindMappingData extends BackwardsMappingData {
+public class RewindMappingData extends BackwardsMappingData {
 
     public RewindMappingData(String unmappedVersion, String mappedVersion) {
         super(unmappedVersion, mappedVersion);
@@ -37,5 +37,4 @@ public final class RewindMappingData extends BackwardsMappingData {
     protected CompoundTag readMappingsFile(String name) {
         return RewindMappingDataLoader.INSTANCE.loadNBTFromDir(name);
     }
-
 }

common/src/main/java/com/viaversion/viarewind/api/data/RewindMappingDataLoader.java

@@ -22,7 +22,7 @@ import com.viaversion.viarewind.ViaRewind;
 import java.io.File;
 import java.util.logging.Logger;
 
-public final class RewindMappingDataLoader extends BackwardsMappingDataLoader {
+public class RewindMappingDataLoader extends BackwardsMappingDataLoader {
 
     public static final RewindMappingDataLoader INSTANCE = new RewindMappingDataLoader();
 
@@ -30,6 +30,7 @@ public final class RewindMappingDataLoader extends BackwardsMappingDataLoader {
         super(RewindMappingDataLoader.class, "assets/viarewind/data/");
     }
 
+
     @Override
     public Logger getLogger() {
         return ViaRewind.getPlatform().getLogger();
@@ -39,5 +40,4 @@ public final class RewindMappingDataLoader extends BackwardsMappingDataLoader {
     public File getDataFolder() {
         return ViaRewind.getPlatform().getDataFolder();
     }
-
 }

common/src/main/java/com/viaversion/viarewind/api/minecraft/entitydata/EntityDataTypes1_7_6_10.java

@@ -23,7 +23,6 @@ import com.viaversion.viaversion.api.type.Type;
 import com.viaversion.viaversion.api.type.Types;
 
 public enum EntityDataTypes1_7_6_10 implements EntityDataType {
-
     BYTE(0, Types.BYTE),
     SHORT(1, Types.SHORT),
     INT(2, Types.INT),
@@ -35,12 +34,12 @@ public enum EntityDataTypes1_7_6_10 implements EntityDataType {
     private final int typeID;
     private final Type<?> type;
 
-    EntityDataTypes1_7_6_10(final int typeID, final Type<?> type) {
+    EntityDataTypes1_7_6_10(int typeID, Type<?> type) {
         this.typeID = typeID;
         this.type = type;
     }
 
-    public static EntityDataTypes1_7_6_10 byId(final int id) {
+    public static EntityDataTypes1_7_6_10 byId(int id) {
         return values()[id];
     }
 
@@ -53,5 +52,4 @@ public enum EntityDataTypes1_7_6_10 implements EntityDataType {
     public Type<?> type() {
         return this.type;
     }
-
 }

common/src/main/java/com/viaversion/viarewind/api/minecraft/math/AABB.java

@@ -17,8 +17,20 @@
  */
 package com.viaversion.viarewind.api.minecraft.math;
 
-import com.viaversion.viaversion.api.minecraft.Vector3d;
+public class AABB {
+    Vector3d min;
+    Vector3d max;
 
-public record Box(Vector3d start, Vector3d end) {
+    public AABB(Vector3d min, Vector3d max) {
+        this.min = min;
+        this.max = max;
+    }
 
+    public Vector3d getMin() {
+        return this.min;
+    }
+
+    public Vector3d getMax() {
+        return this.max;
+    }
 }

common/src/main/java/com/viaversion/viarewind/api/minecraft/math/Ray3d.java

@@ -0,0 +1,36 @@
+/*
+ * This file is part of ViaRewind - https://github.com/ViaVersion/ViaRewind
+ * Copyright (C) 2018-2026 ViaVersion and contributors
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+package com.viaversion.viarewind.api.minecraft.math;
+
+public class Ray3d {
+    Vector3d start;
+    Vector3d dir;
+
+    public Ray3d(Vector3d start, Vector3d dir) {
+        this.start = start;
+        this.dir = dir;
+    }
+
+    public Vector3d getStart() {
+        return this.start;
+    }
+
+    public Vector3d getDir() {
+        return this.dir;
+    }
+}

common/src/main/java/com/viaversion/viarewind/api/minecraft/math/RayTracing.java

@@ -17,58 +17,42 @@
  */
 package com.viaversion.viarewind.api.minecraft.math;
 
-import com.viaversion.viaversion.api.minecraft.Vector3d;
+public class RayTracing {
 
-public final class RayTracing {
+    public static Vector3d trace(Ray3d ray, AABB aabb, double distance) {
+        Vector3d invDir = new Vector3d(1f / ray.dir.x, 1f / ray.dir.y, 1f / ray.dir.z);
 
-    public static Vector3d trace(final Box ray, final Box box, final double distance) {
-        final Vector3d invDir = new Vector3d(1f / ray.end().x(), 1f / ray.end().y(), 1f / ray.end().z());
+        boolean signDirX = invDir.x < 0;
+        boolean signDirY = invDir.y < 0;
+        boolean signDirZ = invDir.z < 0;
 
-        final boolean signDirX = invDir.x() < 0;
-        final boolean signDirY = invDir.y() < 0;
-        final boolean signDirZ = invDir.z() < 0;
+        Vector3d bbox = signDirX ? aabb.max : aabb.min;
+        double tmin = (bbox.x - ray.start.x) * invDir.x;
+        bbox = signDirX ? aabb.min : aabb.max;
+        double tmax = (bbox.x - ray.start.x) * invDir.x;
+        bbox = signDirY ? aabb.max : aabb.min;
+        double tymin = (bbox.y - ray.start.y) * invDir.y;
+        bbox = signDirY ? aabb.min : aabb.max;
+        double tymax = (bbox.y - ray.start.y) * invDir.y;
 
-        Vector3d boundingBox = signDirX ? box.end() : box.start();
-        double tmin = (boundingBox.x() - ray.start().x()) * invDir.x();
-        boundingBox = signDirX ? box.start() : box.end();
-        double tmax = (boundingBox.x() - ray.start().x()) * invDir.x();
-        boundingBox = signDirY ? box.end() : box.start();
-        final double tymin = (boundingBox.y() - ray.start().y()) * invDir.y();
-        boundingBox = signDirY ? box.start() : box.end();
-        final double tymax = (boundingBox.y() - ray.start().y()) * invDir.y();
+        if (tmin > tymax || tymin > tmax) return null;
 
-        if (tmin > tymax || tymin > tmax) {
-            return null;
-        }
+        if (tymin > tmin) tmin = tymin;
 
-        if (tymin > tmin) {
-            tmin = tymin;
-        }
-        if (tymax < tmax) {
-            tmax = tymax;
-        }
+        if (tymax < tmax) tmax = tymax;
 
-        boundingBox = signDirZ ? box.end() : box.start();
-        double tzmin = (boundingBox.z() - ray.start().z()) * invDir.z();
-        boundingBox = signDirZ ? box.start() : box.end();
-        double tzmax = (boundingBox.z() - ray.start().z()) * invDir.z();
+        bbox = signDirZ ? aabb.max : aabb.min;
+        double tzmin = (bbox.z - ray.start.z) * invDir.z;
+        bbox = signDirZ ? aabb.min : aabb.max;
+        double tzmax = (bbox.z - ray.start.z) * invDir.z;
 
-        if (tmin > tzmax || tzmin > tmax) {
-            return null;
-        }
+        if (tmin > tzmax || tzmin > tmax) return null;
 
-        if (tzmin > tmin) {
-            tmin = tzmin;
-        }
-        if (tzmax < tmax) {
-            tmax = tzmax;
-        }
+        if (tzmin > tmin) tmin = tzmin;
 
-        if (tmin <= distance && tmax > 0) {
-            // ray.start().clone().add(ray.dir().clone().normalize().multiply(tmin))
-        }
+        if (tzmax < tmax) tmax = tzmax;
 
-        return null;
+        return tmin <= distance && tmax > 0 ? ray.start.clone().add(ray.dir.clone().normalize().multiply(tmin)) : null;
     }
 
 }

common/src/main/java/com/viaversion/viarewind/api/minecraft/math/Vector3d.java

@@ -0,0 +1,128 @@
+/*
+ * This file is part of ViaRewind - https://github.com/ViaVersion/ViaRewind
+ * Copyright (C) 2018-2026 ViaVersion and contributors
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+package com.viaversion.viarewind.api.minecraft.math;
+
+import java.util.Objects;
+
+public class Vector3d {
+    double x, y, z;
+
+    public Vector3d(double x, double y, double z) {
+        this.x = x;
+        this.y = y;
+        this.z = z;
+    }
+
+    public Vector3d() {
+    }
+
+    public void set(Vector3d vec) {
+        this.x = vec.x;
+        this.y = vec.y;
+        this.z = vec.z;
+    }
+
+    public Vector3d set(double x, double y, double z) {
+        this.x = x;
+        this.y = y;
+        this.z = z;
+        return this;
+    }
+
+    public Vector3d multiply(double a) {
+        this.x *= a;
+        this.y *= a;
+        this.z *= a;
+        return this;
+    }
+
+    public Vector3d add(Vector3d vec) {
+        this.x += vec.x;
+        this.y += vec.y;
+        this.z += vec.z;
+        return this;
+    }
+
+    public Vector3d substract(Vector3d vec) {
+        this.x -= vec.x;
+        this.y -= vec.y;
+        this.z -= vec.z;
+        return this;
+    }
+
+    public double length() {
+        return Math.sqrt(lengthSquared());
+    }
+
+    public double lengthSquared() {
+        return this.x * this.x + this.y * this.y + this.z * this.z;
+    }
+
+    public Vector3d normalize() {
+        double length = length();
+        multiply(1.0 / length);
+        return this;
+    }
+
+    @Override
+    public Vector3d clone() {
+        return new Vector3d(this.x, this.y, this.z);
+    }
+
+    @Override
+    public boolean equals(Object o) {
+        if (this == o) return true;
+        if (o == null || getClass() != o.getClass()) return false;
+        Vector3d vector3d = (Vector3d) o;
+        return Double.compare(vector3d.x, x) == 0 && Double.compare(vector3d.y, y) == 0 && Double.compare(vector3d.z, z) == 0;
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(x, y, z);
+    }
+
+    @Override
+    public String toString() {
+        return "Vector3d{" + "x=" + x + ", y=" + y + ", z=" + z + '}';
+    }
+
+    public double getX() {
+        return this.x;
+    }
+
+    public void setX(double x) {
+        this.x = x;
+    }
+
+    public double getY() {
+        return this.y;
+    }
+
+    public void setY(double y) {
+        this.y = y;
+    }
+
+    public double getZ() {
+        return this.z;
+    }
+
+    public void setZ(double z) {
+        this.z = z;
+    }
+}

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/Protocol1_8To1_7_6_10.java

@@ -130,6 +130,7 @@ public class Protocol1_8To1_7_6_10 extends BackwardsProtocol<ClientboundPackets1
         connection.put(new GameProfileStorage(connection));
         connection.put(new ScoreboardTracker(connection));
         connection.put(new CompressionStatusTracker(connection));
+        connection.put(new WorldBorderEmulator(connection));
     }
 
     @Override

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/data/VirtualHologramEntity.java

@@ -18,14 +18,14 @@
 package com.viaversion.viarewind.protocol.v1_8to1_7_6_10.data;
 
 import com.viaversion.viarewind.api.minecraft.entitydata.EntityDataTypes1_7_6_10;
-import com.viaversion.viarewind.api.minecraft.math.Box;
+import com.viaversion.viarewind.api.minecraft.math.AABB;
+import com.viaversion.viarewind.api.minecraft.math.Vector3d;
 import com.viaversion.viarewind.api.type.RewindTypes;
 import com.viaversion.viarewind.protocol.v1_7_6_10to1_7_2_5.packet.ClientboundPackets1_7_2_5;
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.Protocol1_8To1_7_6_10;
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.rewriter.EntityPacketRewriter1_8;
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage.EntityTracker1_8;
 import com.viaversion.viaversion.api.connection.UserConnection;
-import com.viaversion.viaversion.api.minecraft.Vector3d;
 import com.viaversion.viaversion.api.minecraft.entities.EntityTypes1_8;
 import com.viaversion.viaversion.api.minecraft.entitydata.EntityData;
 import com.viaversion.viaversion.api.minecraft.entitydata.types.EntityDataTypes1_8;
@@ -328,14 +328,14 @@ public class VirtualHologramEntity {
         }
     }
 
-    public Box getBoundingBox() {
+    public AABB getBoundingBox() {
         final double width = this.small ? 0.25 : 0.5;
         final double height = this.small ? 0.9875 : 1.975;
 
         final Vector3d min = new Vector3d(this.locX - width / 2, this.locY, this.locZ - width / 2);
         final Vector3d max = new Vector3d(this.locX + width / 2, this.locY + height, this.locZ + width / 2);
 
-        return new Box(min, max);
+        return new AABB(min, max);
     }
 
     private int additionalEntityId() {

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/rewriter/PlayerPacketRewriter1_8.java

@@ -21,8 +21,10 @@ import com.viaversion.nbt.tag.CompoundTag;
 import com.viaversion.nbt.tag.ListTag;
 import com.viaversion.nbt.tag.StringTag;
 import com.viaversion.viarewind.ViaRewind;
-import com.viaversion.viarewind.api.minecraft.math.Box;
+import com.viaversion.viarewind.api.minecraft.math.AABB;
+import com.viaversion.viarewind.api.minecraft.math.Ray3d;
 import com.viaversion.viarewind.api.minecraft.math.RayTracing;
+import com.viaversion.viarewind.api.minecraft.math.Vector3d;
 import com.viaversion.viarewind.api.type.RewindTypes;
 import com.viaversion.viarewind.protocol.v1_7_6_10to1_7_2_5.packet.ClientboundPackets1_7_2_5;
 import com.viaversion.viarewind.protocol.v1_7_6_10to1_7_2_5.packet.ServerboundPackets1_7_2_5;
@@ -144,8 +146,8 @@ public class PlayerPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
                         return;
                     }
 
-                    // On 1.8 clients when the player respawns it keeps all metadata and sets player back to end HP
-                    // 1.7 clients do not copy the metadata but do set the health to end HP
+                    // On 1.8 clients when the player respawns it keeps all metadata and sets player back to max HP
+                    // 1.7 clients do not copy the metadata but do set the health to max HP
                     // to match the same client logic we remove the HP metadata as otherwise the client would be set to their old HP before respawning
                     final List<EntityData> entityDataList = new ArrayList<>(tracker.getEntityData());
                     entityDataList.removeIf(entityData -> entityData.id() == 6);
@@ -509,7 +511,13 @@ public class PlayerPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
                 String username = message.split(" ")[1];
                 final GameProfileStorage storage = wrapper.user().get(GameProfileStorage.class);
 
-                final GameProfileStorage.GameProfile profile = storage.get(username, true);
+                // Multiple players with the same name may exist. This will only represent
+                // fake players so it's fine to just pick the first one.
+                List<GameProfileStorage.GameProfile> profiles = storage.get(username, true);
+                if (profiles.isEmpty()) {
+                    return;
+                }
+                final GameProfileStorage.GameProfile profile = profiles.get(0);
                 if (profile != null && profile.uuid != null) {
                     wrapper.cancel();
 
@@ -527,9 +535,6 @@ public class PlayerPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
                 map(Types.BYTE, Types.VAR_INT); // Mode
                 handler(wrapper -> {
                     final int mode = wrapper.get(Types.VAR_INT, 1);
-                    if (mode != 0) {
-                        return;
-                    }
                     final EntityTracker1_8 tracker = wrapper.user().getEntityTracker(Protocol1_8To1_7_6_10.class);
                     final int entityId = tracker.getHologramIdWithExtra(wrapper.get(Types.VAR_INT, 0));
                     final PlayerSessionStorage position = wrapper.user().get(PlayerSessionStorage.class);
@@ -539,20 +544,25 @@ public class PlayerPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
                     }
 
                     wrapper.set(Types.VAR_INT, 0, entityId);
-                    final Box boundingBox = tracker.getHolograms().get(entityId).getBoundingBox();
+
+                    if (mode != 0) {
+                        return;
+                    }
+
+                    final AABB boundingBox = tracker.getHolograms().get(entityId).getBoundingBox();
 
                     Vector3d pos = new Vector3d(position.getPosX(), position.getPosY() + 1.8, position.getPosZ());
                     double yaw = Math.toRadians(position.yaw);
                     double pitch = Math.toRadians(position.pitch);
 
                     Vector3d dir = new Vector3d(-Math.cos(pitch) * Math.sin(yaw), -Math.sin(pitch), Math.cos(pitch) * Math.cos(yaw));
-                    Rotation ray = new Rotation(pos, dir);
+                    Ray3d ray = new Ray3d(pos, dir);
                     Vector3d intersection = RayTracing.trace(ray, boundingBox, 5.0);
 
                     if (intersection == null) {
                         return;
                     }
-                    intersection.substract(boundingBox.start());
+                    intersection.substract(boundingBox.getMin());
 
                     wrapper.set(Types.VAR_INT, 1, 2);
                     wrapper.write(Types.FLOAT, (float) intersection.getX());
@@ -606,6 +616,16 @@ public class PlayerPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
             protected void register() {
                 map(Types.VAR_INT); // Status
                 map(RewindTypes.U_BYTE_POSITION, Types.BLOCK_POSITION1_8); // Position
+                handler(wrapper -> {
+                    // 1.7 uses 255 as "no face"
+                    // 1.8 changed this to only be in the range EnumFacing range (0-5) and uses DOWN when no face is used
+                    // This properly rewrites invalid values to the 1.8 equivalent
+                    int face = wrapper.read(Types.UNSIGNED_BYTE);
+                    if (face > 5) {
+                        face = 0;
+                    }
+                    wrapper.write(Types.UNSIGNED_BYTE, (short) face);
+                });
             }
         });
 

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/rewriter/WorldPacketRewriter1_8.java

@@ -24,7 +24,6 @@ import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.Protocol1_8To1_7_6_10;
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.data.Particles1_8;
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage.WorldBorderEmulator;
 import com.viaversion.viarewind.utils.ChatUtil;
-import com.viaversion.viaversion.api.connection.UserConnection;
 import com.viaversion.viaversion.api.minecraft.BlockChangeRecord;
 import com.viaversion.viaversion.api.minecraft.ClientWorld;
 import com.viaversion.viaversion.api.minecraft.chunks.Chunk;
@@ -225,7 +224,12 @@ public class WorldPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
                         columnData[i + 3] = data[column + i * columns];
                     }
 
-                    sendMapItemData(wrapper.user(), id, columnData);
+                    final PacketWrapper mapData = PacketWrapper.create(ClientboundPackets1_8.MAP_ITEM_DATA, wrapper.user());
+                    mapData.write(Types.VAR_INT, id); // map id
+                    mapData.write(Types.SHORT, (short) columnData.length); // data length
+                    mapData.write(new FixedByteArrayType(columnData.length), columnData); // data
+
+                    mapData.send(Protocol1_8To1_7_6_10.class);
                 }
             }
 
@@ -238,7 +242,12 @@ public class WorldPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
                     iconData[i * 3 + 3] = icons[i * 4 + 2];
                 }
 
-                sendMapItemData(wrapper.user(), id, iconData);
+                final PacketWrapper mapData = PacketWrapper.create(ClientboundPackets1_8.MAP_ITEM_DATA, wrapper.user());
+                mapData.write(Types.VAR_INT, id); // map id
+                mapData.write(Types.SHORT, (short) iconData.length); // data length
+                mapData.write(new FixedByteArrayType(iconData.length), iconData); // data
+
+                mapData.send(Protocol1_8To1_7_6_10.class);
             }
 
             // Update scale
@@ -250,55 +259,35 @@ public class WorldPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10>
             mapData.send(Protocol1_8To1_7_6_10.class);
         });
 
-        protocol.registerClientbound(ClientboundPackets1_8.BLOCK_ENTITY_DATA, wrapper -> {
-            wrapper.passthroughAndMap(Types.BLOCK_POSITION1_8, RewindTypes.SHORT_POSITION); // Position
-            wrapper.passthrough(Types.UNSIGNED_BYTE); // Action
-            wrapper.passthroughAndMap(Types.NAMED_COMPOUND_TAG, RewindTypes.COMPRESSED_NBT); // Tag
+        protocol.registerClientbound(ClientboundPackets1_8.BLOCK_ENTITY_DATA, new PacketHandlers() {
+            @Override
+            public void register() {
+                map(Types.BLOCK_POSITION1_8, RewindTypes.SHORT_POSITION); // position
+                map(Types.UNSIGNED_BYTE); // action
+                map(Types.NAMED_COMPOUND_TAG, RewindTypes.COMPRESSED_NBT); // nbt
+            }
         });
 
         protocol.cancelClientbound(ClientboundPackets1_8.CHANGE_DIFFICULTY);
 
         protocol.registerClientbound(ClientboundPackets1_8.SET_BORDER, null, wrapper -> {
-            wrapper.cancel();
-            final int action = wrapper.read(Types.VAR_INT);
-            if (action == 3) { // initialize
-                final double x = wrapper.read(Types.DOUBLE);
-                final double z = wrapper.read(Types.DOUBLE);
-                final double oldRadius = wrapper.read(Types.DOUBLE);
-                final double newRadius = wrapper.read(Types.DOUBLE);
-                final long speed = wrapper.read(Types.VAR_LONG);
-
-                wrapper.user().put(new WorldBorderEmulator(wrapper.user(), x, z, oldRadius, newRadius, speed));
-            }
-
             final WorldBorderEmulator emulator = wrapper.user().get(WorldBorderEmulator.class);
-            if (emulator == null) {
-                return;
-            }
+            wrapper.cancel();
 
+            final int action = wrapper.read(Types.VAR_INT);
             if (action == 0) { // set size
-                final double radius = wrapper.read(Types.DOUBLE);
-                emulator.setSize(radius);
-            } else if (action == 1) { // delta size
-                final double oldRadius = wrapper.read(Types.DOUBLE);
-                final double newRadius = wrapper.read(Types.DOUBLE);
-                final long speed = wrapper.read(Types.VAR_LONG);
-                emulator.updateDeltaTime(oldRadius, newRadius, speed);
+                emulator.setSize(wrapper.read(Types.DOUBLE)); // radius
+            } else if (action == 1) { // lerp size
+                emulator.lerpSize(wrapper.read(Types.DOUBLE), wrapper.read(Types.DOUBLE), wrapper.read(Types.VAR_LONG)); // old radius, new radius, speed
             } else if (action == 2) { // set center
-                final double x = wrapper.read(Types.DOUBLE);
-                final double z = wrapper.read(Types.DOUBLE);
-                emulator.setCenter(x, z);
+                emulator.setCenter(wrapper.read(Types.DOUBLE), wrapper.read(Types.DOUBLE)); // x, z
+            } else if (action == 3) { // initialize
+                emulator.init(
+                    wrapper.read(Types.DOUBLE), wrapper.read(Types.DOUBLE), // x, z
+                    wrapper.read(Types.DOUBLE), wrapper.read(Types.DOUBLE), // old radius, new radius
+                    wrapper.read(Types.VAR_LONG) // speed
+                );
             }
         });
     }
-
-    private void sendMapItemData(final UserConnection connection, final int id, final byte[] columnData) {
-        final PacketWrapper mapData = PacketWrapper.create(ClientboundPackets1_8.MAP_ITEM_DATA, connection);
-        mapData.write(Types.VAR_INT, id); // map id
-        mapData.write(Types.SHORT, (short) columnData.length); // data length
-        mapData.write(new FixedByteArrayType(columnData.length), columnData); // data
-
-        mapData.send(Protocol1_8To1_7_6_10.class);
-    }
-
 }

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/storage/EntityTracker1_8.java

@@ -222,17 +222,16 @@ public class EntityTracker1_8 extends EntityTrackerBase {
 
     public void checkNametagVisbility(final String username) {
         final GameProfileStorage profileStorage = user().get(GameProfileStorage.class);
-        final GameProfileStorage.GameProfile profile = profileStorage.get(username, false);
-        if (profile == null) {
-            return;
-        }
+        final List<GameProfileStorage.GameProfile> profiles = profileStorage.get(username, false);
 
-        final int entityId = getPlayerEntityId(profile.uuid);
-        if (entityId == -1) {
-            return;
-        }
+        for (GameProfileStorage.GameProfile profile : profiles) {
+            final int entityId = getPlayerEntityId(profile.uuid);
+            if (entityId == -1) {
+                return;
+            }
 
-        checkNametagVisibility(entityId);
+            checkNametagVisibility(entityId);
+        }
     }
 
     private boolean isPlayerNametagHidden(final int entityId) {

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/storage/GameProfileStorage.java

@@ -17,6 +17,10 @@
  */
 package com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage;
 
+import com.google.common.collect.ArrayListMultimap;
+import com.google.common.collect.HashMultimap;
+import com.google.common.collect.ListMultimap;
+import com.google.common.collect.Multimap;
 import com.viaversion.nbt.tag.CompoundTag;
 import com.viaversion.nbt.tag.ListTag;
 import com.viaversion.nbt.tag.StringTag;
@@ -34,6 +38,7 @@ import java.util.UUID;
 
 public class GameProfileStorage extends StoredObject {
     private final Map<UUID, GameProfile> properties = new HashMap<>();
+    private final ListMultimap<String, GameProfile> nameToProfile = ArrayListMultimap.create();
 
     public GameProfileStorage(UserConnection user) {
         super(user);
@@ -42,6 +47,7 @@ public class GameProfileStorage extends StoredObject {
     public GameProfile put(UUID uuid, String name, String displayName, int ping, int gamemode) {
         GameProfile gameProfile = new GameProfile(uuid, name, displayName, ping, gamemode);
         properties.put(uuid, gameProfile);
+        nameToProfile.put(name, gameProfile);
         return gameProfile;
     }
 
@@ -49,19 +55,9 @@ public class GameProfileStorage extends StoredObject {
         return properties.get(uuid);
     }
 
-    public GameProfile get(String name, boolean ignoreCase) {
+    public List<GameProfile> get(String name, boolean ignoreCase) {
         if (ignoreCase) name = name.toLowerCase();
-
-        for (GameProfile profile : properties.values()) {
-            if (profile.name == null) continue;
-
-            String n = ignoreCase ? profile.name.toLowerCase() : profile.name;
-
-            if (n.equals(name)) {
-                return profile;
-            }
-        }
-        return null;
+        return nameToProfile.get(name);
     }
 
     public List<GameProfile> getAllWithPrefix(String prefix, boolean ignoreCase) {
@@ -81,7 +77,11 @@ public class GameProfileStorage extends StoredObject {
     }
 
     public GameProfile remove(UUID uuid) {
-        return properties.remove(uuid);
+        GameProfile removedProfile = properties.remove(uuid);
+        if (removedProfile != null) {
+            nameToProfile.get(removedProfile.name).remove(removedProfile);
+        }
+        return removedProfile;
     }
 
 

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/storage/ScoreboardTracker.java

@@ -30,8 +30,7 @@ import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 
-public final class ScoreboardTracker extends StoredObject {
-
+public class ScoreboardTracker extends StoredObject {
     private final HashMap<String, List<String>> teams = new HashMap<>();
     private final HashSet<String> objectives = new HashSet<>();
     private final HashMap<String, ScoreTeam> scoreTeams = new HashMap<>();
@@ -41,74 +40,76 @@ public final class ScoreboardTracker extends StoredObject {
     private final HashMap<String, String> teamNameTagVisibilities = new HashMap<>();
     private String colorIndependentSidebar;
 
-    public ScoreboardTracker(final UserConnection user) {
+    public ScoreboardTracker(UserConnection user) {
         super(user);
     }
 
-    public void addPlayerToTeam(final String player, final String team) {
+    public void addPlayerToTeam(String player, String team) {
         teams.computeIfAbsent(team, key -> new ArrayList<>()).add(player);
     }
 
-    public void setTeamColor(final String team, final Byte color) {
+    public void setTeamColor(String team, Byte color) {
         teamColors.put(team, color);
     }
 
-    public Optional<Byte> getTeamColor(final String team) {
+    public Optional<Byte> getTeamColor(String team) {
         return Optional.ofNullable(teamColors.get(team));
     }
 
-    public void addTeam(final String team) {
+    public void addTeam(String team) {
         teams.computeIfAbsent(team, key -> new ArrayList<>());
     }
 
-    public void removeTeam(final String team) {
+    public void removeTeam(String team) {
         teams.remove(team);
         scoreTeams.remove(team);
         teamColors.remove(team);
         teamNameTagVisibilities.remove(team);
     }
 
-    public boolean teamExists(final String team) {
+    public boolean teamExists(String team) {
         return teams.containsKey(team);
     }
 
     public void removePlayerFromTeam(String player, String team) {
-        final List<String> teamPlayers = teams.get(team);
-        if (teamPlayers != null) {
-            teamPlayers.remove(player);
-        }
+        List<String> teamPlayers = teams.get(team);
+        if (teamPlayers != null) teamPlayers.remove(player);
     }
 
-    public boolean isPlayerInTeam(final String player, final String team) {
-        final List<String> teamPlayers = teams.get(team);
+    public boolean isPlayerInTeam(String player, String team) {
+        List<String> teamPlayers = teams.get(team);
         return teamPlayers != null && teamPlayers.contains(player);
     }
 
-
-    public Optional<Byte> getPlayerTeamColor(final String player) {
-        final Optional<String> team = getTeam(player);
-        return team.flatMap(this::getTeamColor);
+    public boolean isPlayerInTeam(String player) {
+        for (List<String> teamPlayers : teams.values()) {
+            if (teamPlayers.contains(player)) return true;
+        }
+        return false;
     }
 
-    public Optional<String> getTeam(final String player) {
-        for (Map.Entry<String, List<String>> entry : teams.entrySet()) {
-            if (entry.getValue().contains(player)) {
+    public Optional<Byte> getPlayerTeamColor(String player) {
+        Optional<String> team = getTeam(player);
+        return team.isPresent() ? getTeamColor(team.get()) : Optional.empty();
+    }
+
+    public Optional<String> getTeam(String player) {
+        for (Map.Entry<String, List<String>> entry : teams.entrySet())
+            if (entry.getValue().contains(player))
                 return Optional.of(entry.getKey());
-            }
-        }
         return Optional.empty();
     }
 
-    public void setTeamNameTagVisibility(final String team, final String visibility) {
+    public void setTeamNameTagVisibility(String team, String visibility) {
         teamNameTagVisibilities.put(team, visibility);
     }
 
-    public String getTeamNameTagVisibility(final String team) {
+    public String getTeamNameTagVisibility(String team) {
         return teamNameTagVisibilities.getOrDefault(team, "always");
     }
 
-    public boolean isNametagHidden(final String username) {
-        for (final Map.Entry<String, List<String>> entry : teams.entrySet()) {
+    public boolean isNametagHidden(String username) {
+        for (Map.Entry<String, List<String>> entry : teams.entrySet()) {
             if (entry.getValue().contains(username)) {
                 return "never".equalsIgnoreCase(teamNameTagVisibilities.getOrDefault(entry.getKey(), "always"));
             }
@@ -116,15 +117,15 @@ public final class ScoreboardTracker extends StoredObject {
         return false;
     }
 
-    public List<String> getTeamMembers(final String team) {
+    public List<String> getTeamMembers(String team) {
         return teams.getOrDefault(team, new ArrayList<>());
     }
 
-    public void addObjective(final String name) {
+    public void addObjective(String name) {
         objectives.add(name);
     }
 
-    public void removeObjective(final String name) {
+    public void removeObjective(String name) {
         objectives.remove(name);
         colorDependentSidebar.values().remove(name);
         if (name.equals(colorIndependentSidebar)) {
@@ -132,63 +133,58 @@ public final class ScoreboardTracker extends StoredObject {
         }
     }
 
-    public boolean objectiveExists(final String name) {
+    public boolean objectiveExists(String name) {
         return objectives.contains(name);
     }
 
-    public String sendTeamForScore(final String score) {
-        if (score.length() <= 16) {
-            return score;
-        }
+    public String sendTeamForScore(String score) {
+        if (score.length() <= 16) return score;
+        if (scoreTeams.containsKey(score)) return scoreTeams.get(score).name;
 
-        if (scoreTeams.containsKey(score)) {
-            return scoreTeams.get(score).name;
-        }
-
-        int l = 16;
-        int i = Math.min(16, score.length() - 16);
-        String name = score.substring(i, i + l);
-        while (scoreTeamNames.contains(name) || teams.containsKey(name)) {
-            i--;
-            while (score.length() - l - i > 16) {
-                l--;
-                if (l < 1) {
-                    return score;
+        // Find the longest window score[i, i+l) that:
+        // - is not already claimed as a team name
+        // - keeps prefix (score[0,i)) at most 16 chars i <= 16
+        // - keeps suffix (score[i+l, end)) at most 16 chars score.length()-l-i <= 16
+        for (int l = 16; l >= 1; l--) {
+            final int iMax = Math.min(16, score.length() - l);
+            final int iMin = Math.max(0, score.length() - l - 16);
+            for (int i = iMax; i >= iMin; i--) {
+                final String name = score.substring(i, i + l);
+                if (scoreTeamNames.contains(name) || teams.containsKey(name)) {
+                    continue;
                 }
-                i = Math.min(16, score.length() - l);
+
+                final String prefix = score.substring(0, i);
+                final String suffix = i + l >= score.length() ? "" : score.substring(i + l);
+
+                final ScoreTeam scoreTeam = new ScoreTeam(name, prefix, suffix);
+                scoreTeams.put(score, scoreTeam);
+                scoreTeamNames.add(name);
+
+                final PacketWrapper teamPacket = PacketWrapper.create(ClientboundPackets1_7_2_5.SET_PLAYER_TEAM, getUser());
+                teamPacket.write(Types.STRING, name);
+                teamPacket.write(Types.BYTE, (byte) 0);
+                teamPacket.write(Types.STRING, "ViaRewind");
+                teamPacket.write(Types.STRING, prefix);
+                teamPacket.write(Types.STRING, suffix);
+                teamPacket.write(Types.BYTE, (byte) 0);
+                teamPacket.write(Types.SHORT, (short) 1);
+                teamPacket.write(Types.STRING, name);
+                teamPacket.send(Protocol1_8To1_7_6_10.class);
+
+                return name;
             }
-            name = score.substring(i, i + l);
         }
-        final String prefix = score.substring(0, i);
-        final String suffix = i + l >= score.length() ? "" : score.substring(i + l);
 
-        final ScoreTeam scoreTeam = new ScoreTeam(name, prefix, suffix);
-        scoreTeams.put(score, scoreTeam);
-        scoreTeamNames.add(name);
-
-        PacketWrapper teamPacket = PacketWrapper.create(ClientboundPackets1_7_2_5.SET_PLAYER_TEAM, user());
-        teamPacket.write(Types.STRING, name);
-        teamPacket.write(Types.BYTE, (byte) 0);
-        teamPacket.write(Types.STRING, "ViaRewind");
-        teamPacket.write(Types.STRING, prefix);
-        teamPacket.write(Types.STRING, suffix);
-        teamPacket.write(Types.BYTE, (byte) 0);
-        teamPacket.write(Types.SHORT, (short) 1);
-        teamPacket.write(Types.STRING, name);
-        teamPacket.send(Protocol1_8To1_7_6_10.class);
-
-        return name;
+        return score;
     }
 
-    public String removeTeamForScore(final String score) {
-        final ScoreTeam scoreTeam = scoreTeams.remove(score);
-        if (scoreTeam == null) {
-            return score;
-        }
-
+    public String removeTeamForScore(String score) {
+        ScoreTeam scoreTeam = scoreTeams.remove(score);
+        if (scoreTeam == null) return score;
         scoreTeamNames.remove(scoreTeam.name);
 
-        final PacketWrapper teamPacket = PacketWrapper.create(ClientboundPackets1_7_2_5.SET_PLAYER_TEAM, user());
+        PacketWrapper teamPacket = PacketWrapper.create(ClientboundPackets1_7_2_5.SET_PLAYER_TEAM, getUser());
         teamPacket.write(Types.STRING, scoreTeam.name);
         teamPacket.write(Types.BYTE, (byte) 1);
         teamPacket.send(Protocol1_8To1_7_6_10.class);
@@ -200,7 +196,7 @@ public final class ScoreboardTracker extends StoredObject {
         return this.colorIndependentSidebar;
     }
 
-    public void setColorIndependentSidebar(final String colorIndependentSidebar) {
+    public void setColorIndependentSidebar(String colorIndependentSidebar) {
         this.colorIndependentSidebar = colorIndependentSidebar;
     }
 
@@ -208,6 +204,15 @@ public final class ScoreboardTracker extends StoredObject {
         return this.colorDependentSidebar;
     }
 
-    private record ScoreTeam(String name, String prefix, String suffix) {
+    private static class ScoreTeam {
+        private final String prefix;
+        private final String suffix;
+        private final String name;
+
+        public ScoreTeam(String name, String prefix, String suffix) {
+            this.prefix = prefix;
+            this.suffix = suffix;
+            this.name = name;
+        }
     }
 }

common/src/main/java/com/viaversion/viarewind/protocol/v1_8to1_7_6_10/storage/WorldBorderEmulator.java

@@ -19,54 +19,63 @@ package com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage;
 
 import com.viaversion.viaversion.api.connection.StoredObject;
 import com.viaversion.viaversion.api.connection.UserConnection;
-import com.viaversion.viaversion.util.MathUtil;
-
-public final class WorldBorderEmulator extends StoredObject {
 
+public class WorldBorderEmulator extends StoredObject {
     private double x, z;
     private double oldDiameter, newDiameter;
 
-    private long deltaTime;
-    private long deltaStartTime;
+    private long lerpTime;
+    private long lerpStartTime;
 
-    public WorldBorderEmulator(UserConnection user, final double x, final double z, final double oldDiameter, final double newDiameter, final long deltaTime) {
+    private boolean init = false;
+
+    public WorldBorderEmulator(UserConnection user) {
         super(user);
+    }
 
+    public void init(double x, double z, double oldDiameter, double newDiameter, long lerpTime) {
         this.x = x;
         this.z = z;
 
         this.oldDiameter = oldDiameter;
         this.newDiameter = newDiameter;
 
-        this.deltaTime = deltaTime;
+        this.lerpTime = lerpTime;
+
+        init = true;
     }
 
-    public void setCenter(final double x, final double z) {
+    public void setCenter(double x, double z) {
         this.x = x;
         this.z = z;
     }
 
-    public void updateDeltaTime(final double oldDiameter, final double newDiameter, final long deltaTime) {
+    public void lerpSize(double oldDiameter, double newDiameter, long lerpTime) {
         this.oldDiameter = oldDiameter;
         this.newDiameter = newDiameter;
-        this.deltaTime = deltaTime;
-        this.deltaStartTime = System.currentTimeMillis();
+        this.lerpTime = lerpTime;
+        this.lerpStartTime = System.currentTimeMillis();
     }
 
     public double getSize() {
-        if (deltaTime == 0) {
+        if (lerpTime == 0) {
             return newDiameter;
         }
 
-        final double percent = ((double) (System.currentTimeMillis() - deltaStartTime) / (double) (deltaTime));
-        return oldDiameter + (newDiameter - oldDiameter) * MathUtil.clamp(percent, 0D, 1D);
+        double percent = ((double) (System.currentTimeMillis() - lerpStartTime) / (double) (lerpTime));
+
+        // Clamp value
+        if (percent > 1.0D) percent = 1.0d;
+        else if (percent < 0.0D) percent = 0.0d;
+
+        return oldDiameter + (newDiameter - oldDiameter) * percent;
     }
 
-    public void setSize(final double size) {
+    public void setSize(double size) {
         this.oldDiameter = size;
         this.newDiameter = size;
 
-        this.deltaTime = 0;
+        this.lerpTime = 0;
     }
 
     public double getX() {
@@ -77,6 +86,10 @@ public final class WorldBorderEmulator extends StoredObject {
         return z;
     }
 
+    public boolean isInit() {
+        return init;
+    }
+
     public enum Side {
         NORTH(0, -1),
         EAST(1, 0),
@@ -92,4 +105,3 @@ public final class WorldBorderEmulator extends StoredObject {
         }
     }
 }
-

common/src/main/java/com/viaversion/viarewind/protocol/v1_9to1_8/rewriter/EntityPacketRewriter1_9.java

@@ -253,6 +253,23 @@ public class EntityPacketRewriter1_9 extends VREntityRewriter<ClientboundPackets
             }
         });
 
+        protocol.registerClientbound(ClientboundPackets1_9.ANIMATE, new PacketHandlers() {
+            @Override
+            public void register() {
+                map(Types.VAR_INT); // Entity id
+                map(Types.UNSIGNED_BYTE); // Animation
+                handler(wrapper -> {
+                    final short animation = wrapper.get(Types.UNSIGNED_BYTE, 0);
+                    // Action 1 is a hurt duplicate of entity status 2 (only ViaBackwards manufactures it)
+                    // and makes 1.8 clients re-flash mid-window; action 3 is offhand swing on 1.9+ but
+                    // eat food on 1.8. Cancel both, matching ViaVersion's reverse-direction handling.
+                    if (animation == 1 || animation == 3) {
+                        wrapper.cancel();
+                    }
+                });
+            }
+        });
+
         protocol.registerClientbound(ClientboundPackets1_9.MOVE_ENTITY_POS, wrapper -> {
             final int entityId = wrapper.passthrough(Types.VAR_INT);
             final int deltaX = wrapper.read(Types.SHORT);

common/src/main/java/com/viaversion/viarewind/protocol/v1_9to1_8/storage/WindowTracker.java

@@ -52,11 +52,11 @@ public class WindowTracker extends StoredObject {
         openWindow.write(Types.UNSIGNED_BYTE, windowId);
         openWindow.write(Types.STRING, "minecraft:brewing_stand");
 
-        TextComponent title = new StringComponent().
-            append(new TranslationComponent("container.brewing")).
-            append(new StringComponent(": " + TextFormatting.DARK_GRAY)).
-            append(new StringComponent(amount + " " + TextFormatting.DARK_RED)).
-            append(new TranslationComponent("item.blazePowder.name", TextFormatting.DARK_RED));
+        TextComponent title = new StringComponent()
+            .append(new TranslationComponent("container.brewing"))
+            .append(new StringComponent(": ").formatted(TextFormatting.DARK_GRAY))
+            .append(new StringComponent(amount + " ").formatted(TextFormatting.DARK_RED))
+            .append(new TranslationComponent("item.blazePowder.name").formatted(TextFormatting.DARK_RED));
 
         openWindow.write(Types.COMPONENT, TextComponentSerializer.V1_8.serializeJson(title));
         openWindow.write(Types.UNSIGNED_BYTE, (short) 420);

common/src/main/resources/assets/viarewind/config.yml

@@ -5,7 +5,7 @@
 # ONLY DISABLE IF YOU HAVE 1.9 COOLDOWN DISABLED ON YOUR SERVER
 # 1.8 PLAYERS MAY ASK WHY PVP IS NOT WORKING OTHERWISE.
 # REQUIRES A SERVER RESTART TO TAKE EFFECT
-cooldown-indicator: 'TITLE'
+cooldown-indicator: DISABLED
 #
 # Replaces Adventure mode with Survival mode for 1.7.x clients
 # Enable this option if your server is using the 'CanDestroy'
@@ -14,31 +14,31 @@ replace-adventure: false
 #
 # Whether similar ones should replace 1.9 particles in
 # 1.8 and lower
-replace-particles: false
+replace-particles: true
 #
 # Max number of pages for written books before a client gets kicked
-max-book-pages: 100
+max-book-pages: 500
 #
 # Max number of characters in the json (!) string of a book page before a client gets kicked
 max-book-page-length: 5000
 #
 # Whether to emulate the 1.8+ world border for 1.7.x clients
-emulate-world-border: true
+emulate-world-border: false
 #
 # Always shows a mapped mob's original name, and not only when hovering over it with the cursor.
 always-show-original-mob-name: true
 #
 # The particle to show the world border for the 1.8+ world border for 1.7.x clients
-world-border-particle: 'fireworksSpark'
+world-border-particle: fireworksSpark
 #
 # If enabled, 1.8 players on 1.9+ servers can use /offhand to switch items between their main hand and offhand.
-enable-offhand: true
+enable-offhand: false
 #
 # Allows defining the offhand command
-offhand-command: '/offhand'
+offhand-command: /offhand
 #
 # If enabled, 1.8 players on 1.9+ servers will also experience the levitation effect by sending velocity packets.
 emulate-levitation-effect: true
 #
 # If enabled, 1.8 players will handle the player combat packet added in 1.9 by showing the custom death message above the hotbar.
-handle-player-combat-packet: true
+handle-player-combat-packet: false

gradle.properties

@@ -7,7 +7,7 @@ org.gradle.configuration-cache=true
 project_jvm_version=17
 
 project_group=com.viaversion
-project_version=4.1.0-SNAPSHOT
+project_version=4.1.1
 project_description=ViaVersion addon to allow 1.8.x and 1.7.x clients on newer server versions.
 
 publishing_dev_id=florianreuth

gradle/wrapper/gradle-wrapper.properties

@@ -1,8 +1,10 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionSha256Sum=2ab2958f2a1e51120c326cad6f385153bb11ee93b3c216c5fccebfdfbb7ec6cb
-distributionUrl=https\://services.gradle.org/distributions/gradle-9.4.1-bin.zip
+distributionSha256Sum=553c78f50dafcd54d65b9a444649057857469edf836431389695608536d6b746
+distributionUrl=https\://services.gradle.org/distributions/gradle-9.5.0-bin.zip
 networkTimeout=10000
+retries=0
+retryBackOffMs=500
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

gradlew

@@ -57,7 +57,7 @@
 #       Darwin, MinGW, and NonStop.
 #
 #   (3) This script is generated from the Groovy template
-#       https://github.com/gradle/gradle/blob/2d6327017519d23b96af35865dc997fcb544fb40/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
+#       https://github.com/gradle/gradle/blob/3d91ce3b8caaf77ad09f381f43615b715b53f72c/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
 #       within the Gradle project.
 #
 #       You can find Gradle at https://github.com/gradle/gradle/.

gradlew.bat

@@ -23,8 +23,8 @@
 @rem
 @rem ##########################################################################
 
-@rem Set local scope for the variables with windows NT shell
-if "%OS%"=="Windows_NT" setlocal
+@rem Set local scope for the variables, and ensure extensions are enabled
+setlocal EnableExtensions
 
 set DIRNAME=%~dp0
 if "%DIRNAME%"=="" set DIRNAME=.
@@ -51,7 +51,7 @@ echo. 1>&2
 echo Please set the JAVA_HOME variable in your environment to match the 1>&2
 echo location of your Java installation. 1>&2
 
-goto fail
+"%COMSPEC%" /c exit 1
 
 :findJavaFromJavaHome
 set JAVA_HOME=%JAVA_HOME:"=%
@@ -65,7 +65,7 @@ echo. 1>&2
 echo Please set the JAVA_HOME variable in your environment to match the 1>&2
 echo location of your Java installation. 1>&2
 
-goto fail
+"%COMSPEC%" /c exit 1
 
 :execute
 @rem Setup the command line
@@ -73,21 +73,10 @@ goto fail
 
 
 @rem Execute Gradle
-"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -jar "%APP_HOME%\gradle\wrapper\gradle-wrapper.jar" %*
+@rem endlocal doesn't take effect until after the line is parsed and variables are expanded
+@rem which allows us to clear the local environment before executing the java command
+endlocal & "%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -jar "%APP_HOME%\gradle\wrapper\gradle-wrapper.jar" %* & call :exitWithErrorLevel
 
-:end
-@rem End local scope for the variables with windows NT shell
-if %ERRORLEVEL% equ 0 goto mainEnd
-
-:fail
-rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of
-rem the _cmd.exe /c_ return code!
-set EXIT_CODE=%ERRORLEVEL%
-if %EXIT_CODE% equ 0 set EXIT_CODE=1
-if not ""=="%GRADLE_EXIT_CONSOLE%" exit %EXIT_CODE%
-exit /b %EXIT_CODE%
-
-:mainEnd
-if "%OS%"=="Windows_NT" endlocal
-
-:omega
+:exitWithErrorLevel
+@rem Use "%COMSPEC%" /c exit to allow operators to work properly in scripts
+"%COMSPEC%" /c exit %ERRORLEVEL%