Skip to content

Instantly share code, notes, and snippets.

@Rich-Harris
Last active October 12, 2020 15:09
Show Gist options
  • Save Rich-Harris/51e1bf24e7c093469ef7a0983bad94cb to your computer and use it in GitHub Desktop.
Save Rich-Harris/51e1bf24e7c093469ef7a0983bad94cb to your computer and use it in GitHub Desktop.
Don't ship untranspiled code

When Babel 6 came out, it was hard for a lot of packages to upgrade because it was essentially an entirely different category of thing than Babel 5. So what happened was that some packages upgraded, and some didn't — at least not straight away.

Some projects took the prima facie enlightened view that packages should expose untranspiled code, so that the consumers of that code could determine for themselves what needed to get transpiled based on the environments they supported.

That was a costly decision. If I was the author of an app that was using Babel 6, I couldn't import a library that was still using Babel 5 and shipping untranspiled code (because the configs were completely incompatible), and vice versa. Frankly, it was a bloody nuisance. We are bad at anticipating these sorts of issues. It will happen again at some point.

Adding a few extra bytes to pkg.main or pkg.module is a small price to pay for things just working. As well as avoiding the aforementioned headaches, it means that your builds are a lot quicker. I don't want to run my node_modules through Babel.

But it might not be a few bytes!

Right. So you're a library author, and you want to use async/await and all that stuff. You have a few options here:

  1. Don't. It transpiles badly. Honestly, we managed for long enough without it — it's not that hard.
  2. Say that you don't support environments without async/await. That's totally fine. I don't support environments without Array.prototype.filter.
  3. Include your src code in the npm package. If someone really wants to use your futuristic code in an environment you don't support, let them. Just make sure you also include pre-transpiled bundles.

The golden rule

A lot of developers believe that the priority is to ship the leanest, most optimised code possible. I disagree. The priority is to ensure that people can consume code with the least amount of faffery. A novice developer should not have to learn about transpilers in order to use your library.

Shipping code that works out of the box is just basic politeness. I've said this in the past, and I stand by it:

It’s the difference between giving someone raw ingredients and a cooked meal — if you went to a restaurant and ordered a burger, you’d be pretty mad if they gave you half a pound of minced beef and a frying pan instead.

Shipping the leanest, most optimised code is something experts care about. Experts have the knowledge and incentives to invest time in the kinds of workflows that involve transpiling other people's code. That should not be the default.

@sancarn
Copy link

sancarn commented May 6, 2019

@maple3142

because IE and old version node should be abandoned in my opinion

you know there are good reasons many devlopers still use Internet Explorer. It's not for fun or because they are stubborn. In windows software development if ever you want an easy to make, easily integratable GUI which comes native to windows for all users, using the IE ActiveX object is a real winner. It saves you having to ship with e.g. chromium and also gives you an expansive API which allows you to truly integrate every part of IE into your application. Bare in mind, Microsoft could easily fix this for everyone by making an ActiveX edge control, but yeah, in the end if we could migrate away from IE we would. It isn't as simple as it may seem from a web-only developer's perspective :)

@robertknight
Copy link

Don't. It transpiles badly. Honestly, we managed for long enough without it — it's not that hard.

I think that might be an issue with Babel's default approach to transpiling async/await (regenerator) rather than an unavoidable problem. The TypeScript compiler and babel-plugin-async-to-promises can both transpile async/await down to a fairly modest amount of code which isn't a complete PITA to debug.

That said, if a library doesn't make heavy use of async code then just writing code with Promises directly would probably be better.

@steve-taylor
Copy link

steve-taylor commented Jan 3, 2020

Babel 7's lack of support for targeting specific ES versions forces me to go with option #3 sans pre-transpiled bundles. As a library author, the decision about which browsers to support is not mine to make.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment