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",
+    }
+  `)
+})