Use platform services for config dir location

Author: NotRyken

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Added toast notification option
 - Stopped trigger duplicate detection evaluating disabled notifications
+- Improved config directory detection
 
 ## 2.3.10
 

common/src/main/java/dev/terminalmc/chatnotify/config/Config.java

@@ -18,6 +18,7 @@
 
 import com.google.gson.*;
 import dev.terminalmc.chatnotify.ChatNotify;
+import dev.terminalmc.chatnotify.platform.Services;
 import dev.terminalmc.chatnotify.util.JsonUtil;
 import net.minecraft.sounds.SoundSource;
 import org.jetbrains.annotations.NotNull;
@@ -50,7 +51,7 @@
 public class Config {
     public static final int VERSION = 9;
     public final int version = VERSION;
-    private static final Path DIR_PATH = Path.of("config");
+    private static final Path CONFIG_DIR = Services.PLATFORM.getConfigDir();
     public static final String FILE_NAME = ChatNotify.MOD_ID + ".json";
     public static final String UNREADABLE_FILE_NAME = ChatNotify.MOD_ID + ".unreadable.json";
     public static final String OLD_FILE_NAME = ChatNotify.MOD_ID + ".old.json";
@@ -316,7 +317,7 @@ public static Config reload() {
     // Load and save
 
     public static @NotNull Config load() {
-        Path file = DIR_PATH.resolve(FILE_NAME);
+        Path file = CONFIG_DIR.resolve(FILE_NAME);
         Config config = null;
         if (Files.exists(file)) {
             JsonUtil.reset();
@@ -347,8 +348,8 @@ public static Config reload() {
     private static void backup(String path) {
         try {
             ChatNotify.LOG.warn("Copying {} to {}", FILE_NAME, path);
-            if (!Files.isDirectory(DIR_PATH)) Files.createDirectories(DIR_PATH);
-            Path file = DIR_PATH.resolve(FILE_NAME);
+            if (!Files.isDirectory(CONFIG_DIR)) Files.createDirectories(CONFIG_DIR);
+            Path file = CONFIG_DIR.resolve(FILE_NAME);
             Path backupFile = file.resolveSibling(path);
             Files.move(file, backupFile, StandardCopyOption.ATOMIC_MOVE,
                     StandardCopyOption.REPLACE_EXISTING);
@@ -361,8 +362,8 @@ public static void save() {
         if (instance == null) return;
         instance.validate();
         try {
-            if (!Files.isDirectory(DIR_PATH)) Files.createDirectories(DIR_PATH);
-            Path file = DIR_PATH.resolve(FILE_NAME);
+            if (!Files.isDirectory(CONFIG_DIR)) Files.createDirectories(CONFIG_DIR);
+            Path file = CONFIG_DIR.resolve(FILE_NAME);
             Path tempFile = file.resolveSibling(file.getFileName() + ".tmp");
             try (OutputStreamWriter writer = new OutputStreamWriter(
                     new FileOutputStream(tempFile.toFile()), StandardCharsets.UTF_8)) {

common/src/main/java/dev/terminalmc/chatnotify/platform/Services.java

@@ -0,0 +1,18 @@
+package dev.terminalmc.chatnotify.platform;
+
+import dev.terminalmc.chatnotify.ChatNotify;
+import dev.terminalmc.chatnotify.platform.services.IPlatformInfo;
+
+import java.util.ServiceLoader;
+
+public class Services {
+    public static final IPlatformInfo PLATFORM = load(IPlatformInfo.class);
+
+    public static <T> T load(Class<T> clazz) {
+        final T loadedService = ServiceLoader.load(clazz)
+                .findFirst()
+                .orElseThrow(() -> new NullPointerException("Failed to load service for " + clazz.getName()));
+        ChatNotify.LOG.debug("Loaded {} for service {}", loadedService, clazz);
+        return loadedService;
+    }
+}

common/src/main/java/dev/terminalmc/chatnotify/platform/services/IPlatformInfo.java

@@ -0,0 +1,10 @@
+package dev.terminalmc.chatnotify.platform.services;
+
+import java.nio.file.Path;
+
+public interface IPlatformInfo {
+    /**
+     * @return the configuration directory of the instance.
+     */
+    Path getConfigDir();
+}

fabric/src/main/java/dev/terminalmc/chatnotify/platform/FabricPlatformInfo.java

@@ -0,0 +1,13 @@
+package dev.terminalmc.chatnotify.platform;
+
+import dev.terminalmc.chatnotify.platform.services.IPlatformInfo;
+import net.fabricmc.loader.api.FabricLoader;
+
+import java.nio.file.Path;
+
+public class FabricPlatformInfo implements IPlatformInfo {
+    @Override
+    public Path getConfigDir() {
+        return FabricLoader.getInstance().getConfigDir();
+    }
+}

fabric/src/main/resources/META-INF/services/dev.terminalmc.chatnotify.platform.services.IPlatformInfo

@@ -0,0 +1 @@
+dev.terminalmc.chatnotify.platform.FabricPlatformInfo

neoforge/src/main/java/dev/terminalmc/chatnotify/platform/NeoForgePlatformInfo.java

@@ -0,0 +1,13 @@
+package dev.terminalmc.chatnotify.platform;
+
+import dev.terminalmc.chatnotify.platform.services.IPlatformInfo;
+import net.neoforged.fml.loading.FMLPaths;
+
+import java.nio.file.Path;
+
+public class NeoForgePlatformInfo implements IPlatformInfo {
+    @Override
+    public Path getConfigDir() {
+        return FMLPaths.CONFIGDIR.get();
+    }
+}

neoforge/src/main/resources/META-INF/services/dev.terminalmc.framework.platform.services.IPlatformInfo

@@ -0,0 +1 @@
+dev.terminalmc.chatnotify.platform.NeoForgePlatformInfo