Minification generates invalid code containing break to out of scope label

[YarnSphere]

Minimal reproduction:

l: {
  try {
    while (true) {}
    break l;
  } catch {}
}

esbuild 0.24.2 output:

try{e:for(;;);break e}catch{}

Link to reproduction: https://esbuild.github.io/try/#dAAwLjI0LjIALS1taW5pZnkAbDogewogIHRyeSB7CiAgICB3aGlsZSAodHJ1ZSkge30KICAgIGJyZWFrIGw7CiAgfSBjYXRjaCB7fQp9

Hit this when bundling JS code generated by the Kotlin compiler, which at some point contains the above pattern.

[YarnSphere]

It seems like this was introduced in version 0.21.4.

Output of version 0.21.3:

e:try{for(;;);break e}catch{}

At a glance, this could be related to the changelog entry:

Fix for await transform when a label is present

This release fixes a bug where the for await transform, which wraps the loop in a try statement, previously failed to also move the loop's label into the try statement. This bug only affects code that uses both of these features in combination. Here's an example of some affected code:

[evanw]

Thanks for the report. The full commit that introduced this problem was b004bbe. There's no associated issue in this repo because it resulted from another issue I found here: arogozine/LinqToTypeScript#29.