gopls/internal/server: modify checkGoModDeps condition

- checkGoModDeps should only be run once per file.

Change-Id: Ia3df152240c36843507936eddf16a560c82b5a4b
Reviewed-on: https://go-review.googlesource.com/c/tools/+/730163
Reviewed-by: Hongxiang Jiang <hxjiang@golang.org>
Auto-Submit: Ethan Lee <ethanalee@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>

Author: ethanalee-work

gopls/internal/server/text_synchronization.go

@@ -254,14 +254,16 @@ func (s *server) didModifyFiles(ctx context.Context, modifications []file.Modifi
 	// to their files.
 	modifications = s.session.ExpandModificationsToDirectories(ctx, modifications)
 
+	// TODO: also handle go.work changes as well.
+	uris := make(map[protocol.DocumentURI]struct{})
 	for _, m := range modifications {
-		// TODO: also handle go.work changes as well.
-		if strings.HasSuffix(m.URI.Path(), "go.mod") {
-			if m.Action == file.Create || m.Action == file.Change {
-				s.checkGoModDeps(ctx, m.URI)
-			}
+		if strings.HasSuffix(m.URI.Path(), "go.mod") && (m.Action == file.Create || m.Action == file.Change) {
+			uris[m.URI] = struct{}{}
 		}
 	}
+	for uri := range uris {
+		s.checkGoModDeps(ctx, uri)
+	}
 
 	viewsToDiagnose, err := s.session.DidModifyFiles(ctx, modifications)
 	if err != nil {