top level async iterator

[dnalborczyk]

not sure if this is a bug or a feature request - or neither: not sure if async iterators is part of the top-level-await proposal.

const imports = [
  import("./a.js"),
  import("./b.js")
]

for await (const i of imports) {}

// throws: 
// for await (const i of imports) {}
//    ^
// SyntaxError: Invalid or unexpected token
//      at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)

[jdalton]

Patched f13ed28; Related to acornjs/acorn#712.

[dnalborczyk]

@jdalton thanks for the quick implementation!!

uh, I think we need a condition for node v10+ support only in the tests. I'll add it soon...

btw, do you think it needs to be fixed on the babel side of things as well? babel/babel#7637 I can file a bug if needed.

[jdalton]

No need to patch the esm test, I have it fixed locally.
Good catch on the babel side. Pinging them with a bug isn't a bad idea.

Update:

Patched 7abe703.

Update:

v3.0.73 is released 🎉