[Refactor] don't destructure builtins

[MichaelDeBoey]

Introduced in a3c0ffa, but these are the only places we do this.
In all other places we use Object.*

[ljharb]

I actually prefer to do it everywhere - iow, I think this is going the wrong direction.

[MichaelDeBoey]

@ljharb This is always going to come from Object from now on, so it's unnecessary to destruct Object globally imo

[ljharb]

It's not globally, it's at the top level module scope.

[MichaelDeBoey]

@ljharb Yeah that's what I meant, sorry for the miswording.

My opinion is just: from this point onwards, these methods will always come from Object, so I don't see any point to destruct these methods.
People just seeing entries(abc) will ask themselves where entries is coming from and why Object.entries isn't used (they'll think it's probably a special implementation for some reason), only to find out it actually is used.
When making it explicit directly, people will need less cognitive load to understand where everything is coming from

[ljharb]

That's true, and I suppose I've accepted the risk of environment mutation, so that's not really an argument that applies.

[MichaelDeBoey]

If people are breaking their environment outside of this plugin, than it's their own fault for breaking it imo.

But that's out of scope of this PR I think.
My point still stands: making it explicit where entries comes from will improve readability, especially for newer developers in the field.
Which will in it's turn only cause less help from the OSS community to help and fix bugs themselves

[ljharb]

Rebased to clean up remaining examples of this pattern.