…ions
Previously when there was only a single result, we'd bypass VS Code and compute the target editor ourselves which led to #5894
This way, we always let VS Code determine the target editor.
Fixes #5894

We've moved to VS Code's goToLocation command, which flashes the range, but leaves the selection at just the start.

... but looking at current editor first.
We don't have a source document/position when the server calls `goToLocation` so it needs the original implementation. We just shouldn't be doing it in the case where we have a set of results, even if there is only one in there.

We can't try to search for a range in a document if it might not be open yet, so we need to wait on the whole operation not just the equality check.

DanTup deleted the fix-go-to-command branch