The flashing yellow is annoying when food is in the offhand and it can't be eaten. I still want to know how many hunger points/health it grants when it can be eaten, but that info is useless when it cant be eaten.

I understand that this might not be worth it to implement considering the size of the request when you have other things to do.

This makes sense. Goes hand-in-hand with how the health stuff works.

does "inedible" is food level is greater than equal to 20?

does "inedible" is food level is greater than equal to 20?

Yes, but should still display if FoodComponent.isAlwaysEdible() is true.

This is what Minecraft uses in Item.use, we should do the same thing:

if (player.canConsume(item.getFoodComponent().isAlwaysEdible())) {

}