Re: [PATCH] fsmonitor: invalidate untracked cache after losing history

[Derrick Stolee <stolee@gmail.com>]

On 11/5/2026 9:08 AM, Derrick Stolee wrote:
> On 11/4/2026 5:52 PM, Junio C Hamano wrote:
>> "Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes:
> 
>>>     A user of an internal monorepo reported this behavior, including an
>>>     AI-assisted powershell script to prove how it worked. I used an agent to
>>>     assist in porting that to a Git test script and a code fix, but then
>>>     applied significant human editing and validation on top to get it up to
>>>     my standards.
>>>     
>>>     Thanks, -Stolee
>>
>> Would this be a viable replacement for ik/fsmonitor-untracked-cache-trivial-response
>>
>>   https://lore.kernel.org/git/pull.2247.git.1791127155614.gitgitgadget@gmail.com/
>>
>> that has been left unreviewed for a month?
> 
> Thank you for pointing me at this. I didn't realize there was already a
> patch working on a fix for this issue.
> 
> Ilia's patch has the stronger approach of trying to preserve as much of
> the untracked cache as possible, which has some performance improvements
> over my change (in the "rare" case of the fsmonitor failing).
> 
> Let me first see if we can take that patch instead of mine. I'll start
> some review over there.
Review complete. We should take Ilia's patch instead of mine. Consider this
patch retracted.

Thanks,
-Stolee