chore: fix linting errors

.prettierrc.js

@@ -1,4 +1,6 @@
+const rules = require('@netlify/eslint-config-node/.prettierrc.json')
+
 module.exports = {
-  ...require("@netlify/eslint-config-node/.prettierrc.json"),
-  endOfLine: "auto",
+  ...rules,
+  endOfLine: 'auto',
 }

src/__tests__/gatsby-node.ts

@@ -1,30 +1,26 @@
-jest.mock('../plugin-data', () => {
-  return {
-    __esModule: true,
-    default: jest.fn().mockReturnValue({
-      publicFolder: jest.fn().mockReturnValue('mock-file-path'),
-    }),
-  }
-})
-jest.mock('../build-headers-program', () => {
-  return {
-    __esModule: true,
-    default: jest.fn(),
-  }
-})
-jest.mock('fs-extra', () => {
-  return {
-    __esModule: true,
-    default: jest.fn(),
-    existsSync: jest.fn(),
-    readFile: jest.fn(),
-    writeFile: jest.fn(),
-  }
-})
+/* eslint-disable import/first, max-nested-callbacks */
+jest.mock('../plugin-data', () => ({
+  __esModule: true,
+  default: jest.fn().mockReturnValue({
+    publicFolder: jest.fn().mockReturnValue('mock-file-path'),
+  }),
+}))
+jest.mock('../build-headers-program', () => ({
+  __esModule: true,
+  default: jest.fn(),
+}))
+jest.mock('fs-extra', () => ({
+  __esModule: true,
+  default: jest.fn(),
+  existsSync: jest.fn(),
+  readFile: jest.fn(),
+  writeFile: jest.fn(),
+}))
 
 // Importing writeFile here gives us access to the mocked method to assert the correct content is written to the file within test cases
 import { writeFile } from 'fs-extra'
 import { testPluginOptionsSchema } from 'gatsby-plugin-utils'
+
 import { pluginOptionsSchema, onPostBuild } from '../gatsby-node'
 
 describe(`gatsby-node.js`, () => {
@@ -46,7 +42,7 @@ describe(`gatsby-node.js`, () => {
         mergeSecurityHeaders: `this should be a boolean`,
         mergeLinkHeaders: `this should be a boolean`,
         mergeCachingHeaders: `this should be a boolean`,
-        transformHeaders: (too, many, args) => ``,
+        transformHeaders: (too, many, args) => [too, many, args],
         generateMatchPathRewrites: `this should be a boolean`,
       })
 
@@ -115,3 +111,4 @@ describe(`gatsby-node.js`, () => {
     })
   })
 })
+/* eslint-enable import/first, max-nested-callbacks */

src/constants.ts

@@ -1,4 +1,4 @@
-import _ from "lodash"
+import _ from 'lodash'
 
 // Gatsby values
 export const BUILD_HTML_STAGE = `build-html`
@@ -7,17 +7,19 @@ export const BUILD_CSS_STAGE = `build-css`
 // Plugin values
 export const NETLIFY_HEADERS_FILENAME = `_headers`
 
+// Default options including transform to manipulate headers for
+// sorting and rewrites for client only paths
 export const DEFAULT_OPTIONS = {
   headers: {},
   mergeSecurityHeaders: true,
   mergeLinkHeaders: true,
   mergeCachingHeaders: true,
-  transformHeaders: _.identity, // optional transform for manipulating headers for sorting, etc
-  generateMatchPathRewrites: true, // generate rewrites for client only paths
+  transformHeaders: _.identity,
+  generateMatchPathRewrites: true,
 }
 
 export const SECURITY_HEADERS = {
-  "/*": [
+  '/*': [
     `X-Frame-Options: DENY`,
     `X-XSS-Protection: 1; mode=block`,
     `X-Content-Type-Options: nosniff`,
@@ -28,8 +30,8 @@ export const SECURITY_HEADERS = {
 export const IMMUTABLE_CACHING_HEADER = `Cache-Control: public, max-age=31536000, immutable`
 
 export const CACHING_HEADERS = {
-  "/static/*": [IMMUTABLE_CACHING_HEADER],
-  "/sw.js": [`Cache-Control: no-cache`],
+  '/static/*': [IMMUTABLE_CACHING_HEADER],
+  '/sw.js': [`Cache-Control: no-cache`],
 }
 
 export const LINK_REGEX = /^(Link: <\/)(.+)(>;.+)/

src/create-redirects.ts

@@ -38,9 +38,7 @@ export default async function writeRedirectsFile(pluginData: any, redirects: any
       const value = rest[key]
 
       if (typeof value === `string` && value.includes(` `)) {
-        console.warn(
-          `Invalid redirect value "${value}" specified for key "${key}". ` + `Values should not contain spaces.`,
-        )
+        console.warn(`Invalid redirect value "${value}" specified for key "${key}". Values should not contain spaces.`)
       } else if (NETLIFY_REDIRECT_KEYWORDS_ALLOWLIST.has(key)) {
         pieces.push(`${key}=${value}`)
       }
@@ -49,10 +47,7 @@ export default async function writeRedirectsFile(pluginData: any, redirects: any
     return pieces.join(`  `)
   })
 
-  rewrites = rewrites.map(({
-    fromPath,
-    toPath
-  }: any) => `${fromPath}  ${toPath}  200`)
+  rewrites = rewrites.map(({ fromPath, toPath }: any) => `${fromPath}  ${toPath}  200`)
 
   let commentFound = false
 

src/plugin-data.ts

@@ -1,15 +1,17 @@
-import path from "path"
+import path from 'path'
 
-export function buildPrefixer(prefix: any, ...paths: any[]) {
-  return (...subpaths: any[]) => path.join(prefix, ...paths, ...subpaths);
-}
+const buildPrefixer =
+  (prefix: any, ...paths: any[]) =>
+  (...subpaths: any[]) =>
+    path.join(prefix, ...paths, ...subpaths)
 
 // This function assembles data across the manifests and store to match a similar
 // shape of `static-entry.js`. With it, we can build headers that point to the correct
 // hashed filenames and ensure we pull in the componentChunkName.
-export default function makePluginData(store: any, assetsManifest: any, pathPrefix: any) {
+const makePluginData = (store: any, assetsManifest: any, pathPrefix: any) => {
   const { program, pages, components } = store.getState()
   const publicFolder = buildPrefixer(program.directory, `public`)
+  // eslint-disable-next-line node/global-require, import/no-dynamic-require, @typescript-eslint/no-var-requires
   const stats = require(publicFolder(`webpack.stats.json`))
   // Get all the files, not just the first
   const chunkManifest = stats.assetsByChunkName
@@ -25,3 +27,5 @@ export default function makePluginData(store: any, assetsManifest: any, pathPref
     publicFolder,
   }
 }
+
+export { makePluginData as default, buildPrefixer }