Remove next-routes (godaddy#240)

* [minor] drop auto next-routes and old next build

* [doc] drop next-routes notes

packages/gasket-cli/docs/gotchas.md

@@ -19,7 +19,6 @@ CommonJS-style modules because they're imported directly by node:
 * The Gasket config file (`gasket.config.js`)
 * Your store creator (including files it imports, like reducers, action
   constants, etc.) - typically `store.js`.
-* Your routes creator (if using `next-routes`- typically `routes.js`).
 * Lifecycle hooks (`/lifecycles/*.js`)
 * Plugins (`/plugins/*.js`)
 

packages/gasket-plugin-nextjs/docs/deployment.md

@@ -107,7 +107,6 @@ Ensure that the following files are included when you deploy your application:
 - `package-lock.json` *Automatically generated file about the installed dependencies*
 - `.babelrc` *Config file for babel*
 - `gasket.config.js` *Config file for Gasket*
-- `routes.js` *Routes for your application*
 - `store.js` *Scripts for creating a redux store and/or attaching a reducer*
 
 ## Deployment checklist

packages/gasket-plugin-nextjs/docs/routing.md

@@ -89,4 +89,3 @@ your component with the [Next.js router HOC]: `withRouter`.
 [Next.js documentation]:https://github.com/zeit/next.js
 [Next.js routing]:https://github.com/zeit/next.js#routing
 [Next.js router HOC]:https://github.com/zeit/next.js#using-a-higher-order-component
-[next-routes]:https://github.com/fridays/next-routes

packages/gasket-plugin-nextjs/index.js

@@ -128,34 +128,12 @@ module.exports = {
         next();
       });
 
-      // TODO: (@kinetifex) we no longer support next-routes - remove this
-      const { root, routes } = gasket.config || {};
-      const routesModulePath = path.join(root, routes || './routes');
-      let ssr;
-
-      try {
-        let router = require(routesModulePath);
-
-        // Handle ES6-style modules
-        if (router.default) {
-          router = router.default;
-        }
-
-        ssr = router.getRequestHandler(app);
-      } catch (err) {
-        if (err.code !== 'MODULE_NOT_FOUND') {
-          throw err;
-        }
-
-        ssr = app.getRequestHandler();
-      }
-
       //
       // Now that express has been setup, and users have been able to
       // interact with the express router we want to add a last, catch all
       // route that will activate the `next`.
       //
-      expressApp.all('*', ssr);
+      expressApp.all('*', app.getRequestHandler());
 
       return app;
     },
@@ -164,18 +142,7 @@ module.exports = {
       // Don't do a build, use dev server for local
       if ((command.id || command) === 'local') return;
 
-      //
-      // Different versions of Nextjs, have different ways of exporting the builder.
-      // In order to support canary, and other versions of next we need to detect
-      // the different locations.
-      //
-      let builder;
-      try {
-        builder = require('next/dist/server/build').default;
-      } catch (e) {
-        builder = require('next/dist/build').default;
-      }
-
+      const builder = require('next/dist/build').default;
       return await builder(path.resolve('.'), await createConfig(gasket, true));
     },
     /**

packages/gasket-plugin-nextjs/test/index.test.js

@@ -293,19 +293,6 @@ describe('build hook', () => {
     await buildHook({ command: 'local' });
     assume(builderStub).not.called();
   });
-
-  it('supports older next build', async () => {
-    const oldBuilderStub = stub();
-    const buildHook = getMockedBuildHook({
-      'next/dist/server/build': {
-        default: oldBuilderStub
-      }
-    });
-    await buildHook({ command: { id: 'build' } });
-
-    assume(oldBuilderStub).called();
-    assume(builderStub).not.called();
-  });
 });
 
 describe('workbox hook', () => {