feat: Implement ItemStack request processing system and inventory improvements

src/main/java/cn/nukkit/Server.java

@@ -530,6 +530,12 @@ public Level remove(@NotNull Object key) {
      * Server authority block destruction
      */
     public boolean serverAuthoritativeBlockBreaking;
+    /**
+     * Server authoritative inventory mode
+     * When enabled, server has final authority over inventory changes
+     * @since v1.16.100 (protocol 407+)
+     */
+    public boolean serverAuthoritativeInventory;
     /**
      * Network encryption
      */
@@ -3358,6 +3364,7 @@ private void loadSettings() {
             default -> this.serverAuthoritativeMovementMode = 1; // server-auth
         }
         this.serverAuthoritativeBlockBreaking = this.getPropertyBoolean("server-authoritative-block-breaking", true);
+        this.serverAuthoritativeInventory = this.getPropertyBoolean("server-authoritative-inventory", true);
 
         // === Advanced MOT settings from nukkit-mot.yml ===
         ServerConfig config = this.serverConfig;
@@ -3577,6 +3584,7 @@ private static class ServerProperties extends ConfigSection {
 
             put("server-authoritative-movement", "server-auth");
             put("server-authoritative-block-breaking", true);
+            put("server-authoritative-inventory", true);
         }
     }
 

src/main/java/cn/nukkit/block/Block.java

@@ -765,27 +765,13 @@ public Item[] getDrops(@Nullable Player player, Item item) {
         return this.getDrops(item);
     }
 
-    private double customToolBreakTimeBonus(int toolType, @Nullable Integer speed) {
-        if (speed != null) return speed;
-        else if (toolType == ItemTool.TYPE_SWORD) {
-            if (this instanceof BlockCobweb) {
-                return 15.0;
-            } else if (this instanceof BlockBamboo) {
-                return 30.0;
-            } else return 1.0;
-        } else if (toolType == ItemTool.TYPE_SHEARS) {
-            if (this instanceof BlockWool || this instanceof BlockLeaves) {
-                return 5.0;
-            } else if (this instanceof BlockCobweb) {
-                return 15.0;
-            } else return 1.0;
-        } else if (toolType == ItemTool.TYPE_NONE) return 1.0;
-        return 0;
-    }
-
     private double toolBreakTimeBonus0(Item item) {
-        if (item instanceof ItemCustomTool itemCustomTool && itemCustomTool.getSpeed() != null) {
-            return customToolBreakTimeBonus(customToolType(item), itemCustomTool.getSpeed());
+        if (item instanceof ItemCustomTool itemCustomTool) {
+            //按当前方块查 destroy_speeds；未命中则回退原版逻辑（tier 查表 + sword/shears 特殊值）
+            Integer speed = itemCustomTool.getSpeedFor(getId());
+            if (speed != null) {
+                return speed;
+            }
         }
         return toolBreakTimeBonus0(toolType0(item, getId()), item.getTier(), this.getId());
     }
@@ -830,10 +816,6 @@ private static double speedRateByHasteLore0(int hasteLoreLevel) {
         return 1.0 + (0.2 * hasteLoreLevel);
     }
 
-    private int customToolType(Item item) {
-        return toolType0(item, this.getId());
-    }
-
     private static int toolType0(Item item, int blockId) {
         if (item.isHoe()) {
             switch (blockId) {
@@ -857,6 +839,11 @@ private static int toolType0(Item item, int blockId) {
     }
 
     private static boolean correctTool0(int blockToolType, Item item, int blockId) {
+        //自定义工具：digger 含此方块即视为正确工具（让 addExtraBlock 能挖非自身类型的方块）
+        if (item instanceof ItemCustomTool customTool && customTool.getSpeedFor(blockId) != null) {
+            return true;
+        }
+
         boolean isLeaves = blockId == LEAVES || blockId == LEAVES2 || blockId == AZALEA_LEAVES
                 || blockId == AZALEA_LEAVES_FLOWERED || blockId == MANGROVE_LEAVES || blockId == CHERRY_LEAVES || blockId == PALE_OAK_LEAVES;
 

src/main/java/cn/nukkit/block/BlockCartographyTable.java

@@ -1,5 +1,9 @@
 package cn.nukkit.block;
 
+import cn.nukkit.Player;
+import cn.nukkit.inventory.CartographyTableInventory;
+import cn.nukkit.item.Item;
+import cn.nukkit.item.ItemBlock;
 import cn.nukkit.item.ItemTool;
 import cn.nukkit.utils.BlockColor;
 
@@ -39,4 +43,22 @@ public BlockColor getColor() {
     public int getBurnChance() {
         return 5;
     }
+
+    @Override
+    public boolean canBeActivated() {
+        return true;
+    }
+
+    @Override
+    public boolean onActivate(Item item, Player player) {
+        if (player != null) {
+            player.addWindow(new CartographyTableInventory(player.getUIInventory(), this), Player.CARTOGRAPHY_WINDOW_ID);
+        }
+        return true;
+    }
+
+    @Override
+    public Item toItem() {
+        return new ItemBlock(Block.get(this.getId(), 0), 0);
+    }
 }

src/main/java/cn/nukkit/block/BlockChest.java

@@ -54,7 +54,7 @@ public int getId() {
     public Class<? extends BlockEntityChest> getBlockEntityClass() {
         return BlockEntityChest.class;
     }
-    
+
     @NotNull
     @Override
     public String getBlockEntityType() {

src/main/java/cn/nukkit/block/BlockDropper.java

@@ -20,7 +20,7 @@
 import java.util.Map;
 import java.util.concurrent.ThreadLocalRandom;
 
-public class BlockDropper extends BlockSolidMeta implements Faceable {
+public class BlockDropper extends BlockSolidMeta implements Faceable, BlockEntityHolder<BlockEntityDropper> {
 
     protected boolean triggered = false;
 
@@ -42,6 +42,18 @@ public String getName() {
         return "Dropper";
     }
 
+    @NotNull
+    @Override
+    public Class<? extends BlockEntityDropper> getBlockEntityClass() {
+        return BlockEntityDropper.class;
+    }
+
+    @NotNull
+    @Override
+    public String getBlockEntityType() {
+        return BlockEntity.DROPPER;
+    }
+
     @Override
     public double getHardness() {
         return 0.5;

src/main/java/cn/nukkit/block/BlockEntityHolder.java

@@ -58,12 +58,12 @@ default E createBlockEntity(@Nullable CompoundTag initialData, @Nullable Object.
         } else {
             initialData = initialData.copy();
         }
-        BlockEntity created = BlockEntity.createBlockEntity(typeName, chunk, 
+        BlockEntity created = BlockEntity.createBlockEntity(typeName, chunk,
                 initialData
                     .putString("id", typeName)
                     .putInt("x", getFloorX())
                     .putInt("y", getFloorY())
-                    .putInt("z", getFloorZ()), 
+                    .putInt("z", getFloorZ()),
                 args);
 
         Class<? extends E> entityClass = getBlockEntityClass();
@@ -123,7 +123,7 @@ static <E extends BlockEntity, H extends BlockEntityHolder<E>> E setBlockAndCrea
     static <E extends BlockEntity, H extends BlockEntityHolder<E>> E setBlockAndCreateEntity(
             @NotNull H holder, boolean direct, boolean update, @Nullable CompoundTag initialData,
             @Nullable Object... args) {
-        Block block = holder.getBlock(); 
+        Block block = holder.getBlock();
         Level level = block.getLevel();
         Block layer0 = level.getBlock(block, 0);
         Block layer1 = level.getBlock(block, 1);
@@ -137,7 +137,7 @@ static <E extends BlockEntity, H extends BlockEntityHolder<E>> E setBlockAndCrea
                 throw e;
             }
         }
-        
+
         return null;
     }
 

src/main/java/cn/nukkit/block/BlockShulkerBox.java

@@ -20,7 +20,7 @@
 /**
  * Created by PetteriM1
  */
-public class BlockShulkerBox extends BlockTransparentMeta {
+public class BlockShulkerBox extends BlockTransparentMeta implements BlockEntityHolder<BlockEntityShulkerBox> {
 
     public BlockShulkerBox() {
         this(0);
@@ -45,6 +45,18 @@ public String getName() {
         return this.getDyeColor().getName() + " Shulker Box";
     }
 
+    @NotNull
+    @Override
+    public Class<? extends BlockEntityShulkerBox> getBlockEntityClass() {
+        return BlockEntityShulkerBox.class;
+    }
+
+    @NotNull
+    @Override
+    public String getBlockEntityType() {
+        return BlockEntity.SHULKER_BOX;
+    }
+
     @Override
     public double getHardness() {
         return 2.5;
@@ -196,4 +208,4 @@ public Item[] getDrops(@Nullable Player player, Item item) {
     public boolean diffusesSkyLight() {
         return true;
     }
-}
+}

src/main/java/cn/nukkit/blockentity/BlockEntityBeacon.java

@@ -242,7 +242,15 @@ public void setSecondaryPower(int power) {
     }
 
     private static final IntSet ALLOWED_EFFECTS = new IntOpenHashSet(new int[]{Effect.SPEED, Effect.HASTE, Effect.DAMAGE_RESISTANCE, Effect.JUMP, Effect.STRENGTH, Effect.REGENERATION});
-    private static final IntSet ITEMS = new IntOpenHashSet(new int[]{Item.AIR, ItemID.NETHERITE_INGOT, ItemID.EMERALD, ItemID.DIAMOND, ItemID.GOLD_INGOT, ItemID. IRON_INGOT});
+    private static final IntSet PAYMENT_ITEMS = new IntOpenHashSet(new int[]{ItemID.NETHERITE_INGOT, ItemID.EMERALD, ItemID.DIAMOND, ItemID.GOLD_INGOT, ItemID.IRON_INGOT});
+
+    public static boolean isAllowedEffect(int effectId) {
+        return effectId == 0 || ALLOWED_EFFECTS.contains(effectId);
+    }
+
+    public static boolean isPaymentItem(int itemId) {
+        return PAYMENT_ITEMS.contains(itemId);
+    }
 
     @Override
     public boolean updateCompoundTag(CompoundTag nbt, Player player) {
@@ -262,7 +270,7 @@ public boolean updateCompoundTag(CompoundTag nbt, Player player) {
             }
 
             int material = beaconInventory.useMaterial();
-            if (!ITEMS.contains(material)) {
+            if (!isPaymentItem(material)) {
                 Server.getInstance().getLogger().debug(player.getName() + " tried to set effect but there's no payment in beacon inventory");
                 return false;
             }
@@ -273,14 +281,14 @@ public boolean updateCompoundTag(CompoundTag nbt, Player player) {
         }
 
         int primary = nbt.getInt("primary");
-        if (ALLOWED_EFFECTS.contains(primary)) {
+        if (isAllowedEffect(primary)) {
             this.setPrimaryPower(primary);
         } else {
             Server.getInstance().getLogger().debug(player.getName() + " tried to set an invalid primary effect to a beacon: " + primary);
         }
 
         int secondary = nbt.getInt("secondary");
-        if (ALLOWED_EFFECTS.contains(secondary)) {
+        if (isAllowedEffect(secondary)) {
             this.setSecondaryPower(secondary);
         } else {
             Server.getInstance().getLogger().debug(player.getName() + " tried to set an invalid secondary effect to a beacon: " + secondary);

src/main/java/cn/nukkit/entity/passive/EntityDonkey.java

@@ -48,11 +48,11 @@ public float getHeight() {
     public void initEntity() {
         this.setMaxHealth(15);
 
-        super.initEntity();
-
         if (this.namedTag.contains("ChestedHorse")) {
             this.setChested(this.namedTag.getBoolean("ChestedHorse"));
         }
+
+        super.initEntity();
     }
 
     @Override
@@ -115,7 +115,16 @@ public boolean isChested() {
     }
 
     public void setChested(boolean chested) {
+        boolean changed = this.chested != chested;
         this.chested = chested;
         this.setDataFlag(DATA_FLAGS, DATA_FLAG_CHESTED, chested);
+        if (changed) {
+            this.syncHorseInventoryChestSize();
+        }
+    }
+
+    @Override
+    protected int getChestSize() {
+        return this.chested ? 15 : 0;
     }
 }