docs: add SUPPORT.md and clarify known conversion limits - #88
Open
HaoChiBao wants to merge 1 commit into
Open
Conversation
Document per-format keep/drop behavior, note intentional escaping of literal line-start list markers, and record the pdf-inspector 0.1.8 floor that pulls in the RUSTSEC-2026-0187 lopdf fix. Closes firecrawl#2 Closes firecrawl#41 Closes firecrawl#67
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
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.
Summary
SUPPORT.mddocumenting what each input format keeps, reduces, or drops (images as alt text, headers/footers, spreadsheet visibility and number formats, nested tables, PDF model/OCR limits, and more), grounded in current parser behavior.1.in plain paragraphs is escaped as1\.on purpose so Markdown does not invent a list. Real Wordw:numPrlists still render as normal ordered lists. Closes the confusion in docx to md puts a backslash between number and period in ordered lists #41 after inspecting the attached sample (no numbering part / now:numPr).pdf-inspector = 0.1.8floor inCargo.tomlso the RUSTSEC-2026-0187 lopdf fix (0.42.0) is not lowered by accident.mainalready resolves the fixed versions; transitive DoS via pdf-inspector -> lopdf 0.41.0 (RUSTSEC-2026-0187) #67 can close with this clarification.Closes #2
Closes #41
Closes #67
Test plan
SUPPORT.mdagainst README Features / Supported formats for tone and accuracysrc/formats/**(headers/footers, PDFto_document, spreadsheet number formats)SUPPORT.mdresolveCargo.tomlstill requirespdf-inspector = 0.1.8andcargo tree -i lopdf --lockedshows only0.42.01\.output remains expected for literal paragraph textSummary by cubic
Documents per-format conversion limits in
SUPPORT.mdand links it from the README and the convert-documents skill. Clarifies the intentional escaping of literal line-start list markers and records thepdf-inspector0.1.8 floor to keep thelopdfRUSTSEC fix. No behavior change; this documents current behavior.SUPPORT.mdmatches current parsers (images as alt text, headers/footers omitted, nested table flattening, spreadsheet number formats/visibility, PDFto_documentunsupported and no OCR).README.mdandskills/convert-documents-to-markdown/SKILL.mdresolve toSUPPORT.md.Cargo.tomlretainspdf-inspector = "0.1.8"solopdf >= 0.42.0stays resolved.Written for commit f4e2e4c. Summary will update on new commits.