Add details/summary and HTML block support to accepted markdown.

[evgeniysharapov]

Changed markdown conversion features from custom plugin to current visitor architecture:

• convert <details>/<summary> HTML elements to Confluence expand macro
• wrap <svg> and <div> blocks in Confluence HTML macros with CDATA

Type of Change

• New feature (non-breaking change which adds functionality)
• Performance improvement (walker-based vs regex-based)

Implementation Details

changed HTML passthrough in macro-converter.js:

• PASSTHROUGH_BLOCK_RE: Stashes complete <svg>/<div> blocks with all nested content preserved verbatim and not processed
• PASSTHROUGH_TAG_RE: Extended to stash <details>/<summary> tags only, allowing markdown processing of content between tags

changes html-to-storage.js:

• convertDetails() similar to convertBlockquote() pattern - detects structure, extracts parts, emits macro
• convertHtmlBlock() wraps allowlisted tags in HTML macro with CDATA escaping and UUID generation via crypto.randomUUID()
• allowlist HTML_MACRO_TAGS contains tags to wrap into HTML macro to avoid unintended wrapping in HTML macro (now only svg and div get wrapped )

Testing

• Tests pass locally with my changes
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Test coverage:

• Added 17 new tests across html-to-storage.test.js and macro-converter.test.js
• All 669 existing tests pass (no regressions)

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation (inline comments)
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules

Example Usage

Input markdown:

<details>
<summary>View code</summary>

\`\`\`javascript
console.log("Hello");
\`\`\`

</details>

<svg width="100" height="100">
  <circle cx="50" cy="50" r="40"/>
</svg>

Output Confluence storage:

  <ac:structured-macro ac:name="expand">
    <ac:parameter ac:name="title">View code</ac:parameter>
    <ac:rich-text-body>
      <ac:structured-macro ac:name="code">
        <ac:parameter ac:name="language">javascript</ac:parameter>
        <ac:plain-text-body><![CDATA[console.log("Hello");]]></ac:plain-text-body>
      </ac:structured-macro>
    </ac:rich-text-body>
  </ac:structured-macro>

  <ac:structured-macro ac:name="html" ac:schema-version="1" ac:macro-id="...uuid...">
    <ac:plain-text-body><![CDATA[<svg width="100" height="100">
    <circle cx="50" cy="50" r="40"></circle>
  </svg>]]></ac:plain-text-body>
  </ac:structured-macro>

[pchuri]

Thank you for the thoughtful follow-up and for taking the previous review feedback to heart — this is a much cleaner approach.

Highlights:

• Cookie auth correctly dropped (already on main via feat: add cookie authentication for Enterprise SSO (rebased #106) #111)
• Built on top of the htmlparser2 walker from refactor(converter): replace regex-based markdown→storage with htmlparser2 walker #153 instead of replacing it
• Tight HTML_MACRO_TAGS allowlist ({svg, div}) avoids over-wrapping
• convertDetails mirrors the convertBlockquote pattern nicely, with a safe fallback for <details> without a <summary>
• PASSTHROUGH_BLOCK_RE ordered before PASSTHROUGH_TAG_RE so block stashing wins over inline — exactly the right order
• 17 new tests, no regressions in the 669 existing tests

A few non-blocking nits for possible follow-up (no need to address in this PR):

• convertHtmlBlock re-serializes attrs via renderAttrs, so self-closing slashes (<circle r="40"/> → <circle r="40">) are not preserved byte-identically. The tests assert this, so it's intentional — flagging in case strict-XML consumers ever care.
• PASSTHROUGH_BLOCK_RE's lazy match assumes svg/div don't self-nest; rare in practice.
• require('crypto') inside convertHtmlBlock could hoist to module scope.

Approving and merging. Thanks again for the contribution!

[pchuri]

Looks great — clean walker-based approach with solid test coverage. Thanks!

[github-actions]

🎉 This PR is included in version 2.6.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀