Fix bundler bug with var nested inside if (#4348)
This release fixes a bug with the bundler that happens when importing an ES module using require (which causes it to be wrapped) and there's a top-level var inside an if statement without being wrapped in a { ... } block (and a few other conditions). The bundling transform needed to hoist these var declarations outside of the lazy ES module wrapper for correctness. See the issue for details.
Fix minifier bug with for inside try inside label (#4351)
This fixes an old regression from version v0.21.4. Some code was introduced to move the label inside the try statement to address a problem with transforming labeled for await loops to avoid the await (the transformation involves converting the for await loop into a for loop and wrapping it in a try statement). However, it introduces problems for cross-compiled JVM code that uses all three of these features heavily. This release restricts this transform to only apply to for loops that esbuild itself generates internally as part of the for await transform. Here is an example of some affected code:
// Original code
d:{e:{try{while(1){breakd}}catch{breake;}}}// Old output (with --minify)
a:try{e:for(;;)breaka}catch{breake}// New output (with --minify)
a:e:try{for(;;)breaka}catch{breake}
Inline IIFEs containing a single expression (#4354)
Previously inlining of IIFEs (immediately-invoked function expressions) only worked if the body contained a single return statement. Now it should also work if the body contains a single expression statement instead:
// Original code
constfoo=()=>{constcb=()=>{console.log(x())}returncb()}// Old output (with --minify)
constfoo=()=>(()=>{console.log(x())})();// New output (with --minify)
constfoo=()=>{console.log(x())};
The minifier now strips empty finally clauses (#4353)
This improvement means that finally clauses containing dead code can potentially cause the associated try statement to be removed from the output entirely in minified builds:
// Original code
functionfoo(callback){if(DEBUG)stack.push(callback.name);try{callback();}finally{if(DEBUG)stack.pop();}}// Old output (with --minify --define:DEBUG=false)
functionfoo(a){try{a()}finally{}}// New output (with --minify --define:DEBUG=false)
functionfoo(a){a()}
Allow tree-shaking of the Symbol constructor
With this release, calling Symbol is now considered to be side-effect free when the argument is known to be a primitive value. This means esbuild can now tree-shake module-level symbol variables:
// Original code
consta=Symbol('foo')constb=Symbol(bar)// Old output (with --tree-shaking=true)
consta=Symbol("foo");constb=Symbol(bar);// New output (with --tree-shaking=true)
constb=Symbol(bar);
Configuration
📅Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.
🔕Ignore: Close this MR and you won't be reminded about this update again.
If you want to rebase/retry this MR, check this box
This MR contains the following updates:
| Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [esbuild](https://github.com/evanw/esbuild) | [`0.27.0` -> `0.27.1`](https://renovatebot.com/diffs/npm/esbuild/0.27.0/0.27.1) |  |  |
---
### Release Notes
<details>
<summary>evanw/esbuild (esbuild)</summary>
### [`v0.27.1`](https://github.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#0271)
[Compare Source](https://github.com/evanw/esbuild/compare/v0.27.0...v0.27.1)
- Fix bundler bug with `var` nested inside `if` ([#​4348](https://github.com/evanw/esbuild/issues/4348))
This release fixes a bug with the bundler that happens when importing an ES module using `require` (which causes it to be wrapped) and there's a top-level `var` inside an `if` statement without being wrapped in a `{ ... }` block (and a few other conditions). The bundling transform needed to hoist these `var` declarations outside of the lazy ES module wrapper for correctness. See the issue for details.
- Fix minifier bug with `for` inside `try` inside label ([#​4351](https://github.com/evanw/esbuild/issues/4351))
This fixes an old regression from [version v0.21.4](https://github.com/evanw/esbuild/releases/v0.21.4). Some code was introduced to move the label inside the `try` statement to address a problem with transforming labeled `for await` loops to avoid the `await` (the transformation involves converting the `for await` loop into a `for` loop and wrapping it in a `try` statement). However, it introduces problems for cross-compiled JVM code that uses all three of these features heavily. This release restricts this transform to only apply to `for` loops that esbuild itself generates internally as part of the `for await` transform. Here is an example of some affected code:
```js
// Original code
d: {
e: {
try {
while (1) { break d }
} catch { break e; }
}
}
// Old output (with --minify)
a:try{e:for(;;)break a}catch{break e}
// New output (with --minify)
a:e:try{for(;;)break a}catch{break e}
```
- Inline IIFEs containing a single expression ([#​4354](https://github.com/evanw/esbuild/issues/4354))
Previously inlining of IIFEs (immediately-invoked function expressions) only worked if the body contained a single `return` statement. Now it should also work if the body contains a single expression statement instead:
```js
// Original code
const foo = () => {
const cb = () => {
console.log(x())
}
return cb()
}
// Old output (with --minify)
const foo=()=>(()=>{console.log(x())})();
// New output (with --minify)
const foo=()=>{console.log(x())};
```
- The minifier now strips empty `finally` clauses ([#​4353](https://github.com/evanw/esbuild/issues/4353))
This improvement means that `finally` clauses containing dead code can potentially cause the associated `try` statement to be removed from the output entirely in minified builds:
```js
// Original code
function foo(callback) {
if (DEBUG) stack.push(callback.name);
try {
callback();
} finally {
if (DEBUG) stack.pop();
}
}
// Old output (with --minify --define:DEBUG=false)
function foo(a){try{a()}finally{}}
// New output (with --minify --define:DEBUG=false)
function foo(a){a()}
```
- Allow tree-shaking of the `Symbol` constructor
With this release, calling `Symbol` is now considered to be side-effect free when the argument is known to be a primitive value. This means esbuild can now tree-shake module-level symbol variables:
```js
// Original code
const a = Symbol('foo')
const b = Symbol(bar)
// Old output (with --tree-shaking=true)
const a = Symbol("foo");
const b = Symbol(bar);
// New output (with --tree-shaking=true)
const b = Symbol(bar);
```
</details>
---
### Configuration
📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.
♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 **Ignore**: Close this MR and you won't be reminded about this update again.
---
- [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box
---
This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi4zMy4xIiwidXBkYXRlZEluVmVyIjoiNDIuMzMuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
This MR contains the following updates:
0.27.0->0.27.1Release Notes
evanw/esbuild (esbuild)
v0.27.1Compare Source
Fix bundler bug with
varnested insideif(#4348)This release fixes a bug with the bundler that happens when importing an ES module using
require(which causes it to be wrapped) and there's a top-levelvarinside anifstatement without being wrapped in a{ ... }block (and a few other conditions). The bundling transform needed to hoist thesevardeclarations outside of the lazy ES module wrapper for correctness. See the issue for details.Fix minifier bug with
forinsidetryinside label (#4351)This fixes an old regression from version v0.21.4. Some code was introduced to move the label inside the
trystatement to address a problem with transforming labeledfor awaitloops to avoid theawait(the transformation involves converting thefor awaitloop into aforloop and wrapping it in atrystatement). However, it introduces problems for cross-compiled JVM code that uses all three of these features heavily. This release restricts this transform to only apply toforloops that esbuild itself generates internally as part of thefor awaittransform. Here is an example of some affected code:Inline IIFEs containing a single expression (#4354)
Previously inlining of IIFEs (immediately-invoked function expressions) only worked if the body contained a single
returnstatement. Now it should also work if the body contains a single expression statement instead:The minifier now strips empty
finallyclauses (#4353)This improvement means that
finallyclauses containing dead code can potentially cause the associatedtrystatement to be removed from the output entirely in minified builds:Allow tree-shaking of the
SymbolconstructorWith this release, calling
Symbolis now considered to be side-effect free when the argument is known to be a primitive value. This means esbuild can now tree-shake module-level symbol variables:Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this MR and you won't be reminded about this update again.
This MR has been generated by Renovate Bot.
restored source branch
renovate/esbuild-0.xmentioned in issue #49
mentioned in commit
812ea12873