[pluginutils] attachScopes should add scope object to ForStatement.test and ForStatement.update

[eight04]

• Rollup Plugin Name: pluginutils
• Rollup Plugin Version: 4.0.0
• Rollup Version:
• Operating System (or Browser): Windows 10
• Node Version: 13.8.0
• Link to reproduction (⚠️ read below): https://repl.it/repls/WeightyUnnaturalBoolean

Expected Behavior

With the following code,

for (let a = 0; a < 10; a++) {
  console.log(a);
  let b = 10;
}

There are two different scopes. ForStatement.body (i.e. BlockStatement) can access variable a and variable b. However, ForStatement.test and ForStatement.update (i.e. BinaryExpression and UpdateExpression) can only access variable a.

Actual Behavior

attachScopes only attaches scope object to BlockStatement

Additional Information

[shellscape]

Thanks for opening an issue. Pull requests to resolve this issue are welcome. This is unlikely to get a fix without community contribution.

[stale]

Hey folks. This issue hasn't received any traction for 60 days, so we're going to close this for housekeeping. If this is still an ongoing issue, please do consider contributing a Pull Request to resolve it. Further discussion is always welcome even with the issue closed. If anything actionable is posted in the comments, we'll consider reopening it. ⓘ