@netlify/build 26.0.0 → 26.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/build",
-  "version": "26.0.0",
+  "version": "26.1.1",
   "description": "Netlify build module",
   "type": "module",
   "exports": "./src/core/main.js",
@@ -60,10 +60,10 @@
     "@netlify/config": "^17.0.0",
     "@netlify/functions-utils": "^4.0.0",
     "@netlify/git-utils": "^4.0.0",
-    "@netlify/plugin-edge-handlers": "^3.0.0",
-    "@netlify/plugins-list": "^6.2.0",
+    "@netlify/plugin-edge-handlers": "^3.0.2",
+    "@netlify/plugins-list": "^6.2.1",
     "@netlify/run-utils": "^4.0.0",
-    "@netlify/zip-it-and-ship-it": "^5.3.0",
+    "@netlify/zip-it-and-ship-it": "^5.4.0",
     "@sindresorhus/slugify": "^1.1.0",
     "ansi-escapes": "^4.3.2",
     "chalk": "^4.1.2",
@@ -78,7 +78,6 @@
     "keep-func-props": "^3.0.0",
     "locate-path": "^6.0.0",
     "log-process-errors": "^6.0.0",
-    "make-dir": "^3.0.2",
     "map-obj": "^4.0.0",
     "memoize-one": "^6.0.0",
     "os-name": "^4.0.1",

package/src/core/feature_flags.js

@@ -18,6 +18,5 @@ export const DEFAULT_FEATURE_FLAGS = {
   buildbot_es_modules_esbuild: false,
   buildbot_zisi_trace_nft: false,
   buildbot_zisi_esbuild_parser: false,
-  buildbot_scheduled_functions: false,
   zisi_parse_isc: false,
 }

package/src/error/handle.js

@@ -51,7 +51,7 @@ const isCancelCrash = async function (error) {
     const cwd = getCwd()
     return !(await pathExists(cwd))
     // `process.cwd()` fails when the current directory does not exist
-  } catch (error_) {
+  } catch {
     return true
   }
 }

package/src/error/monitor/report.js

@@ -102,7 +102,7 @@ const reportError = async function ({ errorMonitor, error, logs, testOpts, event
   try {
     await promisify(errorMonitor.notify)(error, (event) => onError(event, eventProps))
     // Failsafe
-  } catch (error_) {
+  } catch {
     log(logs, `Error monitor could not notify\n${error.stack}`)
   }
 }

package/src/error/parse/clean_stack.js

@@ -51,7 +51,7 @@ const cleanStackLine = function (lines, line) {
 const getCwd = function () {
   try {
     return cwd()
-  } catch (error) {
+  } catch {
     return ''
   }
 }

package/src/install/missing.js

@@ -1,8 +1,6 @@
-import { writeFile, unlink } from 'fs'
+import { promises as fs } from 'fs'
 import { normalize } from 'path'
-import { promisify } from 'util'
 
-import makeDir from 'make-dir'
 import pathExists from 'path-exists'
 import { isFile } from 'path-type'
 
@@ -10,9 +8,6 @@ import { logInstallMissingPlugins } from '../log/messages/install.js'
 
 import { addExactDependencies } from './main.js'
 
-const pWriteFile = promisify(writeFile)
-const pUnlink = promisify(unlink)
-
 // Automatically install plugins if not already installed.
 // Since this is done under the hood, we always use `npm` with specific `npm`
 // options. We do not allow configure the package manager nor its options.
@@ -45,10 +40,10 @@ const ensureDir = async function (logs, autoPluginsDir) {
   // If `.netlify` exists but is not a directory, we remove it first
   const autoPluginsParent = normalize(`${autoPluginsDir}/..`)
   if (await isFile(autoPluginsParent)) {
-    await pUnlink(autoPluginsParent)
+    await fs.unlink(autoPluginsParent)
   }
 
-  await makeDir(autoPluginsDir)
+  await fs.mkdir(autoPluginsDir, { recursive: true })
 }
 
 // Create a dummy `package.json` so we can run `npm install` and get a lock file
@@ -59,7 +54,7 @@ const createPackageJson = async function (autoPluginsDir) {
   }
 
   const packageJsonContent = JSON.stringify(AUTO_PLUGINS_PACKAGE_JSON, null, 2)
-  await pWriteFile(packageJsonPath, packageJsonContent)
+  await fs.writeFile(packageJsonPath, packageJsonContent)
 }
 
 const AUTO_PLUGINS_PACKAGE_JSON = {

package/src/log/logger.js

@@ -45,7 +45,7 @@ const serializeIndentedArray = function (array) {
 }
 
 const serializeIndentedItem = function (item) {
-  return indentString(item, INDENT_SIZE + 1).trimLeft()
+  return indentString(item, INDENT_SIZE + 1).trimStart()
 }
 
 export const logError = function (logs, string, opts) {

package/src/log/messages/core_steps.js

@@ -21,7 +21,7 @@ export const logBundleResults = ({ logs, results = [] }) => {
     ({ bundler, bundlerWarnings }) => bundler === 'esbuild' && bundlerWarnings && bundlerWarnings.length !== 0,
   )
   const modulesWithDynamicImports = [
-    ...new Set(results.map((result) => result.nodeModulesWithDynamicImports || []).flat()),
+    ...new Set(results.flatMap((result) => result.nodeModulesWithDynamicImports || [])),
   ]
 
   if (resultsWithErrors.length !== 0) {

package/src/log/messages/mutations.js

@@ -1,12 +1,10 @@
-import { readFile } from 'fs'
-import { inspect, promisify } from 'util'
+import { promises as fs } from 'fs'
+import { inspect } from 'util'
 
 import pathExists from 'path-exists'
 
 import { log, logMessage, logSubHeader } from '../logger.js'
 
-const pReadFile = promisify(readFile)
-
 export const logConfigMutations = function (logs, newConfigMutations, debug) {
   const configMutationsToLog = debug ? newConfigMutations : newConfigMutations.filter(shouldLogConfigMutation)
   configMutationsToLog.forEach(({ keysString, value }) => {
@@ -47,7 +45,7 @@ export const logConfigOnUpload = async function ({ logs, configPath, debug }) {
     return
   }
 
-  const configContents = await pReadFile(configPath, 'utf8')
+  const configContents = await fs.readFile(configPath, 'utf8')
   logMessage(logs, configContents.trim())
 }
 
@@ -63,7 +61,7 @@ export const logHeadersOnUpload = async function ({ logs, headersPath, debug })
     return
   }
 
-  const headersContents = await pReadFile(headersPath, 'utf8')
+  const headersContents = await fs.readFile(headersPath, 'utf8')
   logMessage(logs, headersContents.trim())
 }
 
@@ -79,6 +77,6 @@ export const logRedirectsOnUpload = async function ({ logs, redirectsPath, debug
     return
   }
 
-  const redirectsContents = await pReadFile(redirectsPath, 'utf8')
+  const redirectsContents = await fs.readFile(redirectsPath, 'utf8')
   logMessage(logs, `${redirectsContents.trim()}\n`)
 }

package/src/log/serialize.js

@@ -1,7 +1,7 @@
 import { dump } from 'js-yaml'
 
 export const serializeObject = function (object) {
-  return dump(object, { noRefs: true, sortKeys: true, lineWidth: Number.POSITIVE_INFINITY }).trimRight()
+  return dump(object, { noRefs: true, sortKeys: true, lineWidth: Number.POSITIVE_INFINITY }).trimEnd()
 }
 
 export const serializeArray = function (array) {

package/src/log/stream.js

@@ -76,7 +76,7 @@ const pushOutputToLogs = function (childProcess, logs) {
 }
 
 const logsListener = function (logs, chunk) {
-  logs.push(chunk.toString().trimRight())
+  logs.push(chunk.toString().trimEnd())
 }
 
 const unpushOutputToLogs = function (childProcess, logs, { stdoutListener, stderrListener }) {

package/src/plugins/compatibility.js

@@ -109,7 +109,7 @@ const siteDependencyTest = async function ({ dependencyName, allowedVersion, sit
     const packageJsonPath = await resolvePath(`${dependencyName}/package.json`, buildDir)
     const { version } = await importJsonFile(packageJsonPath)
     return semver.satisfies(version, allowedVersion)
-  } catch (error) {
+  } catch {
     return false
   }
 }

package/src/plugins/manifest/load.js

@@ -1,5 +1,4 @@
-import { readFile } from 'fs'
-import { promisify } from 'util'
+import { promises as fs } from 'fs'
 
 import { load as loadYaml, JSON_SCHEMA } from 'js-yaml'
 
@@ -7,8 +6,6 @@ import { addErrorInfo } from '../../error/info.js'
 
 import { validateManifest } from './validate.js'
 
-const pReadFile = promisify(readFile)
-
 // Load "manifest.yml" using its file path
 export const loadManifest = async function ({ manifestPath, packageName, pluginPackageJson, loadedFrom, origin }) {
   try {
@@ -28,7 +25,7 @@ export const loadManifest = async function ({ manifestPath, packageName, pluginP
 
 const loadRawManifest = async function (manifestPath) {
   try {
-    return await pReadFile(manifestPath, 'utf8')
+    return await fs.readFile(manifestPath, 'utf8')
   } catch (error) {
     error.message = `Could not load plugin's "manifest.yml"\n${error.message}`
     throw error

package/src/plugins_core/functions/index.js

@@ -19,7 +19,7 @@ const isUsingEsbuild = (functionsConfig = {}) =>
 // The function configuration keys returned by @netlify/config are not an exact
 // match to the properties that @netlify/zip-it-and-ship-it expects. We do that
 // translation here.
-const normalizeFunctionConfig = ({ buildDir, featureFlags, functionConfig = {}, isRunningLocally }) => ({
+const normalizeFunctionConfig = ({ buildDir, functionConfig = {}, isRunningLocally }) => ({
   externalNodeModules: functionConfig.external_node_modules,
   includedFiles: functionConfig.included_files,
   includedFilesBasePath: buildDir,
@@ -37,7 +37,7 @@ const normalizeFunctionConfig = ({ buildDir, featureFlags, functionConfig = {},
   // build process.
   rustTargetDirectory: isRunningLocally ? undefined : resolve(buildDir, '.netlify', 'rust-functions-cache', '[name]'),
 
-  schedule: featureFlags.buildbot_scheduled_functions ? functionConfig.schedule : undefined,
+  schedule: functionConfig.schedule,
 })
 
 const getZisiParameters = ({

package/src/utils/json.js

@@ -6,7 +6,7 @@ const ROOT_PACKAGE_JSON_PATH = fileURLToPath(new URL('../../package.json', impor
 // TODO: Replace with dynamic `import()` once it is supported without
 // experimental flags
 export const importJsonFile = async function (filePath) {
-  const fileContents = await fs.readFile(filePath, 'utf8')
+  const fileContents = await fs.readFile(filePath)
   return JSON.parse(fileContents)
 }
 

package/src/utils/package.js

@@ -19,5 +19,5 @@ const getPackageObj = async function ({ cwd, normalize = true }) {
     return await readPkgUp({ cwd, normalize })
     // If the `package.json` is invalid and `normalize` is `true`, an error is
     // thrown. We return `undefined` then.
-  } catch (error) {}
+  } catch {}
 }

package/src/utils/resolve.js

@@ -23,7 +23,7 @@ export const resolvePath = async function (path, basedir) {
     // Fallback.
     // `resolve` sometimes gives unhelpful error messages.
     // https://github.com/browserify/resolve/issues/223
-  } catch (error) {
+  } catch {
     return require.resolve(path, { paths: [basedir] })
   }
 }