fix(es/minifier): Do not add vars if eval exists

[kdy1]

Related issue:

• Closes Minify: multiple same temp symbols declarations #8886

[changeset-bot]

⚠️ No Changeset found

Latest commit: 885d299

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Austaras]

Seems like this PR introduced lots of unresolved vars. Maybe we should make name mangler working properly in presence of eval?

[kdy1]

Actually name mangler cannot work properly in presence of eval, because it cannot modify/add/remove a name even theoretically

[kdy1]

swc-bump:

• swc_ecma_minifier
• swc_core

[Austaras]

I see. But there's still lots of changes in absence of eval, why?

[kdy1]

@Austaras Nice catch! Thank you. I was confused about the changes.

[swc-bot]

Automated review comment generated by auto-rebase script

[Austaras]

Wait, I see the problem now.

[Austaras]

This code maybe not of best quality, but I hope you get what I mean.

[kdy1]

Thank you so much 👍

[swc-bot]

Automated review comment generated by auto-rebase script