Looking good! I love the flavour text - Get 'em!
Some questions about the loot sack: is it a container you select in the world (ie, one already in the vicinity) or is it something you need to drop/spawn to get minions to fill? If you give the gather/drop loot commands, does it pop up automatically at your feet? Can you pick it up and take it with you?
Are you going to include configuration via ini file for things like weight/value ratio and drop/keep potion toggles? I'm a chronic restarter (though not recently, strangely) and I hate having to try to remember all my "default" settings for all my mods.
I could include a configuration .ini now you mention it. I hadn't really seen why anyone'd want to use an ini when they can do it at runtime but yeah I can see now why some people'd prefer it =p - the flavour text is changable from the options too. Currently got "goblin" ("Lookin' fer shinies boss!"), "commander" ("Search the area people!"), "animal trainer" ("Fetch!")
At the moment the moment the loot bag is a static container which:
- Appears automaticlly in front of you when you issue a loot related command and there's no existing bag nearby
- Appears by the minions feet if you tell them to drop what they're carrying and there's no bag nearby
But small minions have trouble reaching the container to open it as a static prop so I'm gonna replace it with a movable item that can be grabbed/moved/psuedo-picked up (It'd add a dummy version with added weight to your inventory, difference being that when you next use a loot command the loot container'd retain the items that were inside it because you "carried" the bag with you instead of threw down a new empty one), I might even try your own trick of adding the value to the bag like you did with your storage sacks so you can pick up the entire bag and just hand it to the merchant at the end
Can't really see any refinements to it I can make. My biggest worry is finding a way to balance the distance at which it decides to create a new bag, too far and the minions might deposit loot off-screen and the player isn't sure where it went, too near and the player might accidentlly delete the old bag and lose loot he wanted to claim, the only other solution would be to have the player cast a spell to summon a bag IF he wants one handy but that'll add more clutter to your (already burgeoning) spellbook when "minimal clutter" is the focus of the mod's design