Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] Use Babel assumptions instead of loose mode #37461

Closed
wants to merge 12 commits into from
Closed

[core] Use Babel assumptions instead of loose mode #37461

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6c1a983
Use Babel assumptions instead of `loose` mode
nicolo-ribaudo Jun 1, 2023
ac4be35
Update package.json
nicolo-ribaudo Jun 1, 2023
6469928
Merge branch 'master' into babel-assumptions-instead-of-loose-mode
ZeeshanTamboli Jun 9, 2023
856fadc
fix typo
ZeeshanTamboli Jun 20, 2023
9306e9e
always include @babel/plugin-proposal-class-properties
ZeeshanTamboli Jun 20, 2023
6776ae7
remove @babel/plugin-proposal-class-properties from yarn resolutions
ZeeshanTamboli Jun 20, 2023
bffb244
Merge branch 'master' into babel-assumptions-instead-of-loose-mode
ZeeshanTamboli Jun 20, 2023
44c6844
prettier
ZeeshanTamboli Jun 20, 2023
3f783ba
remove @babel/plugin-proposal-object-rest-spread instead of class-pro…
ZeeshanTamboli Jun 20, 2023
2fb933b
always include @babel/plugin-proposal-object-rest-spread
ZeeshanTamboli Jun 21, 2023
18f25cb
Merge branch 'master' into babel-assumptions-instead-of-loose-mode
ZeeshanTamboli Jun 21, 2023
fcf68f3
prettier
ZeeshanTamboli Jun 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 11 additions & 6 deletions babel.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,10 @@ module.exports = function getBabelConfig(api) {
debug: process.env.MUI_BUILD_VERBOSE === 'true',
modules: useESModules ? false : 'commonjs',
shippedProposals: api.env('modern'),
include: [
'@babel/plugin-proposal-class-properties',
'@babel/plugin-proposal-object-rest-spread',
],
},
],
[
Expand All @@ -62,12 +66,6 @@ module.exports = function getBabelConfig(api) {
},
],
'babel-plugin-optimize-clsx',
// Need the following 3 proposals for all targets in .browserslistrc.
// With our usage the transpiled loose mode is equivalent to spec mode.
['@babel/plugin-proposal-class-properties', { loose: true }],
['@babel/plugin-proposal-private-methods', { loose: true }],
['@babel/plugin-proposal-private-property-in-object', { loose: true }],
['@babel/plugin-proposal-object-rest-spread', { loose: true }],
[
'@babel/plugin-transform-runtime',
{
Expand Down Expand Up @@ -98,8 +96,15 @@ module.exports = function getBabelConfig(api) {
}

return {
// https://babeljs.io/docs/assumptions
assumptions: {
noDocumentAll: true,
// With our case these assumptions are safe, and the
// resulting behavior is equivalent to spec mode.
setPublicClassFields: true,
privateFieldsAsProperties: true,
objectRestNoSymbols: true,
setSpreadProperties: true,
Copy link
Author

Choose a reason for hiding this comment

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

Feel free to go through https://babeljs.io/docs/assumptions and check if there is anything else you want to enable :)

Copy link
Author

Choose a reason for hiding this comment

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

One possible effect of this PR is that setSpreadProperties is applied more, because object spread is also compiled in some cases by the destructuring transform and you didn't enable loose mode in that plugin. Now all the object spreads are transformed with this assumption, regardless of which plugin actually does it.

},
presets,
plugins,
Expand Down