refactor: own shared inventory scanning in TLibs

README.md

@@ -13,6 +13,7 @@ Players encounter its effects through the plugins that depend on it: custom equi
 - **Equipment appearance preservation** — carries supported skin and appearance information through item rebuilding.
 - **Tiered socket handling** — shared rules for socket categories and the gems or runes applied to them.
 - **Equipment events** — exposes armour equip changes so other plugins can respond consistently.
+- **Inventory scanning** — shared periodic and event-driven scans; gameplay plugins register handlers for their own item updates.
 - **Shared persistence and utilities** — provides database support and common text, time, and location helpers for dependent plugins.
 
 TLibs supports the gameplay plugins that provide the player-facing experiences, keeping common behavior in one place.

pom.xml

@@ -97,6 +97,12 @@
             <version>5.10.2</version>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.mockito</groupId>
+            <artifactId>mockito-core</artifactId>
+            <version>5.20.0</version>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-simple</artifactId>
@@ -208,9 +214,11 @@
                 <artifactId>maven-surefire-plugin</artifactId>
                 <version>3.2.5</version>
                 <configuration>
+                    <systemPropertyVariables>
+                        <!-- The provided BungeeCord bundle contains a proxy-specific SLF4J provider. -->
+                        <slf4j.provider>org.slf4j.simple.SimpleServiceProvider</slf4j.provider>
+                    </systemPropertyVariables>
                     <classpathDependencyExcludes>
-                        <classpathDependencyExclude>io.papermc.paper:paper-api</classpathDependencyExclude>
-                        <classpathDependencyExclude>net.md-5:bungeecord-chat</classpathDependencyExclude>
                         <classpathDependencyExclude>local:ItemsAdder</classpathDependencyExclude>
                         <classpathDependencyExclude>local:MMOItems</classpathDependencyExclude>
                         <classpathDependencyExclude>local:MythicLib</classpathDependencyExclude>

src/main/java/net/tfminecraft/tlibs/TLibs.java

@@ -8,6 +8,7 @@
 import net.tfminecraft.tlibs.config.RebuildConfig;
 import net.tfminecraft.tlibs.config.SocketTierConfig;
 import net.tfminecraft.tlibs.enums.APIType;
+import net.tfminecraft.tlibs.itemscan.ItemScanService;
 import net.tfminecraft.tlibs.listener.FurnitureRepairListener;
 import net.tfminecraft.tlibs.mmoitem.GemApplyPrimer;
 import net.tfminecraft.tlibs.mmoitem.MMOItemRebuildRegistrar;
@@ -33,6 +34,7 @@ public void onEnable() {
 		saveDefaultConfig();
 		reloadPluginConfig();
 		initializeAPIs();
+		ItemScanService.start(this);
 		GemApplyPrimer.init(this);
 		ArmorEquipEvent.registerListener(this);
 		Bukkit.getPluginManager().registerEvents(new FurnitureRepairListener(), this);
@@ -48,6 +50,13 @@ public void onEnable() {
 		Bukkit.getLogger().info("[TLibs] Complete!");
 	}
 
+	@Override
+	public void onDisable() {
+		if (ItemScanService.get() != null) {
+			ItemScanService.get().stop();
+		}
+	}
+
 	private void registerRebuildBridge() {
 		getServer().getPluginManager().registerEvents(rebuildRegistrar, this);
 		rebuildRegistrar.tryRegister();

src/main/java/net/tfminecraft/tlibs/itemscan/ItemScanHandler.java

@@ -0,0 +1,12 @@
+package net.tfminecraft.tlibs.itemscan;
+
+import org.bukkit.entity.Player;
+import org.bukkit.inventory.Inventory;
+import org.bukkit.inventory.ItemStack;
+
+public interface ItemScanHandler {
+
+    boolean matches(ItemStack stack);
+
+    void update(Player player, Inventory inventory, int slot, ItemStack stack);
+}

src/main/java/net/tfminecraft/tlibs/itemscan/ItemScanService.java

@@ -0,0 +1,155 @@
+package net.tfminecraft.tlibs.itemscan;
+
+import java.util.ArrayList;
+import java.util.List;
+import java.util.concurrent.CopyOnWriteArrayList;
+
+import org.bukkit.Bukkit;
+import org.bukkit.Material;
+import org.bukkit.block.BlockState;
+import org.bukkit.block.DoubleChest;
+import org.bukkit.entity.Entity;
+import org.bukkit.entity.Item;
+import org.bukkit.entity.Player;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.HandlerList;
+import org.bukkit.event.Listener;
+import org.bukkit.event.entity.EntityPickupItemEvent;
+import org.bukkit.event.inventory.InventoryOpenEvent;
+import org.bukkit.event.inventory.InventoryType;
+import org.bukkit.inventory.Inventory;
+import org.bukkit.inventory.InventoryHolder;
+import org.bukkit.inventory.InventoryView;
+import org.bukkit.inventory.ItemStack;
+import org.bukkit.plugin.java.JavaPlugin;
+import org.bukkit.scheduler.BukkitTask;
+
+public final class ItemScanService implements Listener {
+
+    private static ItemScanService instance;
+
+    private final List<ItemScanHandler> handlers = new CopyOnWriteArrayList<>();
+    private int roundRobin;
+    private BukkitTask task;
+
+    public static ItemScanService get() {
+        return instance;
+    }
+
+    public static void start(JavaPlugin plugin) {
+        if (instance != null) {
+            instance.stop();
+        }
+        instance = new ItemScanService();
+        plugin.getServer().getPluginManager().registerEvents(instance, plugin);
+        instance.task = Bukkit.getScheduler().runTaskTimer(plugin, instance::pulse, 0L, 2L);
+    }
+
+    public void stop() {
+        if (task != null) {
+            task.cancel();
+            task = null;
+        }
+        HandlerList.unregisterAll(this);
+        handlers.clear();
+        if (instance == this) {
+            instance = null;
+        }
+    }
+
+    public void subscribe(ItemScanHandler handler) {
+        if (handler != null && !handlers.contains(handler)) {
+            handlers.add(handler);
+        }
+    }
+
+    public void unsubscribe(ItemScanHandler handler) {
+        handlers.remove(handler);
+    }
+
+    private void pulse() {
+        if (handlers.isEmpty()) {
+            return;
+        }
+        List<? extends Player> players = new ArrayList<>(Bukkit.getOnlinePlayers());
+        if (players.isEmpty()) {
+            roundRobin = 0;
+            return;
+        }
+        if (roundRobin >= players.size()) {
+            roundRobin = 0;
+        }
+        Player player = players.get(roundRobin);
+        roundRobin++;
+        scanPlayerView(player);
+    }
+
+    @EventHandler
+    public void onInventoryOpen(InventoryOpenEvent event) {
+        if (!(event.getPlayer() instanceof Player player)) {
+            return;
+        }
+        scanOpenView(player, event.getView());
+    }
+
+    @EventHandler
+    public void onPickup(EntityPickupItemEvent event) {
+        if (!(event.getEntity() instanceof Player player)) {
+            return;
+        }
+        Item entity = event.getItem();
+        ItemStack stack = entity.getItemStack();
+        applyHandlers(player, null, -1, stack);
+        entity.setItemStack(stack);
+    }
+
+    private void scanPlayerView(Player player) {
+        InventoryView view = player.getOpenInventory();
+        InventoryType topType = view.getTopInventory().getType();
+        if (topType == InventoryType.CRAFTING || topType == InventoryType.CREATIVE) {
+            scanInventory(player, player.getInventory());
+            return;
+        }
+        scanOpenView(player, view);
+    }
+
+    private void scanOpenView(Player player, InventoryView view) {
+        scanInventory(player, view.getTopInventory());
+        scanInventory(player, player.getInventory());
+    }
+
+    private void scanInventory(Player player, Inventory inventory) {
+        if (inventory == null || !isScannable(inventory)) {
+            return;
+        }
+        int size = inventory.getSize();
+        for (int slot = 0; slot < size; slot++) {
+            scanSlot(player, inventory, slot);
+        }
+    }
+
+    private static boolean isScannable(Inventory inventory) {
+        InventoryHolder holder = inventory.getHolder();
+        if (holder == null) {
+            return true;
+        }
+        return holder instanceof Player || holder instanceof BlockState
+                || holder instanceof Entity || holder instanceof DoubleChest;
+    }
+
+    private void scanSlot(Player player, Inventory inventory, int slot) {
+        ItemStack stack = inventory.getItem(slot);
+        applyHandlers(player, inventory, slot, stack);
+    }
+
+    private void applyHandlers(Player player, Inventory inventory, int slot, ItemStack stack) {
+        if (stack == null || stack.getType() == Material.AIR || handlers.isEmpty()) {
+            return;
+        }
+        for (ItemScanHandler handler : handlers) {
+            if (handler.matches(stack)) {
+                handler.update(player, inventory, slot, stack);
+            }
+        }
+    }
+}

src/main/resources/plugin.yml

@@ -3,7 +3,6 @@ api-version: '1.21.10'
 name: TLibs
 version: ${project.version}
 author: Drefvelin
-folia-supported: true
 
 loadbefore: [RPCharacters]
 softdepend: [MMOItems, MythicLib, ItemsAdder, MythicMobs, MMOCore]