Slight Refactor of Exports to Support Externals (#246)

* Refactor export detection to create a mapping from source to exports inside a source
* Tests pass now, next step is sources
* External function export
* More passing tests
* add array and object tests
* Include local name test
* Prepend export {} from '' instead of append
* Enforce yarn usage in pre-push
* Cleanup Rollup copy not used
* broken failing generator
* c8 coverage
* Ensure coverage folder isn't part of git history
* var not let for externs since users may specify es5 with cjs
* Remove dependency on sanitize filename, key is unique enough already
* Revert previous commit
* Add test for multiple default as exports from different sources
* Can now safely remove sanitize-filename since we have a test for its previous usage

Author: kristoferbaxter

package.json

@@ -22,7 +22,7 @@
   "scripts": {
     "pretest": "rimraf dist transpile && tsc -p tsconfig.test.json & wait",
     "test": "ava",
-    "precoverage": "c8 ava",
+    "precoverage": "yarn pretest && c8 ava",
     "coverage": "c8 report --reporter=html",
     "postcoverage": "sirv coverage/",
     "build": "rimraf dist transpile && tsc -p tsconfig.json & wait",
@@ -58,7 +58,6 @@
     "prettier": "1.19.1",
     "rimraf": "3.0.0",
     "rollup": "0.67.4",
-    "rollup-plugin-copy": "3.1.0",
     "sirv-cli": "0.4.5",
     "tslint": "5.20.1",
     "typescript": "3.7.3"
@@ -72,7 +71,7 @@
   },
   "husky": {
     "hooks": {
-      "pre-push": "npm-run-all test build",
+      "pre-push": "yarn npm-run-all test build",
       "pre-commit": "lint-staged"
     }
   },

rollup.config.js

@@ -16,7 +16,6 @@
 
 import pkg from './package.json';
 import builtins from 'builtins';
-import copy from 'rollup-plugin-copy';
 
 export default {
   input: './transpile/index.js',
@@ -25,12 +24,6 @@ export default {
     ...Object.keys(pkg.dependencies || {}),
     ...Object.keys(pkg.peerDependencies || {}),
   ],
-  plugins: [
-    copy({
-      './transpile/index.d.ts': './dist/index.d.ts',
-      verbose: true,
-    }),
-  ],
   output: [
     {
       file: './dist/index.mjs',

src/debug.ts

@@ -28,7 +28,7 @@ export const logSource = (preamble: string, source: string, code?: string) => {
   }
 };
 
-export const log = (preamble: string, message: string): void | null => {
+export const log = (preamble: string, message: string | object): void | null => {
   if (DEBUG_ENABLED) {
     console.log(preamble);
     console.log(message);

src/transformers/arrow-function.ts

@@ -27,6 +27,8 @@ import { parse, walk } from '../acorn';
  * @see https://astexplorer.net/#/gist/20d7fbe32292c6c9f303c1d9643885ec/6c97fabca987814cced795b8d3d078094e642264
  */
 export default class ArrowFunctionTransform extends Transform implements TransformInterface {
+  public name = 'ArrowFunctionTransform';
+
   /**
    * @param code source to parse, and modify
    * @return modified input source with computed literal keys

src/transformers/const.ts

@@ -21,6 +21,8 @@ import { TransformSourceDescription } from 'rollup';
 import MagicString from 'magic-string';
 
 export default class ConstTransform extends Transform implements TransformInterface {
+  public name = 'ConstTransform';
+
   /**
    * When outputting ES2017+ code there is neglagible differences between `const` and `let` for runtime performance.
    * So, we replace all usages of `const` with `let` to enable more variable folding.