Skip to content

Commit b220568

Browse files
watch: allow listening for grouped changes
1 parent 40ef9d5 commit b220568

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

lib/internal/watch_mode/files_watcher.js

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -17,17 +17,18 @@ const { addAbortListener } = require('internal/events/abort_listener');
1717
const { watch } = require('fs');
1818
const { fileURLToPath } = require('internal/url');
1919
const { resolve, dirname } = require('path');
20-
const { setTimeout } = require('timers');
20+
const { setTimeout, clearTimeout } = require('timers');
2121

2222
const supportsRecursiveWatching = process.platform === 'win32' ||
2323
process.platform === 'darwin';
2424

2525
class FilesWatcher extends EventEmitter {
2626
#watchers = new SafeMap();
2727
#filteredFiles = new SafeSet();
28-
#debouncing = new SafeSet();
2928
#depencencyOwners = new SafeMap();
3029
#ownerDependencies = new SafeMap();
30+
#debounceOwners = new SafeSet();
31+
#debounceTimer;
3132
#debounce;
3233
#mode;
3334
#signal;
@@ -75,17 +76,20 @@ class FilesWatcher extends EventEmitter {
7576
}
7677

7778
#onChange(trigger) {
78-
if (this.#debouncing.has(trigger)) {
79-
return;
80-
}
8179
if (this.#mode === 'filter' && !this.#filteredFiles.has(trigger)) {
8280
return;
8381
}
84-
this.#debouncing.add(trigger);
8582
const owners = this.#depencencyOwners.get(trigger);
86-
setTimeout(() => {
87-
this.#debouncing.delete(trigger);
88-
this.emit('changed', { owners });
83+
if (owners) {
84+
for (const owner of owners) {
85+
this.#debounceOwners.add(owner);
86+
}
87+
}
88+
clearTimeout(this.#debounceTimer);
89+
this.#debounceTimer = setTimeout(() => {
90+
this.#debounceTimer = null;
91+
this.emit('changed', { owners: this.#debounceOwners });
92+
this.#debounceOwners.clear();
8993
}, this.#debounce).unref();
9094
}
9195

0 commit comments

Comments
 (0)