.... helpful criticism ....
- I like it .. it is easy to read and to follow ... keep it that way
a few people .. don't like gotos .. I don't care as long as it works

a few things I would like to say, but this is just my opinion

- be careful with setting the wait time to 1, player with higher ping rates might be in trouble with this
- btw wait times .. i like to define a few different Wait vars within the header .. so it is easier to tweak later
eg Wait for Event Macro, Wait for Drag, Wait for Drop ....
- ExEvent's without propper Wait often cause trouble !
- OFFSET clicks .. as you said would be a good idea .. just in csae someone moves the gumps
- WaitGump Failsafe .. how about changing to wait a shorter time .. if it isn't working ..
try to open Gump again instead of stop the script ?
- finditem %ingot C_ , %ResourceSecure
if #findstack <
25 ( 25 - %ingotsinbag )
{
display You have used up all of your ingots.
halt
}
eg
you have .. 20 Ingots in your Backpack ... so you need to grab 5 to get back on 25 in backpack...
so you check your ResourceBox and you see there are only 20 left .... enough to fulfill the needs but
you script stops :-)
- I like version numbers in FileNames to make it easier to handle different versions and to easily see
what is what ... While in Beta I just use build numbers ... later versions ...
- also always a good idea to check, if the char is still alive (#CHARGHOST) and logged in (#CLILOGGED)
... just to make sure the script is not messing around

Keep going ... and have fun