Skip to content

Conversation

@JSerFeng
Copy link
Contributor

@JSerFeng JSerFeng commented Nov 10, 2025

Summary

Should always make sure entry chunk signature won't be ruined by other chunk imports.

When user use splitChunk to split module out from the entry chunk, there is a cycle and other chunks may import the symbol from the entry chunk. The entry chunk has to expose those symbols.

Check if other chunk has modules depends on modules inside entry chunk, if so, generate a chunk to avoid such things

Related links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@JSerFeng JSerFeng requested a review from quininer as a code owner November 10, 2025 02:50
Copilot AI review requested due to automatic review settings November 10, 2025 02:50
@netlify
Copy link

netlify bot commented Nov 10, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 0415db6
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/69117a278d631f0008bd9a41

@JSerFeng JSerFeng requested review from SyMind and ahabhgk November 10, 2025 02:51
@JSerFeng JSerFeng force-pushed the fix/prevent-entry-signiture branch from 79465c3 to bf4e0b6 Compare November 10, 2025 02:52
@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Nov 10, 2025
@JSerFeng JSerFeng enabled auto-merge (squash) November 10, 2025 03:02
@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Nov 10, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@JSerFeng
Copy link
Contributor Author

@copilot

Copy link
Contributor

Copilot AI commented Nov 10, 2025

@JSerFeng I've opened a new pull request, #12134, to work on those changes. Once the pull request is ready, I'll request review from you.

@JSerFeng JSerFeng force-pushed the fix/prevent-entry-signiture branch from 6d28905 to c086536 Compare November 10, 2025 03:32
quininer
quininer previously approved these changes Nov 10, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

📦 Binary Size-limit

Comparing 0415db6 to fix: should not inject css prefetch/preload runtime when only prefetch/preload js chunks (#12112) by harpsealjs

❌ Size increased by 26.50KB from 47.93MB to 47.95MB (⬆️0.05%)

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 10, 2025

CodSpeed Performance Report

Merging #12133 will not alter performance

Comparing fix/prevent-entry-signiture (0415db6) with main (4506d64)

Summary

✅ 17 untouched

@JSerFeng JSerFeng merged commit 0f02db9 into main Nov 10, 2025
92 of 96 checks passed
@JSerFeng JSerFeng deleted the fix/prevent-entry-signiture branch November 10, 2025 06:39
@Timeless0911
Copy link
Contributor

Timeless0911 commented Nov 10, 2025

Fix: web-infra-dev/rsbuild#6524

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants