top-level await not working

[dnalborczyk]

top-level await seems to be not working. not sure if that is a regression (never tried it, until now).

node 9.3.0, std/esm v0.18.0

repro steps:

esmrc:

{
  "await": true
}

index.js:

await 1;

cli:

node --require @std/esm index.js

error:

await 1;
^^^^^

SyntaxError: await is only valid in async function
    at createScript (vm.js:80:10)
    at Object.runInThisContext (vm.js:152:10)
    at Module._compile (module.js:624:28)
    at Object.Module._extensions..js (module.js:671:10)

[jdalton]

It looks like I'm only enabling top-level await for ESM. So if you're using the defaults you'd want to make that an index.mjs. I'll expand it to supporting both CJS and ESM though.

[dnalborczyk]

@jdalton I had a suspicion, since the docs are mentioning

A boolean to support top-level await in the main ES module.

but I thought, if it's not meant to work in CJS, I would have suggested to change the wording to emphasize that the "await" flag only works in ES modules.

Anyhow, I tried the above with ESM (default):

index.mjs

await 1;

esmrc:

{
  "await": true
}

cli:

node --require @std/esm index.mjs

... as well as:

index.js

await 1;

{
  "esm": "js",
  "await": true
}

cli:

node --require @std/esm index.js

and both are still failing. :/

[jdalton]

Patched 5743e31 and 82e88cc.

[dnalborczyk]

just tried top-level await with ESM (default mjs and explicit js) and CJS mode on lastest master - and needless to say, both work now! 😃

Thanks for your amazing work @jdalton !!

[jdalton]

@std/esm v0.19.0 is released 🎉