Change 'Swing_Animation' to a more comprehensive 'Animation' Item Component

Both the animation field in the consumable component and the forward_movement field in the kinetic_weapon component should be removed from those parent components and into the swing_animation component, which be changed to just 'animation'. Within this new animation component, 'type' is changed to 'primary' for swinging, and a new field called 'secondary' is added, setting/overriding the animations for shield blocking, trident charging, eating, spear charging, crossbow charging, goat horn use, etc. (Notably, these are all already animations that can be set within a consumable component, but only in the context of a consumable item! One cannot overwrite the animations of those items without making them consumable. And why?)
The 'Forward_Movement' field in the kinetic_weapon component also makes little sense being there. When set to a custom number, it affects any instance of the stab animation, not just in the kinetic attack spears use. This includes things like opening a chest or left clicking to punch. It would make much more sense in this new 'animation' component. Currently, the only instance of the forward movement tag not affecting the spear animation is when it is applied as a consumable animation. And the only reason for that is because Kinetic_weapon overrides that animation for its behavior anyway. This would be fixed if a consumable's animation and a kinetic_weapon's use animation took from the same field, a simple secondary animation field replacing both.