Conversation
|
|
Preview deployments for this pull request: www - |
There was a problem hiding this comment.
Pull request overview
This PR targets faster apps/www build/prerender times by increasing prerender parallelism and avoiding repeated expensive parsing/compilation work during route loading.
Changes:
- Increased React Router prerender concurrency.
- Avoided full MDX compilation on the home page by extracting blog frontmatter directly.
- Added several process-wide caches for MDX compilation, component prop extraction, story extraction, and component CSS parsing.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/www/react-router.config.ts | Increases prerender concurrency to speed up builds. |
| apps/www/app/routes/home/home.tsx | Replaces bundleMDX usage with a lightweight frontmatter extraction for blog cards. |
| apps/www/app/routes/components/component.tsx | Adds caching for resolving/reading/parsing component CSS. |
| apps/www/app/_utils/get-react-props.server.ts | Reuses a single react-docgen-typescript parser and caches component docs. |
| apps/www/app/_utils/generate-from-mdx.ts | Caches MDX compilation results to avoid recompiling identical sources. |
| apps/www/app/_utils/extract-stories.server.ts | Caches story extraction results per component path + variant. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <copilot@github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| { | ||
| code: string; | ||
| // biome-ignore lint/suspicious/noExplicitAny: this is how frontmatter is typed in mdx-bundler | ||
| frontmatter: { [key: string]: any }; |
There was a problem hiding this comment.
Could use unknown here instead of any?
There was a problem hiding this comment.
This is how it's typed in frontmatter, but I guess we could do unkown - should make no difference
Summary
Checks
pnpm changesetif relevant)