Deduplicate watched file events, avoid MatchesFileName (#1308)

Author: jakebailey

internal/project/configfileregistry.go

@@ -239,7 +239,10 @@ func (c *ConfigFileRegistry) tryInvokeWildCardDirectories(fileName string, path
 	configFiles := c.ConfigFiles.ToMap()
 	for configPath, entry := range configFiles {
 		entry.mu.Lock()
-		hasSet := entry.commandLine != nil && entry.commandLine.MatchesFileName(fileName) && entry.SetPendingReload(PendingReloadFileNames)
+		hasSet := false
+		if entry.commandLine != nil && entry.pendingReload == PendingReloadNone && entry.commandLine.MatchesFileName(fileName) {
+			hasSet = entry.SetPendingReload(PendingReloadFileNames)
+		}
 		var projects map[*Project]struct{}
 		if hasSet {
 			projects = maps.Clone(entry.projects.Keys())

internal/project/service.go

@@ -9,6 +9,7 @@ import (
 	"strings"
 	"sync"
 
+	"github.com/microsoft/typescript-go/internal/collections"
 	"github.com/microsoft/typescript-go/internal/core"
 	"github.com/microsoft/typescript-go/internal/ls"
 	"github.com/microsoft/typescript-go/internal/lsp/lsproto"
@@ -296,9 +297,15 @@ func (s *Service) Close() {
 }
 
 func (s *Service) OnWatchedFilesChanged(ctx context.Context, changes []*lsproto.FileEvent) error {
+	seen := collections.NewSetWithSizeHint[lsproto.FileEvent](len(changes))
+
 	s.projectsMu.RLock()
 	defer s.projectsMu.RUnlock()
 	for _, change := range changes {
+		if !seen.AddIfAbsent(*change) {
+			continue
+		}
+
 		fileName := ls.DocumentURIToFileName(change.Uri)
 		path := s.toPath(fileName)
 		if err, ok := s.configFileRegistry.onWatchedFilesChanged(path, change.Type); ok {