ci: check that API references do not cause Docusaurus build failures#2829
Merged
ci: check that API references do not cause Docusaurus build failures#2829
Conversation
Member
Author
|
I found some simplification opportunities. Putting this in draft. |
This was referenced Feb 12, 2026
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.
Related Issues
Proposed Changes:
I started from the idea that the simplest way to check whether generated Markdown API references are compatible with Docusaurus is to check out the Docs Website and try building it. Then I found better tools, and this idea evolved into something simpler.
websitefolder for testingdocusaurus-mdx-checker, as suggested in Docusaurus docs: this does not build the website but just checks generated Markdown files.How did you test it?
Temporarily added some changes:
Notes for the reviewer
By running the checker but not building the website, we only get MD/MDX errors: for example, we don't get errors for duplicate page ids or broken links. In general, it seems OK since our previous errors were caused by invalid MD/MDX. Plus, the check is much faster than building the website.
With this change, we no longer need to run
hatch run docsin each integration workflow:haystack-core-integrations/.github/workflows/aimlapi.yml
Lines 57 to 59 in d964201
I prefer to remove this step in another PR to keep this focused and minimal.
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:.