improve build

- publish all modules
- add test mods to general runtime classpath

- fix issues with keybindings being registered before the game is initialized

.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

@@ -96,6 +96,15 @@ evaluationDependsOnChildren()
 
 // TODO: add testmods of libraries to general client runtime classpath
 
+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)
+    }
+}
+
 dependencies {
     project.subprojects.filter {
         it.projectDir.resolve("src/main/resources/frog.mod.toml").exists()

gradle/libs.versions.toml

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

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

@@ -15,8 +15,17 @@ public class KeyMappingsImpl implements KeyMappings {
 	}
 
 	public void registerKey(KeyMapping mapping){
-		// TODO maybe synchronize?
+		Runnable register = () -> {
-		Minecraft.getInstance().options.keyMappings = ArrayUtils.add(Minecraft.getInstance().options.keyMappings, mapping);
+			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 {
+			Events.CLIENT_STARTUP.register((client, config) -> register.run());
+		}
 	}
 
 	@Override