fix(ssr): exclude module condition from ssr node resolve

[hi-ogawa]

Description

• closes pluginContainer.resolveId(..., { ssr: true }) works differently from ssrLoadModule resolution for "module" entry/exports #16631

It looks like vite:resolve plugin's tryNodeResolve doesn't set overrideConditions and it will end up including module as conditions and it's impossible to exclude it from user config:

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

Lines 1101 to 1105 in 60824f0

	const additionalConditions = new Set(
	options.overrideConditions || [
	'production',
	'development',
	'module',

This contracts with ssr module loader's node resolve, which explicitly exclude module conditions:

vite/packages/vite/src/node/ssr/ssrModuleLoader.ts

Line 141 in 60824f0

overrideConditions: [...overrideConditions, 'production', 'development'],

todo

• check if the patch fixes the issue on Vitest
• just excluding 'module' when !targetWeb would be too breaking?
• should investigate cache issue?
• what does environment api do?

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[sapphi-red]

I guess this is not needed anymore as #18302 is merged. I'm going to close this, but feel free to re-open it if I'm missing something.