Skip to content

Conversation

@jonathanhefner
Copy link
Member

Mintlify's Markdown parser treats dollar signs as special characters in some cases. The exact cause is not clear, but it might be related to MDX's LaTeX support.

An example of this behavior can be seen in the build failure for #881, in which npm run check:docs:links failed to parse schema.mdx due to newly introduced $schema properties.

This commit modifies the schema.mdx generator to HTML encode dollar signs, thus preventing such parse errors.

Mintlify's Markdown parser treats dollar signs as special characters in
some cases.  The exact cause is not clear, but it might be related to
MDX's [LaTeX support](https://mdxjs.com/guides/math/).

An example of this behavior can be seen in the build failure for
[modelcontextprotocol#881][], in which `npm run check:docs:links` failed to parse
`schema.mdx` due to newly introduced `$schema` properties.

This commit modifies the `schema.mdx` generator to HTML encode dollar
signs, thus preventing such parse errors.

[modelcontextprotocol#881]: modelcontextprotocol#881
Copy link
Contributor

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dsp-ant dsp-ant merged commit 0ff4a9b into modelcontextprotocol:main Sep 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants