Add bounds checks for compartment lookup

This *shouldn't* be needed, as the compartment index is always >= 0, and
Inventory.getItem returns an empty stack if the item is not found.

However, some mods pass a negative compartment index, which causes
getItem to throw an AIOOB instead.

Fixes #2267

Author: SquidDev

projects/common/src/main/java/dan200/computercraft/shared/util/InventoryUtil.java

@@ -54,7 +54,7 @@ public static int getHandSlot(Player player, InteractionHand hand) {
      */
     public static int getInventorySlotFromCompartment(Player player, int slot, ItemStack stack) {
         if (stack.isEmpty()) throw new IllegalArgumentException("Cannot search for empty stack");
-        if (player.getInventory().getItem(slot) == stack) return slot;
+        if (slot >= 0 && slot < Inventory.INVENTORY_SIZE && player.getInventory().getItem(slot) == stack) return slot;
         if (player.getInventory().getItem(Inventory.SLOT_OFFHAND) == stack) return Inventory.SLOT_OFFHAND;
         return -1;
     }