TISUnion / Carpet-TIS-Addition

A Carpet mod (fabric-carpet) extension, a collection of carpet mod style useful tools and interesting features
https://carpet.tis.world
GNU Lesser General Public License v3.0
271 stars 38 forks source link

Crash with Lithium in dev environment #177

Closed zly2006 closed 1 week ago

zly2006 commented 1 week ago

Bug description

Crash

Steps to reproduce

using these dependencies in my mod dev environment:

    modImplementation("maven.modrinth:carpet:1.4.128")
    modImplementation("maven.modrinth:carpet-tis-addition:v1.63.0-mc1.20.4")
    modImplementation("maven.modrinth:lithium:mc1.20.4-0.12.1")

They are all the latest version I can find on modrinth. But when I try to start debugging it crashed.

Relevant logs

the mixin error:

Caused by: org.spongepowered.asm.mixin.injection.throwables.InvalidInjectionException: @At("INVOKE") on net/minecraft/entity/Entity::optimizedFastEntityMovement_dontUseThatLargeBlockCollisions with priority 1000 cannot inject into net/minecraft/entity/Entity::adjustMovementForCollisions(Lnet/minecraft/entity/Entity;Lnet/minecraft/util/math/Vec3d;Lnet/minecraft/util/math/Box;Lnet/minecraft/world/World;Ljava/util/List;)Lnet/minecraft/util/math/Vec3d; merged by me.jellysquid.mods.lithium.mixin.entity.collisions.movement.EntityMixin with priority 1000 [INJECT_PREPARE Applicator Phase -> carpet-tis-addition.mixins.json:rule.optimizedFastEntityMovement.EntityMixin from mod carpet-tis-addition -> Prepare Injections -> wrapOperation$bfb000$carpet-tis-addition$optimizedFastEntityMovement_dontUseThatLargeBlockCollisions(Lnet/minecraft/world/World;Lnet/minecraft/entity/Entity;Lnet/minecraft/util/math/Box;Lcom/llamalad7/mixinextras/injector/wrapoperation/Operation;Lnet/minecraft/util/math/Vec3d;)Ljava/lang/Iterable; -> Prepare ->  -> { target: adjustMovementForCollisions(Lnet/minecraft/entity/Entity;Lnet/minecraft/util/math/Vec3d;Lnet/minecraft/util/math/Box;Lnet/minecraft/world/World;Ljava/util/List;)Lnet/minecraft/util/math/Vec3d; }]

Minecraft version

1.20.4

Carpet TIS Addition version

v1.63.0-mc1.20.4

Fabric Carpet version

1.4.128

Check list

Fallen-Breath commented 1 week ago

Please provide a full log, instead of a log snippet

zly2006 commented 1 week ago

Sorry, I found it is not a bug, in dev environments fabric does not load the nested jars so conditional mixin is not loaded to my classpath.