KineticTurtleUpgrade.useTool should be removed and useUp/useDown/swingUp/swingDown should be implemented

useTool causes this unexpected behavior where a turtle will attempt to destroy itself, and makes mining in general not act as expected.

To clarify what is actually going on here, with the help of TheJebForge's testing and explanations:

The Kinetic Augment when equipped as a Turtle upgrade will act as a tool. It provides behavior for turtle.dig() and turtle.attack(). For the Kinetic Augment, these behaviors delegate to kinetic.swing().

Turtles will try to use the left tool first, and then the right tool. In the event that the left tool does not do anything, the right tool will be attempted. That's what's happening in Allymonies' screenshot: turtle.digUp() is called, the Diamond Pickaxe cannot be used (no block there), and the Kinetic Augment gets tried instead.

The problem here is that the fake player's position for the raycast in KineticTurtleUpgrade is slightly wrong. I was originally going to remove the useTool implementation but after understanding the behavior a bit better I am going to try and fix it instead. I will still add a "direction" argument to kinetic.use/kinetic.swing for turtles, or separate digUp/etc. methods, not sure yet

6_4 suggested that this may have been fixed, possibly by 4077b29 ? Would need to test the repro between 1.4.3 and 1.4.4 to confirm

I just checked and this was not fixed; turtles can still mine themselves. I was probably reproducing it incorrectly in some way last time (maybe forgetting to use digUp?)