From 35592a26f89624d0ac4f0505f61a6386cc339e30 Mon Sep 17 00:00:00 2001
From: Mohamed Boudra <boudra.moha@gmail.com>
Date: Thu, 15 Oct 2026 20:26:35 +0200
Subject: [PATCH] fix(markdown): keep summary block boundaries outside bold
 text

---
 packages/app/src/components/markdown/html.ts       | 14 ++++++++++++--
 .../app/src/components/markdown/renderer.test.ts   |  8 ++++++++
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/packages/app/src/components/markdown/html.ts b/packages/app/src/components/markdown/html.ts
index 467430fb72..9a1a9b9ce0 100644
--- a/packages/app/src/components/markdown/html.ts
+++ b/packages/app/src/components/markdown/html.ts
@@ -6,7 +6,7 @@ import type { Nodes } from "mdast";
 const ATTRIBUTE = String.raw`\s+[A-Za-z_:][\w.:-]*(?:\s*=\s*(?:[^\s"'=<>\x60]+|'[^']*'|"[^"]*"))?`;
 const TAG = String.raw`<(?:[A-Za-z][A-Za-z0-9-]*(?:${ATTRIBUTE})*\s*\/?|\/[A-Za-z][A-Za-z0-9-]*\s*)>`;
 const HTML = new RegExp(
-  String.raw`(?<!\\)(?:<!--[\s\S]*?-->|<code\b[^>]*>[\s\S]*?<\/code\s*>|${TAG})`,
+  String.raw`(?<!\\)(?:<!--[\s\S]*?-->|<code\b[^>]*>[\s\S]*?<\/code\s*>|<summary\b[^>]*>[\s\S]*?<\/summary\s*>|${TAG})`,
   "gi",
 );
 const BLOCK_TAGS = new Set([
@@ -57,6 +57,16 @@ export function normalizeMarkdownHtml(source: string): string {
   const convert = (raw: string, offset: number): string => {
     if (codeRanges.some(({ start, end }) => offset >= start && offset < end)) return raw;
     if (raw.startsWith("<!--")) return "";
+    const summary = /^<summary\b[^>]*>([\s\S]*?)<\/summary\s*>$/i.exec(raw);
+    if (summary) {
+      const title = normalizeMarkdownHtml(summary[1]).trim();
+      // A summary is one bold paragraph, even when its HTML title uses headings
+      // or several block wrappers. Slice AST blocks so inline code stays literal.
+      const inline = fromMarkdown(title)
+        .children.map(({ position }) => title.slice(position!.start.offset!, position!.end.offset!))
+        .join(" ");
+      return `**${inline}**\n\n`;
+    }
     const code = /^<code\b[^>]*>([\s\S]*?)<\/code\s*>$/i.exec(raw);
     if (code)
       return inlineCode(
@@ -83,7 +93,7 @@ export function normalizeMarkdownHtml(source: string): string {
     result += source.slice(cursor, range.start);
     // Inline HTML code has separate opening/closing AST nodes. Convert the pair
     // together so its contents keep backslashes and get a safe backtick delimiter.
-    const code = /^<code\b[^>]*>[\s\S]*?<\/code\s*>/i.exec(source.slice(range.start));
+    const code = /^<(code|summary)\b[^>]*>[\s\S]*?<\/\1\s*>/i.exec(source.slice(range.start));
     const end = code ? Math.max(range.end, range.start + code[0].length) : range.end;
     result += source
       .slice(range.start, end)
diff --git a/packages/app/src/components/markdown/renderer.test.ts b/packages/app/src/components/markdown/renderer.test.ts
index a6ad28725f..08359d046c 100644
--- a/packages/app/src/components/markdown/renderer.test.ts
+++ b/packages/app/src/components/markdown/renderer.test.ts
@@ -222,6 +222,14 @@ describe("GitHub comment HTML normalization", () => {
       "**Review details**\n\nBody **stays**.",
     ],
     ["Before<!-- private -->After", "BeforeAfter"],
+    [
+      "<details><summary><h3>Important Files Changed</h3></summary>Body</details>",
+      "**Important Files Changed**\n\nBody",
+    ],
+    [
+      "<details><summary><p>First</p><p>Second</p></summary>Body</details>",
+      "**First Second**\n\nBody",
+    ],
     ["<span>file.</span>ts", "file.ts"],
     ['<a href="https://example.com">Git</a>Hub', "[Git](<https://example.com>)Hub"],
     ["<span>Git</span><span>Hub</span>", "GitHub"],