refactor: remove handleHotUpdate from watch-package-data plugin

[sapphi-red]

Description

While reading the env API PR, I noticed that this handleHotUpdate hook is not needed for watch-package-data plugin.

watchChange hook is always called before handleHotUpdate hook, so the package data is already invalidated.

vite/packages/vite/src/node/server/index.ts

Lines 764 to 793 in 6dbed4b

	const onFileAddUnlink = async (file: string, isUnlink: boolean) => {
	file = normalizePath(file)
	await container.watchChange(file, { event: isUnlink ? 'delete' : 'create' })
	if (publicDir && publicFiles) {
	if (file.startsWith(publicDir)) {
	const path = file.slice(publicDir.length)
	publicFiles[isUnlink ? 'delete' : 'add'](path)
	if (!isUnlink) {
	const moduleWithSamePath = await moduleGraph.getModuleByUrl(path)
	const etag = moduleWithSamePath?.transformResult?.etag
	if (etag) {
	// The public file should win on the next request over a module with the
	// same path. Prevent the transform etag fast path from serving the module
	moduleGraph.etagToModuleMap.delete(etag)
	}
	}
	}
	}
	if (isUnlink) moduleGraph.onFileDelete(file)
	await onHMRUpdate(isUnlink ? 'delete' : 'create', file)
	}
	watcher.on('change', async (file) => {
	file = normalizePath(file)
	await container.watchChange(file, { event: 'update' })
	// invalidate module graph cache on file change
	moduleGraph.onFileChange(file)
	await onHMRUpdate('update', file)
	})

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.