Started by GitHub push by Geolykt103620ccba675bf76d0ac738235e3c743e352ea563f620ccba675bf76d0ac738235e3c743e352ea563frefs/remotes/origin/main620ccba675bf76d0ac738235e3c743e352ea563f620ccba675bf76d0ac738235e3c743e352ea563frefs/remotes/origin/main620ccba675bf76d0ac738235e3c743e352ea563f620ccba675bf76d0ac738235e3c743e352ea563frefs/remotes/origin/mainhttps://github.com/stianloader/Micromixin.gitfalse#1033887442691micromixin #103103falsefalse103407FAILURE1777827762792https://stianloader.org/jenkins/job/micromixin/103/micromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasWrapOperationAnnotation.javamicromixin-testing-suite-neo/build.gradlemicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/WrapOpMixins.javamicromixin-transformer/build.gradlemicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/WrapOpMixinsTarget.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/HandlerContextHelper.javamicromixin-testing-suite-neo/WrapOpMixinsTarget.classmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/ModifyReturnValue.javamicromixin-backports/build.gradlemicromixin-test-j8/pom.xmlmicromixin-annotations/src/main/java9/module-info.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/MicromixinTestNeo.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/ASMUtil.javamicromixin-testing-suite-neo/micromixin-test-config-eclipse.jsonmicromixin-annotations/build.gradlemicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/Operation.javamicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.javamicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/locals/ArgumentCaptureContext.javamicromixin-testing-suite-neo/micromixin-test-config-gradle.jsonmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperationRuntime.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinMethodStub.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/PrintUtils.javamicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.json4d357f580448eae8b9f91dd9427e0ffaaa90810d1777151694000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.de[CI-SKIP] Initial skeleton work for WrapOperation Let's hope this doesn't trigger a build, but the build should probably fail anyways so maybe that's not a concern in the first place. Technically the only thing remaining is documentation and a lot of testing. Otherwise, this one seems to work right now. Well, at least only for INVOKE instructions. Not for GETx/PUTx et al though - for now (until the need arises). 2026-04-25 23:14:54 +02004d357f580448eae8b9f91dd9427e0ffaaa90810d[CI-SKIP] Initial skeleton work for WrapOperationaddmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javaaddmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/WrapOpMixins.javaeditmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.javaaddmicromixin-testing-suite-neo/WrapOpMixinsTarget.classeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/MicromixinTestNeo.javaeditmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/ModifyReturnValue.javaeditmicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsoneditmicromixin-backports/build.gradleeditmicromixin-transformer/build.gradleaddmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperationRuntime.javaaddmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/WrapOpMixinsTarget.javaeditmicromixin-test-j8/pom.xmleditmicromixin-testing-suite-neo/micromixin-test-config-eclipse.jsoneditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/locals/ArgumentCaptureContext.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/ASMUtil.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinMethodStub.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/PrintUtils.javaeditmicromixin-testing-suite-neo/build.gradleaddmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/Operation.javaeditmicromixin-testing-suite-neo/micromixin-test-config-gradle.jsoneditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/HandlerContextHelper.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasWrapOperationAnnotation.javaeditmicromixin-annotations/build.gradleeditmicromixin-annotations/src/main/java9/module-info.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasWrapOperationAnnotation.javamicromixin-transformer/src/test/java/org/stianloader/micromixin/test/ASMUtilTest.java.gitignoremicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/WrapOpMixins.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/WrapOpMixinsTarget.javamicromixin-testing-suite-neo/WrapOpMixinsTarget.classmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinRedirectAnnotation.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/MicromixinTestNeo.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/TestReport.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/annotations/IncludeClasses.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/ASMUtil.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasModifyReturnValueAnnotation.javamicromixin-annotations/src/main/java/com/llamalad7/mixinextras/sugar/Cancellable.javamicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyConstantAnnotation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/locals/ArgumentCaptureContext.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyVariableAnnotation.javamicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsonmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/annotations/ExpectSignaller.javaa165b394326018dfd9e69a6b03001aac6d7bb8af1777326643000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.de[CI-SKIP] Properly implement Cancellable support for WrapOperation Also implements Cancellable support for the Modify annotations. Also probably fixed argument capture in general (for the Modify annotations and WrapOperation). Still CI-SKIP because javadocs aren't ready yet and there still needs to be some polishing around the rough edges. Also WrapOperation still doesn't support chaining injectors. Will do so later, maybe. 2026-04-27 23:50:43 +0200a165b394326018dfd9e69a6b03001aac6d7bb8af[CI-SKIP] Properly implement Cancellable support for WrapOperationdeletemicromixin-testing-suite-neo/WrapOpMixinsTarget.classeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyVariableAnnotation.javaeditmicromixin-transformer/src/test/java/org/stianloader/micromixin/test/ASMUtilTest.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/locals/ArgumentCaptureContext.javaedit.gitignoreeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasWrapOperationAnnotation.javaeditmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javaeditmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/sugar/Cancellable.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/WrapOpMixins.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasModifyReturnValueAnnotation.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/WrapOpMixinsTarget.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinRedirectAnnotation.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/annotations/ExpectSignaller.javaeditmicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsoneditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyConstantAnnotation.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/ASMUtil.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/MicromixinTestNeo.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/TestReport.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/annotations/IncludeClasses.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinHeader.javamicromixin-transformer/src/test/java/org/stianloader/micromixin/test/SelfmodificationTest.java1a25a7a540bcd9b62518388720f401caf1767f531777378112000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.de[CI-SKIP] Forbid mixin classes from transforming themselves This is the most primitive kind of sanity test that can be done, the rest becomes slightly difficult due to mixins potentially being able to transform other mixin classes - provided someone would wire that correctly, which I doubt anyone already does - but w/e. CI-SKIP since the actual work is still very much WIP. Also, I'm currently committing this commit from my college since I currently have to waste my time doing /something/ - whatever that will be. 2026-04-28 14:08:32 +02001a25a7a540bcd9b62518388720f401caf1767f53[CI-SKIP] Forbid mixin classes from transforming themselvesaddmicromixin-transformer/src/test/java/org/stianloader/micromixin/test/SelfmodificationTest.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinHeader.javamicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/WrapOpMixins.javamicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/WrapOpMixinsTarget.javamicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsoncdd138ccefab590434bc3522b2c29694683758b41777381367000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.de[CI-SKIP] Improve javadocs, add test Well, I'll go home now, so I'll stash the work I have on my laptop onto git so I can pull it back to my PC once I'm back home. Figuring out how to handle argument capture and whether they should be throwing will be something to figure out later. I myself am biased towards printing a warning by default and having some kind of strict mode for Mixin parity. Although ... this is not the first time I've been contemplating this, I need to think about it -.- The way MixinExtras handles it's sugar and forbids it from being intermingled with ordinary arguments kinda feels like an unnecessary requirement in my opinion. 2026-04-28 15:02:47 +0200cdd138ccefab590434bc3522b2c29694683758b4[CI-SKIP] Improve javadocs, add testeditmicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsoneditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/WrapOpMixins.javaeditmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.javaeditmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/WrapOpMixinsTarget.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasWrapOperationAnnotation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/VirtualConstructorMergeAnnotation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasModifyReturnValueAnnotation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/api/MixinTransformer.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/api/MixinVendor.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyConstantAnnotation.javamicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javamicromixin-testing-suite-neo/micromixin-test-config-gradle.jsonmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/locals/ArgumentCaptureContext.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinMethodStub.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyVariableAnnotation.java620ccba675bf76d0ac738235e3c743e352ea563f1777827564000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.deFinalize work on WrapOperation, introduce vendor compatibility system WrapOperation will not support chaining for now, but that can be implemented at a later point in time - when the need arises that is, but that is rather unlikely in an ecosystem where I'm the only person doing anything. 2026-05-03 18:59:24 +0200620ccba675bf76d0ac738235e3c743e352ea563fFinalize work on WrapOperation, introduce vendor compatibility systemeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyConstantAnnotation.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/api/MixinTransformer.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinMethodStub.javaeditmicromixin-testing-suite-neo/micromixin-test-config-gradle.jsoneditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinModifyVariableAnnotation.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasModifyReturnValueAnnotation.javaeditmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/wrapoperation/WrapOperation.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/api/MixinVendor.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/locals/ArgumentCaptureContext.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/mixinsextras/MixinExtrasWrapOperationAnnotation.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/VirtualConstructorMergeAnnotation.javagithttps://stianloader.org/jenkins/user/geolyktGeolykt