Skip to content
This repository has been archived by the owner on Aug 4, 2024. It is now read-only.

[Snyk] Upgrade esbuild from 0.14.54 to 0.15.0 #252

Merged
merged 1 commit into from
Aug 31, 2022

Conversation

snyk-bot
Copy link
Contributor

Snyk has created this PR to upgrade esbuild from 0.14.54 to 0.15.0.

merge advice
ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.


  • The recommended version is 1 version ahead of your current version.
  • The recommended version was released 21 days ago, on 2022-08-10.
Release notes
Package name: esbuild
  • 0.15.0 - 2022-08-10

    This release contains backwards-incompatible changes. Since esbuild is before version 1.0.0, these changes have been released as a new minor version to reflect this (as recommended by npm). You should either be pinning the exact version of esbuild in your package.json file or be using a version range syntax that only accepts patch upgrades such as ~0.14.0. See the documentation about semver for more information.

    • Implement the Yarn Plug'n'Play module resolution algorithm (#154, #237, #1263, #2451)

      Node comes with a package manager called npm, which installs packages into a node_modules folder. Node and esbuild both come with built-in rules for resolving import paths to packages within node_modules, so packages installed via npm work automatically without any configuration. However, many people use an alternative package manager called Yarn. While Yarn can install packages using node_modules, it also offers a different package installation strategy called Plug'n'Play, which is often shortened to "PnP" (not to be confused with pnpm, which is an entirely different unrelated package manager).

      Plug'n'Play installs packages as .zip files on your file system. The packages are never actually unzipped. Since Node doesn't know anything about Yarn's package installation strategy, this means you can no longer run your code with Node as it won't be able to find your packages. Instead, you need to run your code with Yarn, which applies patches to Node's file system APIs before running your code. These patches attempt to make zip files seem like normal directories. When running under Yarn, using Node's file system API to read ./some.zip/lib/file.js actually automatically extracts lib/file.js from ./some.zip at run-time as if it was a normal file. Other file system APIs behave similarly. However, these patches don't work with esbuild because esbuild is not written in JavaScript; it's a native binary executable that interacts with the file system directly through the operating system.

      Previously the workaround for using esbuild with Plug'n'Play was to use the @ yarnpkg/esbuild-plugin-pnp plugin with esbuild's JavaScript API. However, this wasn't great because the plugin needed to potentially intercept every single import path and file load to check whether it was a Plug'n'Play package, which has an unusually high performance cost. It also meant that certain subtleties of path resolution rules within a .zip file could differ slightly from the way esbuild normally works since path resolution inside .zip files was implemented by Yarn, not by esbuild (which is due to a limitation of esbuild's plugin API).

      With this release, esbuild now contains an independent implementation of Yarn's Plug'n'Play algorithm (which is used when esbuild finds a .pnp.js, .pnp.cjs, or .pnp.data.json file in the directory tree). Creating additional implementations of this algorithm recently became possible because Yarn's package manifest format was recently documented: https://yarnpkg.com/advanced/pnp-spec/. This should mean that you can now use esbuild to bundle Plug'n'Play projects without any additional configuration (so you shouldn't need @ yarnpkg/esbuild-plugin-pnp anymore). Bundling these projects should now happen much faster as Yarn no longer even needs to be run at all. Bundling the Yarn codebase itself with esbuild before and after this change seems to demonstrate over a 10x speedup (3.4s to 0.24s). And path resolution rules within Yarn packages should now be consistent with how esbuild handles regular Node packages. For example, fields such as module and browser in package.json files within .zip files should now be respected.

      Keep in mind that this is brand new code and there may be some initial issues to work through before esbuild's implementation is solid. Yarn's Plug'n'Play specification is also brand new and may need some follow-up edits to guide new implementations to match Yarn's exact behavior. If you try this out, make sure to test it before committing to using it, and let me know if anything isn't working as expected. Should you need to debug esbuild's path resolution, you may find --log-level=verbose helpful.

  • 0.14.54 - 2022-08-08
    • Fix optimizations for calls containing spread arguments (#2445)

      This release fixes the handling of spread arguments in the optimization of /* @ __PURE__ */ comments, empty functions, and identity functions:

      // Original code
      function empty() {}
      function identity(x) { return x }
      /* @ PURE / a(...x)
      /
      @ PURE */ new b(...x)
      empty(...x)
      identity(...x)

      // Old output (with --minify --tree-shaking=true)
      ...x;...x;...x;...x;

      // New output (with --minify --tree-shaking=true)
      function identity(n){return n}[...x];[...x];[...x];identity(...x);

      Previously esbuild assumed arguments with side effects could be directly inlined. This is almost always true except for spread arguments, which are not syntactically valid on their own and which have the side effect of causing iteration, which might have further side effects. Now esbuild will wrap these elements in an unused array so that they are syntactically valid and so that the iteration side effects are preserved.

from esbuild GitHub release notes
Commit messages
Package name: esbuild
  • c223771 publish 0.15.0 to npm
  • a0b752e implement the yarn pnp module resolution algorithm (#2451)
  • c58fe49 zip: fix bug with readdir and trailing slashes
  • 4f43888 add simple test coverage for zip and `__virtual__`
  • e870ec5 implement yarn pnp `__virtual__` path mangling
  • a481005 add zip file support
  • 664c8a5 jsx: use first element loc for logs and source map
  • 1114e7d fix dropped errors in `readdir` due to refactor
  • da0e4ca increase node unref test timeout for slow vms

Compare


Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.

For more information:

🧐 View latest project report

🛠 Adjust upgrade PR settings

🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@mergify mergify bot merged commit ecde426 into master Aug 31, 2022
@mergify mergify bot deleted the snyk-upgrade-e433af265ad64bc17f73ff6d426f5b2e branch August 31, 2022 08:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant