Start a new pull request in StackBlitz Codeflow.

• vitejs-vite-ttd4av (Open reproduction)

It feel like counter-intuitive

Hi, let's discuss this bug, when we load some files whose location is outside of the root, we will call watcher.add to include these files, watch the changes of these files.But When we rename these files, we will trigger 'old file delete event' and 'new file create event', because we haven't include the new file in watcher, we can't catch the 'new file create event', and can't trigger HMR.Related code as below:

export function ensureWatchedFile(
  watcher: FSWatcher,
  file: string | null,
  root: string,
): void {
  if (
    file &&
    // only need to watch if out of root
    !file.startsWith(withTrailingSlash(root)) &&
    // some rollup plugins use null bytes for private resolved Ids
    !file.includes('\0') &&
    fs.existsSync(file)
  ) {
    // resolve file to normalized system path
    watcher.add(path.resolve(file))
  }
}

Solution: Add the top directory of the outside file path to chokidar.

It seems to be related to caching, and using the following configuration options can temporarily solve this problem.

export default defineConfig({
    server: {
        fs: {
            cachedChecks: false,
        }
    }
});

The external file-level listener seems to only trigger the unlink event when performing a rename operation, and does not trigger the add event. This may be due to cache invalidation.

Solution: Add the top directory of the outside file path to chokidar.

@HawtinZeng It seems that there is a high cost associated with modifying the monitored path to be the nearest common parent path of the external path and root path. I think that when handling external module paths (ensureWatchedFile), synchronously updating the cache seems to restore everything to normal 🤔.

It seems to be related to caching, and using the following configuration options can temporarily solve this problem.

export default defineConfig({
    server: {
        fs: {
            cachedChecks: false,
        }
    }
});

The external file-level listener seems to only trigger the unlink event when performing a rename operation, and does not trigger the add event. This may be due to cache invalidation.

Solution: Add the top directory of the outside file path to chokidar.

@HawtinZeng It seems that there is a high cost associated with modifying the monitored path to be the nearest common parent path of the external path and root path. I think that when handling external module paths (ensureWatchedFile), synchronously updating the cache seems to restore everything to normal 🤔.

Hi, I have tried your method, it didn't work.Maybe a better solution is offer an option array to define where store external files, then we can watch these paths.

Or we can add the new file path into watcher while resolved new file path, It's much better.

It seems this is caused by #15712. I guess it's happening because of inconsistent values here.

I hope this is related otherwise I'll create a new issue. Vite now breaks when I'm trying to create a new file and import it anywhere. Same for moving files. And it's the same issue with "Failed to resolve import". The last version where this worked correctly is 5.0.13. Anything above(and including) 5.1.7 breaks in this particular way.