Added slot to item highlight, added better error capture for generate…

…d assets
KubeJS-Mods · Sep 7, 2024 · 0b52eed · 0b52eed
1 parent b561674
commit 0b52eed
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 4 deletions.
diff --git a/src/main/java/dev/latvian/mods/kubejs/client/ClientAssetPacks.java b/src/main/java/dev/latvian/mods/kubejs/client/ClientAssetPacks.java
@@ -37,6 +37,15 @@ public ClientAssetPacks() {
 	}
 
 	public List<PackResources> inject(List<PackResources> original) {
+		try {
+			return inject0(original);
+		} catch (Throwable ex) {
+			ConsoleJS.CLIENT.error("Error while generating client assets", ex);
+			return original;
+		}
+	}
+
+	private List<PackResources> inject0(List<PackResources> original) {
 		var packs = new ArrayList<>(original);
 
 		var filePacks = new ArrayList<PackResources>();

diff --git a/src/main/java/dev/latvian/mods/kubejs/client/ModelGenerator.java b/src/main/java/dev/latvian/mods/kubejs/client/ModelGenerator.java
@@ -4,6 +4,7 @@
 import com.google.gson.JsonObject;
 import dev.latvian.mods.kubejs.bindings.AABBWrapper;
 import dev.latvian.mods.kubejs.bindings.DirectionWrapper;
+import dev.latvian.mods.rhino.util.HideFromJS;
 import net.minecraft.core.Direction;
 import net.minecraft.resources.ResourceLocation;
 import net.minecraft.world.phys.AABB;
@@ -169,6 +170,7 @@ public void parent(@Nullable ResourceLocation s) {
 		parent = s;
 	}
 
+	@HideFromJS
 	public void texture(String name, String texture) {
 		textures.put(name, texture);
 	}

diff --git a/src/main/java/dev/latvian/mods/kubejs/kubedex/KubedexPayloadHandler.java b/src/main/java/dev/latvian/mods/kubejs/kubedex/KubedexPayloadHandler.java
@@ -30,6 +30,9 @@
 import java.util.stream.Stream;
 
 public class KubedexPayloadHandler {
+	public record SlotItem(ItemStack item, int slot) {
+	}
+
 	private static ListTag sortedTagList(Stream<? extends TagKey<?>> stream) {
 		return stream
 			.map(TagKey::location)
@@ -132,35 +135,41 @@ public static void entity(ServerPlayer player, int entityId, int flags) {
 	}
 
 	public static void inventory(ServerPlayer player, List<Integer> slots, List<ItemStack> stacks, int flags) {
-		var allStacks = new LinkedHashSet<>(stacks);
+		var allStacks = new LinkedHashSet<SlotItem>();
+
+		for (var s : stacks) {
+			allStacks.add(new SlotItem(s, -1));
+		}
 
 		for (int s : slots) {
 			if (s >= 0 && s < player.getInventory().getContainerSize()) {
 				var item = player.getInventory().getItem(s);
 
 				if (!item.isEmpty()) {
-					allStacks.add(item);
+					allStacks.add(new SlotItem(item, s));
 				}
 			}
 		}
 
 		itemStacks(player, allStacks, flags);
 	}
 
-	public static void itemStacks(ServerPlayer player, Collection<ItemStack> stacks, int flags) {
+	public static void itemStacks(ServerPlayer player, Collection<SlotItem> stacks, int flags) {
 		var ops = player.server.registryAccess().createSerializationContext(NbtOps.INSTANCE);
 
 		var payload = new CompoundTag();
 		payload.put("flags", flags(flags));
 
 		var payloadItems = new ListTag();
 
-		for (var stack : stacks) {
+		for (var slotStack : stacks) {
+			var stack = slotStack.item;
 			var tag = new OrderedCompoundTag();
 			tag.putString("string", stack.kjs$toItemString0(ops));
 			tag.put("item", ItemStack.CODEC.encodeStart(ops, stack).result().get());
 			tag.put("name", ComponentSerialization.FLAT_CODEC.encodeStart(ops, stack.getHoverName()).getOrThrow());
 			tag.putString("icon", stack.kjs$getWebIconURL(ops, 64).toString());
+			tag.putInt("slot", slotStack.slot);
 
 			var patch = stack.getComponentsPatch();