`await` template is not ideal

[teidesu]

few issues with the .await expression i found immediately:

• it doesn't work inside awaited expressions. this is intended looking at the code, but this means i can't do this:

const foo = await getFoo(getBaz().await<tab>)

• it doesn't wrap the awaited expression in parens. i.e.:

const foo = getFoo().await|<tab>

becomes

const foo = await getFoo()|

and i can't chain anything to that awaited value (e.g. (await getFoo()).bar bc it would chain on the getFoo() call instead of the actual awaited value

[ipatalas]

• it doesn't work inside awaited expressions. this is intended looking at the code, but this means i can't do this:

const foo = await getFoo(getBaz().await)

Fair enough. That's a clear bug which I have just fixed. New version should be released within a week or so since I still have few other things to fix there.

• it doesn't wrap the awaited expression in parens. i.e.:
  const foo = getFoo().await|
  becomes

const foo = await getFoo()|
and i can't chain anything to that awaited value (e.g. (await getFoo()).bar bc it would chain on the getFoo() call instead of the actual awaited value

I cannot agree on that. This very much depends on the use case. For example I have never needed that in my experience because I don't like to chain awaited expressions inline. In such case I usually do sth like:

const foo = await getFoo()
const bar = foo.chained().result()

I can imagine people have different opinion on that but I just wanted to show chaining by default is not the best option. Even if you do chain awaited expressions inline I guess it's only small part of all awaited expressions and if parens were always added you'd have to remove them everywhere where they are not needed.

However I have a solution for you anyway. See Custom Templates first and then add a template similar to this:

{
          "name": "chain",
          "description": "",
          "body": "({{expr}}).$1",
          "when": [
              "identifier",
              "expression",
              "function-call"
          ]
},

Then you can do:

const foo = await getFoo().chain|<tab>

which will result in

const foo = (await getFoo()).<cursor>

A small custom snippet solves the problem and you you use it often you can even shorten the name to c or sth.