Add MDX JSX runtime PoC#111
Conversation
|
@codex review and suggest improvements about the DX and API |
💡 Codex ReviewLines 212 to 214 in 3cf476c This branch drops every prop whose value is Lines 224 to 225 in 3cf476c The runtime only keys nodes from the third Line 112 in 3cf476c Lowercasing all unknown attribute names causes case-sensitive SVG attributes that are not explicitly aliased (for example ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
xote/jsx-runtimeandxote/jsx-dev-runtime.Xote.Mdxhelper module for ReScript-native MDX consumption: direct.mdxexternals,Mdx.render, component maps, component adapters, and child normalization helpers.Mdx.components.MdxDocs.resthrough the Xote JSX runtime.rehype-slugso Markdown headings keep stable anchor IDs for the docs table of contents.Validation
npm testnpm run buildnpm run mdx:poccd docs-website && npm run buildgit diff --checkNote: the docs build still reports pre-existing ReScript unused-variable warnings in
ColorMixerDemo.res.