-
Notifications
You must be signed in to change notification settings - Fork 12.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Top Level Await #25988
Comments
@kitsonk I don't think we can safely implement TLA until at least the variant is decided upon - otherwise how exports work in modules with TLA is liable to change (and thereby typechecking). I get that it's great in a REPL context - but at the top level of a module with exports it is much less straightforward! |
I understand that it is likely more complicated than just allowing it, because variant A or variant B would have significant impacts on CFA. From a usefulness perspective, the proposed optional constraint of only allowing it in modules without @weswigham is it worth at least having a tracking issue at this stage of the proposal? |
Has there been an movement on this issue? @RyanCavanaugh, I see you added the "Waiting for TC39" label. What are you waiting on? The advancement to stage 3? The compromise listed above by @kitsonk seems like a good one that would solve the vast majority of use-cases without an impact on type checking semantics. |
We're waiting on stage 3 or 4 since there are possible runtime semantics implications. "This so reasonable that surely TC39 won't come up with some other behavior" has bitten us enough that we're not really willing to risk it when it comes to runtime behavior. |
The proposal has gone Stage 3. |
@RyanCavanaugh Any plans for implementation? |
We can probably put the parsing and typechecking behavior into the backlog at some point, but be aware that ultimately the way TLA executes comes down to the module loader. Webpack can enable it pretty easily by modifying the bundled module loader (although probably not quite to spec, since implementation to spec requires inspecting promise internals to skip event loop turns), but in the context of the cjs loader or pre-TLA esm loader (as we usually consider we emit for), there's really no way to downlevel TLA (although I think We can probably at least tag this as |
Allowing it through, with no downlevel emit would be perfect for my use case. Treating it like |
I agree with @kitsonk - there are use-cases today that would benefit from parsing and type-checking without any downlevel with
|
The part of lexer and parser seems nearly be done... Is there any progress branch or tracking on this one? |
Is this in typescript@next (dev20191004)? It seems like TLA works when I have "target": "esnext" (emits await at top level). It does not work when I have "target": "es5" even though I have "module": "esnext". Is this going to emit TLA downlevel? If it's not, it isn't going to be very useful with webpack 5. Unless I'm missing something. I can ignore the error with // @ts-ignore but it emits "yield" instead of "await". |
@RyanCavanaugh @DanielRosenwasser this mentioned Milestone 3.7. Is this actually available to use in the latest TS builds? If so, is there something i need to do to enable this? Thanks! |
I spoke to Daniel at TSConf. He said it wasn't going to make 3.7 and he needed to update the IP. |
Related: accounting for top-level for-await tc39/proposal-top-level-await#133 |
|
Search Terms
top level await
Suggestion
The ability to utilise
await
directly at the top level of a module or a script. Currently the compiler does not support this, though it is becoming increasing possible to accomplish it in certain runtimes.Top level await is currently a Stage 2 TC39 proposal. (I was unable to find any tracking issue for implementation, therefore this issue)
There was this issue #20923, which was marked as a question, but really was this feature request.
Use Cases
There are many patterns the are emerging that require top level
await
, many which are documented in the TC39 Proposal. The current workaround that is seen a lot in the wild is wrapping anasync
iife. There are lots instances where the main thread of program wants to utiliseawait
to load resources.The value of having top level
await
in Node.js REPL and Chrome debugger is now allowed. There is an outstanding issue to allow it in ts-node.I know the core team is really really adverse to implementing things that haven't reached Stage 3, but there are sufficient patterns in the wild that allow this and an increasing number for situations where it can be allowed, that allowing it under a flag feels like something that might be entertained until it is delivered under Stage 4. If we in user-land choose the potential 👣 🔫 at our own risk, so be it.
Examples
The following:
Could be rewritten as:
Checklist
Technically, this would allow syntax which would change the runtime behaviour of JavaScript and allows new expression level syntax which is currently not permitted (but specifically reserved by TC39 for this purpose).
My suggestion meets these guidelines:
The text was updated successfully, but these errors were encountered: