chore: Port to 1.20.5/1.20.6

build.gradle

@@ -1,10 +1,10 @@
 plugins {
-  id 'fabric-loom' version '1.4.+'
+  id 'fabric-loom' version '1.6.+'
   id 'maven-publish'
 }
 
-sourceCompatibility = JavaVersion.VERSION_17
-targetCompatibility = JavaVersion.VERSION_17
+sourceCompatibility = JavaVersion.VERSION_21
+targetCompatibility = JavaVersion.VERSION_21
 
 archivesBaseName = project.archives_base_name
 version = project.mod_version + "+" + project.minecraft_version
@@ -47,8 +47,6 @@ dependencies {
 
   modImplementation "net.fabricmc.fabric-api:fabric-api:${project.fabric_version}"
 
-  modLocalRuntime "eu.pb4:polymer-reg-sync-manipulator:0.7.0+1.20.3-rc1"
-
   testmodImplementation sourceSets.main.output
 }
 
@@ -62,7 +60,7 @@ processResources {
 
 tasks.withType(JavaCompile).configureEach {
   it.options.encoding = "UTF-8"
-  it.options.release = 17
+  it.options.release = 21
 }
 
 java {

gradle.properties

@@ -3,14 +3,14 @@ org.gradle.jvmargs=-Xmx2G
 
 # Fabric Properties
 
-minecraft_version=1.20.4
-yarn_mappings=1.20.4+build.1
-loader_version=0.15.0
+minecraft_version=1.20.5
+yarn_mappings=1.20.5+build.1
+loader_version=0.15.6
 
 # Mod Properties
 mod_version=0.5.0
 maven_group=xyz.nucleoid
 archives_base_name=fantasy
 
 # Dependencies
-fabric_version=0.91.1+1.20.4
+fabric_version=0.97.5+1.20.5

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.7-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME

src/main/java/xyz/nucleoid/fantasy/RuntimeWorldConfig.java

@@ -72,9 +72,10 @@ public RuntimeWorldConfig setGenerator(ChunkGenerator generator) {
         this.generator = generator;
         return this;
     }
-	
+
     public RuntimeWorldConfig setShouldTickTime(boolean shouldTickTime) {
         this.shouldTickTime = shouldTickTime;
+        this.gameRules.set(GameRules.DO_DAYLIGHT_CYCLE, shouldTickTime);
         return this;
     }
 
@@ -167,7 +168,7 @@ public ChunkGenerator getGenerator() {
     public RuntimeWorld.Constructor getWorldConstructor() {
         return this.worldConstructor;
     }
-	
+
     public boolean shouldTickTime() {
         return this.shouldTickTime;
     }

src/main/java/xyz/nucleoid/fantasy/RuntimeWorldManager.java

@@ -1,16 +1,15 @@
 package xyz.nucleoid.fantasy;
 
-import com.mojang.serialization.Lifecycle;
 import net.fabricmc.fabric.api.event.lifecycle.v1.ServerWorldEvents;
 import net.minecraft.registry.DynamicRegistryManager;
 import net.minecraft.registry.RegistryKey;
 import net.minecraft.registry.RegistryKeys;
 import net.minecraft.registry.SimpleRegistry;
+import net.minecraft.registry.entry.RegistryEntryInfo;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.world.ServerWorld;
 import net.minecraft.text.Text;
 import net.minecraft.util.ProgressListener;
-import net.minecraft.util.math.BlockPos;
 import net.minecraft.world.World;
 import net.minecraft.world.dimension.DimensionOptions;
 import net.minecraft.world.level.storage.LevelStorage;
@@ -43,7 +42,7 @@ RuntimeWorld add(RegistryKey<World> worldKey, RuntimeWorldConfig config, Runtime
 
         var key = RegistryKey.of(RegistryKeys.DIMENSION, worldKey.getValue());
         if(!dimensionsRegistry.contains(key)) {
-            dimensionsRegistry.add(key, options, Lifecycle.stable());
+            dimensionsRegistry.add(key, options, RegistryEntryInfo.DEFAULT);
         }
         ((RemoveFromRegistry<?>) dimensionsRegistry).fantasy$setFrozen(isFrozen);
 

src/main/java/xyz/nucleoid/fantasy/mixin/ServerChunkManagerMixin.java

@@ -14,7 +14,7 @@
 public class ServerChunkManagerMixin {
     @Shadow
     @Final
-    private ServerWorld world;
+    ServerWorld world;
 
     @Inject(method = "executeQueuedTasks", at = @At("HEAD"), cancellable = true)
     private void executeQueuedTasks(CallbackInfoReturnable<Boolean> ci) {

src/main/java/xyz/nucleoid/fantasy/mixin/ServerWorldMixin.java

@@ -19,6 +19,7 @@
 
 @Mixin(ServerWorld.class)
 public abstract class ServerWorldMixin implements FantasyWorldAccess {
+    @Unique
     private static final int TICK_TIMEOUT = 20 * 15;
 
     @Unique
@@ -53,6 +54,7 @@ private void tick(BooleanSupplier shouldKeepTicking, CallbackInfo ci) {
         return shouldTick || this.fantasy$tickTimeout > 0;
     }
 
+    @Unique
     private boolean isWorldEmpty() {
         return this.getPlayers().isEmpty() && this.getChunkManager().getLoadedChunkCount() <= 0;
     }

src/main/java/xyz/nucleoid/fantasy/mixin/ThreadedAnvilChunkStorageMixin.java

@@ -5,7 +5,6 @@
 import net.minecraft.server.world.ThreadedAnvilChunkStorage;
 import net.minecraft.world.gen.chunk.ChunkGenerator;
 import net.minecraft.world.gen.chunk.ChunkGeneratorSettings;
-import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.asm.mixin.injection.At;
@@ -14,7 +13,6 @@
 @Mixin(ThreadedAnvilChunkStorage.class)
 public class ThreadedAnvilChunkStorageMixin {
     @Shadow
-    @Final
     private ChunkGenerator chunkGenerator;
 
     @WrapOperation(method = "<init>", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/gen/chunk/ChunkGeneratorSettings;createMissingSettings()Lnet/minecraft/world/gen/chunk/ChunkGeneratorSettings;"))

src/main/java/xyz/nucleoid/fantasy/mixin/registry/DimensionOptionsMixin.java

@@ -7,10 +7,8 @@
 
 @Mixin(DimensionOptions.class)
 public class DimensionOptionsMixin implements FantasyDimensionOptions {
-    @Unique
-    private boolean fantasy$save = true;
-    @Unique
-    private boolean fantasy$saveProperties = true;
+    @Unique private boolean fantasy$save = true;
+    @Unique private boolean fantasy$saveProperties = true;
 
     @Override
     public void fantasy$setSave(boolean value) {

src/main/java/xyz/nucleoid/fantasy/mixin/registry/DimensionOptionsRegistryHolderMixin.java

@@ -1,9 +1,12 @@
 package xyz.nucleoid.fantasy.mixin.registry;
 
+import com.mojang.logging.LogUtils;
 import net.minecraft.registry.Registry;
 import net.minecraft.world.dimension.DimensionOptions;
 import net.minecraft.world.dimension.DimensionOptionsRegistryHolder;
+import org.slf4j.Logger;
 import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Unique;
 import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.ModifyArg;
 import xyz.nucleoid.fantasy.FantasyDimensionOptions;
@@ -13,6 +16,8 @@
 
 @Mixin(DimensionOptionsRegistryHolder.class)
 public class DimensionOptionsRegistryHolderMixin {
+    @Unique private static final Logger fantasy$LOGGER = LogUtils.getLogger();
+
     @ModifyArg(method = "method_45516", at = @At(value = "INVOKE", target = "Lcom/mojang/serialization/MapCodec;forGetter(Ljava/util/function/Function;)Lcom/mojang/serialization/codecs/RecordCodecBuilder;"))
     private static Function<Object, Registry<DimensionOptions>> fantasy$swapRegistryGetter(Function<Object, Registry<DimensionOptions>> getter) {
         return (x) -> new FilteredRegistry<>(getter.apply(x), FantasyDimensionOptions.SAVE_PROPERTIES_PREDICATE);

src/main/java/xyz/nucleoid/fantasy/mixin/registry/SimpleRegistryMixin.java

@@ -1,43 +1,44 @@
 package xyz.nucleoid.fantasy.mixin.registry;
 
-import com.mojang.serialization.Lifecycle;
+import com.llamalad7.mixinextras.injector.ModifyReturnValue;
+import com.mojang.logging.LogUtils;
 import it.unimi.dsi.fastutil.objects.ObjectList;
 import it.unimi.dsi.fastutil.objects.Reference2IntMap;
+import net.minecraft.registry.MutableRegistry;
+import net.minecraft.registry.Registry;
 import net.minecraft.registry.RegistryKey;
 import net.minecraft.registry.SimpleRegistry;
 import net.minecraft.registry.entry.RegistryEntry;
+import net.minecraft.registry.entry.RegistryEntryInfo;
+import net.minecraft.registry.entry.RegistryEntryOwner;
 import net.minecraft.util.Identifier;
-import org.jetbrains.annotations.Nullable;
+import org.slf4j.Logger;
 import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.Unique;
+import org.spongepowered.asm.mixin.injection.At;
 import xyz.nucleoid.fantasy.RemoveFromRegistry;
 
-import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import java.util.Optional;
+import java.util.UUID;
+import java.util.stream.Stream;
 
 @Mixin(SimpleRegistry.class)
-public abstract class SimpleRegistryMixin<T> implements RemoveFromRegistry<T> {
+public abstract class SimpleRegistryMixin<T> implements RemoveFromRegistry<T>, MutableRegistry<T> {
+    @Unique private static final Logger fantasy$LOGGER = LogUtils.getLogger();
 
     @Shadow @Final private Map<T, RegistryEntry.Reference<T>> valueToEntry;
-
     @Shadow @Final private Map<Identifier, RegistryEntry.Reference<T>> idToEntry;
-
     @Shadow @Final private Map<RegistryKey<T>, RegistryEntry.Reference<T>> keyToEntry;
-
-    @Shadow @Final private Map<T, Lifecycle> entryToLifecycle;
-
+    @Shadow @Final private Map<RegistryKey<T>, RegistryEntryInfo> keyToEntryInfo;
     @Shadow @Final private ObjectList<RegistryEntry.Reference<T>> rawIdToEntry;
-
     @Shadow @Final private Reference2IntMap<T> entryToRawId;
-
-    @Shadow public abstract Optional<RegistryEntry<T>> getEntry(int rawId);
-
+    @Shadow @Final RegistryKey<? extends Registry<T>> key;
     @Shadow private boolean frozen;
 
-    @Shadow @Nullable private List<RegistryEntry.Reference<T>> cachedEntries;
-
     @Override
     public boolean fantasy$remove(T entry) {
         var registryEntry = this.valueToEntry.get(entry);
@@ -47,18 +48,15 @@ public abstract class SimpleRegistryMixin<T> implements RemoveFromRegistry<T> {
         }
 
         try {
-            this.rawIdToEntry.set(rawId, null);
-            this.idToEntry.remove(registryEntry.registryKey().getValue());
             this.keyToEntry.remove(registryEntry.registryKey());
-            this.entryToLifecycle.remove(entry);
+            this.idToEntry.remove(registryEntry.registryKey().getValue());
             this.valueToEntry.remove(entry);
-            if (this.cachedEntries != null) {
-                this.cachedEntries.remove(registryEntry);
-            }
+            this.rawIdToEntry.set(rawId, null);
+            this.keyToEntryInfo.remove(this.key);
 
             return true;
         } catch (Throwable e) {
-            e.printStackTrace();
+            fantasy$LOGGER.error("Could not remove entry", e);
             return false;
         }
     }
@@ -78,4 +76,9 @@ public abstract class SimpleRegistryMixin<T> implements RemoveFromRegistry<T> {
     public boolean fantasy$isFrozen() {
         return this.frozen;
     }
+
+    @ModifyReturnValue(method = "streamEntries", at = @At("RETURN"))
+    public Stream<RegistryEntry.Reference<T>> fixEntryStream(Stream<RegistryEntry.Reference<T>> original) {
+        return original.filter(Objects::nonNull);
+    }
 }

src/main/java/xyz/nucleoid/fantasy/util/FilteredRegistry.java

@@ -34,17 +34,17 @@ public Registry<T> getSource() {
     @Nullable
     @Override
     public Identifier getId(T value) {
-        return check.test(value) ? this.source.getId(value) : null;
+        return this.check.test(value) ? this.source.getId(value) : null;
     }
 
     @Override
     public Optional<RegistryKey<T>> getKey(T entry) {
-        return check.test(entry) ? this.source.getKey(entry) : Optional.empty();
+        return this.check.test(entry) ? this.source.getKey(entry) : Optional.empty();
     }
 
     @Override
     public int getRawId(@Nullable T value) {
-        return check.test(value) ? this.source.getRawId(value) : -1;
+        return this.check.test(value) ? this.source.getRawId(value) : -1;
     }
 
     @Nullable
@@ -67,12 +67,7 @@ public T get(@Nullable RegistryKey<T> key) {
     @Nullable
     @Override
     public T get(@Nullable Identifier id) {
-        return this.get(id);
-    }
-
-    @Override
-    public Lifecycle getEntryLifecycle(T entry) {
-        return this.source.getEntryLifecycle(entry);
+        return this.source.get(id);
     }
 
     @Override
@@ -82,7 +77,7 @@ public Lifecycle getLifecycle() {
 
     @Override
     public Set<Identifier> getIds() {
-        return this.getIds();
+        return this.source.getIds();
     }
 
     @Override
@@ -174,6 +169,6 @@ public void populateTags(Map<TagKey<T>, List<RegistryEntry<T>>> tagEntries) {
     @NotNull
     @Override
     public Iterator<T> iterator() {
-        return Iterators.filter(this.source.iterator(), e -> this.check.test(e));
+        return Iterators.filter(this.source.iterator(), this.check::test);
     }
 }

src/main/java/xyz/nucleoid/fantasy/util/VoidChunkGenerator.java

@@ -1,7 +1,9 @@
 package xyz.nucleoid.fantasy.util;
 
+import com.mojang.datafixers.kinds.Applicative;
 import com.mojang.datafixers.util.Pair;
 import com.mojang.serialization.Codec;
+import com.mojang.serialization.MapCodec;
 import com.mojang.serialization.codecs.RecordCodecBuilder;
 import net.minecraft.block.BlockState;
 import net.minecraft.entity.SpawnGroup;
@@ -46,11 +48,9 @@
 import java.util.stream.Stream;
 
 public class VoidChunkGenerator extends ChunkGenerator {
-    public static final Codec<VoidChunkGenerator> CODEC = RecordCodecBuilder.create(instance -> {
-        return instance.group(
-                Biome.REGISTRY_CODEC.stable().fieldOf("biome").forGetter(g -> g.biome)
-        ).apply(instance, instance.stable(VoidChunkGenerator::new));
-    });
+    public static final MapCodec<VoidChunkGenerator> CODEC = RecordCodecBuilder.mapCodec(instance -> instance.group(
+            Biome.REGISTRY_CODEC.stable().fieldOf("biome").forGetter(VoidChunkGenerator::getBiome)
+    ).apply(instance, instance.stable(VoidChunkGenerator::new)));
 
     private static final VerticalBlockSample EMPTY_SAMPLE = new VerticalBlockSample(0, new BlockState[0]);
 
@@ -82,12 +82,12 @@ public double maxValue() {
 
         @Override
         public CodecHolder<? extends DensityFunction> getCodecHolder() {
-            return CodecHolder.of(Codec.unit(this));
+            return CodecHolder.of(MapCodec.unit(this));
         }
     };
 
     public static final MultiNoiseUtil.MultiNoiseSampler EMPTY_SAMPLER = new MultiNoiseUtil.MultiNoiseSampler(ZERO_DENSITY_FUNCTION, ZERO_DENSITY_FUNCTION, ZERO_DENSITY_FUNCTION, ZERO_DENSITY_FUNCTION, ZERO_DENSITY_FUNCTION, ZERO_DENSITY_FUNCTION, Collections.emptyList());
-    
+
     public VoidChunkGenerator(RegistryEntry<Biome> biome) {
         super(new FixedBiomeSource(biome));
         this.biome = biome;
@@ -115,10 +115,13 @@ public VoidChunkGenerator(MinecraftServer server, Identifier biome) {
         this(server.getRegistryManager().get(RegistryKeys.BIOME), RegistryKey.of(RegistryKeys.BIOME, biome));
     }
     @Override
-    protected Codec<? extends ChunkGenerator> getCodec() {
+    protected MapCodec<? extends ChunkGenerator> getCodec() {
         return CODEC;
     }
-
+
+    protected RegistryEntry<Biome> getBiome() {
+        return this.biome;
+    }
 
     @Override
     public void carve(ChunkRegion chunkRegion, long seed, NoiseConfig noiseConfig, BiomeAccess world, StructureAccessor structureAccessor, Chunk chunk, GenerationStep.Carver carverStep) {

src/main/resources/fabric.mod.json

@@ -21,7 +21,7 @@
   "depends": {
     "fabricloader": ">=0.14.23",
     "minecraft": ">=1.20.3-",
-    "fabric-lifecycle-events-v1": "*",
+    "fabric-api": "*",
     "java": ">=17"
   }
 }