@parcel/transformer-babel: remove core-js dep?

[corwin-of-amber]

🐛 bug report

It seems that this dependency is redundant:

parcel/packages/transformers/babel/package.json

Line 32 in c690f91

"core-js": "^3.2.1",

Searching the code base, I only found core-js in the tests.

🔦 Context

core-js is particularly harmful due to zloirock/core-js#548, zloirock/core-js#611, zloirock/core-js#708.

Any chance it can be removed?

[DeMoorJasper]

@corwin-of-amber there is nothing harmful about asking for some kind of sponsorship for an open source project.

That being said there's the issue of core-js no longer being maintained which will lead to it being replaced at some point...

[corwin-of-amber]

That is a bit subjective, but in that case the decision is yours.

At any rate, I did not find that @parcel/transformer-babel is even utilizing core-js at all. Am I wrong?

[mischnic]

It's used by @babel/preset-env: https://babeljs.io/docs/en/next/babel-preset-env.html#corejs

[corwin-of-amber]

Oh, I missed that! Sorry and thanks for the ref @mischnic!

[DeMoorJasper]

Can we close this, as it's not really an issue and a real dependency? The only way around this I see is to enforce all parcel users to install core-js and babel locally besides parcel which would not change anything.

core-js will eventually be replaced as it's no longer maintained but for now I don't see any issue in leaving it as is. I assumed it was abandoned but apparently someone else took over maintaining it

[mischnic]

core-js will eventually be replaced as it's no longer maintained but for now I don't see any issue in leaving it as is.

If it becomes a problem, this is on Babel to fix.