`export default class {}` breaks scope analysis

[Thomas Chetwin]

The lack of .id in an export ClassDeclaration causes a throw in scope analysis:

https://runkit.com/embed/rz8r6io0kclh (compares with and without providing a name)

The throw originates from:

ast-types/lib/scope.js

Line 240 in a7eaba5

 addPattern(path.get("id"), bindings); 

I suspect the fix might be the same as for FunctionDeclaration a couple of lines before, haven't verified this yet:

ast-types/lib/scope.js

Line 235 in a7eaba5

 node.id !== null) { 

Edit: I have a patch for this locally with a test, seeking approval to send the PR.

[Max Heiber]

pinging about this

[Ashley Claymore]

This appears to have been fixed by #467