Scope construction inside ast visitor

[Boshen]

Inspired by https://blog.sentry.io/how-we-made-javascript-stack-traces-awesome/

I think scope enter / leave callbacks can be moved into the visitor so downstream tools can directly use scopes without effort.

[Boshen]

This is done in commits found in #967