vfs: RealFSProvider.watchFile ignores the listener and unwatchFile leaks it

Offen

#66,223 erstellt am 23.9.2026 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@trivikr arbeitet bereits daran.

Seit 23.9.2026.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Beschreibung

vfs

Version

main

Platform

All

Subsystem

vfs

What steps will reproduce the bug?

import { mkdtempSync, watchFile, writeFileSync, unwatchFile } from "fs";
import { create, RealFSProvider } from "node:vfs";

const d = mkdtempSync("/tmp/v-");
const f = create(new RealFSProvider(d)).mount() + "/f";
const cb = () => console.log("fired");

watchFile(f, { interval:50 }, cb);

setTimeout(() => writeFileSync(f, "x"), 50);
setTimeout(() => {
  unwatchFile(f, cb);
  console.log("done, should exit");
}, 200);

How often does it reproduce? Is there a required condition?

Always

What is the expected behavior? Why is that the expected behavior?

fired
fired
done, should exit

fired prints within ~50–100ms of the write (the { interval: 50 } poll detects the stat change), then unwatchFile(f, cb) at 200ms removes the callback and the process exits cleanly with code 0.

What do you see instead?

done, should exit
<process hangs>

fired never prints - even though the change happens at 50ms and the watcher polls every 50ms for 150ms before unwatch. cb was never registered at all. The hang after done, should exit is the leaked stub keeping the loop alive.

Additional information

No response

Vorherrschende Sprache: JavaScript
Sterne: 122k
Forks: 37.4k
Ø Merge: 4 T. 2 Std.
Gemergte PRs (30 T.): 277

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodejs/node

Stream.prototype.forEach will block in first promise in queue before read more chunk Offen

doc

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

nodejs/node#66106 · Erstellt am 18.9.2026
nodejs 26.8.2 fails to build with clang/LLVM Offen

build

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100

nodejs/node#66076 · 2 Kommentare · Erstellt am 17.9.2026
`TextEncoder.encodeInto()` underfills the destination for some non-ASCII text Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

nodejs/node#65994 · 2 Kommentare · 2 Reaktionen · Erstellt am 11.9.2026
worker: `messageerror` listeners remain during `exit` Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100

nodejs/node#65782 · Erstellt am 04.9.2026
[`util.styleText`] Allow `text` param to be any scalar value (ex. `number`, `bool` etc additionaly to `string`) which will be casted internally to `string` Offen

feature request

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

nodejs/node#63841 · 1 Kommentar · Erstellt am 10.6.2026

Alle Issues in nodejs/node