Switch to esbuild from parcel-bundler

[nikolaik]

Hi there 👋 Please let me know if any of these changes are unwanted or not working as intended.

• Adds a ES module build in dist/index.esm.js
• Remove most babel deps, keeping babel-eslint and @babel/core to keep eslint working
• Should fix webpack dependency warning: Critical dependency: the request of a dependency is an expression #57
• Bump minimum required node version to latest supported
• Update .npmignore to exclude demo and .github dirs

[kadikraman]

Hi @nikolaik thanks so much for the PR! Definitely happy to add this.

There's a bit of an issue with the current script though, it looks like only the last last output of esbuild.build is kept. It's async I think so maybe a race condition? As a result running yarn compile will only have the .esm.js files in the /dist folder.

I've never used esbuild before so not sure on the best practices. Moving the second esbuild.build into a .then block will create all 4 files (though there's probably a nicer way to do this):

esbuild
  .build(
    Object.assign(options, {
      outfile: pkg.main,
      format: 'cjs'
    })
  )
  .then(() => {
    esbuild
      .build(
        Object.assign(options, {
          outfile: pkg.module,
          format: 'esm'
        })
      )
      .catch(() => process.exit(1));
  })
  .catch(() => process.exit(1));

[nikolaik]

Hi @nikolaik thanks so much for the PR! Definitely happy to add this.

Swell!

There's a bit of an issue with the current script though, it looks like only the last last output of esbuild.build is kept. It's async I think so maybe a race condition? As a result running yarn compile will only have the .esm.js files in the /dist folder.

I've never used esbuild before so not sure on the best practices. Moving the second esbuild.build into a .then block will create all 4 files (though there's probably a nicer way to do this):

I'm pretty new to esbuild myself, but found this pattern of doing it in another repository somewhere. Pushed a fix now using two await's which seems to fix things.

(async () => {
  await esbuild
    .build({ ...options, outfile: pkg.main, format: 'cjs' })
    .catch(() => process.exit(1));

  await esbuild
    .build({ ...options, outfile: pkg.module, format: 'esm' })
    .catch(() => process.exit(1));
})();

Bonus: Switched to destructuring syntax from Object.assign

[kadikraman]

Released in v1.5.0 🚀