From 7a8beea66e18fea53f309a659705cc955615a935 Mon Sep 17 00:00:00 2001
From: embeddedt <42941056+embeddedt@users.noreply.github.com>
Date: Sun, 25 Jan 2026 12:35:07 -0500
Subject: [PATCH] Clear encoder cache when configuration finishes & on
 disconnect

Credit to @XFactHD for the suggestion
---
 ...tConfigurationPacketListenerImplMixin.java | 21 ++++++++++++++++
 .../encoder_cache_leak/MinecraftMixin.java    | 25 +++++++++++++++++++
 2 files changed, 46 insertions(+)
 create mode 100644 src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/ClientConfigurationPacketListenerImplMixin.java
 create mode 100644 src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/MinecraftMixin.java

diff --git a/src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/ClientConfigurationPacketListenerImplMixin.java b/src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/ClientConfigurationPacketListenerImplMixin.java
new file mode 100644
index 00000000..deb20085
--- /dev/null
+++ b/src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/ClientConfigurationPacketListenerImplMixin.java
@@ -0,0 +1,21 @@
+package org.embeddedt.modernfix.common.mixin.perf.encoder_cache_leak;
+
+import net.minecraft.client.multiplayer.ClientConfigurationPacketListenerImpl;
+import org.embeddedt.modernfix.annotation.ClientOnlyMixin;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
+
+@Mixin(ClientConfigurationPacketListenerImpl.class)
+@ClientOnlyMixin
+public class ClientConfigurationPacketListenerImplMixin {
+    /**
+     * @author embeddedt
+     * @reason Reset the encoder cache after configuration finishes as the registries are now changing.
+     */
+    @Inject(method = "handleConfigurationFinished", at = @At("RETURN"))
+    private void resetEncoderCache(CallbackInfo ci) {
+        ((EncoderCacheAccessor)DataComponentsAccessor.mfix$getCache()).mfix$getCache().invalidateAll();
+    }
+}
diff --git a/src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/MinecraftMixin.java b/src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/MinecraftMixin.java
new file mode 100644
index 00000000..91fe6326
--- /dev/null
+++ b/src/main/java/org/embeddedt/modernfix/common/mixin/perf/encoder_cache_leak/MinecraftMixin.java
@@ -0,0 +1,25 @@
+package org.embeddedt.modernfix.common.mixin.perf.encoder_cache_leak;
+
+import net.minecraft.client.Minecraft;
+import org.embeddedt.modernfix.annotation.ClientOnlyMixin;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
+
+@Mixin(Minecraft.class)
+@ClientOnlyMixin
+public class MinecraftMixin {
+    /**
+     * @author embeddedt
+     * @reason Make sure the encoder cache is cleared when the client disconnects, as it retains strong references
+     * to registries.
+     */
+    @Inject(method = {
+            "disconnect(Lnet/minecraft/client/gui/screens/Screen;Z)V",
+            "clearClientLevel(Lnet/minecraft/client/gui/screens/Screen;)V"
+    }, at = @At("RETURN"))
+    private void clearEncoderCache(CallbackInfo ci) {
+        ((EncoderCacheAccessor)DataComponentsAccessor.mfix$getCache()).mfix$getCache().invalidateAll();
+    }
+}