fix: remove maybeUnwrapTopLevelPWithA

Author: sy-records

docs/_sidebar.md

@@ -27,5 +27,5 @@
   - [Embed Files](embed-files.md)
   - [UI Kit](ui-kit.md)
 
-- [Awesome docsify](awesome.md)
-- [Changelog](changelog.md)
+* [Awesome docsify](awesome.md)
+* [Changelog](changelog.md)

src/core/render/compiler.js

@@ -5,7 +5,7 @@ import { tree as treeTpl } from './tpl.js';
 import { genTree } from './gen-tree.js';
 import { slugify } from './slugify.js';
 import { emojify } from './emojify.js';
-import { getAndRemoveConfig, maybeUnwrapTopLevelPWithA } from './utils.js';
+import { getAndRemoveConfig } from './utils.js';
 import { imageCompiler } from './compiler/image.js';
 import { headingCompiler } from './compiler/heading.js';
 import { highlightCodeCompiler } from './compiler/code.js';
@@ -193,8 +193,7 @@ export class Compiler {
 
     // compile sidebar from _sidebar.md
     if (text) {
-      html = this.compile(text);
-      return maybeUnwrapTopLevelPWithA(html);
+      return this.compile(text);
     }
     // compile sidebar from content's headings toc
     for (let i = 0; i < toc.length; i++) {

src/core/render/utils.js

@@ -88,40 +88,3 @@ export function getAndRemoveDocsifyIgnoreConfig(content = '') {
 
   return { content, ignoreAllSubs, ignoreSubHeading };
 }
-
-/**
- * Unwraps <p> tags that directly wrap a single <a> tag inside <li> elements at the top level.
- *
- * This function processes an HTML string and removes <p> wrappers inside <li> elements
- * when the <p> contains exactly one <a> element and no other content.
- *
- * It does NOT process nested structures.
- *
- * @see https://github.com/markedjs/marked/issues/1461
- * @param {string} html - The HTML string to transform.
- * @returns {string} - The transformed HTML with <p> removed from <li><p><a></a></p></li> structures.
- */
-export function maybeUnwrapTopLevelPWithA(html) {
-  // Only unwrap if the pattern <li><p><a exists
-  if (!html.includes('<li><p><a')) {
-    return html;
-  }
-
-  const template = document.createElement('template');
-  template.innerHTML = html;
-
-  template.content.querySelectorAll('li').forEach(li => {
-    const first = li.firstElementChild;
-    if (
-      first?.tagName === 'P' &&
-      first.children.length === 1 &&
-      first.firstElementChild.tagName === 'A' &&
-      first.childNodes.length === 1
-    ) {
-      // Replace the <p> with the <a> inside it
-      li.replaceChild(first.firstElementChild, first);
-    }
-  });
-
-  return template.innerHTML;
-}