From 70c2d5bbc066585febd12536a6ab39c4384b277b Mon Sep 17 00:00:00 2001
From: Evan You <yyx990803@gmail.com>
Date: Tue, 17 May 2022 11:13:22 +0800
Subject: [PATCH] fix(ssr): fix ssr render output for fragment in slots

fix #5859
---
 .../server-renderer/__tests__/ssrSlot.spec.ts | 86 +++++++++++++++++++
 .../src/helpers/ssrRenderSlot.ts              |  4 +-
 2 files changed, 88 insertions(+), 2 deletions(-)
 create mode 100644 packages/server-renderer/__tests__/ssrSlot.spec.ts

diff --git a/packages/server-renderer/__tests__/ssrSlot.spec.ts b/packages/server-renderer/__tests__/ssrSlot.spec.ts
new file mode 100644
index 00000000..a68569ca
--- /dev/null
+++ b/packages/server-renderer/__tests__/ssrSlot.spec.ts
@@ -0,0 +1,86 @@
+/**
+ * @jest-environment node
+ */
+
+import { createApp } from 'vue'
+import { renderToString } from '../src/renderToString'
+
+const components = {
+  one: {
+    template: `<div><slot/></div>`
+  }
+}
+
+describe('ssr: slot', () => {
+  test('text slot', async () => {
+    expect(
+      await renderToString(
+        createApp({
+          components,
+          template: `<one>hello</one>`
+        })
+      )
+    ).toBe(`<div><!--[-->hello<!--]--></div>`)
+  })
+
+  test('element slot', async () => {
+    expect(
+      await renderToString(
+        createApp({
+          components,
+          template: `<one><div>hi</div></one>`
+        })
+      )
+    ).toBe(`<div><!--[--><div>hi</div><!--]--></div>`)
+  })
+
+  test('empty slot', async () => {
+    expect(
+      await renderToString(
+        createApp({
+          components: {
+            one: {
+              template: `<div><slot/></div>`
+            }
+          },
+          template: `<one><template v-if="false"/></one>`
+        })
+      )
+    ).toBe(`<div><!--[--><!--]--></div>`)
+  })
+
+  test('multiple elements', async () => {
+    expect(
+      await renderToString(
+        createApp({
+          components,
+          template: `<one><div>one</div><div>two</div></one>`
+        })
+      )
+    ).toBe(`<div><!--[--><div>one</div><div>two</div><!--]--></div>`)
+  })
+
+  test('fragment slot (template v-if)', async () => {
+    expect(
+      await renderToString(
+        createApp({
+          components,
+          template: `<one><template v-if="true">hello</template></one>`
+        })
+      )
+    ).toBe(`<div><!--[--><!--[-->hello<!--]--><!--]--></div>`)
+  })
+
+  test('fragment slot (template v-if + multiple elements)', async () => {
+    expect(
+      await renderToString(
+        createApp({
+          components,
+          template: `<one><template v-if="true"><div>one</div><div>two</div></template></one>`
+        })
+      )
+    ).toBe(
+      `<div><!--[--><!--[--><div>one</div><div>two</div><!--]--><!--]--></div>`
+    )
+  })
+})
diff --git a/packages/server-renderer/src/helpers/ssrRenderSlot.ts b/packages/server-renderer/src/helpers/ssrRenderSlot.ts
index 967b2031..e421578d 100644
--- a/packages/server-renderer/src/helpers/ssrRenderSlot.ts
+++ b/packages/server-renderer/src/helpers/ssrRenderSlot.ts
@@ -82,7 +82,7 @@ export function ssrRenderSlotInner(
   }
 }
 
-const commentRE = /^<!--.*-->$/
+const commentRE = /<!--.*?-->/g
 function isComment(item: SSRBufferItem) {
-  return typeof item === 'string' && commentRE.test(item)
+  return typeof item === 'string' && !item.replace(commentRE, '').trim()
 }