chore: bump spec submodule (section corpus); un-skip heading fixtures - #17
Merged
Conversation
Bumps the spec submodule to markup-carve/carve@2e57817, which carries the section-wrapped heading fixtures (02-headings*, 17-attributes, 19-heading-ids). Removes those six entries from KNOWN_GAPS now that the vendored corpus matches the renderer's PART 9 §13 output. 228/228 tests pass (no skips, no todos).
dereuromark
added a commit
that referenced
this pull request
May 21, 2026
…18) Implements two rules from carve PR #17 (grammar PART 9 §11 + §12). §11 -- LIST MARKER CHANGE STARTS A NEW LIST parseList now captures the first item's unordered marker character and terminates the list when a sibling at the same indent uses a different character (`-` vs `*` vs `+`), so `- a / + b` renders as two lists instead of one. interruptsParagraph is updated in lockstep: a marker line only interrupts a paragraph when the next line shares BOTH the marker character and the task-vs-plain kind -- otherwise the lines are two lone single markers and stay prose (`para / - a / + b` is one paragraph; `para / - a / - b` is paragraph + list). Ordered lists have only the digit `.` dialect here, so no ordered split. §12 -- ADMONITION TITLE A <p class="admonition-title"> is emitted ONLY when the opener carries a double-quoted title; the quote delimiters are stripped (previously they leaked into the rendered text and got smart-curled). Unquoted trailing text is no longer treated as a title. A title followed by an attribute block keeps the title; an explicitly empty "" still emits an empty title element. New test/list-marker-and-admonition.test.ts (13 cases). 241 tests pass; typecheck clean.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Final step of the
<section>wrapper rollout (PART 9 §13).Bumps the spec submodule to markup-carve/carve@2e57817 (#19), which carries the section-wrapped heading fixtures, and removes the six temporary
KNOWN_GAPSentries (02-headings*,17-attributes,19-heading-ids) added in #16 — the vendored corpus now matches the renderer's output.228/228 tests pass (no skips, no todos).
Rollout complete