refactor: normalize cache package dir

[btea]

Description

Additional context

---

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.

[bluwy]

Is there a reason to do so? It's going to affect this:

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

Lines 47 to 49 in 2836276

	const pkgDir = path.dirname(pkgPath)
	packageCache.forEach((pkg, cacheKey) => {
	if (pkg.dir === pkgDir) {

There's also code that will normalize it later:

vite/packages/vite/src/node/plugins/resolve.ts

Lines 930 to 939 in 2836276

	idPkgDir = resolvePackageData(
	pkgName,
	importer,
	preserveSymlinks,
	packageCache,
	)?.dir
	// if still null, it likely means that this id isn't a dep for importer.
	// break to bail early
	if (idPkgDir == null) break
	idPkgDir = normalizePath(idPkgDir)

[patak-dev]

I think it will be good to move to normalizing all paths in our (resolved) configs and public APIs. We have a confusing situation right now with some paths normalized and others that aren't. It would also be good to define a standard regarding trailing slashes for directories.

[btea]

Is there a reason to do so? It's going to affect this:

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

Lines 47 to 49 in 2836276

	const pkgDir = path.dirname(pkgPath)
	packageCache.forEach((pkg, cacheKey) => {
	if (pkg.dir === pkgDir) {

I just think it's better to keep all path information formatted consistent. I did forget to edit it here, I will add it.

There's also code that will normalize it later:

vite/packages/vite/src/node/plugins/resolve.ts

Lines 930 to 939 in 2836276

	idPkgDir = resolvePackageData(
	pkgName,
	importer,
	preserveSymlinks,
	packageCache,
	)?.dir
	// if still null, it likely means that this id isn't a dep for importer.
	// break to bail early
	if (idPkgDir == null) break
	idPkgDir = normalizePath(idPkgDir)

It certainly seems so. However, as in the picture I provided, when debug information is output, the path at this time is not normalized.