I used a decompiler to take a look at your code and there are a few things I can see that could use some fixing up before applying for scripter status.
1) OSbot's API methods can fail and so you should be checking if they return true before sleeping after calling them. Example:
if (bank.depositAll()) {
sleep;
}
2) You should be using conditional sleeps instead of just random sleeps so that the script waits till the condition has passed.
3) I don't know why you're declaring String arrays every time you want to interact with something.
NPC banker = (NPC)this.getNpcs().closest(new String[]{"Banker"});
should be
NPC banker = (NPC)this.getNpcs().closest("Banker");
And even then, it's a good idea to store things as variables and even final variables if you know the value is never going to change.
the keyword final means exactly what it sounds like, whatever value you declare that variable to is FINAL, it can not be changed later on in your code. Most IDEs give an error when you try to redefine a final variable anyway.