chore: refactor

Author: lindsaylevine

helpers/copyUnstableIncludedDirs.js

@@ -0,0 +1,26 @@
+const path = require('path')
+const AdmZip = require('adm-zip')
+const getNetlifyFunctionName = require('../src/lib/helpers/getNetlifyFunctionName')
+
+// This is UNSTABLE support for special use cases needing files served with their Next.js
+// pages that become Netlify functions. Eventually this will be supported internally.
+const copyUnstableIncludedDirs = ({ nextConfig, functionsDist }) => {
+  for (const name in nextConfig.unstableNetlifyFunctionsSupport || {}) {
+    const includeDirs = nextConfig.unstableNetlifyFunctionsSupport[name].includeDirs || []
+    console.log('Processing included dirs for ', name)
+
+    const zipName = path.join(functionsDist, getNetlifyFunctionName(name) + '.zip')
+    const zip = new AdmZip(zipName)
+    includeDirs.forEach((includes) => {
+      if (fs.lstatSync(includes).isDirectory()) {
+        // We add the files at the root of the ZIP because process.cwd()
+        // points to `/` in serverless functions
+        zip.addLocalFolder(includes, includes)
+        console.log(`Added ${includes} to ${zipName}`)
+      }
+    })
+    zip.writeZip(zipName)
+  }
+}
+
+module.exports = copyUnstableIncludedDirs

index.js

@@ -4,11 +4,11 @@ const util = require('util')
 const makeDir = require('make-dir')
 const findUp = require('find-up')
 const nextOnNetlify = require('./src/index.js')
-const AdmZip = require('adm-zip')
 
 const validateNextUsage = require('./helpers/validateNextUsage')
 const doesNotNeedPlugin = require('./helpers/doesNotNeedPlugin')
 const getNextConfig = require('./helpers/getNextConfig')
+const copyUnstableIncludedDirs = require('./helpers/copyUnstableIncludedDirs')
 
 const pWriteFile = util.promisify(fs.writeFile)
 
@@ -66,25 +66,7 @@ module.exports = {
     }
 
     const nextConfig = await getNextConfig(utils.failBuild)
-    const getNetlifyFunctionName = require('./src/lib/helpers/getNetlifyFunctionName')
-
-    // any functions defined in the config need special handling
-    for (const name in nextConfig.unstableNetlifyFunctionsSupport || {}) {
-      const includeDirs = nextConfig.unstableNetlifyFunctionsSupport[name].includeDirs || []
-      console.log('Processing included dirs for ', name)
-
-      const zipName = path.join(FUNCTIONS_DIST, getNetlifyFunctionName(name) + '.zip')
-      const zip = new AdmZip(zipName)
-      includeDirs.forEach((includes) => {
-        if (fs.lstatSync(includes).isDirectory()) {
-          // we add the files at the root of the ZIP because process.cwd()
-          // points to `/` in serverless functions
-          zip.addLocalFolder(includes, includes)
-          console.log(`Added ${includes} to ${zipName}`)
-        }
-      })
-      zip.writeZip(zipName)
-    }
+    copyUnstableIncludedDirs({ nextConfig, functionsDist: FUNCTIONS_DIST })
   },
 }