fix(es/transformation): Fix wrong syntax context of variables with the same names as catch params

[CPunisher]

Description:

When hoisting variables, the names of catch params are saved in catch_param_decls permanently in the process. So the following code skips resolving the variables with the same names.

swc/crates/swc_ecma_transforms_base/src/resolver/mod.rs

Line 1558 in e6fc532

if self.catch_param_decls.contains(&id.sym)

I think this is introduced in #4574. I'm not sure why do that because I didn't find related test cases.

Related issue:

• Closes Wrong syntax context of variables with the same names as catch params #9785

[changeset-bot]

🦋 Changeset detected

Latest commit: 5513a60

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

[codspeed-hq]

CodSpeed Performance Report

Merging #9786 will not alter performance

_{Comparing CPunisher:fix/catch-decl (5513a60) with main (5a44c6b)}

Summary

✅ 194 untouched benchmarks