fix: add ref() and unref() to chokidar.d.ts for typescript build to work.

[saerich]

Description

When attempting to build a vite project after updating from 5.0.8 to 5.0.12, received following error:
ERROR(TypeScript) Class 'import("D:/Projects/PROJECT/node_modules/vite/dist/node/index").FSWatcher' incorrectly implements interface 'import("fs").FSWatcher'.
Type 'FSWatcher' is missing the following properties from type 'FSWatcher': ref, unref
FILE D:/Projects/PROJECT/node_modules/vite/dist/node/index.d.ts:68:15

66 | // Inlined to avoid extra dependency (chokidar is bundled in the published build)
67 |

68 | declare class FSWatcher extends EventEmitter implements fs.FSWatcher {
| ^^^^^^^^^
69 | options: WatchOptions
70 |
71 | /**

Additional context

Not 100% certain if this is the perfect fix, but adding the same two methods to vite/dist/node/index.d.ts allowed build to complete successfully. (Sorry, First time contributing, have read guidelines but nearly certainly have missed something!)

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines, especially the Pull Request Guidelines.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Update the corresponding documentation if needed.
• Ideally, include relevant tests that fail without this PR but pass with it.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[hi-ogawa]

There are some details in:

• Incompatibility of types with recent changes in @types/node paulmillr/chokidar#1299
• [node] fs: add ref/unref to FSWatcher DefinitelyTyped/DefinitelyTyped#68300 (comment)

Probably this happens for users who upgraded to @types/node@20.11.6 without skipLibCheck: true. For example, Vitest's typecheck template also hits this https://stackblitz.com/edit/vitest-dev-vitest-jdc2r7?file=package.json (from https://github.com/vitest-dev/vitest/blob/main/examples/README.md).

[patak-dev]

Thanks for investigating this @hi-ogawa! Let's wait a bit to see what they decide to do in:

• Incompatibility of types with recent changes in @types/node paulmillr/chokidar#1299

[bluwy]

I'm not sure when chokidar will be able to resolve the issue, but I think anyways chokidar will have to do the same fix here as ref/unref had long existed in node12/14, so leaving my approval here for now.

[patak-dev]

Ok, let's merge this then 👍🏼

[mgdodge]

Seeing as the problematic node type update has broken every version of vite that runs with up-to-date node type definitions, is there any chance of this fix getting backported to the 4.x branch? I can't be the only one with multiple build jobs that are still running things in that range.