4.0.4 Release

Closes https://github.com/ViaVersion/ViaRewind/issues/552

.github/dependabot.yml

@@ -0,0 +1,10 @@
+version: 2
+updates:
+  - package-ecosystem: "gradle"
+    directory: "/"
+    schedule:
+      interval: "daily"
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"

.github/workflows/build.yml

@@ -1,4 +1,4 @@
-name: ViaRewind CI
+name: Build
 on: [pull_request, push, workflow_dispatch]
 
 jobs:
@@ -8,7 +8,7 @@ jobs:
       - name: Checkout Repository
         uses: actions/checkout@v4
       - name: Validate Gradle Wrapper
-        uses: gradle/wrapper-validation-action@v3
+        uses: gradle/actions/wrapper-validation@v4
       - name: Set up JDK 17
         uses: actions/setup-java@v4
         with:

.github/workflows/publish.yml

@@ -13,7 +13,7 @@ jobs:
       - name: Checkout Repository
         uses: actions/checkout@v4
       - name: Validate Gradle Wrapper
-        uses: gradle/wrapper-validation-action@v3
+        uses: gradle/actions/wrapper-validation@v4
       - name: Set up JDK 17
         uses: actions/setup-java@v4
         with:

.github/workflows/update-gradle-wrapper.yml

@@ -0,0 +1,14 @@
+name: Update Gradle Wrapper
+on:
+  workflow_dispatch:
+  schedule:
+    - cron: "0 0 * * 0"
+
+jobs:
+  update-gradle-wrapper:
+    runs-on: ubuntu-24.04
+    steps:
+      - name: Checkout Repository
+        uses: actions/checkout@v4
+      - name: Update Gradle Wrapper
+        uses: gradle-update/update-gradle-wrapper-action@v2

.idea/copyright/profiles_settings.xml

@@ -1,5 +1,5 @@
 <component name="CopyrightManager">
-  <settings>
+  <settings default="viarewind-gpl">
     <module2copyright>
       <element module="ViaRewind" copyright="viarewind-gpl" />
     </module2copyright>

build-logic/src/main/groovy/vr.base-conventions.gradle

@@ -22,8 +22,8 @@ base {
 }
 
 dependencies {
-    compileOnly "com.viaversion:viaversion:${project.viaversion_version}"
-    compileOnly "com.viaversion:viabackwards:${project.viabackwards_version}"
+    compileOnly "com.viaversion:viaversion:5.0.4-SNAPSHOT"
+    compileOnly "com.viaversion:viabackwards:5.0.4-SNAPSHOT"
 }
 
 publishing {

build.gradle

@@ -17,6 +17,7 @@ dependencies {
     publishInclude project(":viarewind-common")
     publishInclude project(":viarewind-bukkit")
     publishInclude project(":viarewind-fabric")
+    publishInclude project(":viarewind-sponge")
     publishInclude project(":viarewind-velocity")
 }
 
@@ -51,41 +52,32 @@ idea {
 // -----------------------------------------------------
 // Publishing
 
-def latestCommitHash() {
-    def byteOut = new ByteArrayOutputStream()
-    exec {
-        commandLine 'git', 'rev-parse', '--short', 'HEAD'
-        standardOutput = byteOut
-    }
-    return byteOut.toString('UTF-8').trim()
+Provider<String> latestCommitHash() {
+    return providers.exec {
+        commandLine = ["git", "rev-parse", "--short", "HEAD"]
+    }.standardOutput.getAsText().map(String::trim)
 }
 
-def latestCommitMessage() {
-    def byteOut = new ByteArrayOutputStream()
-    exec {
-        commandLine 'git', 'log', '-1', '--pretty=%B'
-        standardOutput = byteOut
-    }
-    return byteOut.toString('UTF-8').trim()
+Provider<String> latestCommitMessage() {
+    return providers.exec {
+        commandLine = ["git", "log", "-1", "--pretty=%B"]
+    }.standardOutput.getAsText().map(String::trim)
 }
 
-def branchName() {
-    def byteOut = new ByteArrayOutputStream()
-    exec {
-        commandLine 'git', 'rev-parse', '--abbrev-ref', 'HEAD'
-        standardOutput = byteOut
-    }
-    return byteOut.toString('UTF-8').trim()
+Provider<String> branchName() {
+    return providers.exec {
+        commandLine = ["git", "rev-parse", "--abbrev-ref", "HEAD"]
+    }.standardOutput.getAsText().map(String::trim)
 }
 
-def branch = branchName()
+def branch = branchName().get()
 def baseVersion = project.maven_version
 def isRelease = !baseVersion.contains('-')
 def isMainBranch = branch == "master"
 if (!isRelease || isMainBranch) { // Only publish releases from the main branch
     def suffixedVersion = isRelease ? baseVersion : baseVersion + "+" + System.getenv("GITHUB_RUN_NUMBER")
-    def commitHash = latestCommitHash()
-    def changelogContent = "[${commitHash}](https://github.com/ViaVersion/ViaRewind/commit/${commitHash}) ${latestCommitMessage()}"
+    def commitHash = latestCommitHash().get()
+    def changelogContent = "[${commitHash}](https://github.com/ViaVersion/ViaRewind/commit/${commitHash}) ${latestCommitMessage().get()}"
     modrinth {
         def mcVersions = project.mcVersions
                 .split(',')

bukkit/src/main/resources/plugin.yml

@@ -1,6 +1,6 @@
 name: ViaRewind
 main: com.viaversion.viarewind.BukkitPlugin
-authors: [FlorianMichael/EnZaXD, Gerrygames, creeper123321123]
+authors: [FlorianMichael/EnZaXD, Gerrygames, creeper123123321]
 version: ${version}
 description: ${description}
 website: https://viaversion.com/rewind

common/build.gradle

@@ -5,15 +5,45 @@ plugins {
 }
 
 dependencies {
-    compileOnly "io.netty:netty-all:${project.netty_version}"
-    compileOnly "com.google.guava:guava:${project.guava_version}"
+    compileOnly "io.netty:netty-all:4.0.20.Final"
+    compileOnly "com.google.guava:guava:17.0"
 }
 
 sourceSets {
     main {
         classTokenReplacer {
             property("\${version}", rootProject.maven_version)
-            property("\${impl_version}", "git-ViaRewind-${rootProject.maven_version}:${rootProject.latestCommitHash()}")
+            property("\${impl_version}", "git-ViaRewind-${rootProject.maven_version}:${rootProject.latestCommitHash().get()}")
         }
     }
 }
+
+// Task to quickly test/debug code changes using https://github.com/ViaVersion/ViaProxy
+// For further instructions see the ViaProxy repository README
+tasks.register("runViaProxy", JavaExec) {
+    dependsOn tasks.jar
+
+    def viaProxyConfiguration = configurations.create("viaProxy")
+    viaProxyConfiguration.dependencies.add(dependencies.create("net.raphimc:ViaProxy:3.3.5-SNAPSHOT") {
+        transitive = false
+    })
+
+    mainClass = "net.raphimc.viaproxy.ViaProxy"
+    classpath = viaProxyConfiguration
+    workingDir = file("run")
+    jvmArgs = ["-DskipUpdateCheck"]
+    if (System.getProperty("viaproxy.gui.autoStart") != null) {
+        jvmArgs("-Dviaproxy.gui.autoStart")
+    }
+
+    doFirst {
+        def jarsDir = file("$workingDir/jars")
+        jarsDir.mkdirs()
+        file("$jarsDir/${project.name}.jar").bytes = tasks.jar.archiveFile.get().asFile.bytes
+    }
+
+    doLast {
+        file("$workingDir/jars/${project.name}.jar").delete()
+        file("$workingDir/logs").deleteDir()
+    }
+}

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

@@ -40,6 +40,7 @@ public class ViaRewindConfig extends Config implements com.viaversion.viarewind.
 	private boolean enableOffhand;
 	private String offhandCommand;
 	private boolean emulateLevitationEffect;
+	private boolean handlePlayerCombatPacket;
 
     public ViaRewindConfig(File configFile, Logger logger) {
         super(configFile, logger);
@@ -63,6 +64,7 @@ public class ViaRewindConfig extends Config implements com.viaversion.viarewind.
 		enableOffhand = getBoolean("enable-offhand", true);
 		offhandCommand = getString("offhand-command", "/offhand");
 		emulateLevitationEffect = getBoolean("emulate-levitation-effect", true);
+		handlePlayerCombatPacket = getBoolean("handle-player-combat-packet", true);
 	}
 
 	@Override
@@ -120,6 +122,11 @@ public class ViaRewindConfig extends Config implements com.viaversion.viarewind.
 		return emulateLevitationEffect;
 	}
 
+	@Override
+	public boolean handlePlayerCombatPacket() {
+		return handlePlayerCombatPacket;
+	}
+
 	@Override
     public URL getDefaultConfigURL() {
         return getClass().getClassLoader().getResource("assets/viarewind/config.yml");

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

@@ -110,4 +110,11 @@ public interface ViaRewindConfig extends Config {
 	 * @return true if enabled
 	 */
 	boolean emulateLevitationEffect();
+
+	/**
+	 * If enabled, 1.8 players will handle the player combat packet added in 1.9 by showing the custom death message above the hotbar.
+	 *
+	 * @return true if enabled
+	 */
+	boolean handlePlayerCombatPacket();
 }

common/src/main/java/com/viaversion/viarewind/api/rewriter/VREntityRewriter.java

@@ -25,6 +25,7 @@ import com.viaversion.viaversion.api.minecraft.ClientWorld;
 import com.viaversion.viaversion.api.minecraft.entitydata.EntityDataType;
 import com.viaversion.viaversion.api.minecraft.entitydata.types.EntityDataTypes1_8;
 import com.viaversion.viaversion.api.protocol.packet.ClientboundPacketType;
+import com.viaversion.viaversion.api.protocol.remapper.PacketHandler;
 import com.viaversion.viaversion.api.protocol.remapper.PacketHandlers;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.data.entity.EntityTrackerBase;
@@ -47,7 +48,7 @@ public abstract class VREntityRewriter<C extends ClientboundPacketType, T extend
 				map(Types.UNSIGNED_BYTE); // Game mode
 				map(Types.BYTE); // Dimension
 				handler(playerTrackerHandler());
-				handler(wrapper -> wrapper.user().get(ClientWorld.class).setEnvironment(wrapper.get(Types.BYTE, 0)));
+				handler(getDimensionHandler());
 			}
 		});
 	}
@@ -59,6 +60,14 @@ public abstract class VREntityRewriter<C extends ClientboundPacketType, T extend
 		}
 	}
 
+	protected PacketHandler getDimensionHandler() {
+		return wrapper -> {
+			final int dimension = wrapper.get(Types.BYTE, 0);
+			final ClientWorld clientWorld = wrapper.user().getClientWorld(protocol.getClass());
+			clientWorld.setEnvironment(dimension);
+		};
+	}
+
 	@Override
 	protected Object getDisplayVisibilityDataValue() {
 		return (byte) 1;

common/src/main/java/com/viaversion/viarewind/api/type/version/Types1_7_6_10.java

@@ -1,3 +1,20 @@
+/*
+ * This file is part of ViaRewind - https://github.com/ViaVersion/ViaRewind
+ * Copyright (C) 2018-2024 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.type.version;
 
 import com.viaversion.viarewind.api.type.entitydata.EntityDataListType;

common/src/main/java/com/viaversion/viarewind/protocol/v1_7_6_10to1_7_2_5/Protocol1_7_6_10To1_7_2_5.java

@@ -47,7 +47,7 @@ public class Protocol1_7_6_10To1_7_2_5 extends AbstractProtocol<ClientboundPacke
 
 	@Override
 	protected void registerPackets() {
-		this.registerClientbound(State.LOGIN, ClientboundLoginPackets.GAME_PROFILE.getId(), ClientboundLoginPackets.GAME_PROFILE.getId(), new PacketHandlers() {
+		this.registerClientbound(State.LOGIN, ClientboundLoginPackets.GAME_PROFILE, new PacketHandlers() {
 			@Override
 			public void register() {
 				map(Types.STRING, REMOVE_DASHES); // Uuid

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

@@ -61,11 +61,11 @@ public class Protocol1_8To1_7_6_10 extends BackwardsProtocol<ClientboundPackets1
 		itemRewriter.register();
 		entityRewriter.register();
 
-		PlayerPacketRewriter1_8.register(this);
-		ScoreboardPacketRewriter1_8.register(this);
-		WorldPacketRewriter1_8.register(this);
+		new PlayerPacketRewriter1_8(this).register();
+		new ScoreboardPacketRewriter1_8(this).register();
+		new WorldPacketRewriter1_8(this).register();
 
-		this.registerClientbound(State.LOGIN, ClientboundLoginPackets.HELLO.getId(), ClientboundLoginPackets.HELLO.getId(), new PacketHandlers() {
+		this.registerClientbound(State.LOGIN, ClientboundLoginPackets.HELLO, new PacketHandlers() {
 			@Override
 			public void register() {
 				map(Types.STRING); // server hash
@@ -73,7 +73,7 @@ public class Protocol1_8To1_7_6_10 extends BackwardsProtocol<ClientboundPackets1
 				map(Types.BYTE_ARRAY_PRIMITIVE, Types.SHORT_BYTE_ARRAY); // verification token
 			}
 		});
-		this.registerClientbound(State.LOGIN, ClientboundLoginPackets.LOGIN_COMPRESSION.getId(), ClientboundLoginPackets.LOGIN_COMPRESSION.getId(), new PacketHandlers() {
+		this.registerClientbound(State.LOGIN, ClientboundLoginPackets.LOGIN_COMPRESSION, new PacketHandlers() {
 			@Override
 			public void register() {
 				handler(wrapper -> {
@@ -92,7 +92,7 @@ public class Protocol1_8To1_7_6_10 extends BackwardsProtocol<ClientboundPackets1
 			}
 		});
 
-		this.registerServerbound(State.LOGIN, ServerboundLoginPackets.ENCRYPTION_KEY.getId(), ServerboundLoginPackets.ENCRYPTION_KEY.getId(), new PacketHandlers() {
+		this.registerServerbound(State.LOGIN, ServerboundLoginPackets.ENCRYPTION_KEY, new PacketHandlers() {
 			@Override
 			public void register() {
 				map(Types.SHORT_BYTE_ARRAY, Types.BYTE_ARRAY_PRIMITIVE); // shared secret
@@ -118,6 +118,7 @@ public class Protocol1_8To1_7_6_10 extends BackwardsProtocol<ClientboundPackets1
 	@Override
 	public void init(UserConnection connection) {
 		connection.addEntityTracker(this.getClass(), new EntityTracker1_8(connection));
+		connection.addClientWorld(this.getClass(), new ClientWorld());
 
 		connection.put(new InventoryTracker(connection));
 		connection.put(new PlayerSessionStorage(connection));
@@ -125,10 +126,6 @@ public class Protocol1_8To1_7_6_10 extends BackwardsProtocol<ClientboundPackets1
 		connection.put(new ScoreboardTracker(connection));
 		connection.put(new CompressionStatusTracker(connection));
 		connection.put(new WorldBorderEmulator(connection));
-
-		if (!connection.has(ClientWorld.class)) {
-			connection.put(new ClientWorld());
-		}
 	}
 
 	@Override

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

@@ -57,7 +57,7 @@ public class BlockItemPacketRewriter1_8 extends VRBlockItemRewriter<ClientboundP
 			wrapper.write(Types.UNSIGNED_BYTE, windowTypeId); // Window type id
 
 			final JsonElement titleComponent = wrapper.read(Types.COMPONENT); // Title
-			String title = ChatUtil.jsonToLegacy(wrapper.user(), titleComponent);
+			String title = ChatUtil.jsonToLegacy(titleComponent);
 			title = ChatUtil.removeUnusedColor(title, '8');
 			if (title.length() > 32) {
 				title = title.substring(0, 32);
@@ -134,7 +134,7 @@ public class BlockItemPacketRewriter1_8 extends VRBlockItemRewriter<ClientboundP
 						Item[] old = items;
 						items = new Item[old.length - 1];
 						items[0] = old[0];
-						System.arraycopy(old, 2, items, 1, old.length - 3);
+						System.arraycopy(old, 2, items, 1, old.length - 2);
 					}
 					for (int i = 0; i < items.length; i++) {
 						items[i] = handleItemToClient(wrapper.user(), items[i]);
@@ -291,7 +291,7 @@ public class BlockItemPacketRewriter1_8 extends VRBlockItemRewriter<ClientboundP
 			for (StringTag page : pages) {
 				final String value = page.getValue();
 				oldPages.add(new StringTag(value));
-				page.setValue(ChatUtil.jsonToLegacy(connection, value));
+				page.setValue(ChatUtil.jsonToLegacy(value));
 			}
 		}
 		return item;

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

@@ -33,7 +33,6 @@ import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage.GameProfileStora
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage.PlayerSessionStorage;
 import com.viaversion.viarewind.protocol.v1_8to1_7_6_10.storage.ScoreboardTracker;
 import com.viaversion.viaversion.api.minecraft.BlockPosition;
-import com.viaversion.viaversion.api.minecraft.ClientWorld;
 import com.viaversion.viaversion.api.minecraft.entities.EntityTypes1_8;
 import com.viaversion.viaversion.api.minecraft.entities.EntityTypes1_8.EntityType;
 import com.viaversion.viaversion.api.minecraft.item.Item;
@@ -70,6 +69,7 @@ public class EntityPacketRewriter1_8 extends VREntityRewriter<ClientboundPackets
 				read(Types.BOOLEAN); // Reduced debug info
 
 				handler(playerTrackerHandler());
+				handler(getDimensionHandler());
 				handler(wrapper -> {
 					final int entityId = wrapper.get(Types.INT, 0);
 
@@ -81,9 +81,8 @@ public class EntityPacketRewriter1_8 extends VREntityRewriter<ClientboundPackets
 
 					final EntityTracker1_8 tracker = wrapper.user().getEntityTracker(Protocol1_8To1_7_6_10.class);
 					tracker.addPlayer(entityId, wrapper.user().getProtocolInfo().getUuid());
-					tracker.setClientEntityGameMode(wrapper.get(Types.UNSIGNED_BYTE, 0));
 
-					wrapper.user().get(ClientWorld.class).setEnvironment(wrapper.get(Types.BYTE, 0));
+					tracker.setClientEntityGameMode(wrapper.get(Types.UNSIGNED_BYTE, 0));
 
 					// Reset on Velocity server change
 					wrapper.user().put(new ScoreboardTracker(wrapper.user()));
@@ -102,7 +101,7 @@ public class EntityPacketRewriter1_8 extends VREntityRewriter<ClientboundPackets
 			for (List<Integer> part : parts) {
 				final PacketWrapper destroy = PacketWrapper.create(ClientboundPackets1_7_2_5.REMOVE_ENTITIES, wrapper.user());
 				destroy.write(RewindTypes.INT_ARRAY, part.stream().mapToInt(Integer::intValue).toArray());
-				destroy.scheduleSend(Protocol1_8To1_7_6_10.class);
+				destroy.send(Protocol1_8To1_7_6_10.class);
 			}
 		});
 		protocol.registerClientbound(ClientboundPackets1_8.SET_ENTITY_DATA, new PacketHandlers() {

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

@@ -31,11 +31,10 @@ 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.viaversion.api.Via;
-import com.viaversion.viaversion.api.minecraft.Environment;
-import com.viaversion.viaversion.api.minecraft.entities.EntityTypes1_8;
 import com.viaversion.viaversion.api.minecraft.item.Item;
 import com.viaversion.viaversion.api.protocol.packet.PacketWrapper;
 import com.viaversion.viaversion.api.protocol.remapper.PacketHandlers;
+import com.viaversion.viaversion.api.rewriter.RewriterBase;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.libs.gson.JsonElement;
 import com.viaversion.nbt.tag.CompoundTag;
@@ -51,11 +50,17 @@ import io.netty.buffer.Unpooled;
 
 import java.nio.charset.StandardCharsets;
 import java.util.List;
+import java.util.Objects;
 import java.util.UUID;
 
-public class PlayerPacketRewriter1_8 {
+public class PlayerPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10> {
 
-	public static void register(Protocol1_8To1_7_6_10 protocol) {
+	public PlayerPacketRewriter1_8(Protocol1_8To1_7_6_10 protocol) {
+		super(protocol);
+	}
+
+	@Override
+	protected void registerPackets() {
 		protocol.registerClientbound(ClientboundPackets1_8.CHAT, new PacketHandlers() {
 			@Override
 			public void register() {
@@ -101,14 +106,13 @@ public class PlayerPacketRewriter1_8 {
 					final EntityTracker1_8 tracker = wrapper.user().getEntityTracker(Protocol1_8To1_7_6_10.class);
 					tracker.setClientEntityGameMode(wrapper.get(Types.UNSIGNED_BYTE, 1));
 
-					final Environment dimension = Environment.getEnvironmentById(wrapper.get(Types.INT, 0));
+					final int dimension = wrapper.get(Types.INT, 0);
 
 					// Clear entities on dimension change and re-track player
-					final ClientWorld world = wrapper.user().get(ClientWorld.class);
-					if (world.getEnvironment() != dimension) {
-						world.setEnvironment(dimension.id());
+					final ClientWorld world = wrapper.user().getClientWorld(Protocol1_8To1_7_6_10.class);
+					if (world.setEnvironment(dimension)) {
 						tracker.clearEntities();
-						tracker.addEntity(tracker.clientEntityId(), EntityTypes1_8.EntityType.PLAYER);
+						tracker.addPlayer(tracker.clientEntityId(), wrapper.user().getProtocolInfo().getUuid());
 					}
 
 					wrapper.send(Protocol1_8To1_7_6_10.class);
@@ -161,7 +165,7 @@ public class PlayerPacketRewriter1_8 {
 					wrapper.write(Types.BOOLEAN, playerSession.onGround);
 
 					final EntityTracker1_8 tracker = wrapper.user().getEntityTracker(Protocol1_8To1_7_6_10.class);
-					if (tracker.spectatingClientEntityId != tracker.clientEntityId()) {
+					if (!Objects.equals(tracker.spectatingClientEntityId, tracker.clientEntityIdOrNull())) {
 						wrapper.cancel();
 					}
 				});
@@ -251,7 +255,7 @@ public class PlayerPacketRewriter1_8 {
 					gameProfile.gamemode = gamemode;
 					JsonElement displayName = wrapper.read(Types.OPTIONAL_COMPONENT);
 					if (displayName != null) {
-						gameProfile.setDisplayName(ChatUtil.jsonToLegacy(wrapper.user(), displayName));
+						gameProfile.setDisplayName(ChatUtil.jsonToLegacy(displayName));
 					}
 
 					final PacketWrapper playerInfo = PacketWrapper.create(ClientboundPackets1_7_2_5.PLAYER_INFO, wrapper.user());
@@ -314,7 +318,7 @@ public class PlayerPacketRewriter1_8 {
 					packet.scheduleSend(Protocol1_8To1_7_6_10.class);
 				} else if (action == 3) {
 					JsonElement displayNameComponent = wrapper.read(Types.OPTIONAL_COMPONENT);
-					String displayName = displayNameComponent != null ? ChatUtil.jsonToLegacy(wrapper.user(), displayNameComponent) : null;
+					String displayName = displayNameComponent != null ? ChatUtil.jsonToLegacy(displayNameComponent) : null;
 
 					GameProfileStorage.GameProfile gameProfile = gameProfileStorage.get(uuid);
 					if (gameProfile == null || gameProfile.displayName == null && displayName == null) continue;

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

@@ -22,6 +22,7 @@ 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.storage.ScoreboardTracker;
 import com.viaversion.viaversion.api.protocol.packet.PacketWrapper;
 import com.viaversion.viaversion.api.protocol.remapper.PacketHandlers;
+import com.viaversion.viaversion.api.rewriter.RewriterBase;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.protocols.v1_8to1_9.packet.ClientboundPackets1_8;
 import com.viaversion.viaversion.util.ChatColorUtil;
@@ -30,9 +31,14 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
 
-public class ScoreboardPacketRewriter1_8 {
+public class ScoreboardPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10> {
 
-	public static void register(Protocol1_8To1_7_6_10 protocol) {
+	public ScoreboardPacketRewriter1_8(Protocol1_8To1_7_6_10 protocol) {
+		super(protocol);
+	}
+
+	@Override
+	protected void registerPackets() {
 		protocol.registerClientbound(ClientboundPackets1_8.SET_OBJECTIVE, wrapper -> {
 			String name = wrapper.passthrough(Types.STRING);
 			if (name.length() > 16) {

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

@@ -28,6 +28,7 @@ import com.viaversion.viaversion.api.minecraft.BlockChangeRecord;
 import com.viaversion.viaversion.api.minecraft.chunks.Chunk;
 import com.viaversion.viaversion.api.protocol.packet.PacketWrapper;
 import com.viaversion.viaversion.api.protocol.remapper.PacketHandlers;
+import com.viaversion.viaversion.api.rewriter.RewriterBase;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.api.type.types.FixedByteArrayType;
 import com.viaversion.viaversion.api.type.types.chunk.BulkChunkType1_8;
@@ -37,11 +38,16 @@ import com.viaversion.viaversion.protocols.v1_8to1_9.packet.ClientboundPackets1_
 import com.viaversion.viaversion.util.ChatColorUtil;
 import com.viaversion.viaversion.util.IdAndData;
 
-public class WorldPacketRewriter1_8 {
+public class WorldPacketRewriter1_8 extends RewriterBase<Protocol1_8To1_7_6_10> {
 
-	public static void register(Protocol1_8To1_7_6_10 protocol) {
+	public WorldPacketRewriter1_8(Protocol1_8To1_7_6_10 protocol) {
+		super(protocol);
+	}
+
+	@Override
+	protected void registerPackets() {
 		protocol.registerClientbound(ClientboundPackets1_8.LEVEL_CHUNK, wrapper -> {
-			final ClientWorld world = wrapper.user().get(ClientWorld.class);
+			final ClientWorld world = wrapper.user().getClientWorld(Protocol1_8To1_7_6_10.class);
 			final Chunk chunk = wrapper.read(ChunkType1_8.forEnvironment(world.getEnvironment()));
 			protocol.getItemRewriter().handleChunk(chunk);
 
@@ -173,7 +179,7 @@ public class WorldPacketRewriter1_8 {
 				handler(wrapper -> {
 					for (int i = 0; i < 4; i++) {
 						String line = wrapper.read(Types.STRING);
-						line = ChatUtil.jsonToLegacy(wrapper.user(), line);
+						line = ChatUtil.jsonToLegacy(line);
 						line = ChatUtil.removeUnusedColor(line, '0');
 
 						if (line.length() > 15) {

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

@@ -37,10 +37,7 @@ import com.viaversion.viaversion.libs.fastutil.objects.Object2IntMap;
 import com.viaversion.viaversion.libs.fastutil.objects.Object2IntOpenHashMap;
 import com.viaversion.viaversion.protocols.v1_8to1_9.packet.ClientboundPackets1_8;
 
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Map;
-import java.util.UUID;
+import java.util.*;
 import java.util.logging.Level;
 
 public class EntityTracker1_8 extends EntityTrackerBase {
@@ -50,9 +47,9 @@ public class EntityTracker1_8 extends EntityTrackerBase {
 	private final Int2ObjectMap<UUID> entityIdToUUID = new Int2ObjectArrayMap<>();
 	private final Object2IntMap<UUID> entityUUIDToId = new Object2IntOpenHashMap<>();
 
-	private List<EntityData> entityData = new ArrayList<>();
+	private final List<EntityData> entityData = new ArrayList<>();
 
-	public int spectatingClientEntityId = -1;
+	public Integer spectatingClientEntityId;
 	private int clientEntityGameMode;
 
 	public EntityTracker1_8(UserConnection connection) {
@@ -87,12 +84,16 @@ public class EntityTracker1_8 extends EntityTrackerBase {
 
 	@Override
 	public void setClientEntityId(int entityId) {
-		if (this.spectatingClientEntityId == this.clientEntityId()) {
+		if (Objects.equals(this.spectatingClientEntityId, clientEntityIdOrNull())) {
 			this.spectatingClientEntityId = entityId;
 		}
 		super.setClientEntityId(entityId);
 	}
 
+	public Integer clientEntityIdOrNull() {
+		return this.hasClientEntityId() ? this.clientEntityId() : null;
+	}
+
 	public void addPlayer(final int entityId, final UUID uuid) {
 		entityUUIDToId.put(uuid, entityId);
 		entityIdToUUID.put(entityId, uuid);
@@ -188,13 +189,13 @@ public class EntityTracker1_8 extends EntityTrackerBase {
 	}
 
 	public void updateEntityData(List<EntityData> entityData) {
-		entityData.removeIf(first -> entityData.stream().anyMatch(second -> first.id() == second.id()));
+		this.entityData.removeIf(first -> entityData.stream().anyMatch(second -> first.id() == second.id()));
 		for (final EntityData data : entityData) {
 			final Object value = data.value();
 			if (value instanceof Item item) {
-				entityData.add(new EntityData(data.id(), data.dataType(), item.copy()));
+				this.entityData.add(new EntityData(data.id(), data.dataType(), item.copy()));
 			} else {
-				entityData.add(new EntityData(data.id(), data.dataType(), value));
+				this.entityData.add(new EntityData(data.id(), data.dataType(), value));
 			}
 		}
 	}

common/src/main/java/com/viaversion/viarewind/protocol/v1_9to1_8/Protocol1_9To1_8.java

@@ -40,13 +40,13 @@ import java.util.concurrent.TimeUnit;
 
 public class Protocol1_9To1_8 extends BackwardsProtocol<ClientboundPackets1_9, ClientboundPackets1_8, ServerboundPackets1_9, ServerboundPackets1_8> {
 
-	public static final ValueTransformer<Double, Integer> DOUBLE_TO_INT_TIMES_32 = new ValueTransformer<Double, Integer>(Types.INT) {
+	public static final ValueTransformer<Double, Integer> DOUBLE_TO_INT_TIMES_32 = new ValueTransformer<>(Types.INT) {
 		@Override
 		public Integer transform(PacketWrapper wrapper, Double inputValue) {
 			return (int) (inputValue * 32.0D);
 		}
 	};
-	public static final ValueTransformer<Float, Byte> DEGREES_TO_ANGLE = new ValueTransformer<Float, Byte>(Types.BYTE) {
+	public static final ValueTransformer<Float, Byte> DEGREES_TO_ANGLE = new ValueTransformer<>(Types.BYTE) {
 		@Override
 		public Byte transform(PacketWrapper packetWrapper, Float degrees) {
 			return (byte) ((degrees / 360F) * 256);
@@ -66,14 +66,14 @@ public class Protocol1_9To1_8 extends BackwardsProtocol<ClientboundPackets1_9, C
 	protected void registerPackets() {
 		entityRewriter.register();
 		itemRewriter.register();
-
-		PlayerPacketRewriter1_9.register(this);
-		WorldPacketRewriter1_9.register(this);
+		new PlayerPacketRewriter1_9(this).register();
+		new WorldPacketRewriter1_9(this).register();
 	}
 
 	@Override
 	public void init(UserConnection connection) {
 		connection.addEntityTracker(this.getClass(), new EntityTracker1_9(connection));
+		connection.addClientWorld(this.getClass(), new ClientWorld());
 
 		connection.put(new WindowTracker(connection));
 		connection.put(new LevitationStorage());
@@ -81,10 +81,6 @@ public class Protocol1_9To1_8 extends BackwardsProtocol<ClientboundPackets1_9, C
 		connection.put(new CooldownStorage());
 		connection.put(new BlockPlaceDestroyTracker());
 		connection.put(new BossBarStorage(connection));
-
-		if (!connection.has(ClientWorld.class)) {
-			connection.put(new ClientWorld());
-		}
 	}
 
 	@Override

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

@@ -409,11 +409,7 @@ public class EntityPacketRewriter1_9 extends VREntityRewriter<ClientboundPackets
 			final int vehicle = wrapper.read(Types.VAR_INT);
 			final IntList oldPassengers = tracker.getPassengers(vehicle);
 
-			final int count = wrapper.read(Types.VAR_INT);
-			final IntList passengers = new IntArrayList();
-			for (int i = 0; i < count; i++) {
-				passengers.add(wrapper.read(Types.VAR_INT));
-			}
+			final IntList passengers = new IntArrayList(wrapper.read(Types.VAR_INT_ARRAY_PRIMITIVE));
 			tracker.setPassengers(vehicle, passengers);
 
 			if (!oldPassengers.isEmpty()) {
@@ -425,7 +421,7 @@ public class EntityPacketRewriter1_9 extends VREntityRewriter<ClientboundPackets
 					detach.scheduleSend(Protocol1_9To1_8.class);
 				}
 			}
-			for (int i = 0; i < count; i++) {
+			for (int i = 0; i < passengers.size(); i++) {
 				final int attachedEntityId = passengers.getInt(i);
 				final int holdingEntityId = i == 0 ? vehicle : passengers.getInt(i - 1);
 

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

@@ -28,6 +28,7 @@ import com.viaversion.viaversion.api.minecraft.entitydata.EntityData;
 import com.viaversion.viaversion.api.minecraft.entitydata.types.EntityDataTypes1_8;
 import com.viaversion.viaversion.api.protocol.packet.PacketWrapper;
 import com.viaversion.viaversion.api.protocol.remapper.PacketHandlers;
+import com.viaversion.viaversion.api.rewriter.RewriterBase;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.api.type.types.version.Types1_8;
 import com.viaversion.viaversion.libs.gson.JsonElement;
@@ -44,9 +45,14 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.UUID;
 
-public class PlayerPacketRewriter1_9 {
+public class PlayerPacketRewriter1_9 extends RewriterBase<Protocol1_9To1_8> {
 
-	public static void register(final Protocol1_9To1_8 protocol) {
+	public PlayerPacketRewriter1_9(Protocol1_9To1_8 protocol) {
+		super(protocol);
+	}
+
+	@Override
+	protected void registerPackets() {
 		protocol.registerClientbound(ClientboundPackets1_9.BOSS_EVENT, null, wrapper -> {
 			wrapper.cancel();
 			final BossBarStorage bossbar = wrapper.user().get(BossBarStorage.class);
@@ -60,7 +66,7 @@ public class PlayerPacketRewriter1_9 {
 				wrapper.read(Types.VAR_INT); // Division
 				wrapper.read(Types.UNSIGNED_BYTE); // Flags
 
-				bossbar.add(uuid, ChatUtil.jsonToLegacy(wrapper.user(), title), health);
+				bossbar.add(uuid, ChatUtil.jsonToLegacy(title), health);
 			} else if (action == 1 /* remove */) {
 				bossbar.remove(uuid);
 			} else if (action == 2 /* update health */) {
@@ -68,7 +74,7 @@ public class PlayerPacketRewriter1_9 {
 				bossbar.updateHealth(uuid, health);
 			} else if (action == 3 /* update title */) {
 				final JsonElement title = wrapper.read(Types.COMPONENT);
-				bossbar.updateTitle(uuid, ChatUtil.jsonToLegacy(wrapper.user(), title));
+				bossbar.updateTitle(uuid, ChatUtil.jsonToLegacy(title));
 			}
 		});
 
@@ -165,16 +171,24 @@ public class PlayerPacketRewriter1_9 {
 			@Override
 			public void register() {
 				map(Types.INT); // Dimension
-				handler(wrapper -> wrapper.user().get(BossBarStorage.class).reset());
 				handler(wrapper -> {
-					final ClientWorld world = wrapper.user().get(ClientWorld.class);
-					world.setEnvironment(wrapper.get(Types.INT, 0));
+					final ClientWorld world = wrapper.user().getClientWorld(Protocol1_9To1_8.class);
+					final int dimension = wrapper.get(Types.INT, 0);
+
+					if (world.setEnvironment(dimension)) {
+						protocol.getEntityRewriter().clearEntities(wrapper.user());
+						wrapper.user().get(BossBarStorage.class).reset();
+					}
 				});
 			}
 		});
 
 		// Packet implementation doesn't do anything until 1.9
 		protocol.registerClientbound(ClientboundPackets1_9.PLAYER_COMBAT, wrapper -> {
+			if (!ViaRewind.getConfig().handlePlayerCombatPacket()) {
+				return;
+			}
+
 			final int action = wrapper.passthrough(Types.VAR_INT);
 			if (action != 2) {
 				return;
@@ -198,7 +212,7 @@ public class PlayerPacketRewriter1_9 {
 
 			final PacketWrapper chatMessage = wrapper.create(ClientboundPackets1_8.CHAT);
 			chatMessage.write(Types.COMPONENT, message);
-			chatMessage.write(Types.BYTE, (byte) 1); // Position - in chat
+			chatMessage.write(Types.BYTE, (byte) 2); // Position - above hotbar
 			chatMessage.scheduleSend(Protocol1_9To1_8.class);
 		});
 
@@ -421,7 +435,7 @@ public class PlayerPacketRewriter1_9 {
 				map(Types.BLOCK_POSITION1_8); // Position
 				handler(wrapper -> {
 					for (int i = 0; i < 4; i++) {
-						wrapper.write(Types.STRING, ChatUtil.jsonToLegacy(wrapper.user(), wrapper.read(Types.COMPONENT)));
+						wrapper.write(Types.STRING, ChatUtil.jsonToLegacy(wrapper.read(Types.COMPONENT)));
 					}
 				});
 			}
@@ -467,25 +481,30 @@ public class PlayerPacketRewriter1_9 {
 				handlerSoftFail(wrapper -> {
 					final String channel = wrapper.get(Types.STRING, 0);
 					if (channel.equals("MC|BEdit") || channel.equals("MC|BSign")) {
-						Item book = wrapper.passthrough(Types.ITEM1_8);
+						final Item book = wrapper.passthrough(Types.ITEM1_8);
 						book.setIdentifier(386);
-						CompoundTag tag = book.tag();
-						if (tag.contains("pages")) {
-							ListTag<StringTag> pages = tag.getListTag("pages", StringTag.class);
-							if (pages.size() > ViaRewind.getConfig().getMaxBookPages()) {
-								wrapper.user().disconnect("Too many book pages");
+						final CompoundTag tag = book.tag();
+						if (tag == null) {
+							return;
+						}
+						final ListTag<StringTag> pages = tag.getListTag("pages", StringTag.class);
+						if (pages == null) {
+							return;
+						}
+						if (pages.size() > ViaRewind.getConfig().getMaxBookPages()) {
+							wrapper.user().disconnect("Too many book pages");
+							return;
+						}
+						for (int i = 0; i < pages.size(); i++) {
+							final StringTag pageTag = pages.get(i);
+							final String value = pageTag.getValue();
+
+							if (value.length() > ViaRewind.getConfig().getMaxBookPageSize()) {
+								wrapper.user().disconnect("Book page too large");
 								return;
 							}
-							for (int i = 0; i < pages.size(); i++) {
-								StringTag page = pages.get(i);
-								String value = page.getValue();
-								if (value.length() > ViaRewind.getConfig().getMaxBookPageSize()) {
-									wrapper.user().disconnect("Book page too large");
-									return;
-								}
-								value = ChatUtil.jsonToLegacy(wrapper.user(), value);
-								page.setValue(value);
-							}
+
+							pageTag.setValue(ChatUtil.jsonToLegacy(value));
 						}
 					} else if (channel.equals("MC|AdvCdm")) {
 						wrapper.set(Types.STRING, 0, "MC|AdvCmd");

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

@@ -21,7 +21,6 @@ import com.viaversion.nbt.tag.Tag;
 import com.viaversion.viarewind.ViaRewind;
 import com.viaversion.viarewind.protocol.v1_9to1_8.Protocol1_9To1_8;
 import com.viaversion.viarewind.protocol.v1_9to1_8.data.EffectIdMappings1_8;
-import com.viaversion.viaversion.api.connection.UserConnection;
 import com.viaversion.viaversion.api.minecraft.BlockPosition;
 import com.viaversion.viaversion.api.minecraft.Environment;
 import com.viaversion.viaversion.api.minecraft.chunks.BaseChunk;
@@ -32,6 +31,7 @@ import com.viaversion.viaversion.api.minecraft.chunks.DataPalette;
 import com.viaversion.viaversion.api.minecraft.chunks.PaletteType;
 import com.viaversion.viaversion.api.protocol.packet.PacketWrapper;
 import com.viaversion.viaversion.api.protocol.remapper.PacketHandlers;
+import com.viaversion.viaversion.api.rewriter.RewriterBase;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.api.type.types.chunk.ChunkType1_8;
 import com.viaversion.viaversion.api.type.types.chunk.ChunkType1_9_1;
@@ -39,13 +39,17 @@ import com.viaversion.viaversion.protocols.v1_8to1_9.packet.ClientboundPackets1_
 import com.viaversion.viaversion.protocols.v1_8to1_9.packet.ClientboundPackets1_9;
 import com.viaversion.nbt.tag.CompoundTag;
 import com.viaversion.nbt.tag.StringTag;
-import com.viaversion.viaversion.api.minecraft.ClientWorld;
 
 import java.util.ArrayList;
 
-public class WorldPacketRewriter1_9 {
+public class WorldPacketRewriter1_9 extends RewriterBase<Protocol1_9To1_8> {
 
-	public static void register(Protocol1_9To1_8 protocol) {
+	public WorldPacketRewriter1_9(Protocol1_9To1_8 protocol) {
+		super(protocol);
+	}
+
+	@Override
+	protected void registerPackets() {
 		protocol.registerClientbound(ClientboundPackets1_9.BLOCK_ENTITY_DATA, new PacketHandlers() {
 			@Override
 			public void register() {
@@ -107,7 +111,7 @@ public class WorldPacketRewriter1_9 {
 			@Override
 			public void register() {
 				handler(wrapper -> {
-					final Environment environment = wrapper.user().get(ClientWorld.class).getEnvironment();
+					final Environment environment = wrapper.user().getClientWorld(Protocol1_9To1_8.class).getEnvironment();
 
 					final int chunkX = wrapper.read(Types.INT);
 					final int chunkZ = wrapper.read(Types.INT);
@@ -121,7 +125,7 @@ public class WorldPacketRewriter1_9 {
 			@Override
 			public void register() {
 				handler(wrapper -> {
-					final Environment environment = wrapper.user().get(ClientWorld.class).getEnvironment();
+					final Environment environment = wrapper.user().getClientWorld(Protocol1_9To1_8.class).getEnvironment();
 
 					Chunk chunk = wrapper.read(ChunkType1_9_1.forEnvironment(environment));
 

common/src/main/java/com/viaversion/viarewind/utils/ChatUtil.java

@@ -18,7 +18,7 @@
 package com.viaversion.viarewind.utils;
 
 import com.viaversion.viarewind.ViaRewind;
-import com.viaversion.viaversion.api.connection.UserConnection;
+import com.viaversion.viaversion.api.Via;
 import com.viaversion.viaversion.api.protocol.packet.ClientboundPacketType;
 import com.viaversion.viaversion.libs.gson.JsonElement;
 import com.viaversion.viaversion.libs.gson.JsonObject;
@@ -43,29 +43,35 @@ public class ChatUtil {
 		}
 	};
 
-	public static String jsonToLegacy(UserConnection connection, String json) {
-		if (json == null || json.equals("null") || json.isEmpty()) return "";
+	public static String jsonToLegacy(String json) {
+		if (json == null || json.equals("null") || json.isEmpty()) {
+			return "";
+		}
 		try {
-			return jsonToLegacy(connection, JsonParser.parseString(json));
+			return jsonToLegacy(JsonParser.parseString(json));
 		} catch (Exception e) {
-			ViaRewind.getPlatform().getLogger().log(Level.WARNING, "Could not convert component to legacy text: " + json, e);
+			if (!Via.getConfig().isSuppressConversionWarnings()) {
+				ViaRewind.getPlatform().getLogger().log(Level.WARNING, "Could not convert component to legacy text: " + json, e);
+			}
 		}
 		return "";
 	}
 
-	public static String jsonToLegacy(UserConnection connection, JsonElement component) {
+	public static String jsonToLegacy(JsonElement component) {
 		if (component.isJsonNull() || component.isJsonArray() && component.getAsJsonArray().isEmpty() || component.isJsonObject() && component.getAsJsonObject().isEmpty()) {
 			return "";
 		} else if (component.isJsonPrimitive()) {
 			return component.getAsString();
 		} else {
 			try {
-				LEGACY_REWRITER.processText(connection, component);
+				LEGACY_REWRITER.processText(null, component);
 				String legacy = ComponentUtil.jsonToLegacy(component);
 				while (legacy.startsWith("§f")) legacy = legacy.substring(2);
 				return legacy;
 			} catch (Exception ex) {
-				ViaRewind.getPlatform().getLogger().log(Level.WARNING, "Could not convert component to legacy text: " + component, ex);
+				if (!Via.getConfig().isSuppressConversionWarnings()) {
+					ViaRewind.getPlatform().getLogger().log(Level.WARNING, "Could not convert component to legacy text: " + component, ex);
+				}
 			}
 			return "";
 		}

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

@@ -39,4 +39,7 @@ enable-offhand: true
 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
+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

fabric/src/main/resources/fabric.mod.json

@@ -23,9 +23,8 @@
     ]
   },
   "depends": {
-    "fabricloader": ">=0.14.0",
-    "viafabric": ">=0.4.10",
-    "viabackwards": ">=5.0.0"
+    "viafabric": ">=0.4.14",
+    "viabackwards": ">=5.0.4"
   },
   "custom": {
     "modmenu:api": true,

gradle.properties

@@ -4,16 +4,10 @@ org.gradle.parallel=true
 
 # project
 maven_group=com.viaversion
-maven_version=4.0.2
+maven_version=4.0.4
 maven_description=ViaBackwards addon to allow 1.8.x and 1.7.x clients on newer server versions.
 
-# libraries
-viaversion_version=5.0.2
-viabackwards_version=5.0.2
-netty_version=4.0.20.Final
-guava_version=17.0
-
 # Smile emoji
-mcVersions=1.21, 1.20.6, 1.20.5, 1.20.4, 1.20.3, 1.20.2, 1.20.1, 1.20, 1.19.4, 1.19.3, 1.19.2, 1.19.1, 1.19, 1.18.2, 1.18.1, 1.18, 1.17.1, 1.17, 1.16.5, 1.16.4, 1.16.3, 1.16.2, 1.16.1, 1.16, 1.15.2, 1.15.1, 1.15, 1.14.4, 1.14.3, 1.14.2, 1.14.1, 1.14, 1.13.2, 1.13.1, 1.13, 1.12.2, 1.12.1, 1.12, 1.11.2, 1.11.1, 1.11, 1.10.2, 1.10.1, 1.10, 1.9.4, 1.9.3, 1.9.2, 1.9.1, 1.9, 1.8.9, 1.8.8
-mcVersionRange=1.8-1.21
+mcVersions=1.21.4, 1.21.3, 1.21.2, 1.21.1, 1.21, 1.20.6, 1.20.5, 1.20.4, 1.20.3, 1.20.2, 1.20.1, 1.20, 1.19.4, 1.19.3, 1.19.2, 1.19.1, 1.19, 1.18.2, 1.18.1, 1.18, 1.17.1, 1.17, 1.16.5, 1.16.4, 1.16.3, 1.16.2, 1.16.1, 1.16, 1.15.2, 1.15.1, 1.15, 1.14.4, 1.14.3, 1.14.2, 1.14.1, 1.14, 1.13.2, 1.13.1, 1.13, 1.12.2, 1.12.1, 1.12, 1.11.2, 1.11.1, 1.11, 1.10.2, 1.10.1, 1.10, 1.9.4, 1.9.3, 1.9.2, 1.9.1, 1.9, 1.8.9, 1.8.8
+mcVersionRange=1.8-1.21.4
 velocityVersion=3.3

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,7 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-bin.zip
+distributionSha256Sum=f397b287023acdba1e9f6fc5ea72d22dd63669d59ed4a289a29b1a76eee151c6
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.11.1-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME

gradlew

@@ -15,6 +15,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+# SPDX-License-Identifier: Apache-2.0
+#
 
 ##############################################################################
 #
@@ -55,7 +57,7 @@
 #       Darwin, MinGW, and NonStop.
 #
 #   (3) This script is generated from the Groovy template
-#       https://github.com/gradle/gradle/blob/HEAD/subprojects/plugins/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
+#       https://github.com/gradle/gradle/blob/HEAD/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/.
@@ -84,7 +86,8 @@ done
 # shellcheck disable=SC2034
 APP_BASE_NAME=${0##*/}
 # Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036)
-APP_HOME=$( cd "${APP_HOME:-./}" > /dev/null && pwd -P ) || exit
+APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s
+' "$PWD" ) || exit
 
 # Use the maximum available, or set MAX_FD != -1 to use that value.
 MAX_FD=maximum

gradlew.bat

@@ -1,92 +1,94 @@
-@rem
-@rem Copyright 2015 the original author or authors.
-@rem
-@rem Licensed under the Apache License, Version 2.0 (the "License");
-@rem you may not use this file except in compliance with the License.
-@rem You may obtain a copy of the License at
-@rem
-@rem      https://www.apache.org/licenses/LICENSE-2.0
-@rem
-@rem Unless required by applicable law or agreed to in writing, software
-@rem distributed under the License is distributed on an "AS IS" BASIS,
-@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-@rem See the License for the specific language governing permissions and
-@rem limitations under the License.
-@rem
-
-@if "%DEBUG%"=="" @echo off
-@rem ##########################################################################
-@rem
-@rem  Gradle startup script for Windows
-@rem
-@rem ##########################################################################
-
-@rem Set local scope for the variables with windows NT shell
-if "%OS%"=="Windows_NT" setlocal
-
-set DIRNAME=%~dp0
-if "%DIRNAME%"=="" set DIRNAME=.
-@rem This is normally unused
-set APP_BASE_NAME=%~n0
-set APP_HOME=%DIRNAME%
-
-@rem Resolve any "." and ".." in APP_HOME to make it shorter.
-for %%i in ("%APP_HOME%") do set APP_HOME=%%~fi
-
-@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
-set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
-
-@rem Find java.exe
-if defined JAVA_HOME goto findJavaFromJavaHome
-
-set JAVA_EXE=java.exe
-%JAVA_EXE% -version >NUL 2>&1
-if %ERRORLEVEL% equ 0 goto execute
-
-echo. 1>&2
-echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. 1>&2
-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
-
-:findJavaFromJavaHome
-set JAVA_HOME=%JAVA_HOME:"=%
-set JAVA_EXE=%JAVA_HOME%/bin/java.exe
-
-if exist "%JAVA_EXE%" goto execute
-
-echo. 1>&2
-echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% 1>&2
-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
-
-:execute
-@rem Setup the command line
-
-set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
-
-
-@rem Execute Gradle
-"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %*
-
-: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
+@rem
+@rem Copyright 2015 the original author or authors.
+@rem
+@rem Licensed under the Apache License, Version 2.0 (the "License");
+@rem you may not use this file except in compliance with the License.
+@rem You may obtain a copy of the License at
+@rem
+@rem      https://www.apache.org/licenses/LICENSE-2.0
+@rem
+@rem Unless required by applicable law or agreed to in writing, software
+@rem distributed under the License is distributed on an "AS IS" BASIS,
+@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+@rem See the License for the specific language governing permissions and
+@rem limitations under the License.
+@rem
+@rem SPDX-License-Identifier: Apache-2.0
+@rem
+
+@if "%DEBUG%"=="" @echo off
+@rem ##########################################################################
+@rem
+@rem  Gradle startup script for Windows
+@rem
+@rem ##########################################################################
+
+@rem Set local scope for the variables with windows NT shell
+if "%OS%"=="Windows_NT" setlocal
+
+set DIRNAME=%~dp0
+if "%DIRNAME%"=="" set DIRNAME=.
+@rem This is normally unused
+set APP_BASE_NAME=%~n0
+set APP_HOME=%DIRNAME%
+
+@rem Resolve any "." and ".." in APP_HOME to make it shorter.
+for %%i in ("%APP_HOME%") do set APP_HOME=%%~fi
+
+@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
+set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
+
+@rem Find java.exe
+if defined JAVA_HOME goto findJavaFromJavaHome
+
+set JAVA_EXE=java.exe
+%JAVA_EXE% -version >NUL 2>&1
+if %ERRORLEVEL% equ 0 goto execute
+
+echo. 1>&2
+echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. 1>&2
+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
+
+:findJavaFromJavaHome
+set JAVA_HOME=%JAVA_HOME:"=%
+set JAVA_EXE=%JAVA_HOME%/bin/java.exe
+
+if exist "%JAVA_EXE%" goto execute
+
+echo. 1>&2
+echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% 1>&2
+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
+
+:execute
+@rem Setup the command line
+
+set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
+
+
+@rem Execute Gradle
+"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %*
+
+: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

settings.gradle

@@ -5,6 +5,7 @@ includeBuild("build-logic")
 setupViaSubproject("common")
 setupViaSubproject("bukkit")
 setupViaSubproject("fabric")
+setupViaSubproject("sponge")
 setupViaSubproject("velocity")
 
 void setupViaSubproject(String name) {

sponge/build.gradle

@@ -0,0 +1,9 @@
+plugins {
+    id "vr.platform-conventions"
+}
+
+processResources {
+    filesMatching("META-INF/sponge_plugins.json") {
+        expand("version": project.version, "description": project.description)
+    }
+}

sponge/src/main/resources/META-INF/sponge_plugins.json

@@ -0,0 +1,55 @@
+{
+  "loader": {
+    "name": "java_plain",
+    "version": "1.0"
+  },
+  "license": "GNU GPLv3",
+  "global": {
+    "version": "${version}",
+    "links": {
+      "homepage": "https://viaversion.com/rewind",
+      "source": "https://github.com/ViaVersion/ViaRewind",
+      "issues": "https://github.com/ViaVersion/ViaRewind"
+    },
+    "contributors": [
+      {
+        "name": "FlorianMichael/EnZaXD",
+        "description": "Author"
+      },
+      {
+        "name": "Gerrygames",
+        "description": "Author"
+      },
+      {
+        "name": "creeper123123321",
+        "description": "Maintainer"
+      }
+    ],
+    "dependencies": [
+      {
+        "id": "spongeapi",
+        "version": "8.0.0"
+      },
+      {
+        "id": "viasponge",
+        "version": "1.1.0"
+      },
+      {
+        "id": "viaversion",
+        "version": "[5.0.4-SNAPSHOT,)"
+      },
+      {
+        "id": "viabackwards",
+        "version": "[5.0.4-SNAPSHOT,)"
+      }
+    ]
+  },
+  "plugins": [
+    {
+      "id": "viarewind",
+      "name": "ViaRewind",
+      "entrypoint": "com.viaversion.sponge.util.DummyEntrypoint",
+      "description": "${description}"
+    }
+  ]
+}