This area is surprisingly tricky. The problem is that we're going from non-async code into async code and we need to catch any errors that happen within. Async stuff is just syntactic sugar around promises so that's why we call the async run function non-async and use catch explicitly. You can't use async at the top level in a file, or at least if you do it doesn't behave as you'd want it to.

Your solution here looks like it should work, but I'd like to test it to make sure. If errors are still reported correctly then LGTM.

Ok, I've tried it out (didn't take long) and the error handling is working exactly as it was before, so I'm happy with this and it gets us one eslint rule closer.