chore(repo): Improve Typedoc MDX output #5578
Conversation
🦋 Changeset detectedLatest commit: 0fb9ede The changes in this PR will be included in the next version bump. This PR includes changesets to release 22 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| pattern: new RegExp(`\\((?:\\.{1,2}\\/)+.*?${fileName}\\.mdx\\)`, 'g'), | ||
| replace: `(${newPath})`, | ||
| // Match both path and optional anchor | ||
| pattern: new RegExp(`\\((?:(?:\\.{1,2}\\/)+[^()]*?|)${fileName}\\.mdx(#[^)]+)?\\)`, 'g'), |
There was a problem hiding this comment.
The new regex accounts for two bugs I saw:
- A string contained multiple
[]()and previously it replaced parts outside of that link (until the next occurence) [foo](bar.mdx)and[foo](bar.mdx#test)didn't work
| * @param {import('typedoc').SignatureReflection} model | ||
| * @param {{ headingLevel: number }} options | ||
| */ | ||
| signatureReturns: (model, options) => { |
There was a problem hiding this comment.
This was the same as default again, so I can remove it
There was a problem hiding this comment.
I changed the approach to this file completely. Instead of copy/pasting the source and overriding individual parts, I modify the input and output of the default functions to not have to change the inner workings. And to keep it up to date...
Description
The changes in this PR are the result of the review of clerk/clerk-docs#2124.
In this PR:
<code>Fixes ECO-569
Fixes ECO-534
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change