watch: fix null fileName on windows systems

vnc5 · RafaelGSS · commit 56e8969b65d6 · 2024-01-02T15:05:27.000-03:00
PR-URL: #49891 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/lib/internal/watch_mode/files_watcher.js b/lib/internal/watch_mode/files_watcher.js
@@ -98,7 +98,7 @@ class FilesWatcher extends EventEmitter {
     }
     const watcher = watch(path, { recursive, signal: this.#signal });
     watcher.on('change', (eventType, fileName) => this
-      .#onChange(recursive ? resolve(path, fileName) : path));
+      .#onChange(recursive ? resolve(path, fileName ?? '') : path));
     this.#watchers.set(path, { handle: watcher, recursive });
     if (recursive) {
       this.#removeWatchedChildren(path);

Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ class FilesWatcher extends EventEmitter {`
`98`	`98`	`}`
`99`	`99`	`const watcher = watch(path, { recursive, signal: this.#signal });`
`100`	`100`	`watcher.on('change', (eventType, fileName) => this`
`101`		`- .#onChange(recursive ? resolve(path, fileName) : path));`
	`101`	`+ .#onChange(recursive ? resolve(path, fileName ?? '') : path));`
`102`	`102`	`this.#watchers.set(path, { handle: watcher, recursive });`
`103`	`103`	`if (recursive) {`
`104`	`104`	`this.#removeWatchedChildren(path);`