feat: 🔪 remove build script and related configuration

We don't use any of the build-related tooling that this library exports. This
removes the script and all related configuration and dependencies and replaces
the build process with a simple Babel CLI based build.

BREAKING CHANGE: This removes the `hover-scripts build` script, the exported
Babel configuration, and the exported Rollup configuration. It also removes the
built-in Babel integration in the Jest configuation.

Author: jrolfs

.babelrc.json

@@ -0,0 +1,12 @@
+{
+  "presets": [
+    [
+      "@babel/preset-env",
+      {
+        "targets": {
+          "node": "10"
+        }
+      }
+    ]
+  ]
+}

babel.js

@@ -14,7 +14,7 @@
     "test": "node src test",
     "test:update": "node src test --updateSnapshot",
     "build": "run-p build:*",
-    "build:source": "node src build",
+    "build:source": "babel --source-maps --out-dir dist --ignore '**/__tests__/**','**/__mocks__/**' src",
     "build:types": "tsc -p src/",
     "lint": "node src lint",
     "format": "node src format",
@@ -36,7 +36,6 @@
     "api.js",
     "api.d.ts",
     "dist",
-    "babel.js",
     "commitlint.js",
     "eslint.js",
     "eslint-react.js",
@@ -50,33 +49,13 @@
   "author": "Jamie Rolfs <jamie.rolfs@hover.to>",
   "license": "MIT",
   "dependencies": {
-    "@babel/cli": "^7.11.6",
-    "@babel/core": "^7.11.6",
-    "@babel/plugin-proposal-class-properties": "^7.8.3",
-    "@babel/plugin-transform-modules-commonjs": "^7.8.3",
-    "@babel/plugin-transform-runtime": "^7.11.0",
-    "@babel/preset-env": "^7.11.5",
-    "@babel/preset-flow": "^7.8.3",
-    "@babel/preset-react": "^7.8.3",
-    "@babel/runtime": "^7.11.2",
     "@commitlint/cli": "^11.0.0",
     "@commitlint/config-conventional": "^9.1.2",
     "@commitlint/prompt": "^11.0.0",
-    "@rollup/plugin-commonjs": "^15.0.0",
-    "@rollup/plugin-json": "^4.0.2",
-    "@rollup/plugin-node-resolve": "^7.1.1",
-    "@rollup/plugin-replace": "^2.3.1",
     "@types/jest": "^26.0.14",
     "@typescript-eslint/eslint-plugin": "^4.4.0",
     "@typescript-eslint/parser": "^4.3.0",
     "arrify": "^2.0.1",
-    "babel-jest": "^26.3.0",
-    "babel-plugin-macros": "^2.8.0",
-    "babel-plugin-minify-dead-code-elimination": "^0.5.1",
-    "babel-plugin-module-resolver": "^4.0.0",
-    "babel-plugin-transform-inline-environment-variables": "^0.4.3",
-    "babel-plugin-transform-react-remove-prop-types": "^0.4.24",
-    "browserslist": "^4.14.5",
     "chalk": "^4.1.0",
     "commitizen": "^4.2.1",
     "concurrently": "^5.1.0",
@@ -108,12 +87,6 @@
     "read-pkg-up": "^7.0.1",
     "resolve": "^1.15.1",
     "rimraf": "^3.0.2",
-    "rollup": "^2.26.4",
-    "rollup-plugin-babel": "^4.3.3",
-    "rollup-plugin-node-builtins": "^2.1.2",
-    "rollup-plugin-node-globals": "^1.4.0",
-    "rollup-plugin-size-snapshot": "^0.12.0",
-    "rollup-plugin-terser": "^7.0.2",
     "semver": "^7.1.3",
     "typescript": "^4.0.3",
     "which": "^2.0.2",
@@ -147,6 +120,10 @@
   },
   "homepage": "https://github.com/hoverinc/hover-javascript#readme",
   "devDependencies": {
+    "@babel/cli": "^7.11.6",
+    "@babel/core": "^7.11.6",
+    "@babel/preset-env": "^7.11.5",
+    "babel-jest": "^26.5.2",
     "jest-in-case": "^1.0.2",
     "npm-run-all": "^4.1.5",
     "slash": "^3.0.0"

package.json

@@ -14,22 +14,13 @@ Array [
 ]
 `;
 
-exports[`format logs for SIGTERM signal 1`] = `
-Array [
-  Array [
-    The script "build" failed because the process exited too early. Someone might have called \`kill\` or \`killall\`, or the system could be shutting down.,
-  ],
-]
-`;
-
 exports[`format logs help with no args 1`] = `
 Array [
   Array [
     
 Usage: ../ [script] [--flags]
 
 Available Scripts:
-  build
   ci-after-success
   commit-msg
   commit

src/__tests__/__snapshots__/index.js.snap

@@ -73,10 +73,6 @@ cases(
       args: ['lint'],
       signal: 'SIGKILL',
     },
-    'logs for SIGTERM signal': {
-      args: ['build'],
-      signal: 'SIGTERM',
-    },
     'does not log for other signals': {
       args: ['test'],
       signal: 'SIGBREAK',

src/__tests__/index.js

@@ -1,11 +1,7 @@
 test('requiring some files does not blow up', () => {
-  require('../babel-transform')
-  require('../babelrc')
   require('../eslintrc')
   require('../eslintrc-react')
   require('../jest.config')
   require('../lintstagedrc')
   require('../prettierrc')
-  require('../rollup.config')
-  require('../').getRollupConfig()
 })

src/config/__tests__/umbrella.js

@@ -1,9 +1,7 @@
 module.exports = {
-  babel: require('./babelrc'),
   commitlint: require('./commitlint.config'),
   eslint: require('./eslintrc'),
   jest: require('./jest.config'),
   lintStaged: require('./lintstagedrc'),
   prettier: require('./prettierrc'),
-  getRollupConfig: () => require('./rollup.config'),
 }

src/config/babel-transform.js

@@ -1,9 +1,4 @@
-const path = require('path')
-const {ifAnyDep, hasAnyDep, hasFile, hasPkgProp, fromRoot} = require('../utils')
-
-const here = p => path.join(__dirname, p)
-
-const useBuiltInBabelConfig = !hasFile('.babelrc') && !hasPkgProp('babel')
+const {ifAnyDep, hasAnyDep, hasFile, fromRoot} = require('../utils')
 
 const ignores = [
   '/node_modules/',
@@ -50,7 +45,6 @@ const jestConfig = {
     require.resolve('jest-watch-typeahead/testname'),
   ],
   globals: {},
-  transform: {},
 }
 
 if (hasAnyDep('ts-jest')) {
@@ -66,10 +60,4 @@ if (hasFile('tests/setup-env.js')) {
   jestConfig.setupFilesAfterEnv = [fromRoot('tests/setup-env.js')]
 }
 
-if (useBuiltInBabelConfig) {
-  Object.assign(jestConfig.transform, {
-    '^.+\\.js$': here('./babel-transform'),
-  })
-}
-
 module.exports = jestConfig