Cover configuration and faction workflows and preserve state on failure

src/main/java/net/tfminecraft/simplefactions/Cache.java

@@ -17,6 +17,8 @@
 import net.tfminecraft.simplefactions.war.battle.enums.DefenderRespawnMode;
 
 public class Cache {
+    private Cache() {}
+
 	public static String mapRef;
 	/** Chapter slug for a future Archive prefill. Not the live upload folder. */
 	public static String chapterId = "unknown";

src/main/java/net/tfminecraft/simplefactions/SimpleFactions.java

@@ -3,6 +3,8 @@
 
 import net.tfminecraft.simplefactions.vehicles.maintenance.VehicleHealthDecayApi.Vf;
 import java.io.File;
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
 
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.event.EventHandler;
@@ -252,15 +254,24 @@ public class SimpleFactions extends JavaPlugin{
 	private final VehicleMaintenanceDecayTask vehicleMaintenanceDecayTask =
 			new VehicleMaintenanceDecayTask();
 
+	private boolean startupComplete;
+
 	@Override
 	public void onEnable() {
+		startupComplete = false;
 		config = getConfig();
 		plugin = this;
 		FactionManager.inv = inventoryManager;
 		createFolders();
 		createConfigs();
+		try {
+			loadConfigs();
+		} catch (RuntimeException error) {
+			getLogger().log(java.util.logging.Level.SEVERE, "Cannot load SimpleFactions configuration; disabling before restoring saved state", error);
+			getServer().getPluginManager().disablePlugin(this);
+			return;
+		}
 		registerListeners();
-		loadConfigs();
 		registerRpCharactersIntegrationHooks();
 		vehicleRegistryPersistence = new VehicleRegistryPersistence(
 			new File(getDataFolder(), "Cache"),
@@ -365,6 +376,7 @@ public void onEnable() {
 		}
 		inventoryManager.start();
 		vehicleMaintenanceDecayTask.start();
+		startupComplete = true;
 	}
 	@Override
 	public void onDisable() {
@@ -376,8 +388,12 @@ public void onDisable() {
 		CampaignViewRefreshService.stop();
 		BattleManager.shutdown();
 		net.tfminecraft.simplefactions.war.battle.persistence.BattlePersistenceService.stopAutosave();
-		net.tfminecraft.simplefactions.war.battle.persistence.BattlePersistenceService.saveAll();
 		sessionManager.end();
+		if (!startupComplete) {
+			net.tfminecraft.simplefactions.identity.LeaderCharacters.reset();
+			return;
+		}
+		net.tfminecraft.simplefactions.war.battle.persistence.BattlePersistenceService.saveAll();
 		net.tfminecraft.simplefactions.inactivity.InactivityService.save();
 		saveFactionsForShutdown();
 		for(War w : WarManager.get()){
@@ -524,7 +540,30 @@ public static void reloadTitles() {
 	}
 
 	public static void reloadConfigs() {
-		plugin.loadConfigs();
+		var previousRanks = new ArrayList<>(RankLoader.getRanks());
+		var previousTypes = new ArrayList<>(RelationLoader.getTypes());
+		var previousAttitudes = new ArrayList<>(RelationLoader.getAttitudes());
+		var previousTiers = new ArrayList<>(TierLoader.get());
+		var previousTitles = new ArrayList<>(TitleLoader.getTitles());
+		var previousGuildTypes = new LinkedHashMap<>(GuildLoader.get());
+		try {
+			plugin.loadConfigs();
+		} catch (RuntimeException failure) {
+			// Live factions still reference these definitions until every loader succeeds.
+			RankLoader.getRanks().clear();
+			RankLoader.getRanks().addAll(previousRanks);
+			RelationLoader.getTypes().clear();
+			RelationLoader.getTypes().addAll(previousTypes);
+			RelationLoader.getAttitudes().clear();
+			RelationLoader.getAttitudes().addAll(previousAttitudes);
+			TierLoader.get().clear();
+			TierLoader.get().addAll(previousTiers);
+			TitleLoader.getTitles().clear();
+			TitleLoader.getTitles().addAll(previousTitles);
+			GuildLoader.get().clear();
+			GuildLoader.get().putAll(previousGuildTypes);
+			throw failure;
+		}
 		FactionManager.rebindRanks();
 		FactionManager.rebindDiplomacy();
 		// loadConfigs rebuilds every Title, so point factions at the new copies or hasTitle stops matching.

src/main/java/net/tfminecraft/simplefactions/army/Military.java

@@ -273,9 +273,9 @@ public List<LevyEntry> getLevies() {
 				if(total+count >= subject.getMembers().size()) {
 					count = subject.getMembers().size() - total;
 				}
-				total+=count;
-				if(count > 0) r.setSentToOverlord(count);
-				else continue;
+				count = Math.max(0, count);
+				total += count;
+				r.setSentToOverlord(count);
 			}
 			levies.add(new LevyEntry(subject, total));
 			for(LevyEntry e : subjectLevies) {