fix(vite-node): named export should overwrite export all (#7846)

Author: hi-ogawa

packages/vite-node/src/client.ts

@@ -625,7 +625,7 @@ function exportAll(exports: any, sourceModule: any) {
  }
 
  for (const key in sourceModule) {
- if (key !== 'default') {
+ if (key !== 'default' && !(key in exports)) {
  try {
  defineExport(exports, key, () => sourceModule[key])
  }

test/core/test/fixtures/named-overwrite-all/dep1.js

@@ -0,0 +1,4 @@
+export const a = 'dep1-a'
+export const b = 'dep1-b'
+export const c = 'dep1-c'
+export const d = 'dep1-d'

test/core/test/fixtures/named-overwrite-all/dep2.js

@@ -0,0 +1 @@
+export const d = 'dep2-d'

test/core/test/fixtures/named-overwrite-all/main.js

@@ -0,0 +1,4 @@
+export const a = 'main-a'
+export * from './dep1.js'
+export const c = 'main-c'
+export * from './dep2.js'

test/core/test/named-overwrite-all.test.ts

@@ -0,0 +1,14 @@
+import { expect, test } from 'vitest'
+// @ts-expect-error no type
+import * as lib from './fixtures/named-overwrite-all/main.js'
+
+test('named exports overwrite export all', async () => {
+ expect(lib).toMatchInlineSnapshot(`
+ {
+ "a": "main-a",
+ "b": "dep1-b",
+ "c": "main-c",
+ "d": "dep1-d",
+ }
+ `)
+})