Repository navigation
fix: mask empty and unclosed fences, keep require() images, rewrite @site images - #92
Merged
Merged
Conversation
…site images
- maskCodeSegments masks an empty fenced block, and an unclosed fence from
its opener to the end of its container (document, list item, or
blockquote), so cleaning and partial resolution leave its content as
written.
- <img src={require('./x.png').default}> and literal brace src values
become markdown images.
- rewriteImageUrls resolves @site/ image paths against the site directory
like a relative image; one with no bundled asset that the build serves
from a static directory is rewritten to its static URL.
- A front matter description whose first line leaves a code span or link
open drops the dangling opener in llms.txt and ends with '...'.
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.
Remaining content-cleaning edge cases:
<img src={require('./x.png').default}>and other literalsrcexpressions were dropped; they become Markdown images, whichrewriteImageUrlsthen rewrites.@site/image paths weren't rewritten. They resolve against the site directory like a relative image, matching Docusaurus's image handling.llms.txt.tests/test-content-edge-cases.jscovers each case (21 of 29 checks fail onmain). The docs site output is unchanged. On a 3,461-page docs site, the only difference isrequire()images appearing as Markdown images, with no words lost.