Skip to content

fix(converter): keep list soft breaks via hardBreak; relax round-trip whitespace gate#109

Draft
ocavue wants to merge 2 commits into
masterfrom
ocavue/hardbreak-list-roundtrip
Draft

fix(converter): keep list soft breaks via hardBreak; relax round-trip whitespace gate#109
ocavue wants to merge 2 commits into
masterfrom
ocavue/hardbreak-list-roundtrip

Conversation

@ocavue

@ocavue ocavue commented Jun 21, 2026

Copy link
Copy Markdown
Member

Represent soft line breaks with a hardBreak node (dedenting list continuation lines) so list paragraphs round-trip exactly and stop folding in the editor, and classify trailing-whitespace-only round-trip diffs as normalizing instead of lossy.

@vercel

vercel Bot commented Jun 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
meowdown Ready Ready Preview, Comment Jun 21, 2026 2:04pm

@pkg-pr-new

pkg-pr-new Bot commented Jun 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/prosekit/meowdown/@meowdown/core@109
npm i https://pkg.pr.new/prosekit/meowdown/@meowdown/react@109

commit: c12d5f3

@github-actions

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 93.56% 1643 / 1756
🔵 Statements 91.72% 1817 / 1981
🔵 Functions 92.77% 398 / 429
🔵 Branches 83.51% 998 / 1195
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/core/src/converters/check-roundtrip.ts 100% 100% 100% 100%
packages/core/src/converters/md-to-pm.ts 98.13% 82.89% 100% 98.98% 102-104, 146, 200
packages/core/src/converters/pm-to-md.ts 95.67% 88.18% 100% 97.72% 102, 214-215, 229-230, 249, 364, 381
packages/core/src/extensions/extension.ts 100% 100% 100% 100%
packages/core/src/extensions/node-names.ts 100% 100% 100% 100%
Generated in workflow #475 for commit c12d5f3 by the Vitest Coverage Report Action

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.

1 participant