1)Waypoints: Dont use Stand waypoints for everything! Stand waypoints force the character to step exactly the square you made the waypoint. If you're going to the hunting spot or inside of it, use Walk/Nodes. This waypoints are more "randomizable" and if another character steps right in the place where you made the waypoint it will skip to the next one (contrary to the stand waypoint in which your character will wait untill he can reach that exact position).
2)Refill Nuthing: This is a common error from unexperienced scripters and often seen in low level scripts. Withdrawing 0 gold or buying 0 potions. Use if statements to check if you have to refill or not.
By doing this kind of easy questions you will make your script more efficient and less detectable (no one will ever withdraw 0 gold).
//Do i have to withdraw money?
if cash > 0 then
//yes i Do
//Nah! Let's just go back to hunt
3) Extremely complicated Setups: Setups should be clear and straight forward. So this are some things you should avoid doing.
a) creating an action to change the destination of the loot for the name of the backpack. It's always better to use numbers in case inside one backpack you have another of a different color.
b) using complicated variable notation. avoid things like
use short names that best describe the variable and write using camel case!
thenameofthepotion = 'mana potion'
NAMEYOUR_POT = "mana Potion"
are more readable. even separating with "_" like Mana_Potion.
manaPotion = "Strong Mana Potion",
healthPotion = "Ultimate Healing Potion"