Deduplicate watched file events, avoid MatchesFileName

[jakebailey]

These are two tactics which I think will help with #1265.

I've noticed that VS Code can send duplicate events; deduplicate them before handling.

Also, we can avoid the expensive MatchesFileName call if the config file entry is already pending, hopefully saving even more time.

[Copilot]

Pull Request Overview

This PR aims to reduce redundant file event processing and improve performance by skipping unnecessary filename matching when a config entry is already pending reload.

• Deduplicate incoming FileEvent entries before handling them.
• Update ConfigFileRegistry to avoid expensive MatchesFileName calls if a reload is already pending.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
internal/project/service.go	Added sorting and compaction logic in OnWatchedFilesChanged to deduplicate events
internal/project/configfileregistry.go	Refactored tryInvokeWildCardDirectories to only call MatchesFileName once per entry

Comments suppressed due to low confidence (2)

internal/project/service.go:302

• Consider adding unit tests for OnWatchedFilesChanged to verify that duplicate FileEvent entries are properly deduplicated.

		deduped := slices.Clone(changes)

internal/project/configfileregistry.go:242

• Add tests for tryInvokeWildCardDirectories to ensure that MatchesFileName is skipped when pendingReload is not PendingReloadNone and that SetPendingReload behaves correctly.

		hasSet := false