feat: expose subscript and superscript markers in CLI and MCP - #2492
Open
Çağdaş Yürekli (cagdasyurekli) wants to merge 1 commit into
Open
Çağdaş Yürekli (cagdasyurekli) wants to merge 1 commit into
Çağdaş Yürekli (cagdasyurekli) wants to merge 1 commit into
Conversation
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.
Fixes #1217.
The Python API already forwards markdownify's
sub_symbolandsup_symboloptions for DOCX, but the CLI and MCP tool do not expose them. Add--sub-symbol/--sup-symbolto the CLI and optional string arguments with the same names toconvert_to_markdown, and document all three interfaces.For example, a DOCX containing subscript/superscript runs can now produce
H<sub>2</sub>O and x<sup>2</sup>through the CLI or MCP instead ofH2O and x2. Both file and stdin CLI input are supported. Omitting the options preserves current behavior; custom markers and explicit empty strings are supported. This closes the remaining interface/discoverability gap described in the issue discussion without adding another converter or changing PDF/cloud extraction.Validation on Python 3.12:
GITHUB_ACTIONS=true hatch test --python 3.12 -q --tb=short: 871 passed, 43 skipped (the repository's conditional skips).pre-commit run --all-filesandgit diff --check: passed.