Cache reference timestamps behind an `IVsAsyncFileChangeEx2` watcher for snapshot reuse - #20457

Open

xperiandri wants to merge 25 commits into

dotnet:mainfrom

xperiandri:vs/file-change-watcher

Commits on Sep 26, 2026

Record file-watching design review vs Roslyn

xperiandri
and
Copilot
committed
7f287a9
Add IVsAsyncFileChangeEx2 file change watcher adapter

xperiandri
and
Copilot
committed
6246526
Wire reference file watching into FSharpProjectOptionsReactor

xperiandri
and
Copilot
committed
dc01204
Add FileChangeWatcher unit tests and VS release notes

xperiandri
and
Copilot
committed
b762cc4
Link the file-watching release note to its PR
xperiandri
committed
934979f
Address review: mandatory watcher, voption, illib string helpers
xperiandri
committed
a75cac6
Address review: keep applyBatch on a list, Seq.toList
xperiandri
committed
ed72a8b
Address review: ImmutableArray contract, cancellable batch application
xperiandri
committed
38d0eb5
Add String.StartsWithOrdinalIgnoreCase next to EndsWithOrdinalIgnoreCase
xperiandri
committed
e90d29b
Address review: per-sink batching, timer race, diffed watches, tests
xperiandri
committed
2588bff
Use a second constructor instead of an optional delay parameter
xperiandri
committed
f699bac
Wrap the recording service's doc comment in summary for the cref
xperiandri
committed
04fba82
Drop the reference subscription: Roslyn already covers it
xperiandri
committed
2b8b2e6
Keep reference stamps inside the tracker's watch entries
xperiandri
committed
8b8b21e
Watch each project's reference set for its stamps
xperiandri
committed
45f8373
Read snapshot reference stamps from the tracker instead of stat'ing
xperiandri
committed
1d9adec
Document the stamp consumer and add its release note
xperiandri
committed
09eb975
Expose Dispose directly on the tracker and sort the reactor's opens
xperiandri
committed
fa1910f
Dispose the watcher's agent directly
xperiandri
committed
8d75f80
Call String.StartsWith/EndsWith directly instead of the illib helpers

xperiandri
and
claude
committed
1c3d942
Never cache a reference stamp before its watch is confirmed active

xperiandri
and
claude
committed
f7ab57f
Use Seq.tryFindIndexV instead of the option-returning Seq.tryFindIndex

xperiandri
and
claude
committed
41eec4a
Fix the FS0760 that broke every Windows leg, and stop timing the IsActive tests

xperiandri
and
claude
committed
cdf35a7
Keep one release note, in the version in development

xperiandri
and
claude
committed
a3ec7f5
Watch a reference's appearance and removal, not only its writes

xperiandri
and
claude
committed
c1a2d72

Navigation Menu

Cache reference timestamps behind an `IVsAsyncFileChangeEx2` watcher for snapshot reuse - #20457

Cache reference timestamps behind an `IVsAsyncFileChangeEx2` watcher for snapshot reuse#20457
xperiandri wants to merge 25 commits into
dotnet:mainfrom
xperiandri:vs/file-change-watcher

Commits on Sep 26, 2026

Record file-watching design review vs Roslyn

Add IVsAsyncFileChangeEx2 file change watcher adapter

Wire reference file watching into FSharpProjectOptionsReactor

Add FileChangeWatcher unit tests and VS release notes

Link the file-watching release note to its PR

Address review: mandatory watcher, voption, illib string helpers

Address review: keep applyBatch on a list, Seq.toList

Address review: ImmutableArray contract, cancellable batch application

Add String.StartsWithOrdinalIgnoreCase next to EndsWithOrdinalIgnoreCase

Address review: per-sink batching, timer race, diffed watches, tests

Use a second constructor instead of an optional delay parameter

Wrap the recording service's doc comment in summary for the cref

Drop the reference subscription: Roslyn already covers it

Keep reference stamps inside the tracker's watch entries

Watch each project's reference set for its stamps

Read snapshot reference stamps from the tracker instead of stat'ing

Document the stamp consumer and add its release note

Expose Dispose directly on the tracker and sort the reactor's opens

Dispose the watcher's agent directly

Call String.StartsWith/EndsWith directly instead of the illib helpers

Never cache a reference stamp before its watch is confirmed active

Use Seq.tryFindIndexV instead of the option-returning Seq.tryFindIndex

Fix the FS0760 that broke every Windows leg, and stop timing the IsActive tests

Keep one release note, in the version in development

Watch a reference's appearance and removal, not only its writes