Hi Bennyspa - nice effort so far!!
I would highly recommend TM's runebook subs that can be found
Here.
If you would entertain an organizational suggestion I would recommend setting all of your variable values at the beginning of the script to make it a little more user-friendly. Also, consider using subroutines within the script with a loop of Gosubs as the mainloop of the script.
So, for example:
set %chicken C_AG ; NPC to kill
set %weapon XXXXXXX ;Weapon ID
set %featherType VLK
set %featherBag #backpackId
set %RunebookHOME XXXXXXX ;Secure Container Runebook
set %RunebookNPC XXXXXXX ;NPC Location Runebook
set %SecureId XXXXXXX ;House Secure ID
Mainloop:
Gosub Recalltochickenspot
Gosub Findchicken
Gosub Etc,yougetthepoint
goto Mainloop ;yes there are better ways but for illustrative purposes, here is a goto ;)
As for logic, I notice that you don't have anything in place to move to a chicken when you find one, it assumes you are standing next to the chicken when you kill it. You could scan for the chicken, then when you find it pathfind to it then head to your kill/skin/loot loop.
Just a couple of thoughts to start, hope this is helpful!!
X