patch: replace direct config access with a type-safe method for detection items

this is probably extra asl but i hate the little yellow numbers

Author: modpotato

src/main/java/top/modpotato/commands/AntiNetheriteCommand.java

@@ -250,8 +250,7 @@ public boolean onCommand(CommandSender sender, Command command, String label, St
                     String action = args[2].toLowerCase();
                     String item = args[3].toUpperCase();
 
-                    @SuppressWarnings("unchecked")
-                    List<String> items = (List<String>) plugin.getConfigValue("anti-netherite.detection.items");
+                    List<String> items = getDetectionItems();
 
                     if (action.equals("add")) {
                         if (!items.contains(item)) {
@@ -485,7 +484,7 @@ private boolean handleGetCommand(CommandSender sender, String[] args) {
 
         // Handle special case for detection.items
         if (setting.equals("detection.items")) {
-            List<String> items = (List<String>) plugin.getConfigValue("anti-netherite.detection.items");
+            List<String> items = getDetectionItems();
             sender.sendMessage(Component.text("Netherite items list:").color(NamedTextColor.GREEN));
             for (String item : items) {
                 sender.sendMessage(Component.text("- " + item).color(NamedTextColor.YELLOW));
@@ -621,4 +620,26 @@ private List<String> filterCompletions(List<String> completions, String input) {
         }
         return filtered;
     }
+
+    /**
+     * Safely retrieves the detection.items list from the config, ensuring type safety.
+     * Filters out any non-string entries and logs warnings if the data type is unexpected.
+     * @return a mutable List<String> of detection items (never null)
+     */
+    private List<String> getDetectionItems() {
+        Object value = plugin.getConfigValue("anti-netherite.detection.items");
+        List<String> result = new ArrayList<>();
+        if (value instanceof List<?>) {
+            for (Object o : (List<?>) value) {
+                if (o instanceof String s) {
+                    result.add(s);
+                } else if (o != null) {
+                    plugin.getLogger().warning("Non-string entry in detection.items: " + o);
+                }
+            }
+        } else if (value != null) {
+            plugin.getLogger().warning("detection.items config value is not a list: " + value.getClass().getName());
+        }
+        return result;
+    }
 } 

src/main/java/top/modpotato/listeners/InventoryMoveListener.java

@@ -32,6 +32,7 @@ public InventoryMoveListener(NetheriteDetector netheriteDetector, Config config)
         this.config = config;
     }
 
+    @SuppressWarnings("removal")
     @EventHandler
     public void onInventoryClick(InventoryClickEvent event) {
         if (!(event.getWhoClicked() instanceof Player)) {