Fix compilation error and implement action configuration system

Co-authored-by: ptthanh02 <73684260+ptthanh02@users.noreply.github.com>

Author: Copilot

core/src/main/java/github/nighter/smartspawner/spawner/gui/layout/GuiLayout.java

@@ -28,6 +28,11 @@ public Optional<String> getButtonTypeAtSlot(int slot) {
         return Optional.ofNullable(slotToButtonType.get(slot));
     }
 
+    public Optional<GuiButton> getButtonAtSlot(int slot) {
+        String buttonType = slotToButtonType.get(slot);
+        return buttonType != null ? Optional.ofNullable(buttons.get(buttonType)) : Optional.empty();
+    }
+
     public boolean hasButton(String buttonType) {
         return buttons.containsKey(buttonType) && buttons.get(buttonType).isEnabled();
     }

core/src/main/java/github/nighter/smartspawner/spawner/gui/main/SpawnerMenuAction.java

@@ -93,8 +93,16 @@ public void onMenuClick(InventoryClickEvent event) {
             return;
         }
 
-        Material itemType = clickedItem.getType();
+        // Use layout-based action handling
+        int slot = event.getRawSlot();
+        String clickType = getClickTypeString(event.getClick());
+        
+        if (handleLayoutAction(player, spawner, slot, clickType)) {
+            return;
+        }
 
+        // Fallback to legacy material-based handling for backward compatibility
+        Material itemType = clickedItem.getType();
         if (itemType == Material.CHEST) {
             handleChestClick(player, spawner);
         } else if (SPAWNER_INFO_MATERIALS.contains(itemType)) {
@@ -104,6 +112,68 @@ public void onMenuClick(InventoryClickEvent event) {
         }
     }
 
+    private boolean handleLayoutAction(Player player, SpawnerData spawner, int slot, String clickType) {
+        var layoutConfig = plugin.getGuiLayoutConfig();
+        var layout = layoutConfig.getCurrentMainLayout();
+        
+        if (layout == null) {
+            return false;
+        }
+
+        var buttonOpt = layout.getButtonAtSlot(slot);
+        if (buttonOpt.isEmpty()) {
+            return false;
+        }
+
+        var button = buttonOpt.get();
+        String action = button.getAction(clickType);
+        
+        if (action == null) {
+            return false;
+        }
+
+        switch (action) {
+            case "open_storage":
+                handleChestClick(player, spawner);
+                return true;
+            case "toggle_info":
+                if (isClickTooFrequent(player)) {
+                    return true;
+                }
+                // Check stacker permission and open stacker GUI
+                if (!player.hasPermission("smartspawner.stack")) {
+                    messageService.sendMessage(player, "no_permission");
+                    return true;
+                }
+                spawnerStackerUI.openStackerGui(player, spawner);
+                player.playSound(player.getLocation(), Sound.UI_BUTTON_CLICK, 1.0f, 1.0f);
+                return true;
+            case "sell_inventory":
+                if (isClickTooFrequent(player)) {
+                    return true;
+                }
+                // Collect EXP and sell items if shop integration is available
+                handleExpBottleClick(player, spawner, true);
+                handleSellAllItems(player, spawner);
+                return true;
+            case "collect_exp":
+                handleExpBottleClick(player, spawner, false);
+                return true;
+            default:
+                return false;
+        }
+    }
+
+    private String getClickTypeString(ClickType clickType) {
+        return switch (clickType) {
+            case LEFT -> "left_click";
+            case RIGHT -> "right_click";
+            case SHIFT_LEFT -> "shift_left_click";
+            case SHIFT_RIGHT -> "shift_right_click";
+            default -> "left_click";
+        };
+    }
+
     public void handleChestClick(Player player, SpawnerData spawner) {
         String title = languageManager.getGuiTitle("gui_title_storage");
         Inventory pageInventory = spawnerStorageUI.createInventory(spawner, title, 1, -1);

core/src/main/java/github/nighter/smartspawner/spawner/gui/storage/SpawnerStorageAction.java

@@ -139,6 +139,70 @@ private void handleControlSlotClick(Player player, int slot, StoragePageHolder h
             return;
         }
 
+        Optional<GuiButton> buttonOpt = layout.getButtonAtSlot(slot);
+        if (buttonOpt.isEmpty()) {
+            return;
+        }
+
+        GuiButton button = buttonOpt.get();
+        
+        // For now, we'll use left_click as the default action since storage GUI
+        // currently doesn't differentiate between click types in most cases
+        String action = button.getAction("left_click");
+        if (action == null) {
+            // Fallback to legacy button type handling
+            handleLegacyButtonType(player, slot, holder, spawner, inventory, layout);
+            return;
+        }
+
+        switch (action) {
+            case "discard_all":
+                handleDiscardAllItems(player, spawner, inventory);
+                break;
+            case "item_filter":
+                openFilterConfig(player, spawner);
+                break;
+            case "previous_page":
+                if (holder.getCurrentPage() > 1) {
+                    updatePageContent(player, spawner, holder.getCurrentPage() - 1, inventory, true);
+                }
+                break;
+            case "take_all":
+                handleTakeAllItems(player, inventory);
+                break;
+            case "next_page":
+                if (holder.getCurrentPage() < holder.getTotalPages()) {
+                    updatePageContent(player, spawner, holder.getCurrentPage() + 1, inventory, true);
+                }
+                break;
+            case "drop_page":
+                handleDropPageItems(player, spawner, inventory);
+                break;
+            case "sell_all":
+                if (plugin.hasSellIntegration()) {
+                    if (!player.hasPermission("smartspawner.sellall")) {
+                        messageService.sendMessage(player, "no_permission");
+                        return;
+                    }
+                    if (isClickTooFrequent(player)) {
+                        return;
+                    }
+                    player.playSound(player.getLocation(), Sound.UI_BUTTON_CLICK, 1.0f, 1.0f);
+                    spawnerSellManager.sellAllItems(player, spawner);
+                }
+                break;
+            case "return":
+                openMainMenu(player, spawner);
+                break;
+            default:
+                // Fallback to legacy handling for unknown actions
+                handleLegacyButtonType(player, slot, holder, spawner, inventory, layout);
+                break;
+        }
+    }
+
+    private void handleLegacyButtonType(Player player, int slot, StoragePageHolder holder,
+                                       SpawnerData spawner, Inventory inventory, GuiLayout layout) {
         Optional<String> buttonTypeOpt = layout.getButtonTypeAtSlot(slot);
         if (buttonTypeOpt.isEmpty()) {
             return;

core/src/main/java/github/nighter/smartspawner/spawner/gui/storage/SpawnerStorageUI.java

@@ -129,7 +129,6 @@ private void initializeStaticButtons() {
                     languageManager.getGuiItemLoreAsList("sell_button.lore")
             ));
         }
-        }
     }
 
     public Inventory createInventory(SpawnerData spawner, String title, int page, int totalPages) {