Exclude CODEOWNERS from language statistics

Handle Bukkit update events that return BlockState values

.gitattributes

@@ -3,3 +3,4 @@
 # are changed when only line endings change.
 src/generated/**/.cache/cache text eol=lf
 src/generated/**/*.json text eol=lf
+.gitea/CODEOWNERS linguist-detectable=false

.gitea/CODEOWNERS

@@ -0,0 +1,3 @@
+# Pull requests request review from the eligible staff team(s).
+# One approval from either an Admin or Developer satisfies branch protection.
+*   @trueog/Admins @trueog/Developers

src/main/java/no/jckf/dhsupport/core/DhSupport.java

@@ -131,7 +131,7 @@ public class DhSupport implements Configurable
             this.warning("If you understand what this means and would like to disable this warning, set " + DhsConfig.GENERATE_NEW_CHUNKS_WARNING + " to false in your config.");
         }
 
-        if (this.getConfig().getBool(DhsConfig.CHECK_FOR_UPDATES, true)) {
+        if (this.getConfig().getBool(DhsConfig.CHECK_FOR_UPDATES, false)) {
             this.getScheduler().runOnSeparateThread(() -> {
                 this.checkUpdates();
                 return null;
@@ -507,7 +507,16 @@ public class DhSupport implements Configurable
             return;
         }
 
-        this.debug("Touched LOD at " + this.getWorldInterface(worldId).getName() + " " + x + " " + z + (reason == null ? "." : ": " + reason));
+        WorldInterface world = this.getWorldInterface(worldId);
+
+        if (world == null) {
+            this.debug("Ignoring touched LOD for unloaded world " + worldId + " " + x + " " + z
+                + (reason == null ? "." : ": " + reason));
+            return;
+        }
+
+        this.debug("Touched LOD at " + world.getName() + " " + x + " " + z
+            + (reason == null ? "." : ": " + reason));
 
         this.touchedLods.put(key, lodModel);
     }
@@ -527,8 +536,18 @@ public class DhSupport implements Configurable
             LodModel lodModelToDelete = this.touchedLods.get(key);
             this.touchedLods.remove(key);
 
+            if (lodModelToDelete == null) {
+                continue;
+            }
+
             WorldInterface world = this.getWorldInterface(lodModelToDelete.getWorldId());
 
+            if (world == null) {
+                this.debug("Dropping touched LOD for unloaded world " + lodModelToDelete.getWorldId() + " "
+                    + lodModelToDelete.getX() + " " + lodModelToDelete.getZ() + ".");
+                continue;
+            }
+
             this.debug("Changes detected in " + world.getName() + " " + lodModelToDelete.getX() + " " + lodModelToDelete.getZ() + ".");
 
             this.getLodRepository().lodExistsAsync(lodModelToDelete.getWorldId(), lodModelToDelete.getX(), lodModelToDelete.getZ()).thenAccept((exists) -> {
@@ -574,8 +593,15 @@ public class DhSupport implements Configurable
                                 int playersOutOfRangeCount = 0;
                                 int playersWithoutDhCount = 0;
 
+                                org.bukkit.World bukkitWorld = Bukkit.getWorld(newLodModel.getWorldId());
+
+                                if (bukkitWorld == null) {
+                                    this.debug("Skipping real-time LOD update for unloaded world " + newLodModel.getWorldId() + ".");
+                                    return;
+                                }
+
                                 // TODO: Don't use Bukkit classes.
-                                for (Player player : Bukkit.getWorld(newLodModel.getWorldId()).getPlayers()) {
+                                for (Player player : bukkitWorld.getPlayers()) {
                                     Configuration playerConfig = this.getPlayerConfiguration(player.getUniqueId());
 
                                     // No config for this player? Probably not using DH.

src/main/resources/config.yml

@@ -27,7 +27,7 @@ real_time_updates_enabled: true
 # How close to an update players have to be in order to receive it.
 # Required.
 # Default: 128
-real_time_update_radius: 128
+real_time_update_radius: 256
 
 # Let clients ask for updates to LODs they've previously downloaded.
 # Required.
@@ -37,7 +37,7 @@ login_data_sync_enabled: true
 # How far away can a player request updates for during login sync?
 # Required.
 # Default: 128
-login_data_sync_radius: 128
+login_data_sync_radius: 64
 
 # How many login sync requests can a client send at once?
 # Required.
@@ -53,12 +53,12 @@ max_data_transfer_speed: 0
 # If you disable this, the plugin will not be able to create LODs in areas where chunks don't already exist.
 # Optional.
 # Default: true
-#generate_new_chunks: false
+generate_new_chunks: false
 
 # Show warning during startup if chunk generation is enabled?
 # Optional.
 # Default: true
-#generate_new_chunks_warning: false
+generate_new_chunks_warning: true
 
 # How frequently (in seconds) should LODs be updated if world changes are detected?
 # Required.
@@ -117,7 +117,7 @@ worlds:
     # Enable if your world has floating islands or mountains/structures with large overhangs above sea level.
     # Optional.
     # Default: false
-    #scan_to_sea_level: true
+    scan_to_sea_level: true
 
     # When using the FastOverworldBuilder, should the bottom data point be stretched to the lowest permitted build
     # height? This will make the LODs look more complete, and gives a theoretical boost to render performance.
@@ -134,12 +134,12 @@ worlds:
   world_nether:
     # It might make sense to have a lower view distance in the nether, since visibility will be low simply because of
     # the level design anyway.
-    render_distance: 64
+    render_distance: 128
 
     # This is a hack to prevent light from shining through ceilings, mostly noticeable on the nether ceiling.
     # Only works with the FullBuilder, should only be used with builder_resolution=1, and does have some side effects.
-    #perform_underglow_hack: true
-    #builder_resolution: 1
+    perform_underglow_hack: true
+    builder_resolution: 1
 
 # List of events that should trigger LOD updates.
 # Required.
@@ -166,12 +166,12 @@ update_events:
 # Does not affect Folia servers.
 # Required.
 # Default: 4
-scheduler_threads: 4
+scheduler_threads: 8
 
 # Should we notify server admins when a new version of the plugin is available?
 # Optional.
-# Default: true
-#check_for_updates: false
+# Default: false
+check_for_updates: false
 
 # Enable debug logging?
 # Optional.