[pluginutils] attachScopes ignores variables declared in for-of statement

[eight04]

• Rollup Plugin Name: pluginutils
• Rollup Plugin Version: 4.0.0
• Rollup Version:
• Operating System (or Browser): Windows 10 x64
• Node Version: v13.8.0
• Link to reproduction (⚠️ read below): https://repl.it/repls/BasicMidnightblueAmoebas#index.js

Expected Behavior

The local scope should contain variable a.

Actual Behavior

The local scope doesn't contain variable a.

Additional Information

It has been reported once but the issue is closed:
#113

[shellscape]

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

[eight04]

I took a look at the test file, it seems that the ast is hand-crafted. Do I have to find some ways to build them manually? or can we generate the AST by parsing readable source code?

[lukastaegert]

My guess is that they were created by pasting some code here: https://astexplorer.net/ Alternatively of course you could just run acorn in your test.

[eight04]

Can we merge #551 first? I will start working on for-loop issues after that.

[eight04]

Relate: #548, #549