fix: avoid binding ClassExpression (#13572)

vitejs · Jun 20, 2023 · 1a0c806 · 1a0c806
1 parent 435bacc
commit 1a0c806
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/packages/vite/src/node/ssr/__tests__/ssrTransform.spec.ts b/packages/vite/src/node/ssr/__tests__/ssrTransform.spec.ts
@@ -904,3 +904,14 @@ for (const test in tests) {
     }"
   `)
 })
+
+test('avoid binding ClassExpression', async () => {
+  const result = await ssrTransformSimple(
+    `import Foo, {Bar} from './foo';console.log(Foo, Bar);const obj = {foo: class Foo{}, bar: class Bar{}}`,
+  )
+  expect(result?.code).toMatchInlineSnapshot(`
+
+    "const __vite_ssr_import_0__ = await __vite_ssr_import__(\\"./foo\\");
+    console.log(__vite_ssr_import_0__.default, __vite_ssr_import_0__.Bar);const obj = {foo: class Foo{}, bar: class Bar{}}"
+  `)
+})
diff --git a/packages/vite/src/node/ssr/ssrTransform.ts b/packages/vite/src/node/ssr/ssrTransform.ts
@@ -256,7 +256,7 @@ async function ssrTransformScript(
           const topNode = parentStack[parentStack.length - 2]
           s.prependRight(topNode.start, `const ${id.name} = ${binding};\n`)
         }
-      } else {
+      } else if (parent.type !== 'ClassExpression') {
         s.update(id.start, id.end, binding)
       }
     },