Cool, I can't test, but I can give you some pointers.
To fully utilize the power of SRL, you should learn how to implement multiplayer. This way, there will be much less bannage.
Secondly, a couple of procedures can be shortened. For all your Mouse declarations, there is no need to have a random wait after it. Mouse already puts a random wait after the procedure's finished, so it is a bit redudant. For MouseMovements, instead of how you have it now, you can use a case, like so:
SCAR Code:
Procedure MouseMovements;
Begin
Case Random(10) of
1:begin
MMouse(92,130,80,100)
Wait(200+random(200))
end;
2:begin
MMouse(86,351,75,165)
Wait(200+random(200))
end;
3:begin
MMouse(321,378,175,175)
Wait(200+random(200))
end;
4:begin
MMouse(501,210,80,100)
Wait(200+random(200))
end;
5:begin
MMouse(654,374,10,100)
Wait(200+random(200))
end;
End;
Therefore, a lot shorter. Also, I noticed ClickATab and RandomActionProc can be shortened a bit like this too.
Thirdly, Random(x) returns a random value, BUT it returns it from 0 to 1 less than the number you put in. For example, Random(5) will give you a number from 0-4, and NOT 5! Therefore, in AttackMon, RightoLeft will always give a 0or a 1, so we will never left-click on a monster.
Fourth of all, there is already a SetRun procedure in SRL. No need for enablerun.
Sorry if this sounds harsh, but it is meant as constructive critiscism! You will learn from your mistakes
So just take advice, and I know you can become a great scripter!