Immediate thing I notice is you’re using the depreciated animation setup. Many many many people use this, and you could still keep using it, but as it’s depreciated it’d be best to move on. Deprecating LoadAnimation on Humanoid and AnimationController
Don’t set the humanoids health. As a general rule you can observe FF using humanoid:TakeDamage() instead. While maybe you’re not using FF, it’s smart to use that anyway.