fix(ssr): ignore module exports condition (#11409)

Author: bluwy

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

@@ -8,7 +8,7 @@ import {
   usingDynamicImport,
 } from '../utils'
 import { transformRequest } from '../server/transformRequest'
-import type { InternalResolveOptions } from '../plugins/resolve'
+import type { InternalResolveOptionsWithOverrideConditions } from '../plugins/resolve'
 import { tryNodeResolve } from '../plugins/resolve'
 import {
   ssrDynamicImportKey,
@@ -112,10 +112,11 @@ async function instantiateModule(
     root,
   } = server.config
 
-  const resolveOptions: InternalResolveOptions = {
+  const resolveOptions: InternalResolveOptionsWithOverrideConditions = {
     mainFields: ['main'],
     browserField: true,
     conditions: [],
+    overrideConditions: ['production', 'development'],
     extensions: ['.js', '.cjs', '.json'],
     dedupe,
     preserveSymlinks,
@@ -223,7 +224,7 @@ async function instantiateModule(
 async function nodeImport(
   id: string,
   importer: string,
-  resolveOptions: InternalResolveOptions,
+  resolveOptions: InternalResolveOptionsWithOverrideConditions,
 ) {
   let url: string
   if (id.startsWith('node:') || isBuiltin(id)) {

playground/ssr-deps/__tests__/ssr-deps.spec.ts

@@ -116,3 +116,8 @@ test('import css library', async () => {
   await page.goto(url)
   expect(await getColor('.css-lib')).toBe('blue')
 })
+
+test('import css library', async () => {
+  await page.goto(url)
+  expect(await page.textContent('.module-condition')).toMatch('[success]')
+})

playground/ssr-deps/module-condition/import.mjs

@@ -0,0 +1 @@
+export default '[success]'

playground/ssr-deps/module-condition/module.js

@@ -0,0 +1,4 @@
+// this is written in ESM but the file extension implies this is evaluated as CJS.
+// BUT this doesn't matter in practice as the `module` condition is not used in node.
+// hence SSR should not load this file.
+export default '[fail] should not load me'

playground/ssr-deps/module-condition/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@vitejs/test-module-condition",
+  "private": true,
+  "version": "0.0.0",
+  "exports": {
+    ".": {
+      "module": "./module.js",
+      "import": "./import.mjs"
+    }
+  }
+}

playground/ssr-deps/package.json

@@ -29,7 +29,8 @@
     "@vitejs/test-external-using-external-entry": "file:./external-using-external-entry",
     "@vitejs/test-external-entry": "file:./external-entry",
     "@vitejs/test-linked-no-external": "link:./linked-no-external",
-    "@vitejs/test-pkg-exports": "file:./pkg-exports"
+    "@vitejs/test-pkg-exports": "file:./pkg-exports",
+    "@vitejs/test-module-condition": "file:./module-condition"
   },
   "devDependencies": {
     "express": "^4.18.2"

playground/ssr-deps/src/app.js

@@ -15,6 +15,7 @@ import noExternalCjs from '@vitejs/test-no-external-cjs'
 import importBuiltinCjs from '@vitejs/test-import-builtin-cjs'
 import { hello as linkedNoExternal } from '@vitejs/test-linked-no-external'
 import virtualMessage from '@vitejs/test-pkg-exports/virtual'
+import moduleConditionMessage from '@vitejs/test-module-condition'
 import '@vitejs/test-css-lib'
 
 // This import will set a 'Hello World!" message in the nested-external non-entry dependency
@@ -87,5 +88,7 @@ export async function render(url, rootDir) {
 
   html += `\n<p class="css-lib">I should be blue</p>`
 
+  html += `\n<p class="module-condition">${moduleConditionMessage}</p>`
+
   return html + '\n'
 }