chore: update to 1.20.5

build.gradle.kts

@@ -1,16 +1,16 @@
 plugins {
-    id("jfmod") version "1.5-SNAPSHOT"
+    id("jfmod") version "1.6-SNAPSHOT"
 }
 
 allprojects { group = "io.gitlab.jfronny" }
 base.archivesName = "google-chat"
 
-val fabricVersion = "0.89.1+1.20.2"
 jfMod {
-    minecraftVersion = "1.20.2"
+    minecraftVersion = "1.20.5"
     yarn("build.1")
-    loaderVersion = "0.14.22"
-    libJfVersion = "3.13.1"
+    loaderVersion = "0.15.10"
+    libJfVersion = "3.15.3"
+    fabricApiVersion = "0.97.6+1.20.5"
 
     modrinth {
         projectId = "google-chat"
@@ -25,12 +25,19 @@ jfMod {
 }
 
 dependencies {
-    modImplementation("io.gitlab.jfronny.libjf:libjf-config-core-v2:${jfMod.libJfVersion.get()}")
-    modImplementation("io.gitlab.jfronny.libjf:libjf-translate-v1:${jfMod.libJfVersion.get()}")
-    include(modImplementation(fabricApi.module("fabric-message-api-v1", fabricVersion))!!)
+    modImplementation("io.gitlab.jfronny.libjf:libjf-config-core-v2")
+    modImplementation("io.gitlab.jfronny.libjf:libjf-translate-v1")
+    include(modImplementation("net.fabricmc.fabric-api:fabric-message-api-v1")!!)
+    // Keybind
+    modCompileOnly("net.fabricmc.fabric-api:fabric-key-binding-api-v1")
+    modCompileOnly("net.fabricmc.fabric-api:fabric-lifecycle-events-v1")
 
     // Dev env
-    modLocalRuntime("io.gitlab.jfronny.libjf:libjf-config-ui-tiny:${jfMod.libJfVersion.get()}")
-    modLocalRuntime("io.gitlab.jfronny.libjf:libjf-devutil:${jfMod.libJfVersion.get()}")
-    modLocalRuntime("com.terraformersmc:modmenu:8.0.0-beta.2")
+    modLocalRuntime("io.gitlab.jfronny.libjf:libjf-config-ui-tiny")
+    modLocalRuntime("io.gitlab.jfronny.libjf:libjf-devutil")
+    modLocalRuntime("net.fabricmc.fabric-api:fabric-resource-loader-v0")
+    modLocalRuntime("com.terraformersmc:modmenu:10.0.0-beta.1")
+    // for modmenu
+    modLocalRuntime("net.fabricmc.fabric-api:fabric-screen-api-v1")
+    modLocalRuntime("net.fabricmc.fabric-api:fabric-key-binding-api-v1")
 }

src/client/java/io/gitlab/jfronny/googlechat/client/GoogleChatClient.java

@@ -0,0 +1,41 @@
+package io.gitlab.jfronny.googlechat.client;
+
+import io.gitlab.jfronny.googlechat.GoogleChat;
+import io.gitlab.jfronny.googlechat.GoogleChatConfig;
+import io.gitlab.jfronny.libjf.config.api.v2.ConfigHolder;
+import io.gitlab.jfronny.libjf.config.api.v2.ConfigInstance;
+import net.fabricmc.api.ClientModInitializer;
+import net.fabricmc.fabric.api.client.event.lifecycle.v1.ClientTickEvents;
+import net.fabricmc.fabric.api.client.keybinding.v1.KeyBindingHelper;
+import net.fabricmc.loader.api.FabricLoader;
+import net.minecraft.client.option.KeyBinding;
+import net.minecraft.client.util.InputUtil;
+import org.jetbrains.annotations.NotNull;
+
+public class GoogleChatClient implements ClientModInitializer {
+    @Override
+    public void onInitializeClient() {
+        ConfigInstance ci = ConfigHolder.getInstance().get(GoogleChat.MOD_ID);
+        if (ci != null
+                && FabricLoader.getInstance().isModLoaded("fabric-key-binding-api-v1")
+                && FabricLoader.getInstance().isModLoaded("fabric-lifecycle-events-v1")) {
+            setupKeybind(ci);
+        }
+    }
+
+    private static void setupKeybind(@NotNull ConfigInstance ci) {
+        // Factored out to prevent loading classes if mods are not present
+        KeyBinding keyBinding = KeyBindingHelper.registerKeyBinding(new KeyBinding(
+                "key." + GoogleChat.MOD_ID + ".toggle",
+                InputUtil.Type.KEYSYM,
+                -1,
+                KeyBinding.MULTIPLAYER_CATEGORY
+        ));
+        ClientTickEvents.END_CLIENT_TICK.register(client -> {
+            if (keyBinding.wasPressed()) {
+                GoogleChatConfig.General.enabled = !GoogleChatConfig.General.enabled;
+                ci.write();
+            }
+        });
+    }
+}

src/client/java/io/gitlab/jfronny/googlechat/client/mixin/ChatScreenMixin.java

@@ -9,7 +9,7 @@ import org.spongepowered.asm.mixin.injection.ModifyVariable;
 
 @Mixin(ChatScreen.class)
 public class ChatScreenMixin {
-    @ModifyVariable(method = "sendMessage(Ljava/lang/String;Z)Z", at = @At(value = "HEAD"), argsOnly = true, ordinal = 0)
+    @ModifyVariable(method = "sendMessage(Ljava/lang/String;Z)V", at = @At(value = "HEAD"), argsOnly = true, ordinal = 0)
     String googlechat$translateChatText(String chatText) {
         if (chatText.startsWith("/")) return chatText; // Bypass for client-side commands (Carpet, ...)
         return GoogleChat.translateIfNeeded(chatText, TranslationDirection.C2S, true);

src/main/java/io/gitlab/jfronny/googlechat/GoogleChat.java

@@ -1,7 +1,9 @@
 package io.gitlab.jfronny.googlechat;
 
 import io.gitlab.jfronny.commons.io.cache.FixedSizeMap;
-import io.gitlab.jfronny.commons.logging.Logger;
+import io.gitlab.jfronny.commons.logger.SystemLoggerPlus;
+import io.gitlab.jfronny.commons.throwable.Coerce;
+import io.gitlab.jfronny.commons.throwable.Try;
 import io.gitlab.jfronny.libjf.translate.api.Language;
 import io.gitlab.jfronny.libjf.translate.api.TranslateService;
 import net.fabricmc.api.EnvType;
@@ -17,7 +19,7 @@ import java.util.regex.Pattern;
 
 public class GoogleChat implements ModInitializer {
     public static final String MOD_ID = "google-chat";
-    public static final Logger LOGGER = Logger.forName(MOD_ID);
+    public static final SystemLoggerPlus LOGGER = SystemLoggerPlus.forName(MOD_ID);
     public static TranslateService<?> TRANSLATE_SERVICE;
     private static final boolean IS_SERVER = FabricLoader.getInstance().getEnvironmentType() == EnvType.SERVER;
 
@@ -30,7 +32,7 @@ public class GoogleChat implements ModInitializer {
 
     @Override
     public void onInitialize() {
-        ForkJoinPool.commonPool().execute(() -> TRANSLATE_SERVICE = TranslateService.getConfigured());
+        ForkJoinPool.commonPool().execute(Try.handle(Coerce.runnable(() -> TRANSLATE_SERVICE = TranslateService.getConfigured()), e -> LOGGER.error("Could not initialize translation service", e)));
     }
 
     public static void onConfigChange() {
@@ -91,7 +93,7 @@ public class GoogleChat implements ModInitializer {
     }
 
     public static TextContent translateIfNeeded(TextContent source, TranslationDirection direction, boolean respectRegex) {
-        if (source == null) return null;
+        if (source == null || source == PlainTextContent.EMPTY) return source;
         if (direction.shouldSkipOutright()) return source;
         String sourceString = toString(source);
         if (respectRegex && direction.failsRegex(sourceString)) return source;
@@ -108,8 +110,8 @@ public class GoogleChat implements ModInitializer {
                     else args[i] = args[i];
                 }
                 return new TranslatableTextContent(tx.getKey(), translateIfNeeded(tx.getFallback(), direction, false), args);
-            } else if (t instanceof LiteralTextContent tx) {
-                return new LiteralTextContent(translateIfNeeded(tx.string(), direction, false));
+            } else if (t instanceof PlainTextContent.Literal tx) {
+                return new PlainTextContent.Literal(translateIfNeeded(tx.string(), direction, false));
             } else {
 //                LOGGER.info("Unhandled text type: " + source.getClass() + " (" + source + ")");
                 return t;

src/main/java/io/gitlab/jfronny/googlechat/GoogleChatConfig.java

@@ -1,11 +1,17 @@
 package io.gitlab.jfronny.googlechat;
 
-import io.gitlab.jfronny.commons.serialize.gson.api.v2.Ignore;
+import io.gitlab.jfronny.commons.serialize.SerializeReader;
+import io.gitlab.jfronny.commons.serialize.annotations.Ignore;
+import io.gitlab.jfronny.commons.throwable.Assume;
+import io.gitlab.jfronny.commons.throwable.ThrowingConsumer;
 import io.gitlab.jfronny.libjf.config.api.v2.*;
 import io.gitlab.jfronny.libjf.config.api.v2.dsl.ConfigBuilder;
+import io.gitlab.jfronny.libjf.config.api.v2.dsl.Migration;
 import net.fabricmc.api.*;
 import net.fabricmc.loader.api.*;
 
+import java.util.function.Consumer;
+
 @JfConfig(tweaker = GoogleChatConfig.class)
 public class GoogleChatConfig {
     @Category(referencedConfigs = "libjf-translate-v1")
@@ -64,17 +70,28 @@ public class GoogleChatConfig {
 
     public static ConfigBuilder<?> tweak(ConfigBuilder<?> builder) {
         return builder
-                .addMigration("enabled", reader -> General.enabled = reader.nextBoolean())
-                .addMigration("serverLanguage", reader -> General.serverLanguage = reader.nextString())
-                .addMigration("clientLanguage", reader -> General.clientLanguage = reader.nextString())
-                .addMigration("translationTooltip", reader -> General.translationTooltip = reader.nextBoolean())
-                .addMigration("desugar", reader -> Processing.desugar = reader.nextBoolean())
-                .addMigration("receivingRegex", reader -> Processing.receivingRegex = reader.nextString())
-                .addMigration("receivingRegexIsBlacklist", reader -> Processing.receivingRegexIsBlacklist = reader.nextBoolean())
-                .addMigration("sendingRegex", reader -> Processing.sendingRegex = reader.nextString())
-                .addMigration("sendingRegexIsBlacklist", reader -> Processing.sendingRegexIsBlacklist = reader.nextBoolean())
-                .addMigration("cacheSize", reader -> Advanced.cacheSize = reader.nextInt())
-                .addMigration("debugLogs", reader -> Advanced.debugLogs = reader.nextBoolean());
+                .addMigration("enabled", of(reader -> General.enabled = reader.nextBoolean()))
+                .addMigration("serverLanguage", of(reader -> General.serverLanguage = reader.nextString()))
+                .addMigration("clientLanguage", of(reader -> General.clientLanguage = reader.nextString()))
+                .addMigration("translationTooltip", of(reader -> General.translationTooltip = reader.nextBoolean()))
+                .addMigration("desugar", of(reader -> Processing.desugar = reader.nextBoolean()))
+                .addMigration("receivingRegex", of(reader -> Processing.receivingRegex = reader.nextString()))
+                .addMigration("receivingRegexIsBlacklist", of(reader -> Processing.receivingRegexIsBlacklist = reader.nextBoolean()))
+                .addMigration("sendingRegex", of(reader -> Processing.sendingRegex = reader.nextString()))
+                .addMigration("sendingRegexIsBlacklist", of(reader -> Processing.sendingRegexIsBlacklist = reader.nextBoolean()))
+                .addMigration("cacheSize", of(reader -> Advanced.cacheSize = reader.nextInt()))
+                .addMigration("debugLogs", of(reader -> Advanced.debugLogs = reader.nextBoolean()));
+    }
+
+    private static Migration of(ThrowingConsumer<SerializeReader<?, ?>, Exception> dm) {
+        //TODO use the libjf-native method
+        Consumer<SerializeReader<?, ?>> cn = ((ThrowingConsumer<SerializeReader<?, ?>, RuntimeException>) (ThrowingConsumer) dm)::accept;
+        return new Migration() {
+            @Override
+            public <TEx extends Exception, Reader extends SerializeReader<TEx, Reader>> void apply(Reader reader) throws TEx {
+                cn.accept(reader);
+            }
+        };
     }
 
     static {

src/main/resources/assets/google-chat/lang/en_us.json

@@ -29,5 +29,7 @@
   "google-chat.jfconfig.advanced.debugLogs.tooltips": "Log additional information about message processing. Useful for debugging",
 
   "google-chat.jfconfig.client": "Client",
-  "google-chat.jfconfig.server": "Server"
+  "google-chat.jfconfig.server": "Server",
+
+  "key.google-chat.toggle": "Toggle GoogleChat"
 }

src/main/resources/fabric.mod.json

@@ -17,7 +17,8 @@
   "entrypoints": {
     "libjf:config": ["io.gitlab.jfronny.googlechat.JFC_GoogleChatConfig"],
     "server": ["io.gitlab.jfronny.googlechat.server.GoogleChatServer"],
-    "main": ["io.gitlab.jfronny.googlechat.GoogleChat"]
+    "main": ["io.gitlab.jfronny.googlechat.GoogleChat"],
+    "client": ["io.gitlab.jfronny.googlechat.client.GoogleChatClient"]
   },
   "mixins": [
     {
@@ -28,7 +29,7 @@
   "depends": {
     "fabricloader": ">=0.12.12",
     "minecraft": "*",
-    "libjf-config-core-v1": "*",
+    "libjf-config-core-v2": "*",
     "libjf-translate-v1": "*",
     "fabric-message-api-v1": "*"
   }