fix(ssr): ignore module exports condition

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'
 }