Skip to content

fix(angular-rspack): ensure rebuild chunks emitted summary accurate#34979

Merged
FrozenPandaz merged 1 commit into
nrwl:masterfrom
Coly010:angular-rspack/rebuild
Mar 24, 2026
Merged

fix(angular-rspack): ensure rebuild chunks emitted summary accurate#34979
FrozenPandaz merged 1 commit into
nrwl:masterfrom
Coly010:angular-rspack/rebuild

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Mar 24, 2026

Current Behavior

Currently, rebuilds produce a summary table that is not accurate.
A key difference in how Rspack sets chunks.rendered compared to Webpack resulted in the logic for determining rebuilt chunks to be incorrect.

Expected Behavior

Ensure the summary table for emitted chunks is accurate on rebuild

Related Issues

It does not fully solve #34936, however it should be a good first step to finding out if too many chunks are being emitted

@Coly010 Coly010 requested a review from a team as a code owner March 24, 2026 12:42
@Coly010 Coly010 requested a review from FrozenPandaz March 24, 2026 12:42
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 24, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 744ff8c
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69c286ced000b400089207d9
😎 Deploy Preview https://deploy-preview-34979--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 24, 2026

👷 Deploy request for nx-dev pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 744ff8c

@jaysoo
Copy link
Copy Markdown
Member

jaysoo commented Mar 24, 2026

Thanks @Coly010 !

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Mar 24, 2026

View your CI Pipeline Execution ↗ for commit 744ff8c

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 55s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 8s View ↗
nx-cloud record -- pnpm nx conformance:check ✅ Succeeded 9s View ↗
nx build workspace-plugin ✅ Succeeded 2m 12s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-24 16:09:43 UTC

@FrozenPandaz FrozenPandaz merged commit 36534a4 into nrwl:master Mar 24, 2026
20 checks passed
FrozenPandaz pushed a commit that referenced this pull request Mar 26, 2026
…34979)

## Current Behavior
Currently, rebuilds produce a summary table that is not accurate.
A key difference in how Rspack sets `chunks.rendered` compared to
Webpack resulted in the logic for determining rebuilt chunks to be
incorrect.

## Expected Behavior
Ensure the summary table for emitted chunks is accurate on rebuild

## Related Issues

It does not _fully_ solve #34936, however it should be a good first step
to finding out if too many chunks are being emitted

(cherry picked from commit 36534a4)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants