This CL squashes the following commits from master, to fix broken lsp
tests due to error message and parser recovery changes for 1.20.

In order of submission:
 - CL 425362 (mdempsky): call Interface.Complete in unified importer
 - CL 425497 (gri): disable a test so we can change the parser error
 - CL 425956 (adonovan): support regular expressions in Diagnostics
   tests
 - CL 425594 (gri): re-enable a test with adjusted error message
 - CL 427654 (gri): disable some tests for std lib changes
 - CL 427736 (gri): handle new error message
 - CL 428054 (gri): disable some tests for std lib changes
 - CL 428055 (gri): use regexp to match wanted result

Also: add back the x/tools replace directive to gopls/go.mod, so that
tests will pass. It will be removed in a subsequent CL.

For golang/go#54842

Co-authored-by: Matthew Dempsky <mdempsky@google.com>
Co-authored-by: Robert Griesemer <gri@google.com>
Co-authored-by: Alan Donovan <adonovan@google.com>
Change-Id: I125b9c3fe23de8eb2e130f32b1b7b65b2885e0e2
Reviewed-on: https://go-review.googlesource.com/c/tools/+/427904
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Robert Findley <rfindley@google.com>

…n load

CL 417576 externalized shouldLoad tracking into a map, which was used to
trigger a reload and cleared once reload completes. Unfortunately, it
overlooked the fact that we may also reload the entire workspace (via
reinitialization). In this case, we should clear newly loaded IDs from
the shouldLoad map, so that they are not subsequently loaded again.

For golang/go#54842
Updates golang/go#54473

Change-Id: I26f49552cae502644142dc4a4e946294db37f6f7
Reviewed-on: https://go-review.googlesource.com/c/tools/+/424074
Run-TryBot: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Alan Donovan <adonovan@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>
(cherry picked from commit e55fb40)
Reviewed-on: https://go-review.googlesource.com/c/tools/+/428596
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>

For golang/go#54842

Change-Id: I7382b86d1dce34a57631a6276335fded2f42cf1a
Reviewed-on: https://go-review.googlesource.com/c/tools/+/428597
Run-TryBot: Robert Findley <rfindley@google.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Reviewed-by: Nooras Saba‎ <saba@golang.org>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

Remove the replace directive and update x/tools.

For golang/go#54842

Change-Id: Ie68524bd64e8065702c75a3125c4239310f446c5
Reviewed-on: https://go-review.googlesource.com/c/tools/+/428598
gopls-CI: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nooras Saba‎ <saba@golang.org>
Run-TryBot: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

That includes the fix reviewed in https://go.dev/cl/429515

Remove the replace statement again for release.

For golang/go#54842

Change-Id: I80e9f98eadff82eee7d6d82b8a6a4c30ee108ac7
Reviewed-on: https://go-review.googlesource.com/c/tools/+/429516
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>