String index out of range during AP

[shedaniel]

With the fix to #518 (1e1aa7f), Mixin tries to pretty print a field (asm field) with SignaturePrinter (works only with methods)

This doesn't crash in Mixin 0.8.4.

Type.getReturnType(desc) and Type.getArgumentTypes(desc): Expects a description with ):

Mixin/src/main/java/org/spongepowered/asm/util/SignaturePrinter.java

Line 86 in 155314e

this(name, Type.getReturnType(desc), Type.getArgumentTypes(desc));

TypeUtils uses it:

Mixin/src/ap/java/org/spongepowered/tools/obfuscation/mirror/TypeUtils.java

Line 277 in 155314e

return new SignaturePrinter("", descriptor).setFullyQualified(true).toDescriptor();

TypeHandleASM uses it for anonymous classes for fields:

Mixin/src/ap/java/org/spongepowered/tools/obfuscation/mirror/TypeHandleASM.java

Line 224 in 155314e

if (TypeHandleASM.compareElement(field.name, TypeUtils.getJavaSignature(field.desc), name, type, matchCase)) {

Mixin: (0.8.5 on 1.18 ForgeGradle)

@Mixin(targets = "net.minecraft.server.level.ServerPlayer$2")
public class ServerPlayerEntityMixin {
    @Shadow
    ServerPlayer f_143458_;
}

[RubixDev]

I'm encountering this issue with Forge using Architectury. Is there maybe a way I could set it to use version 0.8.4 or the Fabric fork for now?