Remove config builder and make config abstract (broken)

example/src/main/java/me/lortseam/completeconfig/example/ExampleMod.java

@@ -1,6 +1,5 @@
 package me.lortseam.completeconfig.example;
 
-import me.lortseam.completeconfig.data.Config;
 import net.fabricmc.api.ModInitializer;
 
 public class ExampleMod implements ModInitializer {
@@ -9,9 +8,7 @@ public class ExampleMod implements ModInitializer {
 
     @Override
     public void onInitialize() {
-        Config.builder(MOD_ID)
-                .add(new Settings())
-                .build();
+        new Settings();
     }
 
 }

example/src/main/java/me/lortseam/completeconfig/example/Settings.java

@@ -1,15 +1,19 @@
 package me.lortseam.completeconfig.example;
 
-import me.lortseam.completeconfig.api.ConfigContainer;
 import me.lortseam.completeconfig.api.ConfigEntries;
 import me.lortseam.completeconfig.api.ConfigEntry;
 import me.lortseam.completeconfig.api.ConfigGroup;
+import me.lortseam.completeconfig.data.Config;
 import me.shedaniel.math.Color;
 
 import java.util.Arrays;
 import java.util.List;
 
-public final class Settings implements ConfigContainer {
+public final class Settings extends Config {
+
+    protected Settings() {
+        super(ExampleMod.MOD_ID, false);
+    }
 
     @Transitive
     @ConfigEntries

lib/src/main/java/me/lortseam/completeconfig/data/Config.java

@@ -5,19 +5,17 @@ import lombok.Getter;
 import lombok.NonNull;
 import lombok.extern.log4j.Log4j2;
 import me.lortseam.completeconfig.api.ConfigContainer;
-import me.lortseam.completeconfig.text.TranslationKey;
 import me.lortseam.completeconfig.io.ConfigSource;
+import me.lortseam.completeconfig.text.TranslationKey;
 import net.fabricmc.api.EnvType;
 import net.fabricmc.api.Environment;
 import net.fabricmc.loader.api.FabricLoader;
 import net.fabricmc.loader.api.metadata.ModMetadata;
 
 import java.util.Arrays;
-import java.util.LinkedHashSet;
-import java.util.Objects;
 
 @Log4j2(topic = "CompleteConfig")
-public final class Config extends BaseCollection {
+public abstract class Config extends BaseCollection implements ConfigContainer {
 
     static {
         Runtime.getRuntime().addShutdownHook(new Thread(() -> {
@@ -29,28 +27,29 @@ public final class Config extends BaseCollection {
         }));
     }
 
-    /**
-     * Creates a new config builder for the specified mod.
-     *
-     * @param modId the ID of the mod creating the config
-     */
-    public static Builder builder(@NonNull String modId) {
-        if (!FabricLoader.getInstance().isModLoaded(modId)) {
-            throw new IllegalArgumentException("Mod " + modId + " is not loaded");
-        }
-        return new Builder(modId);
-    }
-
     @Getter(AccessLevel.PACKAGE)
     private final ConfigSource source;
     @Environment(EnvType.CLIENT)
     private TranslationKey translation;
     private final boolean saveOnExit;
 
-    private Config(ConfigSource source, ConfigContainer[] children, boolean saveOnExit) {
-        this.source = source;
+    protected Config(@NonNull String modId, @NonNull String[] branch, boolean saveOnExit) {
+        if (!FabricLoader.getInstance().isModLoaded(modId)) {
+            throw new IllegalArgumentException("Mod " + modId + " is not loaded");
+        }
+        source = new ConfigSource(modId, branch);
         this.saveOnExit = saveOnExit;
-        resolve(children);
+        ConfigRegistry.register(this);
+        resolveContainer(this);
+        if (isEmpty()) {
+            logger.warn("Empty config: " + modId + " " + Arrays.toString(branch));
+            return;
+        }
+        load();
+    }
+
+    protected Config(String modId, boolean saveOnExit) {
+        this(modId, new String[0], saveOnExit);
     }
 
     public ModMetadata getMod() {
@@ -81,88 +80,4 @@ public final class Config extends BaseCollection {
         source.save(this);
     }
 
-    @Log4j2(topic = "CompleteConfig")
-    public final static class Builder {
-
-        private final String modId;
-        private String[] branch = new String[0];
-        private final LinkedHashSet<ConfigContainer> children = new LinkedHashSet<>();
-        private boolean main;
-        private boolean saveOnExit;
-
-        private Builder(String modId) {
-            this.modId = modId;
-        }
-
-        /**
-         * Sets the branch. Every config of a mod needs a unique branch, therefore setting a branch is only required
-         * when using more than one config.
-         *
-         * <p>The branch determines the location of the config's save file.
-         *
-         * @param branch the branch
-         * @return this builder
-         */
-        public Builder setBranch(@NonNull String[] branch) {
-            Arrays.stream(branch).forEach(Objects::requireNonNull);
-            this.branch = branch;
-            return this;
-        }
-
-        /**
-         * Adds one or more containers to the config.
-         *
-         * @param containers one or more containers
-         * @return this builder
-         */
-        public Builder add(@NonNull ConfigContainer... containers) {
-            for (ConfigContainer container : containers) {
-                if (!children.add(Objects.requireNonNull(container))) {
-                    throw new IllegalArgumentException("Duplicate container " + container.getClass().getSimpleName());
-                }
-            }
-            return this;
-        }
-
-        /**
-         * Sets a flag to save the config when the game closes.
-         *
-         * @return this builder
-         */
-        public Builder saveOnExit() {
-            saveOnExit = true;
-            return this;
-        }
-
-        /**
-         * Registers the config as main mod config.
-         *
-         * @return this builder
-         */
-        public Builder main() {
-            main = true;
-            return this;
-        }
-
-        /**
-         * Creates and loads the config.
-         *
-         * @return the created config, or null if empty
-         */
-        public Config build() {
-            Config config = null;
-            if (!children.isEmpty()) {
-                config = new Config(new ConfigSource(modId, branch), children.toArray(new ConfigContainer[0]), saveOnExit);
-            }
-            if (config == null || config.isEmpty()) {
-                logger.warn("Empty config: " + modId + " " + Arrays.toString(branch));
-                return null;
-            }
-            ConfigRegistry.register(config, main || branch.length == 0 && !ConfigRegistry.getMainConfig(modId).isPresent());
-            config.load();
-            return config;
-        }
-
-    }
-
 }