Repository navigation
feat: render markdown tables in sidepanel and skill previews - #2845
Conversation
|
@snowyukitty is attempting to deploy a commit to the esokullu's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
thanks for the contribution @snowyukitty i think it's conflicting with an earlier commit but it can stilll merge into skills block - thanks! |
aea37bf to
aa00b04
Compare
|
@snowyukitty I'm sorry I may have accidentally erased your contribution while trying to force-push a fix to merge it. Would you like to take this #2846 instead? I won't override again, promise! :) |
|
No harm done — I checked
There is one real problem in there, and it's mine rather than yours: those two tests are registered twice. On #2846 — it's assigned to you at the moment, so say the word and I'll pick it up. One question before I would: "shorter", "top not bottom", and "same as the background but differentiated with a border" are clear, but "take a look at desktop AI agents for inspiration" could go a few ways. Do you have a specific one in mind, or is choosing the direction part of the task? |
Description
This change adds support for rendering GitHub Flavored Markdown (GFM) tables in both the sidepanel chat UI and the skill preview pane across Chrome and Firefox builds (addressing #2838).
Key Highlights
Pure ES Table Renderer (
markdown-render.js):<div class="markdown-table-wrapper"><table>...</table></div>).:---,:---:,---:), single edge pipes, escaped pipes (\|), and GFM delimiter hyphen validation (>= 3).-/*/+and dot/parenthesized numbered lists1./1)), blockquotes, and ATX headings containing pipes.Protected Inline Formatting & Collision-Safe Sentinels:
renderInlineMarkdownto protect sanitized markdown anchors with unique sentinels during emphasis processing, preserving starred URLs and supporting bold/italic both around and inside link text.CODE,INLINE,TABLE,LINK,QUOTE,LIST).Parity & Comprehensive Testing:
Closes #2838