Add @Accessor and @Invoker mixin support#4
Open
logan-mcduffie wants to merge 1 commit intoBuild-9:mainfrom
Open
Add @Accessor and @Invoker mixin support#4logan-mcduffie wants to merge 1 commit intoBuild-9:mainfrom
logan-mcduffie wants to merge 1 commit intoBuild-9:mainfrom
Conversation
Implement support for Mixin @Accessor annotations, allowing mods to generate getter/setter methods for private instance fields on target classes. Key changes: - Add SyntheticClassRegistry to manage Mixin-generated accessor classes - Update BytecodeProvider to handle synthetic class generation - Update ClassProvider to load synthetic classes at runtime - Update LaunchEnvironment with helper for runtime class loading Limitation: Static accessor methods are not supported due to Java requiring method bodies in interface static methods, which conflicts with Mixin's SubType classification system.
Collaborator
|
Synthetics are one issue, but this is a symptom of a larger known issue. Early plugin code, and by extension Mixin, its dependencies, and any generated classes do not exist on the runtime classloader. This is a known caveat, and something we have been working to address in the background, well before the game launched. The upstream version runs within the normal plugin environment as intended, resolving these issues and allowing normal plugins to use Mixins without being loaded as early plugins. The main drawback is that normal plugins are only available after the majority of the game has already been loaded, so we need to decide the best way to preload plugins with mixins. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@Accessorand@Invokerannotations@Accessorenables mods to generate getter/setter methods for private instance fields@Invokerenables mods to call private methods on target classesSyntheticClassRegistryto manage Mixin-generated synthetic classes that don't exist on diskBytecodeProviderandClassProviderto handle synthetic class generation and loadingTechnical Details
Accessor and invoker mixins work by generating synthetic classes at runtime. Since Hytale's
TransformingClassLoaderonly transforms classes that exist on disk, this PR adds infrastructure to:Known Limitation: Static accessor methods are not supported due to Java requiring method bodies in interface static methods, which conflicts with Mixin's SubType classification system. This is a minor limitation since static fields typically have public getters.