Fix bug where the <Link /> component reloaded the page unecessarily

.eslintignore

@@ -4,6 +4,11 @@ packages/*/
 .editorconfig
 .eslintignore
 .gitignore
+.snyk
+
 LICENSE
 yarn.lock
 yarn-error.log
+lerna-debug.log
+
+*.png

.snyk

@@ -10,4 +10,7 @@ ignore:
   'npm:underscore.string:20170908':
     - underscore.string:
         reason: DoS attack in a devDependency
+  'npm:braces:20180219':
+    - babel-watch > chokidar > anymatch > micromatch > braces:
+        reason: Dos in a dev dependency
 patch: {}

babel.config.js

@@ -0,0 +1,12 @@
+module.exports = {
+  presets: ['@babel/env', '@babel/react', '@babel/flow'],
+  plugins: [
+    '@babel/proposal-object-rest-spread',
+    '@babel/proposal-class-properties',
+  ],
+  env: {
+    es: {
+      presets: [['@babel/env', { modules: false }]],
+    },
+  },
+}

package.json

@@ -24,17 +24,19 @@
     "check": "yarn run is-pretty && yarn run lint && yarn run test -- -- -w 1 && yarn run build && yarn run vulnerabilities"
   },
   "devDependencies": {
-    "babel-cli": "^6.24.0",
-    "babel-core": "^6.24.0",
-    "babel-eslint": "^8.2.6",
-    "babel-loader": "^7.1.1",
-    "babel-plugin-syntax-object-rest-spread": "^6.13.0",
-    "babel-plugin-transform-class-properties": "^6.24.1",
-    "babel-plugin-transform-flow-strip-types": "^6.22.0",
-    "babel-preset-env": "^1.7.0",
-    "babel-preset-flow": "^6.23.0",
-    "babel-preset-react": "^6.24.1",
-    "babel-watch": "^2.0.6",
+    "@babel/cli": "^7.0.0-rc.3",
+    "@babel/core": "^7.0.0-rc.3",
+    "@babel/plugin-proposal-class-properties": "^7.0.0-rc.3",
+    "@babel/plugin-proposal-object-rest-spread": "^7.0.0-rc.3",
+    "@babel/plugin-syntax-dynamic-import": "^7.0.0-rc.3",
+    "@babel/preset-env": "^7.0.0-rc.3",
+    "@babel/preset-flow": "^7.0.0-rc.3",
+    "@babel/preset-react": "^7.0.0-rc.3",
+    "babel-core": "^7.0.0-bridge.0",
+    "babel-eslint": "^9.0.0-beta.3",
+    "babel-jest": "^23.4.2",
+    "babel-loader": "^8.0.0-beta.6",
+    "babel-watch": "git://github.com/hedgepigdaniel/babel-watch.git",
     "commitizen": "^2.9.6",
     "cross-env": "^5.0.1",
     "css-loader": "^1.0.0",
@@ -52,12 +54,13 @@
     "flow-bin": "^0.78.0",
     "flow-copy-source": "^2.0.2",
     "husky": "^0.14.3",
-    "jest": "^23.4.2",
-    "jest-cli": "^23.4.2",
+    "jest": "^23.5.0",
+    "jest-cli": "^23.5.0",
     "jest-localstorage-mock": "^2.2.0",
     "lerna": "^3.1.1",
     "lint-staged": "^7.2.0",
     "prettier": "^1.14.2",
+    "prop-types": "^15.6.2",
     "react": "^16.1.1",
     "react-redux": "^5.0.6",
     "react-test-renderer": "^16.1.1",

packages/boilerplate/babel.config.js

@@ -0,0 +1,3 @@
+module.exports = {
+  extends: '../../babel.config',
+}

packages/boilerplate/package.json

@@ -21,12 +21,12 @@
     "prettify": "prettier --ignore-path=.eslintignore '**/*' --write",
     "eslint": "eslint",
     "lint": "eslint .",
-    "vulnerabilities": "snyk test"
+    "vulnerabilities": "mkdir node_modules; snyk test"
   },
   "dependencies": {
+    "@babel/polyfill": "^7.0.0-rc.3",
+    "@respond-framework/link": "^0.1.0",
     "@respond-framework/rudy": "^0.1.0",
-    "babel-plugin-syntax-dynamic-import": "^6.18.0",
-    "babel-polyfill": "^6.23.0",
     "express": "^4.15.2",
     "react": "^16.1.1",
     "react-dom": "^16.1.1",

packages/boilerplate/server/serveDev.js

@@ -1,7 +1,7 @@
 /* eslint-disable import/no-extraneous-dependencies */
 
 import 'source-map-support/register'
-import 'babel-polyfill'
+import '@babel/polyfill'
 import path from 'path'
 import express from 'express'
 import favicon from 'serve-favicon'

packages/boilerplate/server/serveProd.js

@@ -1,5 +1,5 @@
 import 'source-map-support/register'
-import 'babel-polyfill'
+import '@babel/polyfill'
 import { resolve } from 'path'
 import express from 'express'
 import favicon from 'serve-favicon'

packages/boilerplate/server/webpack.config.babel.js

@@ -25,7 +25,7 @@ export default (env) => {
         isClient &&
           isDev &&
           'webpack-hot-middleware/client?path=/__webpack_hmr&timeout=20000&reload=false&quiet=false&noInfo=false',
-        isClient && 'babel-polyfill',
+        isClient && '@babel/polyfill',
         res(isServer ? '../src/render.server.js' : '../src/render.browser.js'),
       ].filter(Boolean),
     },
@@ -45,18 +45,11 @@ export default (env) => {
           use: {
             loader: 'babel-loader',
             options: {
-              // { modules: false } allows tree shaking, HMR, and import errors/warnings to work correctly
-              // Sadly, node doesn't support ES imports/exports yet, so it can't go in .babelrc
-              babelrc: false,
               cacheDirectory: true,
-              presets: [['env', { modules: false }], 'react'],
               plugins: [
                 'react-hot-loader/babel',
-                'syntax-dynamic-import',
+                '@babel/syntax-dynamic-import',
                 'universal-import',
-                'transform-flow-strip-types',
-                'syntax-object-rest-spread',
-                'transform-class-properties',
               ],
             },
           },

packages/boilerplate/src/components/Sidebar.js

@@ -1,6 +1,6 @@
 import React from 'react'
 import { connect } from 'react-redux'
-import { NavLink, Link } from '@respond-framework/rudy'
+import { NavLink, Link } from '@respond-framework/link'
 import styles from '../css/Sidebar'
 
 // TODO: Use the link package

packages/integration-tests/.eslintignore

@@ -0,0 +1,6 @@
+node_modules/
+yarn-error.log
+
+*.snap
+
+.eslintignore

packages/integration-tests/.eslintrc.js

@@ -0,0 +1,11 @@
+const path = require('path')
+
+function res() {
+  const segments = Array.from(arguments)
+  segments.unshift(__dirname)
+  return path.join.apply(undefined, segments)
+}
+
+module.exports = {
+  extends: [res('../../.eslintrc.js')],
+}

packages/integration-tests/.prettierrc.js

@@ -0,0 +1 @@
+module.exports = require('../../.prettierrc')

packages/rudy/__helpers__/createTest.js → packages/integration-tests/__helpers__/createTest.js

@@ -1,8 +1,11 @@
 import { applyMiddleware, createStore, combineReducers } from 'redux'
+import {
+  get,
+  clear,
+} from '@respond-framework/rudy/src/history/utils/sessionStorage'
+import { locationToUrl } from '@respond-framework/rudy/src/utils'
+import { createRouter } from '@respond-framework/rudy/src'
 import awaitUrlChange from './awaitUrlChange'
-import { get, clear } from '../src/history/utils/sessionStorage'
-import { locationToUrl } from '../src/utils'
-import { createRouter } from '../src'
 
 export default async (...allArgs) => {
   const args = allArgs.filter((arg) => typeof arg !== 'function')

packages/rudy/__test-helpers__/createLink.js → packages/integration-tests/__test-helpers__/createLink.js

@@ -3,9 +3,9 @@ import renderer from 'react-test-renderer'
 import { createStore, applyMiddleware } from 'redux'
 import { Provider } from 'react-redux'
 
-import { createRouter } from '../src'
+import { createRouter } from '@respond-framework/rudy/src'
 
-import Link, { NavLink } from '../src/Link'
+import Link, { NavLink } from '@respond-framework/rudy/src/Link'
 
 const createLink = async (props, initialPath, options, isNavLink) => {
   const link = isNavLink ? <NavLink {...props} /> : <Link {...props} />

packages/rudy/__test-helpers__/createSequence.js → packages/integration-tests/__test-helpers__/createSequence.js

@@ -1,6 +1,6 @@
 import { applyMiddleware, createStore, combineReducers } from 'redux'
-import { createRouter, createScene } from '../src'
-import { NOT_FOUND } from '../src/types'
+import { createRouter, createScene } from '@respond-framework/rudy/src'
+import { NOT_FOUND } from '@respond-framework/rudy/src/types'
 
 import fakeAsyncWork from './fakeAsyncWork'
 

packages/rudy/__test-helpers__/reducerParameters.js → packages/integration-tests/__test-helpers__/reducerParameters.js

@@ -1,6 +1,6 @@
-import createSmartHistory from '../src/history'
-import { createInitialState } from '../src/core/createReducer'
-import { NOT_FOUND } from '../src/types'
+import createSmartHistory from '@respond-framework/rudy/src/history'
+import { createInitialState } from '@respond-framework/rudy/src/core/createReducer'
+import { NOT_FOUND } from '@respond-framework/rudy/src/types'
 
 export default async (type, pathname) => {
   // eslint-disable-line import/prefer-default-export

packages/rudy/__test-helpers__/rudySetup.js → packages/integration-tests/__test-helpers__/rudySetup.js

@@ -1,7 +1,7 @@
 import { applyMiddleware, createStore, combineReducers } from 'redux'
 import reduxThunk from 'redux-thunk'
-import { createRouter, createScene } from '../src'
-import { NOT_FOUND } from '../src/types'
+import { createRouter, createScene } from '@respond-framework/rudy/src'
+import { NOT_FOUND } from '@respond-framework/rudy/src/types'
 
 import fakeAsyncWork from './fakeAsyncWork'
 

packages/rudy/__test-helpers__/setup.js → packages/integration-tests/__test-helpers__/setup.js

@@ -1,6 +1,6 @@
 import { applyMiddleware, createStore } from 'redux'
 import reduxThunk from 'redux-thunk'
-import { createRouter } from '../src'
+import { createRouter } from '@respond-framework/rudy/src'
 
 const setup = (
   path = '/',

packages/rudy/__test-helpers__/setupThunk.js → packages/integration-tests/__test-helpers__/setupThunk.js

@@ -1,5 +1,5 @@
 import { createStore, applyMiddleware, combineReducers } from 'redux'
-import createRouter from '../src/core/createRouter'
+import createRouter from '@respond-framework/rudy/src/core/createRouter'
 
 export default async (
   path = '/',

packages/rudy/__tests__/Link/Link.js → packages/integration-tests/__tests__/Link/Link.js

@@ -1,4 +1,4 @@
-import { NOT_FOUND } from '../../src/types'
+import { NOT_FOUND } from '@respond-framework/rudy/src/types'
 import createLink, { event } from '../../__test-helpers__/createLink'
 
 test('ON_CLICK: dispatches location-aware action', async () => {

packages/rudy/__tests__/integration/SPA.js → packages/integration-tests/__tests__/integration/SPA.js

@@ -1,7 +1,7 @@
 import createTest from '../../__helpers__/createTest'
 
-jest.mock('../../src/utils/isHydrate', () => () => false)
-jest.mock('../../src/utils/isServer', () => () => false)
+jest.mock('@respond-framework/rudy/src/utils/isHydrate', () => () => false)
+jest.mock('@respond-framework/rudy/src/utils/isServer', () => () => false)
 
 createTest('callbacks called on load if SPA', {
   FIRST: {

packages/rudy/__tests__/integration/actions/addRoutes.js → packages/integration-tests/__tests__/integration/actions/addRoutes.js

@@ -1,5 +1,5 @@
+import { addRoutes } from '@respond-framework/rudy/src/actions'
 import createTest from '../../../__helpers__/createTest'
-import { addRoutes } from '../../../src/actions'
 
 createTest(
   'dispatch(addRoutes(routes))',

packages/rudy/__tests__/integration/actions/changeBasename.js → packages/integration-tests/__tests__/integration/actions/changeBasename.js

@@ -1,5 +1,5 @@
+import { changeBasename } from '@respond-framework/rudy/src/actions'
 import createTest from '../../../__helpers__/createTest'
-import { changeBasename } from '../../../src/actions'
 
 createTest(
   'dispatch(changeBasename(name))',

packages/rudy/__tests__/integration/actions/history.js → packages/integration-tests/__tests__/integration/actions/history.js

@@ -1,4 +1,3 @@
-import createTest from '../../../__helpers__/createTest'
 import {
   push,
   replace,
@@ -13,7 +12,8 @@ import {
   setHash,
   setBasename,
   redirect,
-} from '../../../src/actions'
+} from '@respond-framework/rudy/src/actions'
+import createTest from '../../../__helpers__/createTest'
 
 const routes = {
   FIRST: {

packages/rudy/__tests__/integration/actions/notFound.js → packages/integration-tests/__tests__/integration/actions/notFound.js

@@ -1,6 +1,6 @@
+import { notFound } from '@respond-framework/rudy/src/actions'
+import createScene from '@respond-framework/rudy/src/createScene'
 import createTest from '../../../__helpers__/createTest'
-import { notFound } from '../../../src/actions'
-import createScene from '../../../src/createScene'
 
 createTest('dispatch(notFound())', {}, [notFound()])
 

packages/rudy/__tests__/integration/actions/redirect.js → packages/integration-tests/__tests__/integration/actions/redirect.js

@@ -1,5 +1,5 @@
+import { redirect } from '@respond-framework/rudy/src/actions'
 import createTest from '../../../__helpers__/createTest'
-import { redirect } from '../../../src/actions'
 
 createTest('dispatch(redirect(action))', {}, [redirect({ type: 'REDIRECTED' })])
 

packages/rudy/__tests__/integration/browser/actionsInCallbacks/jump.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/jump.js

@@ -1,5 +1,5 @@
+import { jump } from '@respond-framework/rudy/src/actions'
 import createTest, { resetBrowser } from '../../../../__helpers__/createTest'
-import { jump } from '../../../../src/actions'
 
 beforeEach(resetBrowser)
 

packages/rudy/__tests__/integration/browser/actionsInCallbacks/jump2N.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/jump2N.js

@@ -1,8 +1,8 @@
+import { jump } from '@respond-framework/rudy/src/actions'
 import createTest, {
   resetBrowser,
   setupStore,
 } from '../../../../__helpers__/createTest'
-import { jump } from '../../../../src/actions'
 
 beforeEach(async () => {
   await resetBrowser()

packages/rudy/__tests__/integration/browser/actionsInCallbacks/push.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/push.js

@@ -1,5 +1,5 @@
+import { push } from '@respond-framework/rudy/src/actions'
 import createTest, { resetBrowser } from '../../../../__helpers__/createTest'
-import { push } from '../../../../src/actions'
 
 beforeEach(resetBrowser)
 

packages/rudy/__tests__/integration/browser/actionsInCallbacks/regularAction.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/regularAction.js

@@ -1,5 +1,5 @@
+import { push } from '@respond-framework/rudy/src/actions'
 import createTest, { resetBrowser } from '../../../../__helpers__/createTest'
-import { push } from '../../../../src/actions'
 
 beforeEach(resetBrowser)
 

packages/rudy/__tests__/integration/browser/actionsInCallbacks/replace.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/replace.js

@@ -1,5 +1,5 @@
+import { replace } from '@respond-framework/rudy/src/actions'
 import createTest, { resetBrowser } from '../../../../__helpers__/createTest'
-import { replace } from '../../../../src/actions'
 
 beforeEach(resetBrowser)
 

packages/rudy/__tests__/integration/browser/actionsInCallbacks/reset.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/reset.js

@@ -1,6 +1,6 @@
+import { locationToUrl } from '@respond-framework/rudy/src/utils'
+import { reset } from '@respond-framework/rudy/src/actions'
 import createTest, { resetBrowser } from '../../../../__helpers__/createTest'
-import { locationToUrl } from '../../../../src/utils'
-import { reset } from '../../../../src/actions'
 
 beforeEach(resetBrowser)
 

packages/rudy/__tests__/integration/browser/actionsInCallbacks/resetOnLoad.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/resetOnLoad.js

@@ -1,9 +1,9 @@
+import { locationToUrl } from '@respond-framework/rudy/src/utils'
+import { reset } from '@respond-framework/rudy/src/actions'
 import createTest, {
   resetBrowser,
   setupStore,
 } from '../../../../__helpers__/createTest'
-import { locationToUrl } from '../../../../src/utils'
-import { reset } from '../../../../src/actions'
 
 beforeEach(async () => {
   await resetBrowser()

packages/rudy/__tests__/integration/browser/actionsInCallbacks/set.js → packages/integration-tests/__tests__/integration/browser/actionsInCallbacks/set.js

@@ -1,5 +1,5 @@
+import { set, setState } from '@respond-framework/rudy/src/actions'
 import createTest, { resetBrowser } from '../../../../__helpers__/createTest'
-import { set, setState } from '../../../../src/actions'
 
 beforeEach(resetBrowser)
 

packages/rudy/__tests__/integration/browser/history/jump.js → packages/integration-tests/__tests__/integration/browser/history/jump.js

@@ -1,6 +1,6 @@
+import { locationToUrl } from '@respond-framework/rudy/src/utils'
+import { jump } from '@respond-framework/rudy/src/actions'
 import createTest from '../../../../__helpers__/createTest'
-import { locationToUrl } from '../../../../src/utils'
-import { jump } from '../../../../src/actions'
 
 createTest(
   'set(action, n)',

packages/rudy/__tests__/integration/browser/history/jump2N.js → packages/integration-tests/__tests__/integration/browser/history/jump2N.js

@@ -1,6 +1,6 @@
+import { locationToUrl } from '@respond-framework/rudy/src/utils'
+import { jump } from '@respond-framework/rudy/src/actions'
 import createTest from '../../../../__helpers__/createTest'
-import { locationToUrl } from '../../../../src/utils'
-import { jump } from '../../../../src/actions'
 
 createTest(
   'set(action, n)',