next.js
7351d924 - Turbopack: Remove unused `ignored_subpaths` feature from DiskWatcher (#82210)

Commit
153 days ago
Turbopack: Remove unused `ignored_subpaths` feature from DiskWatcher (#82210) This feature is unused, but I think there are also significant problems with it, which would require work for it to be used: - The `Vec` isn't very scalable and that could be a footgun. It should be an `auto_hash_map::AutoSet` or a `FxHashSet`. - It's unclear that _just_ ignoring watch events is a good idea. I understand that we don't want to allow source code to depend on outputs, but it seems like it would be better to prevent reads in the first place in the `DiskFileSystem` (e.g. pretend the file doesn't exist and emit an issue). This would not require support from the watcher, as we just wouldn't create the invalidator at all. - There's no test coverage of this feature.
Author
bgw bgw
Parents
Loading