fix(semver): Minecraft 1.20 crashing the semver parser

Minecraft doesn't use semver, and as such, sometimes it can cause problems, like in this case: `1.20` is not a valid semver version, but we must deal with it anyway, we've made the `<patch>` component optional, defaulting it to `0` if missing

.forgejo/workflows/publish.yml

@@ -19,7 +19,8 @@ jobs:
     - name: Check Gradle Setup
       run: |
         chmod +x ./gradlew
-        cat ./settings.gradle.kts | sed "s/^.*:minecraft.*$//g" > settings.gradle.kts
+        cat ./settings.gradle.kts | sed "s/^.*:minecraft.*$//g" > settings.gradle.kts1
+        mv settings.gradle.kts1 settings.gradle.kts
     - name: Build
       run: |
         ./gradlew :publishMavenJavaPublicationToFrogMCSnapshotsMavenRepository \

build.gradle.kts

@@ -3,11 +3,11 @@ plugins {
     `java-library`
     id("io.freefair.lombok") version "8.+"
     `maven-publish`
-    id("dev.frogmc.meta-update") version "0.0.1-alpha.3"
+    id("dev.frogmc.meta-update") version "0.0.1-alpha.7"
 }
 
 group = "dev.frogmc"
-version = "0.0.1-alpha.2"
+version = "0.0.1-alpha.14"
 
 repositories {
     maven {
@@ -24,14 +24,17 @@ repositories {
 }
 
 dependencies {
-    implementation(libs.thyroxine){
+    implementation(libs.thyroxine)
-        isTransitive = false
-    }
     compileOnly("org.apache.logging.log4j:log4j-slf4j2-impl:2.22.1")
     compileOnly("org.apache.logging.log4j:log4j-api:2.22.1")
     compileOnly("org.apache.logging.log4j:log4j-core:2.22.1")
+    compileOnly("com.google.code.gson:gson:2.0")
+    compileOnly("com.google.guava:guava:21.0") // Lowest possible version for our code
 
-    api(libs.mixin)
+    api(libs.mixin) {
+        exclude(group = "com.google.code.gson")
+        exclude(group = "com.google.guava")
+    }
     api(libs.mixinextras)
     api(libs.nightconfig)
     api(libs.annotations)

gradle/libs.versions.toml

@@ -1,7 +1,7 @@
 [versions]
 
-thyroxine = "0.0.1-alpha.3"
+thyroxine = "0.0.1-alpha.6"
-nightconfig = "3.7.2"
+nightconfig = "3.7.3"
 mixin = "0.14.0+mixin.0.8.6"
 annotations = "24.1.0"
 mixinextras = "0.3.6"
@@ -16,4 +16,4 @@ mixinextras = { module = "io.github.llamalad7:mixinextras-common", version.ref =
 
 [bundles]
 
-[plugins]
+[plugins]

minecraft/build.gradle.kts

@@ -1,6 +1,6 @@
 plugins {
     java
-    id("dev.frogmc.phytotelma") version "0.0.1-alpha.10"
+    id("dev.frogmc.phytotelma") version "0.0.1-alpha.15"
 }
 
 repositories {
@@ -20,7 +20,9 @@ dependencies {
 }
 
 phytotelma {
-    minecraft("1.21", "1.20.6")
+    minecraft {
+        version = "1.21"
+    }
 }
 
 java {

src/main/java/dev/frogmc/frogloader/impl/FrogLoaderImpl.java

@@ -148,7 +148,7 @@ public class FrogLoaderImpl implements FrogLoader {
 
 	@Override
 	public Optional<ModProperties> getModProperties(String id) {
-		return mods.values().stream().flatMap(m -> m.values().stream()).filter(m -> m.id().equals(id)).findFirst();
+		return mods.values().stream().map(m -> m.get(id)).filter(Objects::nonNull).findFirst();
 	}
 
 	private Collection<String> collectModIds() {

src/main/java/dev/frogmc/frogloader/impl/PluginLoader.java

@@ -1,7 +1,9 @@
 package dev.frogmc.frogloader.impl;
 
+import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.*;
+import java.util.stream.Stream;
 
 import dev.frogmc.frogloader.api.FrogLoader;
 import dev.frogmc.frogloader.api.mod.ModProperties;
@@ -24,7 +26,7 @@ public class PluginLoader {
 
 		List<ModProvider> providers = new ArrayList<>(ServiceLoader.load(ModProvider.class).stream().map(ServiceLoader.Provider::get).toList()); // we need mutability & random access
 		Map<String, Collection<ModProperties>> properties = new HashMap<>();
-		int[] size = new int[providers.size()]; // use a separate size variable to not have to iterate over the list over and over again
+		int[] size = new int[]{providers.size()}; // use a separate size variable to not have to iterate over the list over and over again
 		for (int i = 0; i < size[0]; i++) {
 			ModProvider plugin = providers.get(i); // use random access to work around concurrent access (through modifications during iteration)
 			LOGGER.debug("Found mod provider: {}", plugin.getClass().getName());
@@ -33,6 +35,7 @@ public class PluginLoader {
 			}
 			try {
 				LOGGER.debug("Initialising mod provider: {}", plugin.id());
+				Files.createDirectories(gameDir.resolve(plugin.loadDirectory()));
 				Collection<ModProperties> loadedMods = plugin.loadMods(Discovery.find(gameDir.resolve(plugin.loadDirectory()),
 								plugin::isDirectoryApplicable,
 								plugin::isFileApplicable),
@@ -47,12 +50,14 @@ public class PluginLoader {
 					providers.add(provider);
 					size[0]++;
 				}));
+				modProviders.add(plugin);
 			} catch (Throwable e) {
 				LOGGER.error("Error during plugin initialisation: ", e);
 			}
 		}
 
-		Collection<ModProperties> flattened = properties.values().stream().flatMap(Collection::stream).toList();
+		Collection<ModProperties> flattened = Stream.concat(properties.values().stream().flatMap(Collection::stream),
+				mods.values().stream().map(Map::values).flatMap(Collection::stream)).toList();
 		try {
 			Collection<ModProperties> solved = new ModDependencyResolver(flattened).solve();
 			properties.forEach((s, c) -> c.retainAll(solved));
@@ -61,7 +66,6 @@ public class PluginLoader {
 				c.forEach(p -> map.put(p.id(), p));
 			});
 			modIds.addAll(solved.stream().map(ModProperties::id).toList());
-			modProviders.addAll(providers);
 		} catch (ModDependencyResolver.UnfulfilledDependencyException e) {
 			LoaderGui.execUnfulfilledDep(CrashReportGenerator.writeReport(e, flattened), e, false);
 		} catch (ModDependencyResolver.BreakingModException e) {

src/main/java/dev/frogmc/frogloader/impl/SemVerParseException.java

@@ -4,6 +4,6 @@ import java.io.IOException;
 
 public class SemVerParseException extends IOException {
 	public SemVerParseException(String message) {
-		super("Failed to parse SemVer: " + message);
+		super("Failed to parse SemVer: `" + message + "`");
 	}
 }

src/main/java/dev/frogmc/frogloader/impl/gui/page/UnfulfilledDepPage.java

@@ -35,7 +35,7 @@ public class UnfulfilledDepPage extends JScrollPane {
 				} else {
 					description.append("a Mod with id ").append(entry.dependency());
 				}
-				description.append(" with a version matching range:\n").append(entry.range().toString(" or ")).append("\nNo version is currently available.");
+				description.append(" with a version matching range:\n").append(entry.range().toString(" or ")).append("\nNo matching version is currently available.");
 			}
 			description.append("\nSuggested Solution: Install ")
 					.append(

src/main/java/dev/frogmc/frogloader/impl/mod/ModUtil.java

@@ -15,7 +15,14 @@ public class ModUtil {
 			return "No mods loaded.";
 		}
 
-		Map<String, ModProperties> modIdMap = mods.stream().collect(Collectors.toMap(ModProperties::id, m -> m));
+		Map<String, ModProperties> modIdMap = new HashMap<>();
+
+		for (ModProperties p : mods) {
+			if (!modIdMap.containsKey(p.id()) || modIdMap.get(p.id()).version().compareTo(p.version()) < 0) {
+				modIdMap.put(p.id(), p);
+			}
+		}
+
 		AtomicInteger indent = new AtomicInteger(1);
 		Map<ModProperties, Collection<String>> parentToChildIdMap = getParentMods(mods);
 		Collection<String> containedIds = parentToChildIdMap.values().stream().flatMap(Collection::stream).collect(Collectors.toUnmodifiableSet());

src/main/java/dev/frogmc/frogloader/impl/mod/SemVerImpl.java

@@ -9,12 +9,14 @@ import java.util.regex.Pattern;
 import dev.frogmc.frogloader.api.mod.SemVer;
 import dev.frogmc.frogloader.impl.SemVerParseException;
 import lombok.NonNull;
+import lombok.SneakyThrows;
+import org.jetbrains.annotations.Nullable;
 
 public record SemVerImpl(int major, int minor, int patch, String prerelease, String build) implements SemVer {
 	// Adapted from https://semver.org/#is-there-a-suggested-regular-expression-regex-to-check-a-semver-string
 	private static final Pattern SEMVER_PATTERN = Pattern.compile("^(?<major>0|[1-9]\\d*)\\." +
-																  "(?<minor>0|[1-9]\\d*)\\." +
+																  "(?<minor>0|[1-9]\\d*)" +
-																  "(?<patch>0|[1-9]\\d*)" +
+																  "(\\.(?<patch>0|[1-9]\\d*))?" +
 																  "(?:-(?<prerelease>(?:0|[1-9]\\d*|\\d*[a-zA-Z-][0-9a-zA-Z-]*)" +
 																  "(?:\\.(?:0|[1-9]\\d*|\\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?" +
 																  "(?:\\+(?<buildmetadata>[0-9a-zA-Z-]+(?:\\.[0-9a-zA-Z-]+)*))?$");
@@ -27,7 +29,9 @@ public record SemVerImpl(int major, int minor, int patch, String prerelease, Str
 
 		int major = Integer.parseInt(matcher.group("major"));
 		int minor = Integer.parseInt(matcher.group("minor"));
-		int patch = Integer.parseInt(matcher.group("patch"));
+		// minecraft treats the `patch` component as optional...
+		@Nullable String patchString = matcher.group("patch");
+		int patch = patchString == null ? 0 : Integer.parseInt(matcher.group("patch"));
 		String prerelease = matcher.group("prerelease");
 		String buildmetadata = matcher.group("buildmetadata");
 		return new SemVerImpl(major, minor, patch, prerelease, buildmetadata);