Skip to content

Commit

Permalink
chore(deps): update dependency esbuild to ^0.20.0 (#250)
Browse files Browse the repository at this point in the history
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [esbuild](https://togithub.com/evanw/esbuild) | [`^0.19.5` ->
`^0.20.0`](https://renovatebot.com/diffs/npm/esbuild/0.19.5/0.20.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/esbuild/0.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/esbuild/0.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/esbuild/0.19.5/0.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/esbuild/0.19.5/0.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>evanw/esbuild (esbuild)</summary>

###
[`v0.20.0`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#0200)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.12...v0.20.0)

**This release deliberately contains backwards-incompatible changes.**
To avoid automatically picking up releases like this, you should either
be pinning the exact version of `esbuild` in your `package.json` file
(recommended) or be using a version range syntax that only accepts patch
upgrades such as `^0.19.0` or `~0.19.0`. See npm's documentation about
[semver](https://docs.npmjs.com/cli/v6/using-npm/semver/) for more
information.

This time there is only one breaking change, and it only matters for
people using Deno. Deno tests that use esbuild will now fail unless you
make the change described below.

- Work around API deprecations in Deno 1.40.x
([#&#8203;3609](https://togithub.com/evanw/esbuild/issues/3609),
[#&#8203;3611](https://togithub.com/evanw/esbuild/pull/3611))

[Deno 1.40.0](https://deno.com/blog/v1.40) was just released and
introduced run-time warnings about certain APIs that esbuild uses. With
this release, esbuild will work around these run-time warnings by using
newer APIs if they are present and falling back to the original APIs
otherwise. This should avoid the warnings without breaking compatibility
with older versions of Deno.

Unfortunately, doing this introduces a breaking change. The newer child
process APIs lack a way to synchronously terminate esbuild's child
process, so calling `esbuild.stop()` from within a Deno test is no
longer sufficient to prevent Deno from failing a test that uses
esbuild's API (Deno fails tests that create a child process without
killing it before the test ends). To work around this, esbuild's
`stop()` function has been changed to return a promise, and you now have
to change `esbuild.stop()` to `await esbuild.stop()` in all of your Deno
tests.

- Reorder implicit file extensions within `node_modules`
([#&#8203;3341](https://togithub.com/evanw/esbuild/issues/3341),
[#&#8203;3608](https://togithub.com/evanw/esbuild/issues/3608))

In [version
0.18.0](https://togithub.com/evanw/esbuild/releases/v0.18.0), esbuild
changed the behavior of implicit file extensions within `node_modules`
directories (i.e. in published packages) to prefer `.js` over `.ts` even
when the `--resolve-extensions=` order prefers `.ts` over `.js` (which
it does by default). However, doing that also accidentally made esbuild
prefer `.css` over `.ts`, which caused problems for people that
published packages containing both TypeScript and CSS in files with the
same name.

With this release, esbuild will reorder TypeScript file extensions
immediately after the last JavaScript file extensions in the implicit
file extension order instead of putting them at the end of the order.
Specifically the default implicit file extension order is
`.tsx,.ts,.jsx,.js,.css,.json` which used to become
`.jsx,.js,.css,.json,.tsx,.ts` in `node_modules` directories. With this
release it will now become `.jsx,.js,.tsx,.ts,.css,.json` instead.

Why even rewrite the implicit file extension order at all? One reason is
because the `.js` file is more likely to behave correctly than the `.ts`
file. The behavior of the `.ts` file may depend on `tsconfig.json` and
the `tsconfig.json` file may not even be published, or may use `extends`
to refer to a base `tsconfig.json` file that wasn't published. People
can get into this situation when they forget to add all `.ts` files to
their `.npmignore` file before publishing to npm. Picking `.js` over
`.ts` helps make it more likely that resulting bundle will behave
correctly.

###
[`v0.19.12`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#01912)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.11...v0.19.12)

- The "preserve" JSX mode now preserves JSX text verbatim
([#&#8203;3605](https://togithub.com/evanw/esbuild/issues/3605))

The [JSX specification](https://facebook.github.io/jsx/) deliberately
doesn't specify how JSX text is supposed to be interpreted and there is
no canonical way to interpret JSX text. Two most popular interpretations
are Babel and TypeScript. Yes [they are
different](https://twitter.com/jarredsumner/status/1456118847937781764)
(esbuild [deliberately follows
TypeScript](https://twitter.com/evanwallace/status/1456122279453208576)
by the way).

Previously esbuild normalized text to the TypeScript interpretation when
the "preserve" JSX mode is active. However, "preserve" should arguably
reproduce the original JSX text verbatim so that whatever JSX transform
runs after esbuild is free to interpret it however it wants. So with
this release, esbuild will now pass JSX text through unmodified:

    ```jsx
    // Original code
    let el =
      <a href={'/'} title='&apos;&quot;'> some text
        {foo}
          more text </a>

    // Old output (with --loader=jsx --jsx=preserve)
    let el = <a href="/" title={`'"`}>
      {" some text"}
      {foo}
      {"more text "}
    </a>;

    // New output (with --loader=jsx --jsx=preserve)
    let el = <a href={"/"} title='&apos;&quot;'> some text
        {foo}
          more text </a>;
    ```

-   Allow JSX elements as JSX attribute values

JSX has an obscure feature where you can use JSX elements in attribute
position without surrounding them with `{...}`. It looks like this:

    ```jsx
    let el = <div data-ab=<><a/><b/></>/>;
    ```

I think I originally didn't implement it even though it's part of the
[JSX specification](https://facebook.github.io/jsx/) because it
previously didn't work in TypeScript (and potentially also in Babel?).
However, support for it was [silently added in TypeScript
4.8](https://togithub.com/microsoft/TypeScript/pull/47994) without me
noticing and Babel has also since fixed their [bugs regarding this
feature](https://togithub.com/babel/babel/pull/6006). So I'm adding it
to esbuild too now that I know it's widely supported.

Keep in mind that there is some ongoing discussion about [removing this
feature from JSX](https://togithub.com/facebook/jsx/issues/53). I agree
that the syntax seems out of place (it does away with the elegance of
"JSX is basically just XML with `{...}` escapes" for something arguably
harder to read, which doesn't seem like a good trade-off), but it's in
the specification and TypeScript and Babel both implement it so I'm
going to have esbuild implement it too. However, I reserve the right to
remove it from esbuild if it's ever removed from the specification in
the future. So use it with caution.

- Fix a bug with TypeScript type parsing
([#&#8203;3574](https://togithub.com/evanw/esbuild/issues/3574))

This release fixes a bug with esbuild's TypeScript parser where a
conditional type containing a union type that ends with an infer type
that ends with a constraint could fail to parse. This was caused by the
"don't parse a conditional type" flag not getting passed through the
union type parser. Here's an example of valid TypeScript code that
previously failed to parse correctly:

    ```ts
type InferUnion<T> = T extends { a: infer U extends number } | infer U
extends number ? U : never
    ```

###
[`v0.19.11`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#01911)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.10...v0.19.11)

- Fix TypeScript-specific class transform edge case
([#&#8203;3559](https://togithub.com/evanw/esbuild/issues/3559))

The previous release introduced an optimization that avoided
transforming `super()` in the class constructor for TypeScript code
compiled with `useDefineForClassFields` set to `false` if all class
instance fields have no initializers. The rationale was that in this
case, all class instance fields are omitted in the output so no changes
to the constructor are needed. However, if all of this is the case *and*
there are `#private` instance fields with initializers, those private
instance field initializers were still being moved into the constructor.
This was problematic because they were being inserted before the call to
`super()` (since `super()` is now no longer transformed in that case).
This release introduces an additional optimization that avoids moving
the private instance field initializers into the constructor in this
edge case, which generates smaller code, matches the TypeScript
compiler's output more closely, and avoids this bug:

    ```ts
    // Original code
    class Foo extends Bar {
      #private = 1;
      public: any;
      constructor() {
        super();
      }
    }

    // Old output (with esbuild v0.19.9)
    class Foo extends Bar {
      constructor() {
        super();
        this.#private = 1;
      }
      #private;
    }

    // Old output (with esbuild v0.19.10)
    class Foo extends Bar {
      constructor() {
        this.#private = 1;
        super();
      }
      #private;
    }

    // New output
    class Foo extends Bar {
      #private = 1;
      constructor() {
        super();
      }
    }
    ```

- Minifier: allow reording a primitive past a side-effect
([#&#8203;3568](https://togithub.com/evanw/esbuild/issues/3568))

The minifier previously allowed reordering a side-effect past a
primitive, but didn't handle the case of reordering a primitive past a
side-effect. This additional case is now handled:

    ```js
    // Original code
    function f() {
      let x = false;
      let y = x;
      const boolean = y;
let frag = $.template(`<p contenteditable="${boolean}">hello
world</p>`);
      return frag;
    }

    // Old output (with --minify)
function f(){const e=!1;return $.template(`<p
contenteditable="${e}">hello world</p>`)}

    // New output (with --minify)
function f(){return $.template('<p contenteditable="false">hello
world</p>')}
    ```

- Minifier: consider properties named using known `Symbol` instances to
be side-effect free
([#&#8203;3561](https://togithub.com/evanw/esbuild/issues/3561))

Many things in JavaScript can have side effects including property
accesses and ToString operations, so using a symbol such as
`Symbol.iterator` as a computed property name is not obviously
side-effect free. This release adds a special case for known `Symbol`
instances so that they are considered side-effect free when used as
property names. For example, this class declaration will now be
considered side-effect free:

    ```js
    class Foo {
      *[Symbol.iterator]() {
      }
    }
    ```

- Provide the `stop()` API in node to exit esbuild's child process
([#&#8203;3558](https://togithub.com/evanw/esbuild/issues/3558))

You can now call `stop()` in esbuild's node API to exit esbuild's child
process to reclaim the resources used. It only makes sense to do this
for a long-lived node process when you know you will no longer be making
any more esbuild API calls. It is not necessary to call this to allow
node to exit, and it's advantageous to not call this in between calls to
esbuild's API as sharing a single long-lived esbuild child process is
more efficient than re-creating a new esbuild child process for every
API call. This API call used to exist but was removed in [version
0.9.0](https://togithub.com/evanw/esbuild/releases/v0.9.0). This release
adds it back due to a user request.

###
[`v0.19.10`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#01910)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.9...v0.19.10)

- Fix glob imports in TypeScript files
([#&#8203;3319](https://togithub.com/evanw/esbuild/issues/3319))

This release fixes a problem where bundling a TypeScript file containing
a glob import could emit a call to a helper function that doesn't exist.
The problem happened because esbuild's TypeScript transformation removes
unused imports (which is required for correctness, as they may be
type-only imports) and esbuild's glob import transformation wasn't
correctly marking the imported helper function as used. This wasn't
caught earlier because most of esbuild's glob import tests were written
in JavaScript, not in TypeScript.

- Fix `require()` glob imports with bundling disabled
([#&#8203;3546](https://togithub.com/evanw/esbuild/issues/3546))

Previously `require()` calls containing glob imports were incorrectly
transformed when bundling was disabled. All glob imports should only be
transformed when bundling is enabled. This bug has been fixed.

- Fix a panic when transforming optional chaining with `define`
([#&#8203;3551](https://togithub.com/evanw/esbuild/issues/3551),
[#&#8203;3554](https://togithub.com/evanw/esbuild/pull/3554))

This release fixes a case where esbuild could crash with a panic, which
was triggered by using `define` to replace an expression containing an
optional chain. Here is an example:

    ```js
    // Original code
    console.log(process?.env.SHELL)

    // Old output (with --define:process.env={})
    /* panic: Internal error (while parsing "<stdin>") */

    // New output (with --define:process.env={})
    var define_process_env_default = {};
    console.log(define_process_env_default.SHELL);
    ```

This fix was contributed by
[@&#8203;hi-ogawa](https://togithub.com/hi-ogawa).

- Work around a bug in node's CommonJS export name detector
([#&#8203;3544](https://togithub.com/evanw/esbuild/issues/3544))

The export names of a CommonJS module are dynamically-determined at run
time because CommonJS exports are properties on a mutable object. But
the export names of an ES module are statically-determined at module
instantiation time by using `import` and `export` syntax and cannot be
changed at run time.

When you import a CommonJS module into an ES module in node, node scans
over the source code to attempt to detect the set of export names that
the CommonJS module will end up using. That statically-determined set of
names is used as the set of names that the ES module is allowed to
import at module instantiation time. However, this scan appears to have
bugs (or at least, can cause false positives) because it doesn't appear
to do any scope analysis. Node will incorrectly consider the module to
export something even if the assignment is done to a local variable
instead of to the module-level `exports` object. For example:

    ```js
    // confuseNode.js
    exports.confuseNode = function(exports) {
      // If this local is called "exports", node incorrectly
      // thinks this file has an export called "notAnExport".
      exports.notAnExport = function() {
      };
    };
    ```

You can see that node incorrectly thinks the file `confuseNode.js` has
an export called `notAnExport` when that file is loaded in an ES module
context:

    ```console
    $ node -e 'import("./confuseNode.js").then(console.log)'
    [Module: null prototype] {
      confuseNode: [Function (anonymous)],
      default: { confuseNode: [Function (anonymous)] },
      notAnExport: undefined
    }
    ```

To avoid this, esbuild will now rename local variables that use the
names `exports` and `module` when generating CommonJS output for the
`node` platform.

- Fix the return value of esbuild's `super()` shim
([#&#8203;3538](https://togithub.com/evanw/esbuild/issues/3538))

Some people write `constructor` methods that use the return value of
`super()` instead of using `this`. This isn't too common because
[TypeScript doesn't let you do
that](https://togithub.com/microsoft/TypeScript/issues/37847) but it can
come up when writing JavaScript. Previously esbuild's class lowering
transform incorrectly transformed the return value of `super()` into
`undefined`. With this release, the return value of `super()` will now
be `this` instead:

    ```js
    // Original code
    class Foo extends Object {
      field
      constructor() {
        console.log(typeof super())
      }
    }
    new Foo

    // Old output (with --target=es6)
    class Foo extends Object {
      constructor() {
        var __super = (...args) => {
          super(...args);
          __publicField(this, "field");
        };
        console.log(typeof __super());
      }
    }
    new Foo();

    // New output (with --target=es6)
    class Foo extends Object {
      constructor() {
        var __super = (...args) => {
          super(...args);
          __publicField(this, "field");
          return this;
        };
        console.log(typeof __super());
      }
    }
    new Foo();
    ```

- Terminate the Go GC when esbuild's `stop()` API is called
([#&#8203;3552](https://togithub.com/evanw/esbuild/issues/3552))

If you use esbuild with WebAssembly and pass the `worker: false` flag to
`esbuild.initialize()`, then esbuild will run the WebAssembly module on
the main thread. If you do this within a Deno test and that test calls
`esbuild.stop()` to clean up esbuild's resources, Deno may complain that
a `setTimeout()` call lasted past the end of the test. This happens when
the Go is in the middle of a garbage collection pass and has scheduled
additional ongoing garbage collection work. Normally calling
`esbuild.stop()` will terminate the web worker that the WebAssembly
module runs in, which will terminate the Go GC, but that doesn't happen
if you disable the web worker with `worker: false`.

With this release, esbuild will now attempt to terminate the Go GC in
this edge case by calling `clearTimeout()` on these pending timeouts.

- Apply `/* @&#8203;__NO_SIDE_EFFECTS__ */` on tagged template literals
([#&#8203;3511](https://togithub.com/evanw/esbuild/issues/3511))

Tagged template literals that reference functions annotated with a
`@__NO_SIDE_EFFECTS__` comment are now able to be removed via
tree-shaking if the result is unused. This is a convention from
[Rollup](https://togithub.com/rollup/rollup/pull/5024). Here is an
example:

    ```js
    // Original code
const html = /* @&#8203;__NO_SIDE_EFFECTS__ */ (a, ...b) => ({ a, b })
    html`<a>remove</a>`
    x = html`<b>keep</b>`

    // Old output (with --tree-shaking=true)
const html = /* @&#8203;__NO_SIDE_EFFECTS__ */ (a, ...b) => ({ a, b });
    html`<a>remove</a>`;
    x = html`<b>keep</b>`;

    // New output (with --tree-shaking=true)
const html = /* @&#8203;__NO_SIDE_EFFECTS__ */ (a, ...b) => ({ a, b });
    x = html`<b>keep</b>`;
    ```

Note that this feature currently only works within a single file, so
it's not especially useful. This feature does not yet work across
separate files. I still recommend using `@__PURE__` annotations instead
of this feature, as they have wider tooling support. The drawback of
course is that `@__PURE__` annotations need to be added at each call
site, not at the declaration, and for non-call expressions such as
template literals you need to wrap the expression in an IIFE
(immediately-invoked function expression) to create a call expression to
apply the `@__PURE__` annotation to.

- Publish builds for IBM AIX PowerPC 64-bit
([#&#8203;3549](https://togithub.com/evanw/esbuild/issues/3549))

This release publishes a binary executable to npm for IBM AIX PowerPC
64-bit, which means that in theory esbuild can now be installed in that
environment with `npm install esbuild`. This hasn't actually been tested
yet. If you have access to such a system, it would be helpful to confirm
whether or not doing this actually works.

###
[`v0.19.9`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#0199)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.8...v0.19.9)

- Add support for transforming new CSS gradient syntax for older
browsers

The specification called [CSS Images Module Level
4](https://www.w3.org/TR/css-images-4/) introduces new CSS gradient
syntax for customizing how the browser interpolates colors in between
color stops. You can now control the color space that the interpolation
happens in as well as (for "polar" color spaces) control whether hue
angle interpolation happens clockwise or counterclockwise. You can read
more about this in [Mozilla's blog post about new CSS gradient
features](https://developer.mozilla.org/en-US/blog/css-color-module-level-4/).

With this release, esbuild will now automatically transform this syntax
for older browsers in the `target` list. For example, here's a gradient
that should appear as a rainbow in a browser that supports this new
syntax:

    ```css
    /* Original code */
    .rainbow-gradient {
      width: 100px;
      height: 100px;
background: linear-gradient(in hsl longer hue, #&#8203;7ff,
#&#8203;77f);
    }

    /* New output (with --target=chrome99) */
    .rainbow-gradient {
      width: 100px;
      height: 100px;
      background:
        linear-gradient(
          #&#8203;77ffff,
          #&#8203;77ffaa 12.5%,
          #&#8203;77ff80 18.75%,
          #&#8203;84ff77 21.88%,
          #&#8203;99ff77 25%,
          #eeff77 37.5%,
          #fffb77 40.62%,
          #ffe577 43.75%,
          #ffbb77 50%,
          #ff9077 56.25%,
          #ff7b77 59.38%,
          #ff7788 62.5%,
          #ff77dd 75%,
          #ff77f2 78.12%,
          #f777ff 81.25%,
          #cc77ff 87.5%,
          #&#8203;7777ff);
    }
    ```

You can now use this syntax in your CSS source code and esbuild will
automatically convert it to an equivalent gradient for older browsers.
In addition, esbuild will now also transform "double position" and
"transition hint" syntax for older browsers as appropriate:

    ```css
    /* Original code */
    .stripes {
      width: 100px;
      height: 100px;
background: linear-gradient(#e65 33%, #ff2 33% 67%, #&#8203;99e 67%);
    }
    .glow {
      width: 100px;
      height: 100px;
      background: radial-gradient(white 10%, 20%, black);
    }

    /* New output (with --target=chrome33) */
    .stripes {
      width: 100px;
      height: 100px;
      background:
        linear-gradient(
          #e65 33%,
          #ff2 33%,
          #ff2 67%,
          #&#8203;99e 67%);
    }
    .glow {
      width: 100px;
      height: 100px;
      background:
        radial-gradient(
          #ffffff 10%,
          #aaaaaa 12.81%,
          #&#8203;959595 15.62%,
          #&#8203;7b7b7b 21.25%,
          #&#8203;5a5a5a 32.5%,
          #&#8203;444444 43.75%,
          #&#8203;323232 55%,
          #&#8203;161616 77.5%,
          #&#8203;000000);
    }
    ```

You can see visual examples of these new syntax features by looking at
[esbuild's gradient transformation
tests](https://esbuild.github.io/gradient-tests/).

If necessary, esbuild will construct a new gradient that approximates
the original gradient by recursively splitting the interval in between
color stops until the approximation error is within a small threshold.
That is why the above output CSS contains many more color stops than the
input CSS.

Note that esbuild deliberately *replaces* the original gradient with the
approximation instead of inserting the approximation before the original
gradient as a fallback. The latest version of Firefox has multiple
gradient rendering bugs (including incorrect interpolation of
partially-transparent colors and interpolating non-sRGB colors using the
incorrect color space). If esbuild didn't replace the original gradient,
then Firefox would use the original gradient instead of the fallback the
appearance would be incorrect in Firefox. In other words, the latest
version of Firefox supports modern gradient syntax but interprets it
incorrectly.

- Add support for `color()`, `lab()`, `lch()`, `oklab()`, `oklch()`, and
`hwb()` in CSS

CSS has recently added lots of new ways of specifying colors. You can
read more about this in [Chrome's blog post about CSS color
spaces](https://developer.chrome.com/docs/css-ui/high-definition-css-color-guide).

This release adds support for minifying colors that use the `color()`,
`lab()`, `lch()`, `oklab()`, `oklch()`, or `hwb()` syntax and/or
transforming these colors for browsers that don't support it yet:

    ```css
    /* Original code */
    div {
      color: hwb(90deg 20% 40%);
      background: color(display-p3 1 0 0);
    }

    /* New output (with --target=chrome99) */
    div {
      color: #&#8203;669933;
      background: #ff0f0e;
      background: color(display-p3 1 0 0);
    }
    ```

As you can see, colors outside of the sRGB color space such as
`color(display-p3 1 0 0)` are mapped back into the sRGB gamut and
inserted as a fallback for browsers that don't support the new color
syntax.

- Allow empty type parameter lists in certain cases
([#&#8203;3512](https://togithub.com/evanw/esbuild/issues/3512))

TypeScript allows interface declarations and type aliases to have empty
type parameter lists. Previously esbuild didn't handle this edge case
but with this release, esbuild will now parse this syntax:

    ```ts
    interface Foo<> {}
    type Bar<> = {}
    ```

This fix was contributed by
[@&#8203;magic-akari](https://togithub.com/magic-akari).

###
[`v0.19.8`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#0198)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.7...v0.19.8)

- Add a treemap chart to esbuild's bundle analyzer
([#&#8203;2848](https://togithub.com/evanw/esbuild/issues/2848))

The bundler analyzer on esbuild's website
(https://esbuild.github.io/analyze/) now has a treemap chart type in
addition to the two existing chart types (sunburst and flame). This
should be more familiar for people coming from other similar tools, as
well as make better use of large screens.

- Allow decorators after the `export` keyword
([#&#8203;104](https://togithub.com/evanw/esbuild/issues/104))

Previously esbuild's decorator parser followed the original behavior of
TypeScript's experimental decorators feature, which only allowed
decorators to come before the `export` keyword. However, the upcoming
JavaScript decorators feature also allows decorators to come after the
`export` keyword. And with TypeScript 5.0, TypeScript now also allows
experimental decorators to come after the `export` keyword too. So
esbuild now allows this as well:

    ```js
    // This old syntax has always been permitted:
    @&#8203;decorator export class Foo {}
    @&#8203;decorator export default class Foo {}

    // This new syntax is now permitted too:
    export @&#8203;decorator class Foo {}
    export default @&#8203;decorator class Foo {}
    ```

In addition, esbuild's decorator parser has been rewritten to fix
several subtle and likely unimportant edge cases with esbuild's parsing
of exports and decorators in TypeScript (e.g. TypeScript apparently does
automatic semicolon insertion after `interface` and `export interface`
but not after `export default interface`).

-   Pretty-print decorators using the same whitespace as the original

When printing code containing decorators, esbuild will now try to
respect whether the original code contained newlines after the decorator
or not. This can make generated code containing many decorators much
more compact to read:

    ```js
    // Original code
    class Foo {
      @&#8203;a @&#8203;b @&#8203;c abc
      @&#8203;x @&#8203;y @&#8203;z xyz
    }

    // Old output
    class Foo {
      @&#8203;a
      @&#8203;b
      @&#8203;c
      abc;
      @&#8203;x
      @&#8203;y
      @&#8203;z
      xyz;
    }

    // New output
    class Foo {
      @&#8203;a @&#8203;b @&#8203;c abc;
      @&#8203;x @&#8203;y @&#8203;z xyz;
    }
    ```

###
[`v0.19.7`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#0197)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.6...v0.19.7)

- Add support for bundling code that uses import attributes
([#&#8203;3384](https://togithub.com/evanw/esbuild/issues/3384))

JavaScript is gaining new syntax for associating a map of string
key-value pairs with individual ESM imports. The proposal is still a
work in progress and is still undergoing significant changes before
being finalized. However, the first iteration has already been shipping
in Chromium-based browsers for a while, and the second iteration has
landed in V8 and is now shipping in node, so it makes sense for esbuild
to support it. Here are the two major iterations of this proposal (so
far):

    1.  Import assertions (deprecated, will not be standardized)
        -   Uses the `assert` keyword
        -   Does *not* affect module resolution
        -   Causes an error if the assertion fails
        -   Shipping in Chrome 91+ (and in esbuild 0.11.22+)

    2.  Import attributes (currently set to become standardized)
        -   Uses the `with` keyword
        -   Affects module resolution
        -   Unknown attributes cause an error
        -   Shipping in node 21+

You can already use esbuild to bundle code that uses import assertions
(the first iteration). However, this feature is mostly useless for
bundlers because import assertions are not allowed to affect module
resolution. It's basically only useful as an annotation on external
imports, which esbuild will then preserve in the output for use in a
browser (which would otherwise refuse to load certain imports).

With this release, esbuild now supports bundling code that uses import
attributes (the second iteration). This is much more useful for bundlers
because they are allowed to affect module resolution, which means the
key-value pairs can be provided to plugins. Here's an example, which
uses esbuild's built-in support for the upcoming [JSON module
standard](https://togithub.com/tc39/proposal-json-modules):

    ```js
    // On static imports
    import foo from './package.json' with { type: 'json' }
    console.log(foo)

    // On dynamic imports
const bar = await import('./package.json', { with: { type: 'json' } })
    console.log(bar)
    ```

One important consequence of the change in semantics between import
assertions and import attributes is that two imports with identical
paths but different import attributes are now considered to be different
modules. This is because the import attributes are provided to the
loader, which might then use those attributes during loading. For
example, you could imagine an image loader that produces an image of a
different size depending on the import attributes.

Import attributes are now reported in the
[metafile](https://esbuild.github.io/api/#metafile) and are now provided
to [on-load plugins](https://esbuild.github.io/plugins/#on-load) as a
map in the `with` property. For example, here's an esbuild plugin that
turns all imports with a `type` import attribute equal to `'cheese'`
into a module that exports the cheese emoji:

    ```js
    const cheesePlugin = {
      name: 'cheese',
      setup(build) {
        build.onLoad({ filter: /.*/ }, args => {
          if (args.with.type === 'cheese') return {
            contents: `export default "🧀"`,
          }
        })
      }
    }

    require('esbuild').build({
      bundle: true,
      write: false,
      stdin: {
        contents: `
import foo from 'data:text/javascript,' with { type: 'cheese' }
          console.log(foo)
        `,
      },
      plugins: [cheesePlugin],
    }).then(result => {
      const code = new Function(result.outputFiles[0].text)
      code()
    })
    ```

Warning: It's possible that the second iteration of this feature may
change significantly again even though it's already shipping in real
JavaScript VMs (since it has already happened once before). In that
case, esbuild may end up adjusting its implementation to match the
eventual standard behavior. So keep in mind that by using this, you are
using an unstable upcoming JavaScript feature that may undergo breaking
changes in the future.

- Adjust TypeScript experimental decorator behavior
([#&#8203;3230](https://togithub.com/evanw/esbuild/issues/3230),
[#&#8203;3326](https://togithub.com/evanw/esbuild/issues/3326),
[#&#8203;3394](https://togithub.com/evanw/esbuild/issues/3394))

With this release, esbuild will now allow TypeScript experimental
decorators to access both static class properties and `#private` class
names. For example:

    ```js
    const check =
      <T,>(a: T, b: T): PropertyDecorator =>
        () => console.log(a === b)

    async function test() {
      class Foo {
        static #foo = 1
        static bar = 1 + Foo.#foo
        @&#8203;check(Foo.#foo, 1) a: any
        @&#8203;check(Foo.bar, await Promise.resolve(2)) b: any
      }
    }

    test().then(() => console.log('pass'))
    ```

This will now print `true true pass` when compiled by esbuild.
Previously esbuild evaluated TypeScript decorators outside of the class
body, so it didn't allow decorators to access `Foo` or `#foo`. Now
esbuild does something different, although it's hard to concisely
explain exactly what esbuild is doing now (see the background section
below for more information).

Note that TypeScript's experimental decorator support is currently
buggy: TypeScript's compiler passes this test if only the first `@check`
is present or if only the second `@check` is present, but TypeScript's
compiler fails this test if both checks are present together. I haven't
changed esbuild to match TypeScript's behavior exactly here because I'm
waiting for TypeScript to fix these bugs instead.

Some background: TypeScript experimental decorators don't have
consistent semantics regarding the context that the decorators are
evaluated in. For example, TypeScript will let you use `await` within a
decorator, which implies that the decorator runs outside the class body
(since `await` isn't supported inside a class body), but TypeScript will
also let you use `#private` names, which implies that the decorator runs
inside the class body (since `#private` names are only supported inside
a class body). The value of `this` in a decorator is also buggy (the
run-time value of `this` changes if any decorator in the class uses a
`#private` name but the type of `this` doesn't change, leading to the
type checker no longer matching reality). These inconsistent semantics
make it hard for esbuild to implement this feature as decorator
evaluation happens in some superposition of both inside and outside the
class body that is particular to the internal implementation details of
the TypeScript compiler.

- Forbid `--keep-names` when targeting old browsers
([#&#8203;3477](https://togithub.com/evanw/esbuild/issues/3477))

The `--keep-names` setting needs to be able to assign to the `name`
property on functions and classes. However, before ES6 this property was
non-configurable, and attempting to assign to it would throw an error.
So with this release, esbuild will no longer allow you to enable this
setting while also targeting a really old browser.

###
[`v0.19.6`](https://togithub.com/evanw/esbuild/blob/HEAD/CHANGELOG.md#0196)

[Compare
Source](https://togithub.com/evanw/esbuild/compare/v0.19.5...v0.19.6)

-   Fix a constant folding bug with bigint equality

This release fixes a bug where esbuild incorrectly checked for bigint
equality by checking the equality of the bigint literal text. This is
correct if the bigint doesn't have a radix because bigint literals
without a radix are always in canonical form (since leading zeros are
not allowed). However, this is incorrect if the bigint has a radix (e.g.
`0x123n`) because the canonical form is not enforced when a radix is
present.

    ```js
    // Original code
    console.log(!!0n, !!1n, 123n === 123n)
    console.log(!!0x0n, !!0x1n, 123n === 0x7Bn)

    // Old output
    console.log(false, true, true);
    console.log(true, true, false);

    // New output
    console.log(false, true, true);
    console.log(!!0x0n, !!0x1n, 123n === 0x7Bn);
    ```

-   Add some improvements to the JavaScript minifier

This release adds more cases to the JavaScript minifier, including
support for inlining `String.fromCharCode` and
`String.prototype.charCodeAt` when possible:

    ```js
    // Original code
document.onkeydown = e => e.keyCode === 'A'.charCodeAt(0) &&
console.log(String.fromCharCode(55358, 56768))

    // Old output (with --minify)

document.onkeydown=o=>o.keyCode==="A".charCodeAt(0)&&console.log(String.fromCharCode(55358,56768));

    // New output (with --minify)
    document.onkeydown=o=>o.keyCode===65&&console.log("🧀");
    ```

In addition, immediately-invoked function expressions (IIFEs) that
return a single expression are now inlined when minifying. This makes it
possible to use IIFEs in combination with `@__PURE__` annotations to
annotate arbitrary expressions as side-effect free without the IIFE
wrapper impacting code size. For example:

    ```js
    // Original code
const sideEffectFreeOffset = /* @&#8203;__PURE__ */ (() =>
computeSomething())()
    use(sideEffectFreeOffset)

    // Old output (with --minify)
    const e=(()=>computeSomething())();use(e);

    // New output (with --minify)
    const e=computeSomething();use(e);
    ```

- Automatically prefix the `mask-composite` CSS property for WebKit
([#&#8203;3493](https://togithub.com/evanw/esbuild/issues/3493))

The `mask-composite` property will now be prefixed as
`-webkit-mask-composite` for older WebKit-based browsers. In addition to
prefixing the property name, handling older browsers also requires
rewriting the values since WebKit uses non-standard names for the mask
composite modes:

    ```css
    /* Original code */
    div {
      mask-composite: add, subtract, intersect, exclude;
    }

    /* New output (with --target=chrome100) */
    div {
      -webkit-mask-composite:
        source-over,
        source-out,
        source-in,
        xor;
      mask-composite:
        add,
        subtract,
        intersect,
        exclude;
    }
    ```

- Avoid referencing `this` from JSX elements in derived class
constructors
([#&#8203;3454](https://togithub.com/evanw/esbuild/issues/3454))

When you enable `--jsx=automatic` and `--jsx-dev`, the JSX transform is
supposed to insert `this` as the last argument to the `jsxDEV` function.
I'm not sure exactly why this is and I can't find any specification for
it, but in any case this causes the generated code to crash when you use
a JSX element in a derived class constructor before the call to
`super()` as `this` is not allowed to be accessed at that point. For
example

    ```js
    // Original code
    class ChildComponent extends ParentComponent {
      constructor() {
        super(<div />)
      }
    }

    // Problematic output (with --loader=jsx --jsx=automatic --jsx-dev)
    import { jsxDEV } from "react/jsx-dev-runtime";
    class ChildComponent extends ParentComponent {
      constructor() {
        super(/* @&#8203;__PURE__ */ jsxDEV("div", {}, void 0, false, {
          fileName: "<stdin>",
          lineNumber: 3,
          columnNumber: 15
        }, this)); // The reference to "this" crashes here
      }
    }
    ```

The TypeScript compiler doesn't handle this at all while the Babel
compiler just omits `this` for the entire constructor (even after the
call to `super()`). There seems to be no specification so I can't be
sure that this change doesn't break anything important. But given that
Babel is pretty loose with this and TypeScript doesn't handle this at
all, I'm guessing this value isn't too important. React's blog post
seems to indicate that this value was intended to be used for a
React-specific migration warning at some point, so it could even be that
this value is irrelevant now. Anyway the crash in this case should now
be fixed.

- Allow package subpath imports to map to node built-ins
([#&#8203;3485](https://togithub.com/evanw/esbuild/issues/3485))

You are now able to use a [subpath
import](https://nodejs.org/api/packages.html#subpath-imports) in your
package to resolve to a node built-in module. For example, with a
`package.json` file like this:

    ```json
    {
      "type": "module",
      "imports": {
        "#stream": {
          "node": "stream",
          "default": "./stub.js"
        }
      }
    }
    ```

    You can now import from node's `stream` module like this:

    ```js
    import * as stream from '#stream';
    console.log(Object.keys(stream));
    ```

This will import from node's `stream` module when the platform is `node`
and from `./stub.js` otherwise.

- No longer throw an error when a `Symbol` is missing
([#&#8203;3453](https://togithub.com/evanw/esbuild/issues/3453))

Certain JavaScript syntax features use special properties on the global
`Symbol` object. For example, the asynchronous iteration syntax uses
`Symbol.asyncIterator`. Previously esbuild's generated code for older
browsers required this symbol to be polyfilled. However, starting with
this release esbuild will use
[`Symbol.for()`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Symbol/for)
to construct these symbols if they are missing instead of throwing an
error about a missing polyfill. This means your code no longer needs to
include a polyfill for missing symbols as long as your code also uses
`Symbol.for()` for missing symbols.

- Parse upcoming changes to TypeScript syntax
([#&#8203;3490](https://togithub.com/evanw/esbuild/issues/3490),
[#&#8203;3491](https://togithub.com/evanw/esbuild/pull/3491))

With this release, you can now use `from` as the name of a default
type-only import in TypeScript code, as well as `of` as the name of an
`await using` loop iteration variable:

    ```ts
    import type from from 'from'
    for (await using of of of) ;
    ```

This matches similar changes in the TypeScript compiler
([#&#8203;56376](https://togithub.com/microsoft/TypeScript/issues/56376)
and
[#&#8203;55555](https://togithub.com/microsoft/TypeScript/issues/55555))
which will start allowing this syntax in an upcoming version of
TypeScript. Please never actually write code like this.

The type-only import syntax change was contributed by
[@&#8203;magic-akari](https://togithub.com/magic-akari).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/JoshuaKGoldberg/refined-saved-replies).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNTMuMiIsInVwZGF0ZWRJblZlciI6IjM3LjE1My4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  • Loading branch information
renovate[bot] authored Jan 30, 2024
1 parent 2b9684a commit 3f12333
Show file tree
Hide file tree
Showing 2 changed files with 241 additions and 3 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
"@vitest/coverage-v8": "^1.0.0",
"console-fail-test": "^0.2.3",
"cspell": "^8.0.0",
"esbuild": "^0.19.5",
"esbuild": "^0.20.0",
"eslint": "^8.53.0",
"eslint-plugin-deprecation": "^2.0.0",
"eslint-plugin-eslint-comments": "^3.2.0",
Expand Down
Loading

0 comments on commit 3f12333

Please sign in to comment.