Make hasInterpretedResults not be async

[robertbrignull]

A tiny improvement I thought of when doing #1697 but didn't / forgot to include in that PR.

This is something I did when trying to pull out the getContextValue method and mock things, which didn't turn out well (see comment on that PR) but this small bit could be nice and could make things easier to mock and test in the future.

Checklist

• CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
• Issues have been created for any UI or other user-facing changes made by this pull request.
• [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

[robertbrignull]

Thanks for the review but I've changed my mind about whether changing async to sync is actually an improvement. I now don't think it's worth it. It either makes no difference or it removes flexibility. Also if we do ever want to do this change in the future it's very easy, so I don't think it's worth merging this now.