chore: add eslint configs

Author: lindsaylevine

.editorconfig

@@ -0,0 +1,10 @@
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+max_line_length = 120
+trim_trailing_whitespace = true
+insert_final_newline = true

.eslintignore

@@ -0,0 +1,3 @@
+.next
+node_modules
+test

.eslintrc.js

@@ -0,0 +1,35 @@
+const { overrides } = require("@netlify/eslint-config-node/react_config");
+
+module.exports = {
+  extends: "@netlify/eslint-config-node/react_config",
+  rules: {
+    "max-depth": 0,
+    complexity: 0,
+    "fp/no-let": 0,
+    "fp/no-loops": 0,
+    "fp/no-mutation": 0,
+    "fp/no-mutating-methods": 0,
+    "id-length": 0,
+    "max-statements": 0,
+    "no-await-in-loop": 0,
+    "node/exports-style": 0,
+    "node/global-require": 0,
+    "node/prefer-global/process": 0,
+    "no-magic-numbers": 0,
+    "no-param-reassign": 0,
+    "no-promise-executor-return": 0,
+    "no-prototype-builtins": 0,
+    "no-shadow": 0,
+    "no-unused-vars": 0,
+    "prefer-regex-literals": 0,
+    "promise/prefer-await-to-callbacks": 0,
+    "require-await": 0,
+    "unicorn/consistent-function-scoping": 0,
+    "unicorn/filename-case": 0,
+    "unicorn/no-array-push-push": 0,
+  },
+  env: {
+    jest: true,
+  },
+  overrides: [...overrides],
+};

commitlint.config.js

@@ -1 +1 @@
-module.exports = { extends: ['@commitlint/config-conventional'] }
+module.exports = { extends: ["@commitlint/config-conventional"] };

package.json

@@ -57,6 +57,7 @@
     "is-svg": "^4.3.1",
     "mime-types": "^2.1.30",
     "moize": "^6.0.0",
+    "next": "^11.1.0",
     "node-fetch": "^2.6.1",
     "semver": "^7.3.5",
     "sharp": "^0.29.0"
@@ -66,7 +67,6 @@
     "eslint-config-next": "^11.0.0",
     "husky": "^4.3.0",
     "jest": "^27.0.0",
-    "next": "^11.1.0",
     "prettier": "^2.1.2",
     "react": "^17.0.1",
     "react-dom": "^17.0.1"

src/copyNextDist.js

@@ -1,15 +1,16 @@
 const { join } = require("path");
-const { copySync, existsSync } = require('fs-extra')
 
-const copyNextDist = async (siteRoot) => {
+const { copySync, existsSync } = require("fs-extra");
+
+const copyNextDist = (siteRoot) => {
   // TO-DO: get distDir from next.config.js
-  const distPath = join(siteRoot, '.next')
+  const distPath = join(siteRoot, ".next");
   if (!existsSync(distPath)) {
-    throw new Error(`No dist found in ${distPath}`)
+    throw new Error(`No dist found in ${distPath}`);
   }
-  copySync(distPath, join('.', '.next'), {
-    overwrite: true
-  })
-}
+  copySync(distPath, join(".", ".next"), {
+    overwrite: true,
+  });
+};
 
-module.exports = copyNextDist
+module.exports = copyNextDist;

src/getHandler.js

@@ -1,10 +1,13 @@
 const { Server } = require("http");
+
 const { Bridge } = require("@vercel/node/dist/bridge");
 // This path is specific to next@canary. In a live version we'd resolve various versions of next
 const NextServer = require("next/dist/server/next-server").default;
-function makeHandler() {
+
+const makeHandler =
+  () =>
   // We return a function and then call `toString()` on it to serialise it as the launcher function
-  return (conf) => {
+  (conf) => {
     const nextServer = new NextServer({
       conf,
       dir: ".",
@@ -14,27 +17,25 @@ function makeHandler() {
     const server = new Server(async (req, res) => {
       try {
         await requestHandler(req, res);
-      } catch (err) {
-        console.error(err);
-        process.exit(1);
+      } catch (error) {
+        console.error(error);
+        throw new Error("server function error");
       }
     });
     const bridge = new Bridge(server);
     bridge.listen();
 
     return async (event, context) => {
-      let result = await bridge.launcher(event, context);
+      const result = await bridge.launcher(event, context);
       /** @type import("@netlify/functions").HandlerResponse */
       return {
         ...result,
         isBase64Encoded: result.encoding === "base64",
       };
     };
   };
-}
 
-exports.getHandler = async function (isODB = false) {
-  return `
+const getHandler = (isODB = false) => `
 const { Server } = require("http");
 // We copy the file here rather than requiring from the node module
 const { Bridge } = require("./bridge");
@@ -45,9 +46,8 @@ const { builder } = require("@netlify/functions");
 const { config }  = require(process.cwd() + "/.next/required-server-files.json")
 
 exports.handler = ${
-   isODB
-      ? `builder((${makeHandler().toString()})(config));`
-      : `(${makeHandler().toString()})(config);`
-  }
+  isODB ? `builder((${makeHandler().toString()})(config));` : `(${makeHandler().toString()})(config);`
+}
 `;
-};
+
+module.exports = getHandler;

src/getNextRoot.js

@@ -1,20 +1,20 @@
-const { existsSync } = require('fs')
-const path = require('path')
+const { existsSync } = require("fs");
+const path = require("path");
 
 /**
  * If we're in a monorepo then the Next root may not be the same as the base directory
  * If there's no next.config.js in the root, we instead look for it as a sibling of the publish dir
  */
 const getNextRoot = ({ netlifyConfig }) => {
-  let nextRoot = process.cwd()
+  let nextRoot = process.cwd();
   if (
-    !existsSync(path.join(nextRoot, 'next.config.js')) &&
+    !existsSync(path.join(nextRoot, "next.config.js")) &&
     netlifyConfig.build.publish &&
     netlifyConfig.build.publish !== nextRoot
   ) {
-    nextRoot = path.dirname(netlifyConfig.build.publish)
+    nextRoot = path.dirname(netlifyConfig.build.publish);
   }
-  return nextRoot
-}
+  return nextRoot;
+};
 
-module.exports = getNextRoot
+module.exports = getNextRoot;

src/index.js

@@ -1,12 +1,13 @@
 // @ts-check
-const { getHandler } = require("./getHandler");
-const fs = require("fs").promises;
-const { ensureDir, existsSync, remove } = require("fs-extra");
 const path = require("path");
+
+const { copyFile, ensureDir, writeFile } = require("fs-extra");
+
 const DEFAULT_FUNCTIONS_SRC = "netlify/functions";
-const { writeRedirects } = require("./writeRedirects");
-const getNextRoot = require("./getNextRoot")
-const copyNextDist = require("./copyNextDist")
+const copyNextDist = require("./copyNextDist");
+const getHandler = require("./getHandler");
+const getNextRoot = require("./getNextRoot");
+const writeRedirects = require("./writeRedirects");
 
 const HANDLER_FUNCTION_NAME = "___netlify-handler";
 const ODB_FUNCTION_NAME = "___netlify-odb-handler";
@@ -22,35 +23,24 @@ module.exports = {
         netlifyConfig.functions[functionName].included_files = [];
       }
       // TO-DO: get distDir from next.config.js
-      netlifyConfig.functions[functionName].included_files.push(
-        `.next/server/**`,
-        `.next/*.json`,
-        `.next/BUILD_ID`
-      );
+      netlifyConfig.functions[functionName].included_files.push(`.next/server/**`, `.next/*.json`, `.next/BUILD_ID`);
     });
 
     console.log({ netlifyConfig });
   },
 
   async onBuild({
     netlifyConfig,
-    constants: {
-      PUBLISH_DIR,
-      FUNCTIONS_SRC = DEFAULT_FUNCTIONS_SRC,
-      INTERNAL_FUNCTIONS_SRC,
-    },
+    constants: { PUBLISH_DIR, FUNCTIONS_SRC = DEFAULT_FUNCTIONS_SRC, INTERNAL_FUNCTIONS_SRC },
   }) {
     const FUNCTION_DIR = INTERNAL_FUNCTIONS_SRC || FUNCTIONS_SRC;
     const bridgeFile = require.resolve("@vercel/node/dist/bridge");
 
     const writeHandler = async (func, isODB) => {
       const handlerSource = await getHandler(isODB);
       await ensureDir(path.join(FUNCTION_DIR, func));
-      await fs.writeFile(
-        path.join(FUNCTION_DIR, func, `${func}.js`),
-        handlerSource
-      );
-      await fs.copyFile(bridgeFile, path.join(FUNCTION_DIR, func, "bridge.js"));
+      await writeFile(path.join(FUNCTION_DIR, func, `${func}.js`), handlerSource);
+      await copyFile(bridgeFile, path.join(FUNCTION_DIR, func, "bridge.js"));
     };
 
     await writeHandler(HANDLER_FUNCTION_NAME, false);

src/writeRedirects.js

@@ -1,8 +1,8 @@
 // @ts-check
-const { writeFile } = require("fs").promises;
-const { readJSON } = require("fs-extra");
 const path = require("path");
 
+const { readJSON, writeFile } = require("fs-extra");
+
 const ODB_FUNCTION_PATH = "/.netlify/functions/___netlify-odb-handler";
 const HANDLER_FUNCTION_PATH = "/.netlify/functions/___netlify-handler";
 
@@ -27,41 +27,26 @@ const getNetlifyRoutes = (nextRoute) => {
     // create an incorrect route for the data route. For example, it creates
     // /_next/data/%BUILDID%.json, but NextJS looks for
     // /_next/data/%BUILDID%/index.json
-    netlifyRoute = netlifyRoute.replace(
-      /(\/_next\/data\/[^/]+).json/,
-      "$1/index.json"
-    );
+    netlifyRoute = netlifyRoute.replace(/(\/_next\/data\/[^/]+).json/, "$1/index.json");
 
     // Add second route to the front of the array
     netlifyRoutes.unshift(netlifyRoute);
   }
 
   // Replace catch-all, e.g., [...slug]
-  netlifyRoutes = netlifyRoutes.map((route) =>
-    route.replace(CATCH_ALL_REGEX, "/:$1/*")
-  );
+  netlifyRoutes = netlifyRoutes.map((route) => route.replace(CATCH_ALL_REGEX, "/:$1/*"));
 
   // Replace optional catch-all, e.g., [[...slug]]
-  netlifyRoutes = netlifyRoutes.map((route) =>
-    route.replace(OPTIONAL_CATCH_ALL_REGEX, "/*")
-  );
+  netlifyRoutes = netlifyRoutes.map((route) => route.replace(OPTIONAL_CATCH_ALL_REGEX, "/*"));
 
   // Replace dynamic parameters, e.g., [id]
-  netlifyRoutes = netlifyRoutes.map((route) =>
-    route.replace(DYNAMIC_PARAMETER_REGEX, "/:$1")
-  );
+  netlifyRoutes = netlifyRoutes.map((route) => route.replace(DYNAMIC_PARAMETER_REGEX, "/:$1"));
 
   return netlifyRoutes;
 };
 
-exports.writeRedirects = async function ({
-  publishDir = "out",
-  nextRoot = process.cwd(),
-  netlifyConfig,
-}) {
-  const { dynamicRoutes } = await readJSON(
-    path.join(nextRoot, ".next", "prerender-manifest.json")
-  );
+const writeRedirects = async ({ publishDir = "out", nextRoot = process.cwd(), netlifyConfig }) => {
+  const { dynamicRoutes } = await readJSON(path.join(nextRoot, ".next", "prerender-manifest.json"));
 
   const redirects = [];
   Object.entries(dynamicRoutes).forEach(([route, { dataRoute, fallback }]) => {
@@ -93,3 +78,5 @@ exports.writeRedirects = async function ({
   //     `;
   // await writeFile(path.join(nextRoot, publishDir, "_redirects"), odbRedirects);
 };
+
+module.exports = writeRedirects;