Wandering Trader no longer despawns if it has chat data

CHANGELOG.md

@@ -15,6 +15,7 @@ All notable changes to **CreatureChat** are documented in this file. The format
 ### Fixed
 - Bees no longer forget their chat data when entering/leaving hives (writeNbt & readNbt modified)
 - Vexes no longer take damage when chat data exists
+- Wandering Trader no longer despawns if it has chat data
 
 ## [1.3.0] - 2025-01-14
 

src/main/java/com/owlmaddie/mixin/MixinWanderingTrader.java

+package com.owlmaddie.mixin;
+
+import com.owlmaddie.chat.ChatDataManager;
+import com.owlmaddie.chat.EntityChatData;
+import net.minecraft.entity.passive.WanderingTraderEntity;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+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;
+
+/**
+ * Prevents WanderingTraderEntity from despawning if it has chat data or a character sheet.
+ */
+@Mixin(WanderingTraderEntity.class)
+public abstract class MixinWanderingTrader {
+    private static final Logger LOGGER = LoggerFactory.getLogger("creaturechat");
+
+    @Inject(method = "tickDespawnDelay", at = @At("HEAD"), cancellable = true)
+    private void preventTraderDespawn(CallbackInfo ci) {
+        WanderingTraderEntity trader = (WanderingTraderEntity) (Object) this;
+
+        // Get chat data for this trader
+        EntityChatData chatData = ChatDataManager.getServerInstance().getOrCreateChatData(trader.getUuidAsString());
+
+        // If the character sheet is not empty, cancel the function to prevent despawning
+        if (!chatData.characterSheet.isEmpty()) {
+            ci.cancel();
+        }
+    }
+}

src/main/resources/creaturechat.mixins.json

@@ -10,6 +10,7 @@
 		"MixinEntityChatData",
 		"MixinWitherEntity",
 		"MixinVexEntity",
+		"MixinWanderingTrader",
 		"MixinVillagerEntity",
 		"MixinOnChat"
 	],