Migrate old bungee configs

Author: JulianVennen

README.md

@@ -54,15 +54,15 @@ apiToken = 'example-token'
 
 # Watch servers in the proxy config and automatically remove them when they go offline
 # Note that this only works if you use .exaroton.me addresses in your velocity config.
-watch-servers = true
+watchServers = true
 
 # Automatically start servers when the proxy starts
-[auto-start]
+[autoStartServers]
 enabled = false
 servers = ["example.exaroton.me"]
 
 # Automatically stop servers when the proxy stops
-[auto-stop]
+[autoStopServers]
 enabled = false
 servers = ["example.exaroton.me"]
 ```

buildSrc/src/main/groovy/multiloader-base.gradle

@@ -6,7 +6,7 @@ plugins {
 }
 
 base {
-    archivesName = "${plugin_id}-${project.name}-${minecraft_version}"
+    archivesName = "${plugin_id}-${project.name}"
 }
 
 java {

bungeecord/src/main/java/com/exaroton/proxy/ProxyPluginImpl.java

@@ -2,7 +2,12 @@
 
 import com.exaroton.proxy.servers.ProxyServerManager;
 import net.md_5.bungee.api.CommandSender;
+import net.md_5.bungee.config.Configuration;
+import net.md_5.bungee.config.ConfigurationProvider;
+import net.md_5.bungee.config.YamlConfiguration;
 
+import java.io.IOException;
+import java.nio.file.Files;
 import java.util.Collection;
 import java.util.stream.Collectors;
 
@@ -32,4 +37,46 @@ public ProxyServerManager<?> createProxyServerManager() {
     public Collection<String> getPlayers() {
         return bungeePlugin.getProxy().getPlayers().stream().map(CommandSender::getName).collect(Collectors.toList());
     }
+
+    @Override
+    protected void migrateOldConfigFields() {
+        if (configFile.isEmpty()) {
+            var oldPluginDir = bungeePlugin.getProxy().getPluginsFolder().toPath().resolve("ExarotonBungeePlugin");
+            var oldConfigFile = oldPluginDir.resolve("config.yml").toFile();
+
+            if (oldConfigFile.exists()) {
+                Constants.LOG.info("Migrating old YAML config to TOML");
+                var configProvider = ConfigurationProvider.getProvider(YamlConfiguration.class);
+                try {
+                    var oldConfig = configProvider.load(oldConfigFile);
+
+                    writeConfig(oldConfig, "");
+
+                    Files.delete(oldConfigFile.toPath());
+                    try (var children = Files.list(oldPluginDir)) {
+                        if (children.findAny().isPresent()) {
+                            Constants.LOG.warn("Can't delete old plugin directory as it contains other files.");
+                        } else {
+                            Files.delete(oldPluginDir);
+                        }
+                    }
+                } catch (IOException e) {
+                    Constants.LOG.error("Failed to migrate old YAML config: {}", e.getMessage(), e);
+                }
+            }
+        }
+
+        super.migrateOldConfigFields();
+    }
+
+    private void writeConfig(Object value, String key) {
+        if (value instanceof Configuration) {
+            var config = (Configuration) value;
+            for (String otherKey : config.getKeys()) {
+                writeConfig(config.get(otherKey), (key.isEmpty() ? "" : (key + ".")) + otherKey);
+            }
+        } else {
+            configFile.set(key, value);
+        }
+    }
 }

gradle.properties

@@ -2,7 +2,7 @@
 # Every field you add must be added to the root build.gradle expandProps map.
 
 # Project
-version=2.0.0
+version=2.0.0-SNAPSHOT
 group=com.exaroton.proxy
 java_version=21
 

proxy/src/main/java/com/exaroton/proxy/CommonProxyPlugin.java

@@ -54,7 +54,7 @@ public abstract class CommonProxyPlugin {
      * @return a future that completes when setup is done
      */
     public CompletableFuture<Void> setUp() {
-        initializeConfig();
+        loadConfig();
 
         if (config.apiToken == null || config.apiToken.isEmpty() || config.apiToken.equals("example-token")) {
             throw new IllegalStateException("No API token provided. Please set the API token in the configuration file.");
@@ -154,13 +154,25 @@ public final ProxyServerManager<?> getProxyServerManager() {
      */
     public abstract Collection<String> getPlayers();
 
-    protected void initializeConfig() {
-        configFile = Services.platform().getConfig()
-                .autoreload()
-                .onAutoReload(this::onConfigLoaded)
-                .autosave()
-                .writingMode(WritingMode.REPLACE_ATOMIC)
+    /**
+     * Initializes the configFile field
+     */
+    protected void initializeConfigFile() {
+        if (configFile == null) {
+            configFile = Services.platform().getConfig()
+                    .autoreload()
+                    .onAutoReload(this::onConfigLoaded)
+                    .autosave()
+                    .writingMode(WritingMode.REPLACE_ATOMIC)
                 .build();
+        }
+    }
+
+    /**
+     * Loads the config, migrates old fields and updates the config file
+     */
+    protected void loadConfig() {
+        initializeConfigFile();
         configFile.load();
         migrateOldConfigFields();
         onConfigLoaded(false);