Vite resolves file from cwd instead of node module if the names are identical

[sheremet-va]

Describe the bug

This is based on vitest-dev/vitest#4860

I was able to reduce reproduction to a few lines:

import { createServer } from 'vite';
const s = await createServer();
console.log(await s.pluginContainer.resolveId('tsoa'));
s.close();

The expectation is that tsoa will be resolved to the package in node modules, not to tsoa.json file in the root of the project.

Reproduction

https://stackblitz.com/edit/vitejs-vite-pivsa8?file=vite.mjs,package.json

Steps to reproduce

node vite.mjs

System Info

Stackblitz

Used Package Manager

npm

Logs

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• vitejs-vite-pivsa8 (Open reproduction)

[XiSenao]

If the importer is not specified when using the pluginContainer.resolveId hook, Vite will default to treating the importer as the entry module (index.html).

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

Line 668 in f43945c

async resolveId(rawId, importer = join(root, 'index.html'), options) {

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

Lines 253 to 257 in f43945c

	if (
	id[0] === '.' ||
	((preferRelative || importer?.endsWith('.html')) &&
	startsWithWordCharRE.test(id))
	) {

Using it in the following way may meet your expectations.

await s.pluginContainer.resolveId('tsoa’, null)

refs: #8122 #8808

[sheremet-va]

If the importer is not specified when using the pluginContainer.resolveId hook, Vite will default to treating the importer as the entry module (index.html).

Thank you for investigating. I can confirm that passing down the importer correctly resolves the path.