fix: suppress addWatchFile invalid phase error

[bluwy]

Description

Fix https://github.com/vitejs/vite/actions/runs/6624843925/job/17994626864?pr=11151

The error happens at watchFile

vite/packages/vite/src/node/packages.ts

Lines 233 to 236 in 14966a9

	packageCache.set = (id, pkg) => {
	if (!isInNodeModules(pkg.dir) && !watchedDirs.has(pkg.dir)) {
	watchedDirs.add(pkg.dir)
	watchFile(path.join(pkg.dir, 'package.json'))

While we do swap to a stub after the build ends at

vite/packages/vite/src/node/packages.ts

Lines 248 to 250 in 14966a9

	buildEnd() {
	watchFile = watchFileStub
	},

Rollup only calls buildEnd here https://github.com/rollup/rollup/blob/de1c7b6b30f44047026922c168d3a876fdd5514f/src/rollup/rollup.ts#L83, but the "generate phase" (that's used to track and trigger the error) is set at https://github.com/rollup/rollup/blob/de1c7b6b30f44047026922c168d3a876fdd5514f/src/Graph.ts#L120 (inside graph.build(), which is called in the code above the first Rollup link. Additionally, here's addWatchFIle's error guard https://github.com/rollup/rollup/blob/de1c7b6b30f44047026922c168d3a876fdd5514f/src/utils/PluginContext.ts#L56-L61

So there's a short in-between phase where Rollup declares the start of "generate phase" but haven't ended the build with buildEnd yet. This PR suppresses the error for now as there's no way to detect the phase.

Additional context

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.