You're injecting this method into the head of findAmmo, above the vanilla check that the passed itemstack is an instance of shootableitem.

This causes minecraft to crash if a non-shootableitem is passed to findAmmo. myFindAmmo should return an empty itemstack in this case to preserve vanilla behavior.