wrong scope for destructuring catch leads to incorrect treeshake

[kzc]

• Rollup Version: rollup v2.50.0
• Operating System (or Browser): n/a
• Node Version (if applicable): v16.1.0

Repro

$ cat destructuring-catch-scope.js 
let a = 1;         // mistakenly dropped
let def = "PASS2"; // mistakenly dropped
try {
    throw [ "FAIL2", "PASS1" ];
} catch ({ [a]: b, 3: d = def }) {
    let a = 0, def = "FAIL3";
    console.log(b, d);
}

Expected Behavior

$ cat destructuring-catch-scope.js | node
PASS1 PASS2

Actual Behavior

$ cat destructuring-catch-scope.js | rollup --silent
try {
    throw [ "FAIL2", "PASS1" ];
} catch ({ [a]: b, 3: d = def }) {
    let a = 0, def = "FAIL3";
    console.log(b, d);
}

$ cat destructuring-catch-scope.js | rollup --silent | node
[stdin]:3
} catch ({ [a]: b, 3: d = def }) {
                ^

ReferenceError: a is not defined

[lukastaegert]

Good catch 🙄 Fix at #4108. That one will also fix the other catch scoping issue.