Carlos Fernandez

creating a feat section for this purpose probably isn't correct, allowing a reselection of the slot and showing an "empty slot" for the current grants if there's no selection will...

You should be able to paste a gif directly and it'll play inline

Is there anything else this is good for? For example, can this handle the witches cauldron feat? A rule element needs to be as general as possible, otherwise hardcoding (such...

The concern would be that if it can't be CraftingEntry and must be its own entry, making it a rule element that can alter the quantity of a crafting attempt,...

Forgetting that the same argument applies to munitions crafter.... If all you want is mundane ammo crafting, and want to take the easiest route to it, you could add "munitions-crafter"...

More so that you don't technically *need* automation for munitions crafter either. You can add 10 bullets to your inventory, give them the infused trait, and manually deduct the reagent....

Oh don't worry, that already has wonky behavior. But field discovery is assigned via a path, so you'd assign the field discovery to the munitions crafter entry, not the alchemist...

you'll probably need to special case ammo to default to the batch size (or perhaps all newly added items can do that?). But also I guess make sure only mundane...

Looks fine to me mostly, only gotcha is that if you're going to add min level and mundane restrictions in the RE instead of in code, it might better to...

Hmm, on second though, a predicate could also predicate on the item type. If we're going that far, we might as well add a migration for the other field crafts...