You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
perf(start-plugin-core): speed up rsbuild client build normalization - #8588
normalizeRspackClientBuild converts the Rspack client compilation into the data used to build the Start manifest. It runs on every Rsbuild build and on every dev rebuild. This PR makes it faster for builds with many chunks. The output does not change.
Chunk and chunk group properties (files, auxiliaryFiles, groupsIterable, chunks, childrenIterable) are now read once per call. Rspack builds a new collection on every read, and the previous code read them many times per chunk.
Modules are now read only from chunks that contain a route-split or hydration module. The previous code called getChunkModules for every chunk.
Benchmarks
Median of 21 runs of normalizeRspackClientBuild on generated apps, Rspack 2.2.3, production unless noted:
App
Chunks
Chunk groups
main
This PR
Speedup
Routes and shared vendor chunks
760
481
61.1 ms
14.3 ms
4.3x
Large
2,456
901
131.8 ms
36.5 ms
3.6x
Large (development)
2,456
901
138.6 ms
37.3 ms
3.7x
Shared modules in route chunks
1,397
901
97.0 ms
39.4 ms
2.5x
Many non-route async chunks
3,245
3,101
235.1 ms
53.4 ms
4.4x
For each app, the output was compared with main on Rspack 2.0.0, 2.1.1 and 2.2.3 and was identical.
Tests
New unit tests in normalized-client-build.test.ts use a small fake chunk graph. They check that:
imports and dynamic imports are listed in the same order as before (this test also passes on main);
modules are read only from chunks with a route-split or hydration module;
each chunk and chunk group property is read at most once.
@tanstack/start-plugin-coretest:unit, test:types and test:eslint pass.
I have tested code changes locally with the relevant test commands, or tests do not apply to this pull request.
I fully understand the code in this pull request, including any code generated with AI assistance.
🚀 Release Impact
This change affects published code, and I have generated a changeset.
This change is docs/CI/dev-only (no release).
Summary by CodeRabbit
Performance
Improved Start manifest generation speed for Rsbuild client builds with many chunks, reducing build-time overhead in larger applications. Generated manifests continue to include route and hydration information, with static and dynamic imports kept in traversal order. This helps larger builds complete more efficiently while preserving the manifest details and dependency ordering applications rely on.
Remove the real Rspack compile fixture, its helper and snapshots. Cover the
same behavior in the existing normalized-client-build unit tests with a small
fake chunk graph: output order of imports and dynamic imports, module reads
only for route-split and hydration chunks, and one read per chunk graph
property.
…eader
Clarify comments on the chunk scan and read helpers, read child groups
directly since their merged JS files are already cached per group, keep the
unrelated isManifestJsAsset line unchanged, and assert the set of chunks whose
modules are read.
No actionable comments were generated in the recent review.
Walkthrough
Rsbuild client build normalization now caches chunk and group graph reads for asset and dependency collection. It selects chunks containing tagged modules for route and hydration metadata. Tests cover normalized output, graph-read limits, and metadata in concatenated modules.
Changes: Rsbuild Client Build Normalization
Layer / File(s)
Summary
Cached graph reads and metadata selection packages/start-plugin-core/src/rsbuild/normalized-client-build.ts
A per-normalization reader caches chunk and group data. It derives ordered, deduplicated JavaScript, CSS, and dynamic-import lists. Route and hydration metadata extraction targets chunks containing tagged modules, including nested concatenated modules.
Dependency and normalization integration packages/start-plugin-core/src/rsbuild/normalized-client-build.ts, packages/start-plugin-core/tests/rsbuild/normalized-client-build.test.ts, .changeset/rsbuild-client-build-capture.md
Entrypoints, static dependencies, CSS-only dependencies, and RSC assets use the shared reader. Tests cover normalized output, graph and module reads, and concatenated-module metadata. The changeset describes faster manifest generation for Rsbuild client builds with many chunks.
This change speeds up Rsbuild client build normalization without changing its output. No merge-blocking risk was identified.
Pre-merge checks: 4
Check name
Status
Explanation
Title check
The title names the scope (start-plugin-core), the change type (perf), and the target (rsbuild client build normalization). It matches the main change: normalizeRspackClientBuild now reads chunk prope…
Description check
The description follows the template. It has the Changes section with motivation, benchmark data, and test notes. The checklist is filled in, and both release-impact boxes are handled (changeset is ma…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Title check
Explanation
The title names the scope (start-plugin-core), the change type (perf), and the target (rsbuild client build normalization). It matches the main change: normalizeRspackClientBuild now reads chunk properties once and reads modules only from tagged chunks. The title is specific and short, so it passes.
Full details: Description check
Explanation
The description follows the template. It has the Changes section with motivation, benchmark data, and test notes. The checklist is filled in, and both release-impact boxes are handled (changeset is marked as generated). It discloses no AI authorship, which the template's AI-authored rules ask for. The PR objectives say the author used AI assistance in part, but the description does not state that. The description is otherwise complete.
Finishing Touches
Generate unit tests (beta)
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.
Merging this PR will degrade performance by 14.02%
⚠️Different runtime environments detected
Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.
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
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.
🎯 Changes
normalizeRspackClientBuildconverts the Rspack client compilation into the data used to build the Start manifest. It runs on every Rsbuild build and on every dev rebuild. This PR makes it faster for builds with many chunks. The output does not change.files,auxiliaryFiles,groupsIterable,chunks,childrenIterable) are now read once per call. Rspack builds a new collection on every read, and the previous code read them many times per chunk.getChunkModulesfor every chunk.Benchmarks
Median of 21 runs of
normalizeRspackClientBuildon generated apps, Rspack 2.2.3, production unless noted:mainFor each app, the output was compared with
mainon Rspack 2.0.0, 2.1.1 and 2.2.3 and was identical.Tests
normalized-client-build.test.tsuse a small fake chunk graph. They check that:main);@tanstack/start-plugin-coretest:unit,test:typesandtest:eslintpass.rsbuild,rsc-rsbuildandhmrdeferred-hydration(production and dev)deferred-hydrationcss-inline, with and without transform assets✅ Checklist
🚀 Release Impact
Summary by CodeRabbit