It might be simpler if the Keep keyword was only used for cards that stay in hand, since the majority of attachments are NOT discarded when they trigger. As far as I could see, only Altruism is an exeption (Mad Dog, Festering Guts and Death Meld as well but I do not think they're worth mentionning). And the description of Armor Of Faith seems clear enough for me. Edit: I forgot Surestrike Blessing
And Force Field is an edge case, since it discards itself by reducing its duration. I'd like to see I'm Here About The Job added to the exception list (discard when you play an attack card). That would let us remove the discard line from Backstab and make IHAtJ a little less annoying in EttSC modules. (And now I have to bug the wiki folks to add the redirect so that card link works.)
Dislike -- unless "Attach" becomes a keyword which explains via tooltip that it essentially Keeps until duration ends (or someone manually removes it), like you suggest, unless of course the card says otherwise. (I don't have any suggested exact wording off the top of my head.) Expanding "Attach" into such a keyword could greatly simplify things, that's for sure -- both for char *and* terrain attachments -- and in that case, I'd say Like. $F^ }