Render helper update

[feruzm]

Summary by CodeRabbit

• Bug Fixes

  • HTML parsing now tolerates and deduplicates duplicate attributes, reducing render failures from malformed markup.
  • Markdown-to-HTML conversion is more robust in primary and recovery paths.

• Chores

  • Added development-time warnings to surface parsing diagnostics.
  • Package updated to patch release 2.4.14 with accompanying changelog entry.

[coderabbitai]

📝 Walkthrough

Walkthrough

Preprocess rendered HTML to remove duplicate attributes before DOM parsing; add exported removeDuplicateAttributes and apply it in createDoc and markdown-to-HTML flows. Extend lenientErrorHandler signature to accept context: unknown and make it consistently return undefined with dev-only warnings.

Changes

Cohort / File(s)	Summary
Error handler packages/render-helper/src/consts/dom-parser.const.ts	Extended lenientErrorHandler signature to include context: unknown; handler now consistently returns undefined and emits development-only console.warn messages.
Duplicate-attribute dedupe & usage packages/render-helper/src/helper.ts, packages/render-helper/src/methods/markdown-to-html.method.ts	Added exported removeDuplicateAttributes(html: string) to strip duplicate attributes (keep first occurrence). Integrated into createDoc and both markdown-to-HTML parsing branches (initial and fallback) to clean HTML before DOM parsing/serialization.
Package metadata & changelog packages/render-helper/package.json, packages/render-helper/CHANGELOG.md	Bumped package version to 2.4.14 and added a short changelog entry documenting the patch release.

Sequence Diagram(s)

sequenceDiagram
participant Renderer as Renderer (markdown -> HTML)
participant Dedupe as removeDuplicateAttributes
participant Parser as DOM Parser (`@xmldom/htmlparser2` / `@xmldom/xmldom`)
participant Fallback as htmlparser2 + serializer
Renderer->>Dedupe: rendered HTML
Dedupe->>Parser: cleaned HTML
Parser-->>Renderer: DOMDocument
alt parsing fails / recovery
Parser->>Fallback: attempt reparsing/serialize
Fallback->>Dedupe: serialized HTML
Dedupe->>Parser: cleaned serialized HTML
Parser-->>Renderer: final DOMDocument
end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Fixes and render-helper #617: Modifies the same packages/render-helper modules (helper.ts, markdown-to-html.method.ts) and likely overlaps with duplicate-attribute preprocessing and parsing behavior changes.

Poem

🐰 I nibble tags and toss the clones away,
One attribute stays, the rest don't stay.
I whisper warnings when parsers groan,
I tidy HTML, then hop on home. 🥕

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Render helper update' is vague and generic, using non-descriptive language that doesn't convey specific information about what was changed.	Consider using a more specific title that describes the main change, such as 'Add duplicate attribute removal in HTML parser' or 'Improve HTML parsing with attribute deduplication'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch render

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}