laantungir/amethyst
1
0
Fork 0
mirror of https://github.com/vitorpamplona/amethyst.git synced 2026-08-09 08:04:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3344 from davotoula/fix/markdown-detect-heading-after-blank-line

Browse Source 
fix: render markdown when a heading/list follows a blank line
This commit is contained in:
Vitor Pamplona
2026-06-23 12:58:14 -04:00
committed by GitHub
parent 7d77a057d6 a97529d7af
commit ff2c6f6313
2 changed files with 48 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
amethyst/src/main/java/com/vitorpamplona/amethyst/service/CachedRichTextParser.kt
+8 -1
View File
@@ -207,7 +207,14 @@ object CachedRichTextParser {
// checks that need to know "how much non-space text has
// appeared on the current line".
if (isNewLine) {
if (c != ' ' && c != '\t') {
// A blank line is still "at line start": skipping '\n'/'\r'
// here keeps isNewLine true so a heading/blockquote/list
// marker on the next line (the standard `\n\n#` spacing) is
// still recognized as line-leading. Without the newline
// exclusion, the second '\n' of a blank line flipped
// isNewLine to false and ATX headings after a blank line
// went undetected.
if (c != ' ' && c != '\t' && c != '\n' && c != '\r') {
isNewLine = false
nonSpaceCharCountOnLine = 1
lastNonSpaceChar = c
amethyst/src/test/java/com/vitorpamplona/amethyst/CachedRichTextParserMarkdownTest.kt
+40
View File
@@ -68,6 +68,46 @@ class CachedRichTextParserMarkdownTest {
@Test fun atxAfterNewlineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n# Heading"))
// ---- Line-leading markers after a BLANK line ------------------------
// Regression: the standard `\n\n#` spacing (a blank line before a
// heading) was undetected because the blank line's second '\n' flipped
// the line-start tracker off. NIP-23 long-form articles that are pure
// prose plus `## Section` headings rendered as raw text as a result.
@Test fun atxAfterBlankLineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n\n# Heading"))
@Test fun atxH3AfterBlankLineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n\n### Heading"))
@Test fun atxAfterMultipleBlankLinesIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n\n\n\n## Heading"))
@Test fun atxAfterCrlfBlankLineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\r\n\r\n## Heading"))
@Test fun blockquoteAfterBlankLineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n\n> quoted"))
@Test fun bulletAfterBlankLineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n\n- item"))
@Test fun orderedListAfterBlankLineIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("intro\n\n1. first"))
@Test
fun proseArticleWithHeadingAfterBlankLineIsMarkdown() =
assertTrue(
CachedRichTextParser.isMarkdown(
"The cryptographic part of commerce has been solved.\n\n" +
"What follows is a history.\n\n" +
"### The merchant posts of the Hansa\n\n" +
"Long before any king claimed a monopoly on letters.",
),
)
// Blank lines alone must NOT promote ordinary prose to markdown.
@Test
fun plainProseWithBlankLinesIsNotMarkdown() =
assertFalse(
CachedRichTextParser.isMarkdown(
"Just a normal sentence.\n\nAnother paragraph with no markdown at all.",
),
)
// ---- Blockquote -----------------------------------------------------
@Test fun blockquoteIsMarkdown() = assertTrue(CachedRichTextParser.isMarkdown("> a quote"))