Skip to content

Conversation

@kevinkim-ogp
Copy link
Contributor

@kevinkim-ogp kevinkim-ogp commented Nov 10, 2025

TL;DR

Added support for returning Markdown from RTE instead of HTML.
In preparation of Pair action as Markdown is more token-efficient and easier for LLMs to parse and generate consistently than HTML's verbose tag-based syntax.

What changed?

  • Added the tiptap-markdown package as a dependency
  • Added a new returnMarkdown boolean option to rich text field configurations
  • Modified the editor to return Markdown content when returnMarkdown is enabled

How to test?

  1. Create or edit an action with a rich text field
  2. Set the returnMarkdown option to true in the field configuration
  3. Enter content in the rich text editor
  4. Verify that the field returns Markdown instead of HTML when submitted

Copy link
Contributor Author

kevinkim-ogp commented Nov 10, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedtiptap-markdown@​0.8.101001009181100

View full report

@kevinkim-ogp kevinkim-ogp changed the title feat: rte return markdown [PAIR-2]: Add tiptap-markdown to get markdown Nov 10, 2025
@kevinkim-ogp kevinkim-ogp marked this pull request as ready for review November 10, 2025 08:49
@kevinkim-ogp kevinkim-ogp requested a review from a team as a code owner November 10, 2025 08:49
@kevinkim-ogp kevinkim-ogp changed the title [PAIR-2]: Add tiptap-markdown to get markdown [AI-ACTION-2]: Add tiptap-markdown to get markdown Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants