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

Open

xperiandri wants to merge 22 commits into

dotnet:mainfrom

xperiandri:vs/file-change-watcher

Commits on Sep 9, 2026

Record file-watching design review vs Roslyn

xperiandri
and
Copilot
committed
3c0b044
Add IVsAsyncFileChangeEx2 file change watcher adapter

xperiandri
and
Copilot
committed
655fde6
Wire reference file watching into FSharpProjectOptionsReactor

xperiandri
and
Copilot
committed
868829c
Add FileChangeWatcher unit tests and VS release notes

xperiandri
and
Copilot
committed
ff16eba
Link the file-watching release note to its PR
xperiandri
committed
7492e5f
Address review: mandatory watcher, voption, illib string helpers
xperiandri
committed
eaf9b9a
Address review: keep applyBatch on a list, Seq.toList
xperiandri
committed
b6ec1fc
Address review: ImmutableArray contract, cancellable batch application
xperiandri
committed
00aa0e1
Add String.StartsWithOrdinalIgnoreCase next to EndsWithOrdinalIgnoreCase
xperiandri
committed
d489320
Address review: per-sink batching, timer race, diffed watches, tests
xperiandri
committed
fccdb18
Use a second constructor instead of an optional delay parameter
xperiandri
committed
a41d8bb
Wrap the recording service's doc comment in summary for the cref
xperiandri
committed
86b0178
Drop the reference subscription: Roslyn already covers it
xperiandri
committed
6038036
Keep reference stamps inside the tracker's watch entries
xperiandri
committed
8665b27
Watch each project's reference set for its stamps
xperiandri
committed
9edc359
Read snapshot reference stamps from the tracker instead of stat'ing
xperiandri
committed
0968ee6
Document the stamp consumer and add its release note
xperiandri
committed
711a612
Expose Dispose directly on the tracker and sort the reactor's opens
xperiandri
committed
aa2feb5
Dispose the watcher's agent directly
xperiandri
committed
6c20e94
Call String.StartsWith/EndsWith directly instead of the illib helpers

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

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

xperiandri
and
claude
committed
6617485

Navigation Menu

Uh oh!

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 22 commits into
dotnet:mainfrom
xperiandri:vs/file-change-watcher

Commits on Sep 9, 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