Just started using the bot again since it is out of beta. Running on the trial now to test it out some more.
Regarding what people have said about afk logging, I believe there should be a toggle for people to decide. If the option is Enabled, then the bot uses current logic and automatically logs out after however many minutes of downtime you have it set to. However, if disabled, let the bot sit there and just do nothing until the salvage spot comes back. With the Logout Timer plugin, we can extend our afk time from the 5 minute limit to 25 minutes and this plugin doesn't typically interfere with RL bots from what I have seen. Let the user decide if they want the bot to continue even though the salvage is down for 10 / 15/ 20 minutes if unlucky.
I am not sure what the bot has been coded to do in regards to the cargo hold on the ship. I haven't personally seen it do anything with the cargo hold in the 50 minutes of the trial I have run. I do have a few suggestions to add to the bot to improve the efficiency of the bot related to the cargo hold.
My suggestion for how to handle cargo hold has three different options.
1. Bot salvages and has NPC salvage until the cargo hold is full. Once the cargo hold is full, bot finishes it's inventory of salvage and cleaning. Then it proceeds to pull out inventories of salvage from the cargo hold until it is all cleaned and fully empty.
2. Bot salvages and has NPC salvage until the salvage spot(s) are down. If the bot detects spot(s) down, it cleans what it has in it's inventory and then proceeds to clear out and clean the cargo hold salvage.
3. Implement logic for both the above
I am just tossing out suggestions here. Again, I do not know if you already have this addressed or not as I have just started the bot. But I wanted to get the suggestions in while they are on the top of my head.