yeah, that is bad. Should we change/allow to have multiple <metadata> elements perhaps to allow chaining?
That would make sense. Otherwise model properties would have to be used.
Speaking of which I would still advocate for allowing properties to be specified by content in addition to the value attribute:
This makes it easier to do escaping.