.forgejo/workflows/publish.yml

@@ -19,6 +19,6 @@ jobs:
     - name: Build
       run: |
         chmod +x ./gradlew
-        ./gradlew publishMavenJavaPublicationToFrogMCSnapshotsMavenRepository \
+        ./gradlew :publishMavenJavaPublicationToFrogMCSnapshotsMavenRepository \
         -PFrogMCSnapshotsMavenUsername=${{ secrets.MAVEN_PUSH_USER }} \
         -PFrogMCSnapshotsMavenPassword=${{ secrets.MAVEN_PUSH_TOKEN }} --stacktrace

build.gradle.kts

@@ -35,7 +35,7 @@ allprojects {
 
     phytotelma {
         loader(rootProject.libs.versions.frogloader)
-        minecraft(rootProject.libs.versions.minecraft, provider { "1.20.6" }) // since parchment isn't published for 1.21 yet
+        minecraft(rootProject.libs.versions.minecraft)
     }
 
     tasks.processResources {
@@ -94,14 +94,7 @@ allprojects {
 
 evaluationDependsOnChildren()
 
-tasks.runClient {
-    classpath(sourceSets.test.get().runtimeClasspath)
-    project.subprojects.filter {
-        it.projectDir.resolve("src/main/resources/frog.mod.toml").exists()
-    }.forEach {
-        classpath(it.sourceSets.test.get().output)
-    }
-}
+// TODO: add testmods of libraries to general client runtime classpath
 
 dependencies {
     project.subprojects.filter {

gradle/libs.versions.toml

@@ -1,7 +1,7 @@
 [versions]
 
-minecraft = "1.21"
+minecraft = "1.20.6"
 frogloader = "0.0.1-SNAPSHOT"
-phytotelma = "0.0.1-alpha.9"
+phytotelma = "0.0.1-alpha.4"
 lombok = "8.6"
 

library/core/src/main/java/dev/frogmc/froglib/events/api/Events.java

@@ -5,7 +5,7 @@ import dev.frogmc.froglib.events.api.types.ClientTickEvent;
 import dev.frogmc.froglib.events.api.types.ServerStartupEvent;
 import dev.frogmc.froglib.events.api.types.ServerTickEvent;
 
-public final class LifecycleEvents {
+public class Events {
 	public static final Event<ClientStartupEvent> CLIENT_STARTUP = Event.of(ClientStartupEvent.class);
 	public static final Event<ServerStartupEvent> SERVER_STARTUP = Event.of(ServerStartupEvent.class);
 

library/core/src/main/java/dev/frogmc/froglib/events/impl/mixin/MinecraftServerMixin.java

@@ -1,6 +1,6 @@
 package dev.frogmc.froglib.events.impl.mixin;
 
-import dev.frogmc.froglib.events.api.LifecycleEvents;
+import dev.frogmc.froglib.events.api.Events;
 import net.minecraft.server.MinecraftServer;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.injection.At;
@@ -12,11 +12,11 @@ public abstract class MinecraftServerMixin {
 
 	@Inject(method = "runServer", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/MinecraftServer;tickServer(Ljava/util/function/BooleanSupplier;)V"))
 	private void preServerTick(CallbackInfo ci) {
-		LifecycleEvents.PRE_SERVER_TICK.invoker().onTick((MinecraftServer) (Object) this);
+		Events.PRE_SERVER_TICK.invoker().onTick((MinecraftServer) (Object) this);
 	}
 
 	@Inject(method = "runServer", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/MinecraftServer;tickServer(Ljava/util/function/BooleanSupplier;)V", shift = At.Shift.AFTER))
 	private void postServerTick(CallbackInfo ci) {
-		LifecycleEvents.POST_SERVER_TICK.invoker().onTick((MinecraftServer) (Object) this);
+		Events.POST_SERVER_TICK.invoker().onTick((MinecraftServer) (Object) this);
 	}
 }

library/core/src/main/java/dev/frogmc/froglib/events/impl/mixin/client/MinecraftMixin.java

@@ -1,6 +1,6 @@
 package dev.frogmc.froglib.events.impl.mixin.client;
 
-import dev.frogmc.froglib.events.api.LifecycleEvents;
+import dev.frogmc.froglib.events.api.Events;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.main.GameConfig;
 import org.spongepowered.asm.mixin.Mixin;
@@ -13,16 +13,16 @@ public abstract class MinecraftMixin {
 
 	@Inject(method = "<init>", at = @At("TAIL"))
 	private void postClientInit(GameConfig gameConfig, CallbackInfo ci) {
-		LifecycleEvents.CLIENT_STARTUP.invoker().onClientInit((Minecraft) (Object) this, gameConfig);
+		Events.CLIENT_STARTUP.invoker().onClientInit((Minecraft) (Object) this, gameConfig);
 	}
 
 	@Inject(method = "tick", at = @At("HEAD"))
 	private void preClientTick(CallbackInfo ci) {
-		LifecycleEvents.PRE_CLIENT_TICK.invoker().onTick((Minecraft) (Object) this);
+		Events.PRE_CLIENT_TICK.invoker().onTick((Minecraft) (Object) this);
 	}
 
 	@Inject(method = "tick", at = @At("TAIL"))
 	private void postClientTick(CallbackInfo ci) {
-		LifecycleEvents.POST_CLIENT_TICK.invoker().onTick((Minecraft) (Object) this);
+		Events.POST_CLIENT_TICK.invoker().onTick((Minecraft) (Object) this);
 	}
 }

library/core/src/main/java/dev/frogmc/froglib/events/impl/mixin/server/DedicatedServerMixin.java

@@ -1,6 +1,6 @@
 package dev.frogmc.froglib.events.impl.mixin.server;
 
-import dev.frogmc.froglib.events.api.LifecycleEvents;
+import dev.frogmc.froglib.events.api.Events;
 import net.minecraft.server.dedicated.DedicatedServer;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.injection.At;
@@ -12,6 +12,6 @@ public abstract class DedicatedServerMixin {
 
 	@Inject(method = "initServer", at = @At(value = "INVOKE", target = "Lorg/slf4j/Logger;info(Ljava/lang/String;)V", ordinal = 0))
 	private void initializeServer(CallbackInfoReturnable<Boolean> cir) {
-		LifecycleEvents.SERVER_STARTUP.invoker().onServerInit((DedicatedServer) (Object) this);
+		Events.SERVER_STARTUP.invoker().onServerInit((DedicatedServer) (Object) this);
 	}
 }

library/keybinds/src/main/java/dev/frogmc/froglib/keybinds/impl/KeyMappingsImpl.java

@@ -1,6 +1,6 @@
 package dev.frogmc.froglib.keybinds.impl;
 
-import dev.frogmc.froglib.events.api.LifecycleEvents;
+import dev.frogmc.froglib.events.api.Events;
 import dev.frogmc.froglib.keybinds.api.KeyListener;
 import dev.frogmc.froglib.keybinds.api.KeyMappings;
 import net.minecraft.client.KeyMapping;
@@ -15,23 +15,14 @@ public class KeyMappingsImpl implements KeyMappings {
 	}
 
 	public void registerKey(KeyMapping mapping){
-		Runnable register = () -> {
-			synchronized (Minecraft.getInstance().options) {
-				Minecraft.getInstance().options.keyMappings = ArrayUtils.add(Minecraft.getInstance().options.keyMappings, mapping);
-			}
-		};
-		//noinspection ConstantValue (This inspection is just wrong)
-		if (Minecraft.getInstance() != null && Minecraft.getInstance().options != null) {
-			register.run();
-		} else {
-			LifecycleEvents.CLIENT_STARTUP.register((client, config) -> register.run());
-		}
+		// TODO maybe synchronize?
+		Minecraft.getInstance().options.keyMappings = ArrayUtils.add(Minecraft.getInstance().options.keyMappings, mapping);
 	}
 
 	@Override
 	public void registerKeyWithListener(KeyMapping mapping, KeyListener listener) {
 		registerKey(mapping);
-		LifecycleEvents.POST_CLIENT_TICK.register(c -> {
+		Events.POST_CLIENT_TICK.register(c -> {
 			while (mapping.consumeClick()){
 				listener.onPress(mapping);
 			}

library/player/src/main/java/dev/frogmc/froglib/player/api/PlayerEvents.java

@@ -1,17 +0,0 @@
-package dev.frogmc.froglib.player.api;
-
-import java.util.function.BiConsumer;
-
-import dev.frogmc.froglib.events.api.Event;
-import dev.frogmc.froglib.player.api.types.StopSleepEvent;
-import net.minecraft.core.BlockPos;
-import net.minecraft.world.damagesource.DamageSource;
-import net.minecraft.world.entity.Entity;
-import net.minecraft.world.entity.player.Player;
-
-public final class PlayerEvents {
-	public static final Event<BiConsumer<Player, DamageSource>> DEATH = Event.biConsumer();
-	public static final Event<BiConsumer<Player, Entity>> ATTACK = Event.biConsumer();
-	public static final Event<BiConsumer<Player, BlockPos>> START_SLEEP = Event.biConsumer();
-	public static final Event<StopSleepEvent> STOP_SLEEP = Event.of(StopSleepEvent.class);
-}

library/player/src/main/java/dev/frogmc/froglib/player/api/types/StopSleepEvent.java

@@ -1,7 +0,0 @@
-package dev.frogmc.froglib.player.api.types;
-
-import net.minecraft.world.entity.player.Player;
-
-public interface StopSleepEvent {
-	void onSleepStop(Player player, boolean wakeImmediately, boolean updateLevelForSleepingPlayers);
-}

library/player/src/main/java/dev/frogmc/froglib/player/impl/mixin/PlayerMixin.java

@@ -1,38 +0,0 @@
-package dev.frogmc.froglib.player.impl.mixin;
-
-import com.mojang.datafixers.util.Either;
-import dev.frogmc.froglib.player.api.PlayerEvents;
-import net.minecraft.core.BlockPos;
-import net.minecraft.util.Unit;
-import net.minecraft.world.damagesource.DamageSource;
-import net.minecraft.world.entity.Entity;
-import net.minecraft.world.entity.player.Player;
-import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.injection.At;
-import org.spongepowered.asm.mixin.injection.Inject;
-import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
-import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
-
-@Mixin(Player.class)
-public class PlayerMixin {
-
-	@Inject(method = "die", at = @At("TAIL"))
-	private void onDeath(DamageSource cause, CallbackInfo ci) {
-		PlayerEvents.DEATH.invoker().accept((Player) (Object) this, cause);
-	}
-
-	@Inject(method = "attack", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/entity/Entity;hurt(Lnet/minecraft/world/damagesource/DamageSource;F)Z"))
-	private void onAttack(Entity target, CallbackInfo ci){
-		PlayerEvents.ATTACK.invoker().accept((Player) (Object) this, target);
-	}
-
-	@Inject(method = "startSleepInBed", at = @At("TAIL"))
-	private void onStartSleep(BlockPos bedPos, CallbackInfoReturnable<Either<Player.BedSleepingProblem, Unit>> cir) {
-		PlayerEvents.START_SLEEP.invoker().accept((Player) (Object) this, bedPos);
-	}
-
-	@Inject(method = "stopSleepInBed", at = @At("TAIL"))
-	private void onStopSleep(boolean wakeImmediately, boolean updateLevelForSleepingPlayers, CallbackInfo ci) {
-		PlayerEvents.STOP_SLEEP.invoker().onSleepStop((Player) (Object) this, wakeImmediately, updateLevelForSleepingPlayers);
-	}
-}

library/player/src/main/resources/frog.mod.toml

@@ -1,19 +0,0 @@
-[frog]
-format_version = "1.0.0"
-
-[frog.mod]
-id = "froglib_player"
-name = "FrogLib Player"
-version = "$version"
-license = "Apache-2.0"
-credits = [
-    { name = "FrogMC Team", roles = ["author"] }
-]
-
-[frog.dependencies]
-depends = [
-    { id = "minecraft", versions = "~$game_version", name = "Minecraft" }
-]
-
-[frog.extensions]
-mixin = "froglib.player.mixins.json"

library/player/src/main/resources/froglib.player.mixins.json

@@ -1,17 +0,0 @@
-{
-	"required": true,
-	"minVersion": "0.8",
-	"package": "dev.frogmc.froglib.player.impl.mixin",
-	"compatibilityLevel": "JAVA_21",
-	"mixins": [
-		"PlayerMixin"
-	],
-	"server": [
-	],
-	"client": [
-
-	],
-	"injectors": {
-		"defaultRequire": 1
-	}
-}

library/registries/src/main/java/dev/frogmc/froglib/registries/api/RegistryEvents.java

@@ -4,7 +4,7 @@ import java.util.Collection;
 
 import dev.frogmc.froglib.events.api.Event;
 
-public final class RegistryEvents {
+public class RegistryEvents {
 	public static final Event<BlockRegistration> BLOCK_REGISTRATION = Event.of(list -> () -> list.stream().map(BlockRegistration::register).flatMap(Collection::stream).toList());
 	public static final Event<ItemRegistration> ITEM_REGISTRATION = Event.of(list -> () -> list.stream().map(ItemRegistration::register).flatMap(Collection::stream).toList());
 }

library/registries/src/test/java/dev/frogmc/froglib/registries/test/BlockTestBlocks.java

@@ -9,7 +9,7 @@ import net.minecraft.world.level.block.StainedGlassBlock;
 import net.minecraft.world.level.block.state.BlockBehaviour;
 
 public class BlockTestBlocks {
-	public static final ResourceLocation loc = ResourceLocation.fromNamespaceAndPath("froglib_block_test", "testblock");
+	public static final ResourceLocation loc = new ResourceLocation("froglib_block_test", "testblock");
 	public static final Block b = new StainedGlassBlock(DyeColor.MAGENTA, BlockBehaviour.Properties.of());
 	public static final Item bItem = new BlockItem(b, new Item.Properties());
 }

library/resourceloader/src/main/java/dev/frogmc/froglib/resources/impl/ResourceLoaderClient.java

@@ -1,14 +1,14 @@
 package dev.frogmc.froglib.resources.impl;
 
 import dev.frogmc.froglib.entrypoints.api.ClientExtension;
-import dev.frogmc.froglib.events.api.LifecycleEvents;
+import dev.frogmc.froglib.events.api.Events;
 import dev.frogmc.frogloader.api.mod.ModProperties;
 import net.minecraft.server.packs.resources.ReloadableResourceManager;
 
 public class ResourceLoaderClient implements ClientExtension {
 	@Override
 	public void onClientInit(ModProperties mod) {
-		LifecycleEvents.CLIENT_STARTUP.register((m, c) ->
+		Events.CLIENT_STARTUP.register((m, c) ->
 				((ReloadableResourceManager) m.getResourceManager()).registerReloadListener(new ResourceLoaderReloadListener()));
 	}
 }

library/resourceloader/src/main/java/dev/frogmc/froglib/resources/impl/ResourceLoaderServer.java

@@ -1,14 +1,14 @@
 package dev.frogmc.froglib.resources.impl;
 
 import dev.frogmc.froglib.entrypoints.api.ServerExtension;
-import dev.frogmc.froglib.events.api.LifecycleEvents;
+import dev.frogmc.froglib.events.api.Events;
 import dev.frogmc.frogloader.api.mod.ModProperties;
 import net.minecraft.server.packs.resources.ReloadableResourceManager;
 
 public class ResourceLoaderServer implements ServerExtension {
 	@Override
 	public void onServerInit(ModProperties mod) {
-		LifecycleEvents.SERVER_STARTUP.register(server ->
+		Events.SERVER_STARTUP.register(server ->
 				((ReloadableResourceManager)server.getResourceManager()).registerReloadListener(new ResourceLoaderReloadListener()));
 	}
 }

settings.gradle.kts

@@ -25,5 +25,3 @@ include("library:resourceloader")
 findProject(":library:resourceloader")?.name = "resourceloader"
 include("library:registries")
 findProject(":library:registries")?.name = "registries"
-include("library:player")
-findProject(":library:player")?.name = "player"