integrate CLI

[Rich-Harris]

fixes #1360

[Rich-Harris]

This is failing because the CLI uses async/await. So we need to decide: do we

a) rewrite using regular promises
b) transpile, or
c) drop support for Node 6, seeing as it's no longer 'active LTS'

Transpiling wouldn't be that onerous I suppose, would just need to port it to TypeScript. Probably the lowest friction approach

[codecov-io]

Codecov Report

Merging #1389 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1389      +/-   ##
==========================================
+ Coverage   91.19%   91.24%   +0.04%     
==========================================
  Files         122      122              
  Lines        4465     4465              
  Branches     1377     1377              
==========================================
+ Hits         4072     4074       +2     
+ Misses        156      154       -2     
  Partials      237      237

Impacted Files	Coverage Δ
src/validate/js/propValidators/onrender.ts	100% <0%> (+100%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d010aff...c9494d9. Read the comment docs.

[Conduitry]

What does TypeScript compile async/await to? Having semi-legible transpiled code is nice, which in this case would mean hopefully generators (supported in Node 6) as opposed to impenetrable switch statements.

[Rich-Harris]

It compiles surprisingly well, though it's still not what you'd call human-readable. Though I just looked again, and we're actually barely using async/await at all in the CLI, so I just rewrote it with normal promises for now.

[Conduitry]

Do we want to do whatever string replacement we do in other tests here, so that this comment won't have to change each version?

[Rich-Harris]

good idea — have done that

[Conduitry]

I think you need to add a "bin" entry to package.json? And you can just point that right at the compiled index.ts.js - no need for a separate svelte file that requires that.

[Rich-Harris]

D'oh, of course. I'm not sure you can just point it at the dist file though — I've had issues with permissions in the past where I needed to chmod u+x the file each time it was rebuilt.