Skip to content

Fix singlejar issue#8

Open
Xytronix wants to merge 3 commits into
IroriPowered:pre-releasefrom
Xytronix:pre-release
Open

Fix singlejar issue#8
Xytronix wants to merge 3 commits into
IroriPowered:pre-releasefrom
Xytronix:pre-release

Conversation

@Xytronix
Copy link
Copy Markdown
Contributor

@Xytronix Xytronix commented May 7, 2026

Summary

When an earlyplugin JAR also declares a Main class in a singlejar, MixinPluginClassLoader.hyinit$prioritizeLocalModClass was forcing a re-load of every class
through the mod's PluginClassLoader. Because HyinitClassLoader had already defined the same class, this produced two copies with separate static state.

My change skips the prioritization for singlejars so the OrbisGuard earlyplugin still gets their classes prioritized, but singlejars don't get duplicated.

Hytale dependency bumped to latest pre-release 2026.04.30-b4f6a911e and the project version to 0.1.4.

Type of change

  • Bug fix (non-breaking)
  • New feature (non-breaking)
  • Breaking change (requires documentation/migration)
  • Chore / refactor
  • Documentation only

Implementation details

  • src/main/java/cc/irori/hyinit/mixin/impl/MixinPluginClassLoader.java extend the existing OrbisGuard-compat condition from meta.isEarlyPlugin() to meta.isEarlyPlugin() && !meta.hasMainClass().

Checklist

  • Code compiles successfully
  • Pull request does not contain unrelated changes
  • Changes follow this project's code style (Use spotlessApply when necessary)
  • Changes are tested or will be tested after submission

@Xytronix Xytronix changed the title Pre release Fix singlejar issue May 7, 2026
@Xytronix Xytronix closed this May 7, 2026
@Xytronix Xytronix reopened this May 7, 2026
@Xytronix Xytronix force-pushed the pre-release branch 3 times, most recently from aa85d9a to 66ac236 Compare May 17, 2026 10:01
- Auto-patch legacy YYYY.MM.DD-sha ServerVersion manifests on load
- Strip whitespace from semver strings via @ModifyVariable
- Catch legacy format in SemverRange.fromString, return wildcard
- Use correct server version 0.5.0-pre.8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants