fix: don't parse rich content in display name (#449)

Co-authored-by: 三咲智子 <sxzz@sxzz.moe>

components/account/AccountHeader.vue

@@ -81,6 +81,7 @@ watchEffect(() => {
                 font-bold sm:text-2xl text-xl
                 :content="getDisplayName(account, { rich: true })"
                 :emojis="account.emojis"
+                :markdown="false"
               />
               <AccountBotIndicator v-if="account.bot" />
             </div>

components/content/ContentRich.setup.ts

@@ -5,13 +5,17 @@ defineOptions({
   name: 'ContentRich',
 })
 
-const props = defineProps<{
+const { content, emojis, markdown = true } = defineProps<{
   content: string
+  markdown?: boolean
   emojis?: Emoji[]
 }>()
 
 export default () => h(
   'span',
   { class: 'content-rich' },
-  contentToVNode(props.content, emojisArrayToObject(props.emojis || [])),
+  contentToVNode(content, {
+    emojis: emojisArrayToObject(emojis || []),
+    markdown,
+  }),
 )

composables/content-parse.ts

@@ -12,7 +12,7 @@ function decode(text: string) {
  * Parse raw HTML form Mastodon server to AST,
  * with interop of custom emojis and inline Markdown syntax
  */
-export function parseMastodonHTML(html: string, customEmojis: Record<string, Emoji> = {}) {
+export function parseMastodonHTML(html: string, customEmojis: Record<string, Emoji> = {}, markdown = true) {
   let processed = html
     // custom emojis
     .replace(/:([\w-]+?):/g, (_, name) => {
@@ -21,45 +21,49 @@ export function parseMastodonHTML(html: string, customEmojis: Record<string, Emo
         return `<img src="${emoji.url}" alt=":${name}:" class="custom-emoji" data-emoji-id="${name}" />`
       return `:${name}:`
     })
+
+  if (markdown) {
     // handle code blocks
-    .replace(/>(```|~~~)(\w*)([\s\S]+?)\1/g, (_1, _2, lang, raw) => {
-      const code = htmlToText(raw)
-      const classes = lang ? ` class="language-${lang}"` : ''
-      return `><pre><code${classes}>${code}</code></pre>`
-    })
+    processed = processed
+      .replace(/>(```|~~~)(\w*)([\s\S]+?)\1/g, (_1, _2, lang, raw) => {
+        const code = htmlToText(raw)
+        const classes = lang ? ` class="language-${lang}"` : ''
+        return `><pre><code${classes}>${code}</code></pre>`
+      })
 
-  walkSync(parse(processed), (node) => {
-    if (node.type !== TEXT_NODE)
-      return
-    const replacements = [
-      [/\*\*\*(.*?)\*\*\*/g, '<b><em>$1</em></b>'],
-      [/\*\*(.*?)\*\*/g, '<b>$1</b>'],
-      [/\*(.*?)\*/g, '<em>$1</em>'],
-      [/~~(.*?)~~/g, '<del>$1</del>'],
-      [/`([^`]+?)`/g, '<code>$1</code>'],
-      [/&[^;]+;/g, (val: string) => decode(val)],
-    ] as any
+    walkSync(parse(processed), (node) => {
+      if (node.type !== TEXT_NODE)
+        return
+      const replacements = [
+        [/\*\*\*(.*?)\*\*\*/g, '<b><em>$1</em></b>'],
+        [/\*\*(.*?)\*\*/g, '<b>$1</b>'],
+        [/\*(.*?)\*/g, '<em>$1</em>'],
+        [/~~(.*?)~~/g, '<del>$1</del>'],
+        [/`([^`]+?)`/g, '<code>$1</code>'],
+        [/&[^;]+;/g, (val: string) => decode(val)],
+      ] as any
 
-    for (const [re, replacement] of replacements) {
-      for (const match of node.value.matchAll(re)) {
-        if (node.loc) {
-          const start = match.index! + node.loc[0].start
-          const end = start + match[0].length + node.loc[0].start
-          processed = processed.slice(0, start) + match[0].replace(re, replacement) + processed.slice(end)
-        }
-        else {
-          processed = processed.replace(match[0], match[0].replace(re, replacement))
+      for (const [re, replacement] of replacements) {
+        for (const match of node.value.matchAll(re)) {
+          if (node.loc) {
+            const start = match.index! + node.loc[0].start
+            const end = start + match[0].length + node.loc[0].start
+            processed = processed.slice(0, start) + match[0].replace(re, replacement) + processed.slice(end)
+          }
+          else {
+            processed = processed.replace(match[0], match[0].replace(re, replacement))
+          }
         }
       }
-    }
-  })
+    })
+  }
 
   return parse(processed)
 }
 
-export async function convertMastodonHTML(html: string, customEmojis: Record<string, Emoji> = {}) {
+export function convertMastodonHTML(html: string, customEmojis: Record<string, Emoji> = {}) {
   const tree = parseMastodonHTML(html, customEmojis)
-  return await render(tree)
+  return render(tree)
 }
 
 export function htmlToText(html: string) {

composables/content-render.ts

@@ -13,9 +13,12 @@ import AccountHoverWrapper from '~/components/account/AccountHoverWrapper.vue'
 */
 export function contentToVNode(
   content: string,
-  customEmojis: Record<string, Emoji> = {},
+  { emojis = {}, markdown = true }: {
+    emojis?: Record<string, Emoji>
+    markdown?: boolean
+  } = {},
 ): VNode {
-  const tree = parseMastodonHTML(content, customEmojis)
+  const tree = parseMastodonHTML(content, emojis, markdown)
   return h(Fragment, (tree.children as Node[]).map(n => treeToVNode(n)))
 }
 

tests/content-rich.test.ts

@@ -55,7 +55,7 @@ describe('content-rich', () => {
 })
 
 async function render(content: string, emojis?: Record<string, Emoji>) {
-  const vnode = contentToVNode(content, emojis)
+  const vnode = contentToVNode(content, { emojis })
   const html = (await renderToString(vnode))
     .replace(/<!--[\[\]]-->/g, '')
   let formatted = ''