fix(clientmod): null-guard entity data scan in ravengard session logger

getNonDefaultValues returns null rather than an empty list when an entity
carries no non-default data, and the rig discovery scan touches every
display in the level, so the first plain display crashed the client tick.
Guards all three call sites and wraps the tick body so a logger fault
degrades to a chat error instead of a crash.

Author: Swofty-Developments

clientmod/src/main/java/gg/itzkatze/thehypixelrecreationmod/features/packetlog/RavengardSessionLogger.java

@@ -122,7 +122,14 @@ public static void tick() {
         if (!active) {
             return;
         }
+        try {
+            tickInternal();
+        } catch (RuntimeException exception) {
+            ChatUtils.error("Ravengard session logger tick failed: " + exception);
+        }
+    }
 
+    private static void tickInternal() {
         Minecraft client = Minecraft.getInstance();
         if (client.level == null) {
             StopResult result = stop();
@@ -165,7 +172,11 @@ private static void discoverRigs(Minecraft client) {
     }
 
     private static String rigModel(Entity entity) {
-        for (SynchedEntityData.DataValue<?> value : entity.getEntityData().getNonDefaultValues()) {
+        List<SynchedEntityData.DataValue<?>> values = entity.getEntityData().getNonDefaultValues();
+        if (values == null) {
+            return null;
+        }
+        for (SynchedEntityData.DataValue<?> value : values) {
             if (value.value() instanceof ItemStack stack && !stack.isEmpty()) {
                 var model = stack.get(DataComponents.ITEM_MODEL);
                 if (model != null && model.toString().startsWith(RIG_MODEL_PREFIX)) {
@@ -188,8 +199,11 @@ private static void addEntity(Entity entity, String reason) {
                 + " uuid=" + entity.getUUID()
                 + " pos=vec3(" + entity.getX() + "," + entity.getY() + "," + entity.getZ() + ")");
         writeLine("#   reason: " + reason);
-        for (SynchedEntityData.DataValue<?> value : entity.getEntityData().getNonDefaultValues()) {
-            writeLine("#   data " + value.id() + " = " + formatValue(value.value()));
+        List<SynchedEntityData.DataValue<?>> values = entity.getEntityData().getNonDefaultValues();
+        if (values != null) {
+            for (SynchedEntityData.DataValue<?> value : values) {
+                writeLine("#   data " + value.id() + " = " + formatValue(value.value()));
+            }
         }
 
         for (Entity passenger : entity.getPassengers()) {
@@ -234,8 +248,10 @@ private static void sampleTrack(Minecraft client) {
                     .append(",\"z\":").append(fmt(entity.getZ()))
                     .append(",\"yaw\":").append(fmt(entity.getYRot()));
 
-            if (entity instanceof Display display) {
-                for (SynchedEntityData.DataValue<?> value : display.getEntityData().getNonDefaultValues()) {
+            List<SynchedEntityData.DataValue<?>> displayValues =
+                    entity instanceof Display display ? display.getEntityData().getNonDefaultValues() : null;
+            if (displayValues != null) {
+                for (SynchedEntityData.DataValue<?> value : displayValues) {
                     Object raw = value.value();
                     if (raw instanceof Vector3fc vector) {
                         json.append(",\"d").append(value.id()).append("\":[")