fix: fix scope lookup with optional catch binding (#2)

pionxzh · web-flow · commit 673da0e1567e · 2024-07-18T20:53:17.000+08:00
diff --git a/src/scope.ts b/src/scope.ts
@@ -209,7 +209,7 @@ export default function scopePlugin(fork: Fork) {
       }
     }
     if (ScopeType.check(node)) {
-      if (namedTypes.CatchClause.check(node)) {
+      if (namedTypes.CatchClause.check(node) && node.param !== null) {
         // A catch clause establishes a new scope but the only variable
         // bound in that scope is the catch parameter. Any other
         // declarations create bindings in the outer scope.

Original file line number	Diff line number	Diff line change
`@@ -209,7 +209,7 @@ export default function scopePlugin(fork: Fork) {`
`209`	`209`	`}`
`210`	`210`	`}`
`211`	`211`	`if (ScopeType.check(node)) {`
`212`		`- if (namedTypes.CatchClause.check(node)) {`
	`212`	`+ if (namedTypes.CatchClause.check(node) && node.param !== null) {`
`213`	`213`	`// A catch clause establishes a new scope but the only variable`
`214`	`214`	`// bound in that scope is the catch parameter. Any other`
`215`	`215`	`// declarations create bindings in the outer scope.`