Get rid of separate `metadata_pkg`

[ablaom]

I don't understand why we need a separate metadata_pkg when we have metadata_model, as both take a model type as first argument. We should just move all the traits into the latter.

I was recently confused to find is_wrapper trait, in metadata_pkg, before I realised "pkg" does not necessarily mean global to the package, despite the name.