feat: implement module.generator.json.JSONParse#19320
Merged
alexander-akait merged 1 commit intowebpack:mainfrom Mar 13, 2025
Merged
feat: implement module.generator.json.JSONParse#19320alexander-akait merged 1 commit intowebpack:mainfrom
module.generator.json.JSONParse#19320alexander-akait merged 1 commit intowebpack:mainfrom
Conversation
For large `.json` modules, webpack will generate `JSON.parse` by default for better performance. But there are some circumstances that `JSON.parse` is not a good choice (e.g.: when doing AOT compilation). Thus, a new generator option `module.generator.json.JSONParse` is added to disable generating `JSON.parse` for `.json` module. The default value is kept as `true` and can be opt-out by custom rules. fix: webpack#19319
The committers listed above are authorized under a signed CLA. |
alexander-akait
approved these changes
Mar 13, 2025
colinaaa
added a commit
to colinaaa/rspack
that referenced
this pull request
Mar 13, 2025
For large `.json` modules, webpack will generate `JSON.parse` by default for better performance. But there are some circumstances that `JSON.parse` is not a good choice (e.g.: when doing AOT compilation). Thus, a new generator option `module.generator.json.JSONParse` is added to disable generating `JSON.parse` for `.json` module. The default value is kept as `true` and can be opt-out by custom rules. This is a port of webpack/webpack#19320.
2 tasks
ahabhgk
pushed a commit
to web-infra-dev/rspack
that referenced
this pull request
Mar 14, 2025
For large `.json` modules, webpack will generate `JSON.parse` by default for better performance. But there are some circumstances that `JSON.parse` is not a good choice (e.g.: when doing AOT compilation). Thus, a new generator option `module.generator.json.JSONParse` is added to disable generating `JSON.parse` for `.json` module. The default value is kept as `true` and can be opt-out by custom rules. This is a port of webpack/webpack#19320.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
For large
.jsonmodules, webpack will generateJSON.parseby default forbetter performance. But there are some circumstances that
JSON.parseis not agood choice (e.g.: when doing AOT compilation).
Thus, a new generator option
module.generator.json.JSONParseis added todisable generating
JSON.parsefor.jsonmodule.The default value is kept as
trueand can be opt-out by custom rules.fix: #19319
What kind of change does this PR introduce?
feature
Did you add tests for your changes?
yes
Does this PR introduce a breaking change?
no
What needs to be documented once your changes are merged?
The new option should be documented https://webpack.js.org/configuration/module/#modulegenerator