Fix 10: Add copied version jar to classpath for securejarhandler minecraft module

The copied version jar (1.20.1-forge-47.4.20.jar) is now on -cp so
securejarhandler can scan it and create the minecraft module. The jar
name does not match the ignoreList pattern forge- (starts with 1),
so it is included in the module layer. This ensures PreparableReloadListener
and other vanilla classes are findable in the minecraft module.
This commit is contained in:
SashegDev
2026-07-13 16:05:28 +00:00
parent d4b6ea081e
commit 6cbf8a6549
@@ -63,19 +63,18 @@ public class LaunchCommandBuilder {
// always use ModLauncher as main class (version.json might have // always use ModLauncher as main class (version.json might have
// BootstrapLauncher which breaks with Java module system) // BootstrapLauncher which breaks with Java module system)
if ("forge".equals(loaderType) || "neoforge".equals(loaderType)) { if ("forge".equals(loaderType) || "neoforge".equals(loaderType)) {
// Ensure securejarhandler can create the "minecraft" module: // Ensure securejarhandler can create the "minecraft" module.
// It looks for versions/<versionId>/<versionId>.jar. // The Forge installer only creates versions/<versionId>/<versionId>.json,
// The Forge installer only creates the JSON there, not the jar. // NOT the jar. We copy the vanilla version jar there so
// We copy the vanilla version jar (e.g. versions/1.20.1/1.20.1.jar) // securejarhandler finds it and creates the "minecraft" module.
// to that location so securejarhandler finds it and creates the
// "minecraft" module (identified by Automatic-Module-Name: minecraft
// in the jar's MANIFEST.MF).
ensureVersionJarForForge(); ensureVersionJarForForge();
// Build classpath from manifest libraries only (no version jar on -cp). // Build classpath from manifest libraries + the copied version jar.
// Putting the vanilla jar on -cp would create automatic module _1._20._1 // The copied jar (e.g. 1.20.1-forge-47.4.20.jar) does NOT match the
// (from filename 1.20.1.jar) which conflicts with the "minecraft" module // "-DignoreList" pattern "forge-" (it starts with "1", not "forge-"),
// created by securejarhandler → split-package ResolutionException. // so securejarhandler will include it in the module layer.
// It also won't create automatic module "_1._20._1" because the
// filename is "1.20.1-forge-47.4.20.jar", not "1.20.1.jar".
String classpath; String classpath;
if (manifest != null) { if (manifest != null) {
classpath = buildClasspathFromManifest(manifest, false); classpath = buildClasspathFromManifest(manifest, false);
@@ -86,6 +85,18 @@ public class LaunchCommandBuilder {
classpath = buildVanillaClasspath(); classpath = buildVanillaClasspath();
} }
// Add the copied version jar to classpath (if it exists).
// securejarhandler scans classpath JARs and creates modules.
// The jar name "1.20.1-forge-47.4.20.jar" is NOT in ignoreList,
// so it becomes a proper module with Automatic-Module-Name: minecraft.
Path copiedVersionJar = instance.getPath().resolve("versions")
.resolve(getVersionId()).resolve(getVersionId() + ".jar");
if (Files.exists(copiedVersionJar) && isValidJar(copiedVersionJar)) {
String sep = System.getProperty("os.name").toLowerCase().contains("win") ? ";" : ":";
classpath = copiedVersionJar.toAbsolutePath().toString() + sep + classpath;
System.out.println(ZAnsi.green(" Added version jar for minecraft module: " + copiedVersionJar.getFileName()));
}
command.add("-cp"); command.add("-cp");
String cpFile = writeClasspathFile(classpath); String cpFile = writeClasspathFile(classpath);
command.add(cpFile); command.add(cpFile);