Upgrade dependencies and node

Author: nhunzaker

.circleci/config.yml

@@ -2,7 +2,7 @@ version: 2
 
 defaults: &defaults
   docker:
-    - image: circleci/node:7
+    - image: circleci/node:10
   working_directory: ~/repo
 
 restore-npm-cache: &restore-npm-cache

.eslintrc.json

@@ -23,6 +23,9 @@
   "settings": {
     "flowtype": {
       "onlyFilesWithFlowAnnotation": true
+    },
+    "react": {
+      "version": "15.0"
     }
   },
   "extends": ["eslint:recommended", "plugin:flowtype/recommended"],

.nvmrc

@@ -1 +1 @@
-6.10.0
+10.14.2

docs/guides/architecture.md

@@ -100,7 +100,13 @@ function PlanetsList({ planets = [] }) {
     return <p>No Planets</p>
   }
 
-  return <ul>{planets.map(p => <li key={p.id}>{p.name}</li>)}</ul>
+  return (
+    <ul>
+      {planets.map(p => (
+        <li key={p.id}>{p.name}</li>
+      ))}
+    </ul>
+  )
 }
 ```
 

docs/guides/quickstart.md

@@ -257,7 +257,13 @@ And in our PlanetList:
 import React from 'react'
 
 export default function PlanetList({ planets = [] }) {
-  return <ul>{planets.map(p => <li key={p}>{p}</li>)}</ul>
+  return (
+    <ul>
+      {planets.map(p => (
+        <li key={p}>{p}</li>
+      ))}
+    </ul>
+  )
 }
 ```
 
@@ -462,7 +468,11 @@ import { addPlanet } from '../actions/planets'
 export default function PlanetList({ planets = [] }) {
   return (
     <div>
-      <ul>{planets.map(p => <li key={p}>{p}</li>)}</ul>
+      <ul>
+        {planets.map(p => (
+          <li key={p}>{p}</li>
+        ))}
+      </ul>
 
       <ActionButton action={addPlanet} value="Alpha Centauri">
         Add Planet

examples/painter/app/views/canvas.js

@@ -14,7 +14,9 @@ export default function Canvas({ pixels, height, width, send }) {
         strokeWidth={0.25 / scaleX}
         strokeOpacity="0.2"
       >
-        {pixels.map((row, y) => <Row key={y} cells={row} y={y} />)}
+        {pixels.map((row, y) => (
+          <Row key={y} cells={row} y={y} />
+        ))}
       </g>
     </svg>
   )

package.json

@@ -32,13 +32,14 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "form-serialize": "^0.7.2"
+    "form-serialize": "^0.7.2",
+    "rollup-plugin-unassert": "^0.2.0"
   },
   "devDependencies": {
-    "axios": "^0.16.2",
+    "axios": "^0.18.0",
     "babel-core": "^6.26.0",
-    "babel-eslint": "^7.2.3",
-    "babel-loader": "^7.1.2",
+    "babel-eslint": "^10.0.1",
+    "babel-loader": "^8.0.4",
     "babel-plugin-transform-runtime": "^6.23.0",
     "babel-polyfill": "^6.26.0",
     "babel-preset-es2015": "^6.24.1",
@@ -48,40 +49,40 @@
     "babel-register": "^6.26.0",
     "body-parser": "^1.17.2",
     "browser-sync": "^2.18.13",
-    "codecov": "^2.3.0",
-    "console.table": "^0.9.1",
+    "codecov": "^3.1.0",
+    "console.table": "^0.10.0",
     "elizabot": "^0.0.2",
-    "enzyme": "^2.9.1",
-    "eslint": "^4.5.0",
-    "eslint-plugin-flowtype": "^2.35.0",
+    "enzyme": "^3.8.0",
+    "enzyme-adapter-react-15": "^1.2.0",
+    "eslint": "^5.10.0",
+    "eslint-plugin-flowtype": "^3.2.0",
     "eslint-plugin-flowtype-errors": "^3.3.1",
     "eslint-plugin-react": "^7.3.0",
-    "flow-bin": "^0.53.1",
+    "flow-bin": "^0.89.0",
     "friendly-errors-webpack-plugin": "^1.6.1",
-    "html-webpack-plugin": "^2.30.1",
-    "jest": "^20.0.3",
+    "html-webpack-plugin": "^3.2.0",
+    "jest": "^23.6.0",
     "jsdoc": "^3.5.4",
-    "jsdoc-babel": "^0.3.0",
+    "jsdoc-babel": "^0.5.0",
     "microcosm-debugger": "^3.0.0",
     "minimist": "^1.2.0",
     "prettier": "^1.13.6",
-    "react": "^15.6.1",
-    "react-dom": "^15.6.1",
-    "react-hot-loader": "next",
+    "react": "^15.0.0",
+    "react-dom": "^15.0.0",
+    "react-hot-loader": "^4.5.3",
     "react-router-dom": "^4.2.2",
-    "react-test-renderer": "^15.6.1",
-    "rollup": "^0.49.2",
-    "rollup-plugin-buble": "^0.15.0",
-    "rollup-plugin-filesize": "^1.4.2",
+    "react-test-renderer": "^15.0.0",
+    "rollup": "^0.67.4",
+    "rollup-plugin-buble": "^0.19.6",
+    "rollup-plugin-filesize": "^5.0.1",
     "rollup-plugin-flow": "^1.1.1",
-    "rollup-plugin-json": "^2.3.0",
-    "rollup-plugin-node-resolve": "^3.0.0",
-    "rollup-plugin-strip": "^1.1.1",
-    "rollup-plugin-uglify": "^2.0.1",
+    "rollup-plugin-json": "^3.1.0",
+    "rollup-plugin-node-resolve": "^4.0.0",
+    "rollup-plugin-uglify": "^6.0.0",
     "uid": "^0.0.2",
-    "webpack": "^3.5.5",
-    "webpack-dev-server": "^2.6.1",
-    "zen-observable": "^0.5.2"
+    "webpack": "^4.27.1",
+    "webpack-dev-server": "^3.1.10",
+    "zen-observable": "~0.5.2"
   },
   "prettier": {
     "semi": false,

scripts/bundle

@@ -5,9 +5,9 @@
 const rollup = require('rollup')
 const buble = require('rollup-plugin-buble')
 const flow = require('rollup-plugin-flow')
-const strip = require('rollup-plugin-strip')
 const uglify = require('rollup-plugin-uglify')
 const node = require('rollup-plugin-node-resolve')
+const unassert = require('rollup-plugin-unassert')
 const json = require('rollup-plugin-json')
 const filesize = require('rollup-plugin-filesize')
 const { Dictionary } = require('uglify-js')
@@ -46,13 +46,7 @@ const configs = {
 }
 
 if (!options.strict) {
-  plugins.push(
-    strip({
-      debugger: true,
-      functions: ['console.assert'],
-      sourceMap: true
-    })
-  )
+  plugins.push( plugins.push(unassert()))
 }
 
 if (options.minify) {

src/action.js

@@ -7,6 +7,7 @@
 import Emitter, { type Callback } from './emitter'
 import tag from './tag'
 import { uid } from './utils'
+import assert from 'assert'
 
 type ActionUpdater = (payload?: mixed) => *
 type Revision = { status: string, payload: *, timestamp: number }
@@ -167,10 +168,7 @@ class Action extends Emitter {
     if (this.parent) {
       this.parent.parent = null
     } else {
-      console.assert(
-        false,
-        'Unable to prune action. It is already disconnected.'
-      )
+      assert(false, 'Unable to prune action. It is already disconnected.')
     }
   }
 
@@ -207,7 +205,7 @@ class Action extends Emitter {
     if (this.parent) {
       this.parent.abandon(this)
     } else {
-      console.assert(false, 'Action has already been removed.')
+      assert(false, 'Action has already been removed.')
     }
 
     this.removeAllListeners()
@@ -237,7 +235,7 @@ class Action extends Emitter {
    */
   _callOrSubscribeOnce(status: Status, callback: Callback, scope: ?Object) {
     if (callback) {
-      console.assert(
+      assert(
         typeof callback === 'function',
         `Expected a function when subscribing to ${status}` +
           `instead got ${typeof callback}`

src/addons/action-form.js

@@ -2,6 +2,7 @@
  * @flow
  */
 
+import assert from 'assert'
 import React from 'react'
 import DOM from 'react-dom'
 import { Action, merge } from '../microcosm'
@@ -82,7 +83,7 @@ class ActionForm extends React.PureComponent<Props> {
   submit(event: Event) {
     let form = this.form
 
-    console.assert(
+    assert(
       form,
       'ActionForm has no form reference and can not submit. This can happen',
       'if submit() is called after the parent component has unmounted.'

src/compare-tree/node.js

@@ -4,6 +4,7 @@
  * @flow
  */
 
+import assert from 'assert'
 import type Query from './query'
 
 class Node {
@@ -32,15 +33,12 @@ class Node {
    * the list of edges.
    */
   connect(node: Node | Query) {
-    console.assert(
+    assert(
       this.edges.indexOf(node) <= 0,
       node.id + ' is already connected to ' + this.id
     )
 
-    console.assert(
-      node !== this,
-      'Unable to connect node ' + node.id + ' to self.'
-    )
+    assert(node !== this, 'Unable to connect node ' + node.id + ' to self.')
 
     this.edges.push(node)
   }

src/coroutine.js

@@ -4,6 +4,7 @@
 
 import Action from './action'
 import { isFunction, isPromise, isGeneratorFn } from './utils'
+import assert from 'assert'
 
 /**
  * Provide support for generators, performing a sequence of actions in
@@ -32,7 +33,7 @@ function processGenerator(action: Action, body: GeneratorAction, repo: *) {
       return progress(repo.parallel(subAction))
     }
 
-    console.assert(
+    assert(
       subAction instanceof Action,
       `Iteration of generator expected an Action. Instead got ${typeof subAction}`
     )

src/domain-engine.js

@@ -6,6 +6,7 @@ import MetaDomain from './meta-domain'
 import getRegistration from './get-registration'
 import { get, set, merge, createOrClone } from './utils'
 import { castPath, type KeyPath } from './key-path'
+import assert from 'assert'
 
 import type Action from './action'
 import type Microcosm from './microcosm'
@@ -79,11 +80,11 @@ class DomainEngine {
   }
 
   add(key: string | KeyPath, config: *, options?: Object) {
-    console.assert(
+    assert(
       !options || options.constructor === Object,
-      'addDomain expected a plain object as the second argument.',
-      'Instead got',
-      get(options, 'constructor.name', 'Unknown')
+      'addDomain expected a plain object as the second argument. ' +
+        'Instead got ' +
+        get(options, 'constructor.name', 'Unknown')
     )
 
     let deepOptions = merge(

src/effect-engine.js

@@ -4,7 +4,7 @@
 
 import getRegistration from './get-registration'
 import { get, merge, createOrClone } from './utils'
-
+import assert from 'assert'
 import type Action from './action'
 import type Microcosm from './microcosm'
 
@@ -18,10 +18,10 @@ class EffectEngine {
   }
 
   add(config: Object | Function, options?: Object) {
-    console.assert(
+    assert(
       !options || options.constructor === Object,
-      'addEffect expected a plain object as the second argument. Instead got',
-      get(options, 'constructor.name', 'Unknown')
+      'addEffect expected a plain object as the second argument. Instead got ' +
+        get(options, 'constructor.name', 'Unknown')
     )
 
     let deepOptions = merge(this.repo.options, config.defaults, options)

src/emitter.js

@@ -5,6 +5,7 @@
  */
 
 import { isFunction } from './utils'
+import assert from 'assert'
 
 export type Callback = (...args: *[]) => *
 
@@ -30,7 +31,7 @@ class Emitter<Event: string> {
    * Add an event listener.
    */
   on(event: Event, fn: Callback, scope?: any) {
-    console.assert(
+    assert(
       isFunction(fn),
       `Expected ${event} listener to be function, instead got ${typeof fn}`
     )
@@ -47,7 +48,7 @@ class Emitter<Event: string> {
    * automatically removed.
    */
   once(event: Event, fn: Callback, scope?: any) {
-    console.assert(
+    assert(
       isFunction(fn),
       `Expected ${event} listener to be function, instead got ${typeof fn}`
     )

src/get-registration.js

@@ -1,5 +1,6 @@
 // @flow
 import { isObject } from './utils'
+import assert from 'assert'
 
 export const STATUSES = {
   inactive: 'inactive',
@@ -30,8 +31,8 @@ function getRegistration(pool: ?Object, command: Tagged, status: Status) {
 
   let alias = STATUSES[status]
 
-  console.assert(alias, 'Invalid action status ' + status)
-  console.assert(
+  assert(alias, 'Invalid action status ' + status)
+  assert(
     command.__tagged,
     `Unable to register ${command.name || 'action'}(). It has not been tagged.`
   )
@@ -48,12 +49,12 @@ function getRegistration(pool: ?Object, command: Tagged, status: Status) {
     /**
      * Throw in strict mode if a nested registration is undefined. This is usually a typo.
      */
-    console.assert(
+    assert(
       !(alias in nest) || answer !== undefined,
       `The "${alias}" key within a nested registration for ${command.name ||
         'an action'} is ${answer}. Is it being referenced correctly?`
     )
-    console.assert(
+    assert(
       !(status in nest) || answer !== undefined,
       `The "${status}" key within a nested registration for ${command.name ||
         'an action'} is ${answer}. Is it being referenced correctly?`
@@ -64,7 +65,7 @@ function getRegistration(pool: ?Object, command: Tagged, status: Status) {
     /**
      * Similarly, throw in strict mode if a regular registration is undefined. This is usually a typo.
      */
-    console.assert(
+    assert(
       !(type in pool) || answer !== undefined,
       `${command.name ||
         'action'} key within a registration is ${answer}. Is it being referenced correctly?`