chore: better track mutations to internal item stacks

for REI "quick craft" compat

Author: marcus8448

src/main/java/dev/galacticraft/machinelib/api/menu/ConfiguredMenu.java

@@ -333,6 +333,16 @@ protected boolean moveItemStackTo(ItemStack stack, int startIndex, int endIndex,
         return super.moveItemStackTo(stack, startIndex, endIndex, reverse);
     }
 
+    @Override
+    public void sendAllDataToRemote() {
+        for (int i = 0; i < this.internalSlots; i++) {
+            if (this.slots.get(i) instanceof StorageSlot storageSlot) {
+                storageSlot.handleStackMutation();
+            }
+        }
+        super.sendAllDataToRemote();
+    }
+
     /**
      * Cycles the I/O configuration of a machine face.
      *

src/main/java/dev/galacticraft/machinelib/impl/compat/vanilla/StorageSlot.java

@@ -86,6 +86,18 @@ public boolean allowModification(Player player) {
         return this.watchedStack;
     }
 
+    @ApiStatus.Internal
+    public void handleStackMutation() {
+        if (this.watchedStack != null && this.watchModCount == this.slot.getModifications()
+                && (this.watchedStack.getCount() != this.slot.getAmount()
+                || !this.slot.getComponents().equals(this.watchedStack.getComponentsPatch())
+                || !Utils.itemsEqual(this.slot.getResource(), this.watchedStack.getItem()))) {
+            this.set(this.watchedStack);
+            this.watchModCount = this.slot.getModifications();
+            this.setChanged();
+        }
+    }
+
     @Override
     public boolean hasItem() {
         return !this.slot.isEmpty();
@@ -109,7 +121,6 @@ public void setChanged() {
                     || !this.slot.getComponents().equals(this.watchedStack.getComponentsPatch())
                     || !Utils.itemsEqual(this.slot.getResource(), this.watchedStack.getItem())
             ) {
-                if (true) throw new AssertionError();
                 this.set(this.watchedStack);
                 this.slot.markModified();
             }