perf: improve perf with more delegation - #1090
Conversation
🚀 Deploying Preview to Cloudflare 🚀Preview Deployments by commit
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| ...imports, | ||
| `export const headings = ${JSON.stringify(headings)};`, | ||
| `export const content = () => ${content};`, | ||
| `export default () => renderToStringAsync(<${JSX_IMPORTS.Layout.name} metadata={${JSON.stringify(metadata)}} headings={headings} readingTime={${JSON.stringify(readingTime?.text)}}>{content()}</${JSX_IMPORTS.Layout.name}>);`, |
| ...imports, | ||
| `export const headings = ${JSON.stringify(headings)};`, | ||
| `export const content = () => ${content};`, | ||
| `export default () => renderToStringAsync(<${JSX_IMPORTS.Layout.name} metadata={${JSON.stringify(metadata)}} headings={headings} readingTime={${JSON.stringify(readingTime?.text)}}>{content()}</${JSX_IMPORTS.Layout.name}>);`, |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1090 +/- ##
==========================================
+ Coverage 90.60% 90.68% +0.08%
==========================================
Files 217 219 +2
Lines 20802 21127 +325
Branches 1974 1999 +25
==========================================
+ Hits 18847 19159 +312
- Misses 1948 1961 +13
Partials 7 7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
| File | Main | PR | Change |
|---|---|---|---|
orama-db.json |
9.36 MB | 9.36 MB | +111.00 B (+0.0%) |
Performance estimate (single CI run)
- Generation time: 72.1% faster (20.69 s → 5.77 s)
- Peak memory: 21.1% lower (2.04 GB → 1.61 GB)
web Generator
Output size: 2 files changed · net -49.00 B
File size details
| File | Main | PR | Change |
|---|---|---|---|
all.html |
32.46 MB | 32.46 MB | -47.00 B (-0.0%) |
404.html |
21.90 KB | 21.90 KB | -2.00 B (-0.0%) |
Performance estimate (single CI run)
- Generation time: 68.0% faster (130.11 s → 41.64 s)
- Peak memory: 28.0% lower (5.50 GB → 3.96 GB)
Could you expand what these "various performance improvements" are? The PR descriptions should clearly state what the PR does. |
| const configExplorer = cosmiconfig('doc-kit'); | ||
|
|
||
| // The default `threads` ceiling; `--threads` raises it explicitly. | ||
| const MAX_THREADS = 4; |
There was a problem hiding this comment.
nit: That should be in a constant file
| * every module a second time here. | ||
| */ | ||
| const buildSyntheticDescriptors = input => { | ||
| const buildSyntheticDescriptors = () => { |
There was a problem hiding this comment.
This name.... doesn't mean much, now this could just be buildNotFoundPage imo
Fixes #1008
Various performance improvements