incompatibility with Pufferfish's Attributes

[Boshigiri]

An error happened when applying mod mixins. Specifically, the HungerManagerMixin in puffish_attributes.mixins.json failed to apply during the DEFAULT phase.

The puffish_attributes mod attempted to inject code using the @at("INVOKE") annotation into the method net.minecraft.world.food.FoodData::tick(Lnet/minecraft/world/entity/player/Player;)V. However, this method had already been merged by com.minenash.action_hunger.mixin.HungerManagerMixin with a higher priority (1000), while the injection in the puffish_attributes mod had a priority of 900. This resulted in the injection failure and subsequently caused the crash.

When the game started, the MixinProcessor tried to apply various mod mixins.
When processing the HungerManagerMixin of the puffish_attributes mod, since the target method tick had been merged by other mods with a higher priority, an InvalidInjectionException was thrown.
This exception was not properly handled in the MixinProcessor and propagated upwards, ultimately leading to the game crash.

[Boshigiri]

Pufferfish's Attributes have solved this issue

[Boshigiri]

No fix

[pufmat]

That incompatibility is caused by mixin injection made by ActionHunger mod that moves some vanilla code. Pufferfish's Attributes mod expects call to PlayerEntity.heal(float) to exist in HungerManager.update(PlayerEntity), but due to @Overwrite it is replaced by a call to a helper method which then calls it:

ActionHunger/src/main/java/com/minenash/action_hunger/mixin/HungerManagerMixin.java

Lines 125 to 129 in 49bd641

	private void heal(PlayerEntity player, String source, float amount) {
	if (Config.debug)
	System.out.println("Heal from " + source + ": " + amount);
	player.heal(amount);
	}

That issue can be easily fixed on the ActionHunger mod side by moving calls to PlayerEntity.heal(float) from helper method to the @Overwrite mixin injection method.