My question is if we want to preserve the descriptor?

This is all in our code. However, I think for-in  + assign would be a bit more useful than only doing own properties. Still it is not clear how the metadata propagates from the loader to the internal loader and back out. I think the metadata object is supposed to be kept around (so that === would work).

In this PR I would just change this to for-in + assign.