Didn't nullcheck fishing spot
You can check !myPlayer().isAnimating() before fishing
Don't need to have the condition on the else if statement because you already check it with the first condition.
conditional sleeps that last 250_000 is generally bad practice. I'd do 10-20 seconds max.
To clean up your sleep you can check out the ConditionalSleep2 class https://osbot.org/api/org/osbot/rs07/utility/ConditionalSleep2.html