Improve ChunkLoadError message for loaded chunks#20942
Improve ChunkLoadError message for loaded chunks#20942muskaan1212 wants to merge 1 commit intowebpack:mainfrom
Conversation
|
|
|
|
👋 Hi there! This PR was automatically flagged and closed by our quality checks. If you believe this was a mistake, please review our contributing guidelines |
|
Hi maintainers, thanks for the automated checks. I noticed this PR was closed because my CLA was not signed and possibly due to quality/AI policy checks. I will complete the CLA authorization first. For context, my intention with this PR was to make the Could you please let me know whether this wording-only change is acceptable after CLA completion, or whether I should instead investigate a way to preserve/report the original error? |
|
Hi maintainers, I have completed the CLA process. I understand the PR was automatically closed by quality checks. I’ve also clarified the AI usage section to explain that AI was used only for learning/navigation support, and that I manually reviewed the code change and verified the diff before submission. Could you please let me know if this PR can be reopened, or if you would prefer that I investigate a deeper approach for #16618, such as preserving/reporting the original runtime or parsing error instead of only changing the wording? |
1 similar comment
|
Hi maintainers, I have completed the CLA process. I understand the PR was automatically closed by quality checks. I’ve also clarified the AI usage section to explain that AI was used only for learning/navigation support, and that I manually reviewed the code change and verified the diff before submission. Could you please let me know if this PR can be reopened, or if you would prefer that I investigate a deeper approach for #16618, such as preserving/reporting the original runtime or parsing error instead of only changing the wording? |
Summary
This PR improves the
ChunkLoadErrormessage for cases where the scriptloadevent fires, but the chunk is still not registered as loaded.Previously, webpack reported this case as
missing, which can be misleading because the chunk script may have loaded successfully but failed during parsing, evaluation, or execution. This change updates the message toloaded but not executedto make the failure reason easier to understand while debugging.Related to #16618
What kind of change does this PR introduce?
fix
Did you add tests for your changes?
No tests have been added yet. This is a small runtime error-message improvement, and I would appreciate maintainer guidance on the preferred test location or whether an existing runtime test should be updated.
Does this PR introduce a breaking change?
No. This only changes the wording of the
ChunkLoadErrortype/message for this specific load-event case.If relevant, what needs to be documented once your changes are merged or what have you already documented?
No documentation changes are required at this time.
Use of AI
I used AI assistance for learning support while understanding the issue discussion and navigating the webpack codebase. I manually reviewed the relevant file, made the code change myself, checked the diff, and verified it with
git diff --checkbefore submission. AI was not used to generate unreviewed code.