VulcanJS / vulcan-npm

The full-stack JavaScript App Framework
https://vulcan-docs.vercel.app
MIT License
30 stars 8 forks source link

Faster build with SWC or Esbuild #85

Open eric-burel opened 2 years ago

eric-burel commented 2 years ago

Is your feature request related to a problem? Please describe. I am exploring modern alternatives to Babel/Webpack for a faster build time.

First experiment is on the react-ui package.

Describe the solution you'd like - Need to be able to output code for Node (during SSR) and the browser (for React components) - Need to generate .d.ts?

Describe alternatives you've considered

SWC

Next.js is backing SWC, so it's expected to have a lot of updates.

Be careful that SWC is a parser, so a replacement to Babel, not to Webpack. They are building "SWCpack" for this purpose but it's not there yet.

Dumping Babel may have some consequences, see https://github.com/vercel/next.js/discussions/30174, as SWC doesn't support all packages yet and have a smaller ecosystem.

It proposes a Webpack loader => great for fast setup by replacing babel-loader.

We need to figure the .d.ts generation thing.

Esbuild

Seems to work ok, very fast. But since Next is preferring SWC, that is written in Rust (Esbuild is in Go, so great language but less relevant for a build tool), we might go the Next.js way.

Not completely sure how it replace Babel/Webpack. But I am sure it doesn't correctly support .d.ts either: https://github.com/evanw/esbuild/issues/95

Additional context Add any other context or screenshots about the feature request here.

eric-burel commented 2 years ago

This article describe an alternative, using TypeScript "build" option. Maybe it could help getting rid of Babel and Webpack? Or at least improve our Lerna workflow: https://betterstack.dev/blog/lerna-typescript-monorepo/

eric-burel commented 2 years ago

About the Express starter

Converting "require" to "esm" is currently not supported
{
  column: 27,
  file: '/code/vulcan-npm/starters/express/.yalc/@vulcanjs/mongo/dist/index.js',
  length: 7,
  line: 36,
  lineText: 'var import_merge = __toESM(require("lodash/merge.js"));',
  namespace: '',
  suggestion: ''
}
Converting "require" to "esm" is currently not supported
{
  column: 19,
  file: '/code/vulcan-npm/starters/express/.yalc/@vulcanjs/mongo/dist/index.js',
  length: 7,
  line: 37,
  lineText: 'var import_utils = require("@vulcanjs/utils");',
  namespace: '',
  suggestion: ''
}
Converting "require" to "esm" is currently not supported
{
  column: 30,
  file: '/code/vulcan-npm/starters/express/.yalc/@vulcanjs/mongo/dist/index.js',
  length: 7,
  line: 38,
  lineText: 'var import_mongoose = __toESM(require("mongoose"));',
  namespace: '',
  suggestion: ''
}

It's often an issue in the way we use the library (eg mongoose should maybe use import * as mongoose because it's default is not setup correctly), but still difficult to bypass

eric-burel commented 2 years ago

Relevant thread on making the libraries tree-shakeable when they are rebuilt in the end user app (Next) and not just tree-shaked during the publication of the NPM package: https://github.com/egoist/tsup/issues/578