Mixins silently fail to apply to libraries such as Brigadier and DFU

Mixins which target libraries such as Authlib, Brigadier and DataFixerUpper do not apply.

A simple example of how to reproduce this is below:

private int testCmd(CommandContext<ServerCommandSource> ctx) throws CommandSyntaxException {
        ServerPlayerEntity spe = ctx.getSource().getPlayer();
        ctx.getSource().sendFeedback(new LiteralText(((GameProfileTest) spe.getGameProfile()).getUuid().toString()), false);
        return 1;
}

public interface GameProfileTest {
    UUID getUuid();
    Optional<String> getName();
}

@Mixin(GameProfile.class)
public abstract class GameProfileMixin implements GameProfileTest {
    @Shadow public abstract UUID shadow$getId();
    @Shadow public abstract String shadow$getName();

    @Override
    public UUID getUuid() {
        return this.shadow$getId();
    }

    @Override
    public Optional<String> getName() {
        return Optional.ofNullable(this.shadow$getName());
    }
}

When the example command is executed, it will fail because of a ClassCastException which means the mixin into GameProfile failed.

My testing has shown this occurs on both the client and server environment.

Main on the client seems to refer to Authlib before MinecraftClient is started in Main.

[LoganDark]

@LoganDark Is this fixed now?

Indeed - #630 completely fixed all issues mixing into game libraries. In fact, https://github.com/LoganDark/fabric-command-hider was developed on a draft version of the PR.

I guess I’m just lucky as heck.

This issue is now as good as resolved 😃

[liach]

@LoganDark Is this fixed now?