[Chore]: Remove `metadata` fields from user API classes where it no longer exists in the spec protobuf

[kabir]

Following on from #484 (more specifically #484 (comment)), we should remove metadata fields from all spec/ classes (and any MapStruct Mappers where we might have some ignore entries) that don't have such a field in the corresponding protobuf