Repro

{
  "rules": {
    "@typescript-eslint/return-await": ["error", "always"],
  }
}

async function one() {}
async function one() {}
async function two() {}
async function three() {}

async function problem() {
  return (await one()) 
    ? two()
    : three();
}

Expected Result

async function problem() {
  return await ((await one()) 
    ? two()
    : three());
}

or:

async function problem() {
  return (await one()) 
    ? await two()
    : await three();
}

I think I prefer the second?

Actual Result

return await await await await await await await await await await (await one()) ? two() : three();

Yes, that's ten of them.

Additional Info

This new code is valid, possibly not even wrong? I don't know the binding order on await foo ? bar : baz;. It does, however, trigger "unnecessary await", pretty amusingly.

Full eslintrc / package.json (templates, without this specific lint enabled, at least, at time of writing):
https://github.com/snyk/unless-overloaded/blob/master/.eslintrc.json
https://github.com/snyk/unless-overloaded/blob/master/package.json

Versions