Skip to content

fix(compose): repair the reference pages and revert two wrong fixes - #103

Merged
a-essawy merged 1 commit into
mainfrom
fix/compose-reference-braces
Sep 24, 2026
Merged

a-essawy merged 1 commit into
mainfrom
fix/compose-reference-braces

Conversation

@a-essawy

Copy link
Copy Markdown
Contributor

The compose section has been serving its landing page as the body of all 24 child pages since #100 merged. Root cause, found with mint validate:

warning - parsing error jobs/compose/reference/timeline.mdx:52:159 - Could not parse expression with acorn
error Build validation failed with 1 warning(s).

A generated table cell contained [{time:0,value:0},{time:1,value:1}]. MDX reads a bare {...} as a JSX expression. The generator escaped pipes but not braces (fixed upstream in rendobar/rendobar#798); these are the regenerated pages.

Mintlify reported all three deploys a success, listed every page in the sitemap and llms.txt, and served the section root for each one. llms-full.txt carried none of the 93 video embeds. mint dev does not reproduce it.

Reverting two fixes I shipped on a wrong diagnosis

I read the symptom as a routing collision between jobs/compose.mdx and jobs/compose/, and shipped two changes for it. Neither was the cause:

The check that would have caught it

validate:build runs mint validate. It is the only one of the six that sees a build error, and it is the one that was missing.

A generated table cell held `[{time:0,value:0},{time:1,value:1}]`; MDX reads a
bare `{...}` as a JSX expression, the page failed to parse, and Mintlify served
the section root as the body of all 24 child pages while reporting every deploy
a success. Generator fixed in rendobar/rendobar#798; these are the regenerated
pages.

Also reverts two changes I shipped on a wrong diagnosis: #101 (moving the
overview to index.mdx and dropping `root`, which cost the nav behaviour) and
#102 (a validator rule for a collision that was never the cause).

Adds `validate:build` = `mint validate`, the only check that sees a build
error, and the one that was missing.
@a-essawy
a-essawy merged commit a5545b0 into main Sep 24, 2026
1 check passed
@a-essawy
a-essawy deleted the fix/compose-reference-braces branch September 24, 2026 13:53
a-essawy added a commit that referenced this pull request Sep 24, 2026
#103 fixed the MDX parse error and the children still served the section
landing page. Two variables had changed together, so the brace fix never got a
clean test against the nav shape.

Braces stay fixed; the group drops `root` for `jobs/compose/index` as its
first page, which is what storage/ and automation/ use and what resolves
children correctly on the live site today.

`/docs/jobs/compose` still serves the overview, so no inbound link moves.
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.

1 participant