Modernization

[jcorbin]

• updated eslint; Update eslint #28
• switch to nyc; Switch to nyc #30
• make things pass minimal tsserver-active $EDITOR-ing
  • dropped unused symbols; Drop unused variables throughout #20
  • address code lens feedback, mostly about classifying all the things; Classify all the things #21
  • excepting external deps that are missing type definitions, discussed below
• switch to github actions; Switch to github actions; goodbye travis #33
• comply to eslint:recommended; Use eslint:recommended base (modulo sundered-vars) #46
• add integration test coverage over thekni.js command module; part of Improve test coverage #49
• add minimal tsconfig and tsc test/lint pass to CI; Add tsc #54
• ratchet up tsconfig strictness
• gradually @ts-check everything
  • scanner scanner: ts-check #57
  • outline-lexer outline-lexer: ts-check #58
  • inline-lexer inline-lexer: ts-check #59
  • parser parser: ts-check #60
  • path path: ts-check #55
  • story story: ts-check #56
  • scope
  • grammar
  • engine
  • ...rest
• audit dependencies, updating critical ones, dropping, or subsuming so that they all type check
  • xorshift -- probably subsume ; e.g. the copy from https://github.com/borkshop/js/blob/main/cdom/prng.js
  • system -- switch to rollup and/or snowpack ; need investigation
  • shon -- probably update upstream ?
  • tee -- update, find a replacement, or subsume
  • table -- check for update
  • pop-equals -- TBD it's a test dependency, so could be shifted to devDeps?
• emit ts definitions and figure out the (pre-)publish story
• switch to promises
• use async / await
• switch to ESM

[jcorbin]

Update: istanbul's use of esprima blocks #21 similarly to eslint's parser issues; will update to nyc next, and sort out it's parsing deal

[kriskowal]

pop-equals could probably be inline or simplified. It’s one of my own dead-ends.

tee could probably be inlined and simplified. Just need an object that forwards to two.

shon is also an ill-advised kris special but one I like if you’re up for lifting it upstream.

[jcorbin]

Updated now that we're past the eslint:recommended gate

Next step is a side quest for kni.js test coverage, as revealed by the latent bug discovered in #37

Considering doing a patch release for v3 and v4 including that fix plus the ceil/round bugs fixed in #17

[jcorbin]

Also will probably drop all the // istanbul ignore ... comments, and track towards enabling "all sources" mode to make our coverage numbers more honest as speculated in #34.