Fix case renames not marking previous casing as missing in time entries #143
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As noticed in vercel/next.js#10351 when a filename's casing is changed both variants of the filename are reported from
getTimeInfoEntries
. This makes users unable rely ongetTimeInfoEntries
as a safe source of truth on case insensitive file systems.This PR attempts to correct this by when an event is triggered for a file that is already in the watcher's files it re-checks that the file is still present with exact casing by doing a
readdir
on the directory the file resides in.The added test cases were tested and previously failing on:
I also tested against the added test case in vercel/next.js#10351 to double check this has the expected behavior.