ESBuild define setting transformed incorrectly

[jimrandomh]

Our ESBuild config for a client bundle contains an option that looks like:

  define: {
    "global": "window"
  }

When using ESBuild directly, this works as expected (the global identifier is replaced with window in some libraries that had weird expectations about their environment). Estrella, however, transforms the settings like this (estrella.js:575):

  for (let k in define) {
    define[k] = json(define[k])
  }

This causes global to be replaced with the string "window" instead of the token window, causing a crash on start.

[rsms]

Thanks for the issue report. Fix will be included in the next release.

[rsms]

Shipped with release 1.4.0