netlify-cli 8.1.3 → 8.1.4

package/src/lib/fs.js

@@ -1,22 +1,22 @@
-const fs = require('fs')
-const { promisify } = require('util')
+// @ts-check
+const {
+  constants,
+  promises: { access, readFile, rm, stat },
+} = require('fs')
+const { version } = require('process')
 
 const del = require('del')
-const makeDir = require('make-dir')
-const pathType = require('path-type')
+const { gte, parse } = require('semver')
 
-const statAsync = promisify(fs.stat)
-const readFileAsync = promisify(fs.readFile)
-const writeFileAsync = promisify(fs.writeFile)
-const rmFileAsync = promisify(fs.unlink)
-const copyFileAsync = promisify(fs.copyFile)
-const accessAsync = promisify(fs.access)
-const mkdtempAsync = promisify(fs.mkdtemp)
-const appendFileAsync = promisify(fs.appendFile)
+const NODE_VERSION = parse(version)
 
-const readFileAsyncCatchError = async (filepath) => {
+/**
+ * reads a file async and catches potential errors
+ * @param {string} filePath
+ */
+const readFileAsyncCatchError = async (filePath) => {
   try {
-    return { content: await readFileAsync(filepath) }
+    return { content: await readFile(filePath, 'utf-8') }
   } catch (error) {
     return { error }
   }
@@ -24,42 +24,60 @@ const readFileAsyncCatchError = async (filepath) => {
 
 const fileExistsAsync = async (filePath) => {
   try {
-    await accessAsync(filePath, fs.F_OK)
+    await access(filePath, constants.F_OK)
     return true
   } catch {
     return false
   }
 }
 
-const isDirectoryAsync = (filePath) => pathType.isDirectory(filePath)
-const isFileAsync = (filePath) => pathType.isFile(filePath)
-
-const mkdirRecursiveSync = (path) => makeDir.sync(path)
-const mkdirRecursiveAsync = (path) => makeDir(path)
+/**
+ * Removes a directory recursively and async
+ * @param {string} path
+ * @returns {Promise<void>}
+ */
+const rmdirRecursiveAsync = async (path) => {
+  if (gte(NODE_VERSION, '14.14.0')) {
+    return await rm(path, { force: true, recursive: true })
+  }
+  await del(path, { force: true })
+}
 
-const rmdirRecursiveAsync = (path) => del(path, { force: true })
-const safelyRmdirRecursiveAsync = async (path) => {
+/**
+ * calls stat async with a function and catches potential errors
+ * @param {string} filePath
+ * @param {keyof import('fs').StatsBase<number>} type For example `isDirectory` or `isFile`
+ */
+const isType = async (filePath, type) => {
   try {
-    await rmdirRecursiveAsync(path)
-  } catch (error) {
-    // no-op
+    const stats = await stat(filePath)
+    // @ts-ignore
+    return typeof stats[type] === 'function' ? stats[type]() : stats[type]
+  } catch (error_) {
+    if (error_.code === 'ENOENT') {
+      return false
+    }
+
+    throw error_
   }
 }
 
+/**
+ * Checks if the provided filePath is a file
+ * @param {string} filePath
+ */
+const isFileAsync = (filePath) => isType(filePath, 'isFile')
+
+/**
+ * Checks if the provided filePath is a directory
+ * @param {string} filePath
+ */
+const isDirectoryAsync = (filePath) => isType(filePath, 'isDirectory')
+
 module.exports = {
-  statAsync,
-  readFileAsync,
-  readFileAsyncCatchError,
-  writeFileAsync,
-  rmFileAsync,
-  copyFileAsync,
   fileExistsAsync,
   isDirectoryAsync,
   isFileAsync,
-  mkdirRecursiveSync,
-  mkdirRecursiveAsync,
+  readFileAsyncCatchError,
   rmdirRecursiveAsync,
-  safelyRmdirRecursiveAsync,
-  mkdtempAsync,
-  appendFileAsync,
 }

package/src/lib/functions/background.js

@@ -1,4 +1,4 @@
-const { NETLIFYDEVERR, NETLIFYDEVLOG } = require('../../utils/logo')
+const { NETLIFYDEVERR, NETLIFYDEVLOG } = require('../../utils')
 
 const { formatLambdaError, styleFunctionName } = require('./utils')
 

package/src/lib/functions/form-submissions-handler.js

@@ -1,11 +1,12 @@
+// @ts-check
 const { Readable } = require('stream')
 
 const { parse: parseContentType } = require('content-type')
 const multiparty = require('multiparty')
 const getRawBody = require('raw-body')
 
+const { BACKGROUND } = require('../../utils')
 const { warn } = require('../../utils/command-helpers')
-const { BACKGROUND } = require('../../utils/get-functions')
 const { capitalize } = require('../string')
 
 const createFormSubmissionHandler = function ({ functionsRegistry, siteUrl }) {

package/src/lib/functions/local-proxy.js

@@ -1,8 +1,9 @@
+// @ts-check
 const { stdout } = require('process')
 
 const { getBinaryPath: getFunctionsProxyPath } = require('@netlify/local-functions-proxy')
 
-const execa = require('../../utils/execa')
+const { execa } = require('../../utils')
 
 const runFunctionsProxy = ({ binaryPath, context, directory, event, name, timeout }) => {
   const functionsProxyPath = getFunctionsProxyPath()

package/src/lib/functions/netlify-function.js

@@ -1,8 +1,11 @@
+// @ts-check
 const { error: errorExit } = require('../../utils/command-helpers')
-const { difference } = require('../../utils/difference')
 
 const BACKGROUND_SUFFIX = '-background'
 
+// Returns a new set with all elements of `setA` that don't exist in `setB`.
+const difference = (setA, setB) => new Set([...setA].filter((item) => !setB.has(item)))
+
 class NetlifyFunction {
   constructor({ config, directory, mainFile, name, projectRoot, runtime, timeoutBackground, timeoutSynchronous }) {
     this.config = config

package/src/lib/functions/registry.js

@@ -1,10 +1,8 @@
+// @ts-check
+const { mkdir } = require('fs').promises
 const { env } = require('process')
 
-const chalk = require('chalk')
-
-const { log, warn } = require('../../utils/command-helpers')
-const { NETLIFYDEVERR, NETLIFYDEVLOG } = require('../../utils/logo')
-const { mkdirRecursiveAsync } = require('../fs')
+const { NETLIFYDEVERR, NETLIFYDEVLOG, chalk, log, warn } = require('../../utils')
 const { getLogMessage } = require('../log')
 
 const { NetlifyFunction } = require('./netlify-function')
@@ -45,7 +43,7 @@ class FunctionsRegistry {
   }
 
   static async prepareDirectoryScan(directory) {
-    await mkdirRecursiveAsync(directory)
+    await mkdir(directory, { recursive: true })
 
     // We give runtimes the opportunity to react to a directory scan and run
     // additional logic before the directory is read. So if they implement a

package/src/lib/functions/runtimes/go/index.js

@@ -1,3 +1,4 @@
+// @ts-check
 const { dirname, extname } = require('path')
 const { platform } = require('process')
 
@@ -5,7 +6,7 @@ const tempy = require('tempy')
 
 const isWindows = platform === 'win32'
 
-const execa = require('../../../../utils/execa')
+const { execa } = require('../../../../utils')
 const { runFunctionsProxy } = require('../../local-proxy')
 
 const build = async ({ binaryPath, functionDirectory }) => {

package/src/lib/functions/runtimes/js/builders/netlify-lambda.js

@@ -1,9 +1,11 @@
+// @ts-check
+const { readFile } = require('fs').promises
 const { resolve } = require('path')
 
-const execa = require('execa')
 const minimist = require('minimist')
 
-const { fileExistsAsync, readFileAsync } = require('../../../../fs')
+const { execa } = require('../../../../../utils')
+const { fileExistsAsync } = require('../../../../fs')
 const { memoizedBuild } = require('../../../memoized-build')
 
 const detectNetlifyLambda = async function ({ packageJson } = {}) {
@@ -59,8 +61,8 @@ module.exports = async function handler() {
     return false
   }
 
-  const content = await readFileAsync('package.json')
-  const packageJson = JSON.parse(content, { encoding: 'utf8' })
+  const content = await readFile('package.json', 'utf-8')
+  const packageJson = JSON.parse(content)
   return detectNetlifyLambda({ packageJson })
 }
 module.exports.detectNetlifyLambda = detectNetlifyLambda

package/src/lib/functions/runtimes/js/builders/zisi.js

@@ -1,3 +1,4 @@
+const { writeFile } = require('fs').promises
 const path = require('path')
 
 const { zipFunction } = require('@netlify/zip-it-and-ship-it')
@@ -6,8 +7,7 @@ const makeDir = require('make-dir')
 const readPkgUp = require('read-pkg-up')
 const sourceMapSupport = require('source-map-support')
 
-const { NETLIFYDEVERR } = require('../../../../../utils/logo')
-const { writeFileAsync } = require('../../../../fs')
+const { NETLIFYDEVERR } = require('../../../../../utils')
 const { getPathInProject } = require('../../../../settings')
 const { normalizeFunctionsConfig } = require('../../../config')
 const { memoizedBuild } = require('../../../memoized-build')
@@ -47,7 +47,7 @@ const buildFunction = async ({ cache, config, directory, func, hasTypeModule, pr
   // some projects include a package.json with "type=module", forcing Node to interpret every descending file
   // as ESM. ZISI outputs CJS, so we emit an overriding directive into the output directory.
   if (hasTypeModule) {
-    await writeFileAsync(
+    await writeFile(
       path.join(functionPath, `package.json`),
       JSON.stringify({
         type: 'commonjs',

package/src/lib/functions/runtimes/rust/index.js

@@ -1,3 +1,5 @@
+// @ts-check
+const { readFile } = require('fs').promises
 const { dirname, extname, join, resolve } = require('path')
 const { platform } = require('process')
 
@@ -6,8 +8,7 @@ const toml = require('toml')
 
 const isWindows = platform === 'win32'
 
-const execa = require('../../../../utils/execa')
-const { readFileAsync } = require('../../../fs')
+const { execa } = require('../../../../utils')
 const { getPathInProject } = require('../../../settings')
 const { runFunctionsProxy } = require('../../local-proxy')
 
@@ -36,7 +37,7 @@ const getBuildFunction =
 
 const getCrateName = async (cwd) => {
   const manifestPath = await findUp('Cargo.toml', { cwd, type: 'file' })
-  const manifest = await readFileAsync(manifestPath)
+  const manifest = await readFile(manifestPath, 'utf-8')
   const { package } = toml.parse(manifest)
 
   return package.name

package/src/lib/functions/server.js

@@ -1,8 +1,7 @@
+// @ts-check
 const jwtDecode = require('jwt-decode')
 
-const { error: errorExit, log } = require('../../utils/command-helpers')
-const { getInternalFunctionsDir } = require('../../utils/functions')
-const { NETLIFYDEVERR, NETLIFYDEVLOG } = require('../../utils/logo')
+const { NETLIFYDEVERR, NETLIFYDEVLOG, error: errorExit, getInternalFunctionsDir, log } = require('../../utils')
 
 const { handleBackgroundFunction, handleBackgroundFunctionResult } = require('./background')
 const { createFormSubmissionHandler } = require('./form-submissions-handler')

package/src/lib/functions/synchronous.js

@@ -1,6 +1,7 @@
+// @ts-check
 const { Buffer } = require('buffer')
 
-const { NETLIFYDEVERR } = require('../../utils/logo')
+const { NETLIFYDEVERR } = require('../../utils')
 
 const { detectAwsSdkError } = require('./utils')
 

package/src/lib/functions/utils.js

@@ -1,6 +1,5 @@
-const chalk = require('chalk')
-
-const { warn } = require('../../utils/command-helpers')
+// @ts-check
+const { chalk, warn } = require('../../utils')
 const { getLogMessage } = require('../log')
 
 const DEFAULT_LAMBDA_OPTIONS = {

package/src/lib/functions/watcher.js

@@ -1,3 +1,4 @@
+// @ts-check
 const chokidar = require('chokidar')
 const decache = require('decache')
 const debounce = require('lodash/debounce')

package/src/lib/http-agent.js

@@ -1,10 +1,10 @@
+// @ts-check
+const { readFile } = require('fs').promises
+
 const { HttpsProxyAgent } = require('https-proxy-agent')
 const waitPort = require('wait-port')
 
-const { exit, log } = require('../utils/command-helpers')
-const { NETLIFYDEVERR, NETLIFYDEVWARN } = require('../utils/logo')
-
-const fs = require('./fs')
+const { NETLIFYDEVERR, NETLIFYDEVWARN, exit, log } = require('../utils')
 
 // https://github.com/TooTallNate/node-https-proxy-agent/issues/89
 class HttpsProxyAgentWithCA extends HttpsProxyAgent {
@@ -66,7 +66,7 @@ const tryGetAgent = async ({ certificateFile, httpProxy }) => {
   let certificate
   if (certificateFile) {
     try {
-      certificate = await fs.readFileAsync(certificateFile)
+      certificate = await readFile(certificateFile)
     } catch (error) {
       response = { warning: `Could not read certificate file '${certificateFile}'.`, message: error.message }
     }

package/src/lib/log.js

@@ -1,6 +1,7 @@
-const chalk = require('chalk')
 const dotProp = require('dot-prop')
 
+const { chalk } = require('../utils')
+
 const RED_BACKGROUND = chalk.red('-background')
 const [PRO, BUSINESS, ENTERPRISE] = ['Pro', 'Business', 'Enterprise'].map((plan) => chalk.magenta(plan))
 const BACKGROUND_FUNCTIONS_WARNING = `A serverless function ending in \`${RED_BACKGROUND}\` was detected.

package/src/lib/settings.js

@@ -6,17 +6,32 @@ const envPaths = require('env-paths')
 const OSBasedPaths = envPaths('netlify', { suffix: '' })
 const NETLIFY_HOME = '.netlify'
 
-// Deprecated method to get netlify's home config - ~/.netlify/...
+/**
+ * Deprecated method to get netlify's home config - ~/.netlify/...
+ * @deprecated
+ * @param {string[]} paths
+ * @returns {string}
+ */
 const getLegacyPathInHome = (paths) => {
   const pathInHome = path.join(os.homedir(), NETLIFY_HOME, ...paths)
   return pathInHome
 }
 
+/**
+ * get a global path on the os base path
+ * @param {string[]} paths
+ * @returns {string}
+ */
 const getPathInHome = (paths) => {
   const pathInHome = path.join(OSBasedPaths.config, ...paths)
   return pathInHome
 }
 
+/**
+ * get a path inside the project folder
+ * @param {string[]} paths
+ * @returns {string}
+ */
 const getPathInProject = (paths) => {
   const pathInProject = path.join(NETLIFY_HOME, ...paths)
   return pathInProject

package/src/lib/spinner.js

@@ -1,15 +1,31 @@
+// @ts-check
 const logSymbols = require('log-symbols')
 const ora = require('ora')
 
+/**
+ * Creates a spinner with the following text
+ * @param {object} config
+ * @param {string} config.text
+ * @returns {ora.Ora}
+ */
 const startSpinner = ({ text }) =>
   ora({
     text,
   }).start()
 
+/**
+ * Stops the spinner with the following text
+ * @param {object} config
+ * @param {ora.Ora} config.spinner
+ * @param {object} [config.error]
+ * @param {string} [config.text]
+ * @returns {void}
+ */
 const stopSpinner = ({ error, spinner, text }) => {
   if (!spinner) {
     return
   }
+  // TODO: refactor no package needed `log-symbols` for that
   const symbol = error ? logSymbols.error : logSymbols.success
   spinner.stopAndPersist({
     text,
@@ -17,6 +33,12 @@ const stopSpinner = ({ error, spinner, text }) => {
   })
 }
 
+/**
+ * Clears the spinner
+ * @param {object} config
+ * @param {ora.Ora} config.spinner
+ * @returns {void}
+ */
 const clearSpinner = ({ spinner }) => {
   if (spinner) {
     spinner.stop()

package/src/utils/addons/diffs/index.js

@@ -1,3 +1,4 @@
+// @ts-check
 const concordance = require('concordance')
 
 const { concordanceDiffOptions, concordanceOptions } = require('./options')

package/src/utils/addons/diffs/options.js

@@ -1,5 +1,7 @@
+// @ts-check
 const ansiStyles = require('ansi-styles')
-const chalk = require('chalk')
+
+const { chalk } = require('../../command-helpers')
 
 const forceColor = new chalk.Instance({ level: 1 })
 

package/src/utils/addons/prepare.js

@@ -1,6 +1,5 @@
-const chalk = require('chalk')
-
-const { error, exit, log, warn } = require('../command-helpers')
+// @ts-check
+const { chalk, error, exit, log, warn } = require('../command-helpers')
 
 const ADDON_VALIDATION = {
   EXISTS: 'EXISTS',
@@ -81,17 +80,25 @@ const getAddons = async ({ api, siteId }) => {
   return addons
 }
 
-const prepareAddonCommand = async ({ addonName, context, validation }) => {
-  const { netlify } = context
+/**
+ *
+ * @param {object} config
+ * @param {import('../../commands/base-command').BaseCommand} config.command
+ * @param {string} [config.addonName]
+ * @param {keyof ADDON_VALIDATION} [config.validation]
+ */
+const prepareAddonCommand = async ({ addonName, command, validation }) => {
+  const { netlify } = command
   const { api, site } = netlify
   const siteId = site.id
   if (!siteId) {
     error('No site id found, please run inside a site folder or `netlify link`')
   }
 
-  await context.authenticate()
+  await command.authenticate()
 
   const [manifest, siteData, addons] = await Promise.all([
+    // TODO: check as `getAddonManifest` did not accept a parameter error
     addonName ? getAddonManifest({ api, addonName, error }) : Promise.resolve(),
     getSiteData({ api, siteId }),
     getAddons({ api, siteId }),

package/src/utils/addons/prompts.js

@@ -1,4 +1,5 @@
-const chalk = require('chalk')
+// @ts-check
+const { chalk } = require('../command-helpers')
 
 /* programmatically generate CLI prompts */
 module.exports = function generatePrompts(settings) {

package/src/utils/addons/render.js

@@ -1,5 +1,7 @@
+// @ts-check
 const AsciiTable = require('ascii-table')
-const chalk = require('chalk')
+
+const { chalk } = require('../command-helpers')
 
 const missingValues = function (values, manifest) {
   const display = values