top-level-var will silent circular dependency TDZ error

* Describe what esbuild is doing incorrectly and what it should be doing instead.
doing: esbuild's top-level-var will silence circular dependency tdz
should be doing: support not enable top-level-var or disable top-level-var by default

* Provide a way to reproduce the issue. The best way to do this is to demonstrate the issue on the playground (https://esbuild.github.io/try/) and paste the URL here. A link to a minimal code sample with instructions for how to reproduce the issue may also work. Issues without a way to reproduce them may be closed.
https://github.com/hardfist/circular-dependency-cases/pull/1 lots of edge cases here

Start with the circular-dependency reproduction in hardfist/circular-dependency-cases, especially the linked pull request and its edge cases. Then trace how esbuild handles the top-level-var option for circular JavaScript modules; done means the option no longer silently hides TDZ errors, or is disabled by default as the issue requests.

Written by the indexing model from the issue text.