Merge pull request 'chore(deps): update gradle docker tag to v9' (#6) from renovate/major-docker-images into main

Reviewed-on: #6

Dockerfile

@@ -1,4 +1,4 @@
-FROM gradle:8.6-jdk21
+FROM gradle:9.3-jdk21
 
 WORKDIR /workspace
 

README.md

@@ -51,6 +51,7 @@ http:
 metrics:
   enable_jvm: true
   enable_process: true
+  entity_snapshot_interval_seconds: 15
 movement:
   count_block_changes_only: true
 ```
@@ -77,7 +78,7 @@ The Gitea workflow expects these secrets/variables:
 
 ## Metrics
 
-Some metrics depend on server implementations (TPS/MSPT, player ping, tile entities). If a method is unavailable on your server build, those metrics are omitted.
+Some metrics depend on server implementations (TPS/MSPT, player ping, tile entities). If a method is unavailable on your server build, those metrics are omitted. Entity and tile counts are updated on a periodic main-thread snapshot.
 
 | Metric | Type | Labels | Description |
 | --- | --- | --- | --- |

src/main/java/com/prometheus/spigot/PrometheusSpigotPlugin.java

@@ -2,11 +2,14 @@ package com.prometheus.spigot;
 
 import com.prometheus.spigot.http.MetricsHttpHandler;
 import com.prometheus.spigot.listeners.BlockBreakListener;
+import com.prometheus.spigot.listeners.BlockPlaceListener;
 import com.prometheus.spigot.listeners.ChatListener;
 import com.prometheus.spigot.listeners.ChunkListener;
 import com.prometheus.spigot.listeners.CommandListener;
 import com.prometheus.spigot.listeners.ConnectionListener;
+import com.prometheus.spigot.listeners.CraftListener;
 import com.prometheus.spigot.listeners.DeathListener;
+import com.prometheus.spigot.listeners.EntityKillListener;
 import com.prometheus.spigot.listeners.MoveListener;
 import com.prometheus.spigot.listeners.PluginListener;
 import com.prometheus.spigot.metrics.MetricsRegistry;
@@ -17,7 +20,9 @@ import java.net.InetSocketAddress;
 import java.util.Locale;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
+import org.bukkit.Bukkit;
 import org.bukkit.plugin.java.JavaPlugin;
+import org.bukkit.scheduler.BukkitTask;
 
 public final class PrometheusSpigotPlugin extends JavaPlugin {
     private HttpServer httpServer;
@@ -25,6 +30,7 @@ public final class PrometheusSpigotPlugin extends JavaPlugin {
     private MetricsRegistry metricsRegistry;
     private String bearerToken;
     private boolean countBlockMovementOnly;
+    private BukkitTask snapshotTask;
 
     @Override
     public void onEnable() {
@@ -43,6 +49,8 @@ public final class PrometheusSpigotPlugin extends JavaPlugin {
 
         metricsRegistry = new MetricsRegistry(getServer(), enableJvm, enableProcess);
 
+        scheduleSnapshotTask();
+
         registerListeners();
         startHttpServer();
     }
@@ -57,11 +65,16 @@ public final class PrometheusSpigotPlugin extends JavaPlugin {
             httpExecutor.shutdownNow();
             httpExecutor = null;
         }
+        if (snapshotTask != null) {
+            snapshotTask.cancel();
+            snapshotTask = null;
+        }
     }
 
     private void registerListeners() {
         var pluginManager = getServer().getPluginManager();
         pluginManager.registerEvents(new BlockBreakListener(metricsRegistry), this);
+        pluginManager.registerEvents(new BlockPlaceListener(metricsRegistry), this);
         pluginManager.registerEvents(new DeathListener(metricsRegistry), this);
         pluginManager.registerEvents(new ChatListener(metricsRegistry), this);
         pluginManager.registerEvents(new MoveListener(metricsRegistry, countBlockMovementOnly), this);
@@ -69,6 +82,8 @@ public final class PrometheusSpigotPlugin extends JavaPlugin {
         pluginManager.registerEvents(new CommandListener(metricsRegistry), this);
         pluginManager.registerEvents(new ChunkListener(metricsRegistry), this);
         pluginManager.registerEvents(new PluginListener(metricsRegistry), this);
+        pluginManager.registerEvents(new CraftListener(metricsRegistry), this);
+        pluginManager.registerEvents(new EntityKillListener(metricsRegistry), this);
     }
 
     private void startHttpServer() {
@@ -95,4 +110,12 @@ public final class PrometheusSpigotPlugin extends JavaPlugin {
         httpServer.start();
         getLogger().info(String.format(Locale.ROOT, "Prometheus endpoint listening on http://%s:%d%s", bindAddress, port, path));
     }
+
+    private void scheduleSnapshotTask() {
+        int intervalSeconds = getConfig().getInt("metrics.entity_snapshot_interval_seconds", 15);
+        long intervalTicks = Math.max(20L, intervalSeconds * 20L);
+
+        metricsRegistry.refreshEntitySnapshots();
+        snapshotTask = Bukkit.getScheduler().runTaskTimer(this, metricsRegistry::refreshEntitySnapshots, intervalTicks, intervalTicks);
+    }
 }

src/main/java/com/prometheus/spigot/listeners/BlockPlaceListener.java

@@ -0,0 +1,19 @@
+package com.prometheus.spigot.listeners;
+
+import com.prometheus.spigot.metrics.MetricsRegistry;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.Listener;
+import org.bukkit.event.block.BlockPlaceEvent;
+
+public final class BlockPlaceListener implements Listener {
+    private final MetricsRegistry registry;
+
+    public BlockPlaceListener(MetricsRegistry registry) {
+        this.registry = registry;
+    }
+
+    @EventHandler
+    public void onBlockPlace(BlockPlaceEvent event) {
+        registry.incrementBlocksPlaced(event.getBlockPlaced().getType());
+    }
+}

src/main/java/com/prometheus/spigot/listeners/CraftListener.java

@@ -0,0 +1,25 @@
+package com.prometheus.spigot.listeners;
+
+import com.prometheus.spigot.metrics.MetricsRegistry;
+import org.bukkit.Material;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.Listener;
+import org.bukkit.event.inventory.CraftItemEvent;
+import org.bukkit.inventory.ItemStack;
+
+public final class CraftListener implements Listener {
+    private final MetricsRegistry registry;
+
+    public CraftListener(MetricsRegistry registry) {
+        this.registry = registry;
+    }
+
+    @EventHandler
+    public void onCraft(CraftItemEvent event) {
+        ItemStack result = event.getRecipe() != null ? event.getRecipe().getResult() : null;
+        if (result == null || result.getType() == Material.AIR) {
+            return;
+        }
+        registry.incrementItemsCrafted(result.getType());
+    }
+}

src/main/java/com/prometheus/spigot/listeners/EntityKillListener.java

@@ -0,0 +1,22 @@
+package com.prometheus.spigot.listeners;
+
+import com.prometheus.spigot.metrics.MetricsRegistry;
+import org.bukkit.entity.Player;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.Listener;
+import org.bukkit.event.entity.EntityDeathEvent;
+
+public final class EntityKillListener implements Listener {
+    private final MetricsRegistry registry;
+
+    public EntityKillListener(MetricsRegistry registry) {
+        this.registry = registry;
+    }
+
+    @EventHandler
+    public void onEntityDeath(EntityDeathEvent event) {
+        Player killer = event.getEntity().getKiller();
+        String killerLabel = killer != null ? killer.getName() : "null";
+        registry.incrementEntityKill(event.getEntity().getType(), killerLabel);
+    }
+}

src/main/java/com/prometheus/spigot/metrics/MetricsRegistry.java

@@ -6,6 +6,7 @@ import java.lang.management.MemoryUsage;
 import java.lang.management.ThreadMXBean;
 import java.lang.reflect.Method;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.Comparator;
 import java.util.HashMap;
 import java.util.List;
@@ -31,6 +32,8 @@ public final class MetricsRegistry {
     private final Method serverMsptMethod;
     private final Method playerPingMethod;
     private final Method chunkTileEntitiesMethod;
+    private volatile Map<String, Long> entityCountsSnapshot = Collections.emptyMap();
+    private volatile Map<String, Long> tileEntityCountsSnapshot = Collections.emptyMap();
 
     private final LongAdder chatMessages = new LongAdder();
     private final LongAdder playerMovements = new LongAdder();
@@ -41,11 +44,14 @@ public final class MetricsRegistry {
     private final LongAdder chunkGenerations = new LongAdder();
     private final LongAdder pluginEnables = new LongAdder();
     private final LongAdder pluginDisables = new LongAdder();
+    private final ConcurrentHashMap<String, LongAdder> blocksPlacedByMaterial = new ConcurrentHashMap<>();
     private final ConcurrentHashMap<String, LongAdder> blocksBrokenByMaterial = new ConcurrentHashMap<>();
     private final ConcurrentHashMap<String, LongAdder> deathsByCause = new ConcurrentHashMap<>();
     private final ConcurrentHashMap<String, LongAdder> loginFailuresByReason = new ConcurrentHashMap<>();
     private final ConcurrentHashMap<String, LongAdder> commandsBlocked = new ConcurrentHashMap<>();
     private final ConcurrentHashMap<LabelPair, LongAdder> commandsExecuted = new ConcurrentHashMap<>();
+    private final ConcurrentHashMap<String, LongAdder> itemsCraftedByMaterial = new ConcurrentHashMap<>();
+    private final ConcurrentHashMap<LabelPair, LongAdder> entityKills = new ConcurrentHashMap<>();
 
     public MetricsRegistry(Server server, boolean enableJvm, boolean enableProcess) {
         this.server = server;
@@ -82,6 +88,10 @@ public final class MetricsRegistry {
         incrementLabeledCounter(blocksBrokenByMaterial, material.name());
     }
 
+    public void incrementBlocksPlaced(Material material) {
+        incrementLabeledCounter(blocksPlacedByMaterial, material.name());
+    }
+
     public void incrementPlayerDeath(EntityDamageEvent.DamageCause cause) {
         incrementLabeledCounter(deathsByCause, cause.name());
     }
@@ -120,6 +130,47 @@ public final class MetricsRegistry {
         pluginDisables.increment();
     }
 
+    public void incrementItemsCrafted(Material material) {
+        incrementLabeledCounter(itemsCraftedByMaterial, material.name());
+    }
+
+    public void incrementEntityKill(EntityType entityType, String killer) {
+        String normalizedEntity = normalizeLabel(entityType.name());
+        String normalizedKiller = normalizeLabel(killer);
+        entityKills.computeIfAbsent(new LabelPair(normalizedEntity, normalizedKiller), key -> new LongAdder()).increment();
+    }
+
+    public void refreshEntitySnapshots() {
+        Map<String, Long> entityCounts = new HashMap<>();
+        Map<String, Long> tileCounts = new HashMap<>();
+
+        for (var world : server.getWorlds()) {
+            for (Entity entity : world.getEntities()) {
+                EntityType type = entity.getType();
+                String label = normalizeLabel(type.name());
+                entityCounts.merge(label, 1L, Long::sum);
+            }
+
+            if (chunkTileEntitiesMethod != null) {
+                for (Chunk chunk : world.getLoadedChunks()) {
+                    Object result = invoke(chunkTileEntitiesMethod, chunk);
+                    if (!(result instanceof Object[] states)) {
+                        continue;
+                    }
+                    for (Object state : states) {
+                        if (state instanceof BlockState blockState) {
+                            String label = normalizeLabel(blockState.getType().name());
+                            tileCounts.merge(label, 1L, Long::sum);
+                        }
+                    }
+                }
+            }
+        }
+
+        entityCountsSnapshot = Collections.unmodifiableMap(entityCounts);
+        tileEntityCountsSnapshot = Collections.unmodifiableMap(tileCounts);
+    }
+
     public String render() {
         StringBuilder builder = new StringBuilder();
 
@@ -134,10 +185,13 @@ public final class MetricsRegistry {
         appendCounter(builder, "spigot_plugin_disable_total", "Total plugin disable events", pluginDisables.sum());
 
         appendLabeledCounterAdder(builder, "spigot_blocks_broken_total", "Blocks broken by material", "material", blocksBrokenByMaterial);
+        appendLabeledCounterAdder(builder, "spigot_blocks_placed_total", "Blocks placed by material", "material", blocksPlacedByMaterial);
         appendLabeledCounterAdder(builder, "spigot_player_deaths_total", "Player deaths by cause", "cause", deathsByCause);
         appendLabeledCounterAdder(builder, "spigot_player_logins_failed_total", "Failed player logins by reason", "reason", loginFailuresByReason);
         appendLabeledCounterAdder(builder, "spigot_commands_blocked_total", "Blocked commands by name", "command", commandsBlocked);
         appendLabeledCounterAdder(builder, "spigot_commands_executed_total", "Commands executed by source", "command", "source", commandsExecuted);
+        appendLabeledCounterAdder(builder, "spigot_items_crafted_total", "Items crafted by material", "material", itemsCraftedByMaterial);
+        appendLabeledCounterAdder(builder, "spigot_entities_killed_total", "Entities killed by player", "entity", "killer", entityKills);
 
         appendGauge(builder, "spigot_players_online", "Online player count", server.getOnlinePlayers().size());
         appendGauge(builder, "spigot_players_max", "Max player slots", server.getMaxPlayers());
@@ -226,37 +280,14 @@ public final class MetricsRegistry {
     }
 
     private void appendEntityMetrics(StringBuilder builder) {
-        Map<String, Long> entityCounts = new HashMap<>();
+        appendLabeledGauge(builder, "spigot_entities_total", "Entities by type", "type", entityCountsSnapshot);
-        for (var world : server.getWorlds()) {
-            for (Entity entity : world.getEntities()) {
-                EntityType type = entity.getType();
-                String label = normalizeLabel(type.name());
-                entityCounts.merge(label, 1L, Long::sum);
-            }
-        }
-        appendLabeledGauge(builder, "spigot_entities_total", "Entities by type", "type", entityCounts);
     }
 
     private void appendTileEntityMetrics(StringBuilder builder) {
         if (chunkTileEntitiesMethod == null) {
             return;
         }
-        Map<String, Long> tileCounts = new HashMap<>();
+        appendLabeledGauge(builder, "spigot_tile_entities_total", "Tile entities by type", "type", tileEntityCountsSnapshot);
-        for (var world : server.getWorlds()) {
-            for (Chunk chunk : world.getLoadedChunks()) {
-                Object result = invoke(chunkTileEntitiesMethod, chunk);
-                if (!(result instanceof Object[] states)) {
-                    continue;
-                }
-                for (Object state : states) {
-                    if (state instanceof BlockState blockState) {
-                        String label = normalizeLabel(blockState.getType().name());
-                        tileCounts.merge(label, 1L, Long::sum);
-                    }
-                }
-            }
-        }
-        appendLabeledGauge(builder, "spigot_tile_entities_total", "Tile entities by type", "type", tileCounts);
     }
 
     private void appendPlayerPingMetrics(StringBuilder builder) {

src/main/resources/config.yml

@@ -6,5 +6,6 @@ http:
 metrics:
   enable_jvm: true
   enable_process: true
+  entity_snapshot_interval_seconds: 15
 movement:
   count_block_changes_only: true