Keep existing provider settings

Author: SkyBlade1978

.github/workflows/ci.yml

@@ -126,9 +126,9 @@ jobs:
           if-no-files-found: error
           retention-days: 30
           path: |
-            build/libs/OreSpawn-4.1.0.112021.jar
-            build/libs/OreSpawn-4.1.0.112021-sources.jar
-            build/libs/OreSpawn-4.1.0.112021-javadoc.jar
+            build/libs/OreSpawn-4.1.1.112021.jar
+            build/libs/OreSpawn-4.1.1.112021-sources.jar
+            build/libs/OreSpawn-4.1.1.112021-javadoc.jar
             build/release/SHA256SUMS
             CHANGELOG.txt
 

CHANGELOG.txt

@@ -1,3 +1,10 @@
+Version 4.1.1.112021
+
+* Keep existing provider settings when importing older ore files for the first
+  time, instead of replacing them with the older settings.
+* Describe preserved settings as retained rather than verified in the upgrade
+  report. Invalid files are still checked by the normal provider loader.
+
 Version 4.1.0.112021
 
 * Clarify dimension-wide material controls and replace light-equivalent custom

README.md

@@ -10,7 +10,7 @@ It gives mods and modpacks one place to configure ores, deposit shapes, optional
 rock strata and geomes, provider-owned underground fluid deposits, biome
 palettes and world materials, flat bedrock, and bounded ore retrogen.
 
-This branch builds target-qualified version `4.1.0.112021`: the OreSpawn 4.1.0
+This branch builds target-qualified version `4.1.1.112021`: the OreSpawn 4.1.1
 feature set for Minecraft 1.12.2 and Forge. It adds stable generation identity,
 non-loading geology sampling, per-resource background control, existing-world
 provider merge policy, an Ore Sources arbiter for equivalent provider ores, a

docs/MIGRATION.md

@@ -81,6 +81,12 @@ legacy-ID behaviour.
 Migration is non-destructive. OreSpawn writes `config/orespawn-worldgen.json`
 only when that target does not already exist and retains every source file.
 
+An existing `config/<modid>-orespawn.json` override is also kept unchanged,
+even when legacy files are imported for the first time. The upgrade report
+records that the existing provider settings were retained, not validated.
+An unfinished or invalid override is left for the normal provider validator;
+migration does not replace it with defaults or older settings.
+
 When `config/mineralogy-geomes.json` exists, OreSpawn imports the Mineralogy 6
 profile directly, updates its schema marker, and records `migrated_from`.
 

docs/VERSIONS.md

@@ -52,7 +52,7 @@ Examples:
 | Minecraft | Loader | Target | Example full OreSpawn version |
 | --- | --- | ---: | --- |
 | 1.10.2 | Forge | `110021` | `4.0.6.110021` |
-| 1.12.2 | Forge | `112021` | `4.1.0.112021` |
+| 1.12.2 | Forge | `112021` | `4.1.1.112021` |
 | 1.13.2 | Forge | `113021` | `4.0.6.113021` |
 | 1.20.6 | Forge | `120061` | `4.0.6.120061` |
 | 1.21.11 | Forge | `121111` | `4.0.6.121111` |

gradle.properties

@@ -30,7 +30,7 @@ mod_vendor=Minecraft Mod Development
 mod_license=LGPL-2.1
 mod_license_name=GNU Lesser General Public License, Version 2.1
 mod_license_url=https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt
-mod_version=4.1.0.112021
+mod_version=4.1.1.112021
 mod_group=zone.moddev.mc.orespawn
 mod_authors=SkyBlade1978, dshadowwolf, the MMD Team
 mod_description=Configurable, provider-driven terrain, ore, and deposit generation.

gradle/release/artifacts.gradle

@@ -81,7 +81,7 @@ tasks.register('verifyReleaseConfiguration') {
     description = 'Validates the target-qualified release, API, schemas, reports, and publishing identity.'
 
     doLast {
-        if (project.mod_version != '4.1.0.112021'
+        if (project.mod_version != '4.1.1.112021'
                 || project.mod_group != expectedMavenGroup) {
             throw new GradleException("Unexpected OreSpawn release version: ${project.mod_version}")
         }
@@ -108,8 +108,8 @@ tasks.register('verifyReleaseConfiguration') {
                 'src/main/java/zone/moddev/mc/orespawn/worldgen/LegacyOs3ProfileMigration.java',
                 'README.md', 'CHANGELOG.txt'
         ].each { path ->
-            if (!file(path).getText('UTF-8').contains('4.1.0.112021')) {
-                throw new GradleException("Authoritative release location does not contain 4.1.0.112021: ${path}")
+            if (!file(path).getText('UTF-8').contains('4.1.1.112021')) {
+                throw new GradleException("Authoritative release location does not contain 4.1.1.112021: ${path}")
             }
         }
         if (!file('docs/API.md').getText('UTF-8').contains('orespawn@[4.0.6,5.0.0)')) {
@@ -173,7 +173,7 @@ tasks.register('verifyReleaseArtifacts') {
                 }
                 ['src/test/', 'src/biomeIntegrationTest/', 'src/migrationIntegrationTest/',
                  'src/clientIntegrationTest/', 'src/os1AbiFixture/', 'src/os3AbiFixture/',
-                 'agent-notes/', 'surfaceprobe', 'migrationprobe', 'clientprobe',
+                 'agent-notes/', 'AGENTS.md', 'AGENT.md', 'surfaceprobe', 'migrationprobe', 'clientprobe',
 				 'codec-conflict/', 'LegacyApiConsumer', 'OreSourceEditorProbeBridge',
                  'org/junit/', 'org/mockito/', 'net/bytebuddy/'].each { forbidden ->
                     if (candidateZip.entries().any { it.name.contains(forbidden) }) {

src/main/java/com/mcmoddev/orespawn/compat/LegacyOs3Bridge.java

@@ -335,7 +335,7 @@ private static void migrateConfigDirectory(Path configDirectory) {
 		}
 		for (Map.Entry<String, JsonObject> provider : providers.entrySet()) {
 			try {
-				writeAtomicIfChanged(configDirectory.resolve(provider.getKey() + "-orespawn.json"), provider.getValue());
+				writeProviderIfAbsent(configDirectory.resolve(provider.getKey() + "-orespawn.json"), provider.getValue());
 			} catch (IOException failure) {
 				REPORT.add("provider_write_failed=" + provider.getKey() + ":" + failure.getClass().getSimpleName());
 				LOGGER.error("Could not write migrated OS3 provider {}", provider.getKey(), failure);
@@ -792,6 +792,16 @@ private static int metadata(String block, String state) {
 		return 0;
 	}
 
+	static void writeProviderIfAbsent(Path destination, JsonObject value) throws IOException {
+		// An existing OS4 file is a pack's choice, even on its first legacy import.
+		if (Files.isRegularFile(destination)) {
+			REPORT.add("provider_preserved=" + destination.getFileName());
+			return;
+		}
+
+		writeAtomicIfChanged(destination, value);
+	}
+
 	private static void writeAtomicIfChanged(Path destination, JsonObject value) throws IOException {
 		byte[] data = (GSON.toJson(value) + System.lineSeparator()).getBytes(StandardCharsets.UTF_8);
 		if (Files.isRegularFile(destination) && Arrays.equals(Files.readAllBytes(destination), data)) {
@@ -833,7 +843,8 @@ private static void writeHumanUpgradeReport(Path destination) throws IOException
 			String lower = row.toLowerCase(java.util.Locale.ROOT);
 			if (lower.startsWith("config_source=") || lower.startsWith("resource_loaded=")
 					|| lower.startsWith("config_registered=")) sources.add(row);
-			if ((lower.startsWith("provider_written=") || lower.startsWith("provider_unchanged="))
+			if ((lower.startsWith("provider_written=") || lower.startsWith("provider_unchanged=")
+					|| lower.startsWith("provider_preserved="))
 					&& !lower.contains("migration-report")) providers.add(row);
 			if (lower.contains("_failed=") || lower.contains("_rejected=")
 					|| lower.contains("_ignored=") || lower.contains("_unresolved=")
@@ -842,12 +853,12 @@ private static void writeHumanUpgradeReport(Path destination) throws IOException
 			}
 		}
 		List<String> lines = new ArrayList<>();
-		lines.add("OreSpawn 4.1.0.112021 Upgrade Report");
+		lines.add("OreSpawn 4.1.1.112021 Upgrade Report");
 		lines.add("================================");
 		lines.add("");
 		lines.add("RESULT: Legacy OreSpawn configuration was consumed and translated for OS4.");
 		lines.add("- Legacy sources read: " + sources.size());
-		lines.add("- OS4 provider files written or verified: " + providers.size());
+		lines.add("- OS4 provider files written or retained: " + providers.size());
 		lines.add("- Unique items requiring review: " + warnings.size());
 		lines.add("");
 		lines.add(warnings.isEmpty()

src/main/java/zone/moddev/mc/orespawn/OreSpawn.java

@@ -52,7 +52,7 @@ public class OreSpawn {
 
 	public static final String MODID = "orespawn";
 	public static final String NAME = "OreSpawn";
-	public static final String VERSION = "4.1.0.112021";
+	public static final String VERSION = "4.1.1.112021";
 
 	private static final Logger LOGGER = LogManager.getLogger();
 

src/main/java/zone/moddev/mc/orespawn/worldgen/LegacyMineralogyProfileMigration.java

@@ -152,7 +152,7 @@ private static void writeUpgradeReport(Path worldRoot, Path configDirectory,
 		Path report = worldRoot.resolve("serverconfig/orespawn-upgrade-report.txt");
 		List<String> missing = missingBlocks(igneous, metamorphic, sedimentary);
 		List<String> lines = new ArrayList<>();
-		lines.add("OreSpawn 4.1.0.112021 Upgrade Report");
+		lines.add("OreSpawn 4.1.1.112021 Upgrade Report");
 		lines.add("================================");
 		lines.add("");
 		lines.add("RESULT: Existing Mineralogy " + identity.version + " world detected.");