A user can't actually create an instance of a FSWatcher by themselves, they get it as a result of calling fs.watch, which will call start when a valid filename is given. If the user calls start again on an already started watcher, then it is a noop. Calling start on a closed watcher also does nothing.

So it’s essentially always a no-op from the user’s perspective? I wouldn’t document it as public API then, and instead look into making it private.

So it’s essentially always a no-op from the user’s perspective? I wouldn’t document it as public API then, and instead look into making it private.

That's what it looks like, unless there is some edge case where it is needed that i can't see. Also, there is this issue, #17430, which sort of talks about how it might not be needed.

If we do make it a private function, then what would be the process here. First, add the new private function, and also doc-deprecate the current function?

First, add the new private function, and also doc-deprecate the current function?

People might disagree with me on this but I’d just make it private as a semver-major PR, given that it seems useless otherwise, which makes it very likely that nobody uses it.

If we do follow a deprecation cycle on this, we could start with a runtime deprecation, because it’s undocumented to begin with.

People might disagree with me on this but I’d just make it private as a semver-major PR, given that it seems useless otherwise, which makes it very likely that nobody uses it.

This would probably be my preference here, but i'll wait to here more feed back. Is there someone/team that would be helpful to ping?

@nodejs/tsc ^^^

I agree with @addaleax in #29872 (comment).

@addaleax @cjihrig I'll probably give this until the end of today for others to comment, and if no one objects, i'll probably create another PR, referencing this one, that changes watcher.start to watcher._start.

should there be something added to the deprecations.md file for this(as a runtime depcrecation) or just make sure to specify that it is a semver-major pr?

closing this now since FSWatcher.start has been removed in this PR , #29905