feat: add stricter eslint rules for typescript

[ajhenry]

Pull Request

Proposed Changes

While we were pairing on #162, we found that we didn't add an await keyword to the function running the auth check. To prevent this from happening in the future, I added stricter eslint rules for typescript

This also fixes all the problems that arise from the addition of these rules. In general, we're looking pretty good 🥨

Readiness Checklist

Author/Contributor

• If documentation is needed for this change, has that been included in this pull request
• run npm run lint and fix any linting issues that have been introduced
• run npm run test and run tests
• If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

• Label as either bug, documentation, enhancement, infrastructure, maintenance, or breaking

[jmeridth]

😍

[zkoppert]

✨
My only question on this one was whether this just ran locally or was enforced by a linting workflow but I found the npm lint job in the status checks so looks good! 👍🏻

[ajhenry]

@zkoppert luckily next comes with eslint support 😄