@netlify/build 26.3.0-rc → 26.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/build",
-  "version": "26.3.0-rc",
+  "version": "26.3.3",
   "description": "Netlify build module",
   "type": "module",
   "exports": "./src/core/main.js",
@@ -60,50 +60,50 @@
     "@netlify/config": "^17.0.0",
     "@netlify/functions-utils": "^4.0.0",
     "@netlify/git-utils": "^4.0.0",
-    "@netlify/plugin-edge-handlers": "^3.0.4",
-    "@netlify/plugins-list": "^6.3.0",
+    "@netlify/plugin-edge-handlers": "^3.0.6",
+    "@netlify/plugins-list": "^6.11.0",
     "@netlify/run-utils": "^4.0.0",
-    "@netlify/zip-it-and-ship-it": "5.5.0",
-    "@sindresorhus/slugify": "^1.1.0",
+    "@netlify/zip-it-and-ship-it": "5.8.0",
+    "@sindresorhus/slugify": "^2.0.0",
     "@types/node": "^16.0.0",
-    "ansi-escapes": "^4.3.2",
-    "chalk": "^4.1.2",
-    "clean-stack": "^3.0.1",
-    "execa": "^5.1.1",
-    "figures": "^3.2.0",
-    "filter-obj": "^2.0.1",
+    "ansi-escapes": "^5.0.0",
+    "chalk": "^5.0.0",
+    "clean-stack": "^4.0.0",
+    "execa": "^6.0.0",
+    "figures": "^4.0.0",
+    "filter-obj": "^3.0.0",
     "got": "^10.0.0",
-    "indent-string": "^4.0.0",
-    "is-plain-obj": "^3.0.0",
+    "indent-string": "^5.0.0",
+    "is-plain-obj": "^4.0.0",
     "js-yaml": "^4.0.0",
-    "keep-func-props": "^3.0.0",
-    "locate-path": "^6.0.0",
-    "log-process-errors": "^6.0.0",
-    "map-obj": "^4.0.0",
+    "keep-func-props": "^4.0.0",
+    "locate-path": "^7.0.0",
+    "log-process-errors": "^7.0.0",
+    "map-obj": "^5.0.0",
     "memoize-one": "^6.0.0",
-    "os-name": "^4.0.1",
-    "p-event": "^4.1.0",
+    "os-name": "^5.0.0",
+    "p-event": "^5.0.0",
     "p-every": "^2.0.0",
-    "p-filter": "^2.1.0",
-    "p-locate": "^5.0.0",
-    "p-reduce": "^2.1.0",
-    "path-exists": "^4.0.0",
-    "path-type": "^4.0.0",
-    "pkg-dir": "^5.0.0",
+    "p-filter": "^3.0.0",
+    "p-locate": "^6.0.0",
+    "p-reduce": "^3.0.0",
+    "path-exists": "^5.0.0",
+    "path-type": "^5.0.0",
+    "pkg-dir": "^6.0.0",
     "pretty-ms": "^7.0.0",
-    "ps-list": "^7.0.0",
-    "read-pkg-up": "^7.0.1",
+    "ps-list": "^8.0.0",
+    "read-pkg-up": "^9.0.0",
     "readdirp": "^3.4.0",
     "resolve": "^2.0.0-next.1",
     "rfdc": "^1.3.0",
     "safe-json-stringify": "^1.2.0",
     "semver": "^7.0.0",
     "statsd-client": "0.4.7",
-    "string-width": "^4.2.0",
-    "strip-ansi": "^6.0.0",
-    "supports-color": "^8.0.0",
+    "string-width": "^5.0.0",
+    "strip-ansi": "^7.0.0",
+    "supports-color": "^9.0.0",
     "tmp-promise": "^3.0.2",
-    "ts-node": "^10.4.0",
+    "ts-node": "10.4.0",
     "typescript": "^4.5.4",
     "update-notifier": "^5.0.0",
     "uuid": "^8.0.0",
@@ -116,16 +116,16 @@
     "cpy": "^8.1.0",
     "del": "^6.0.0",
     "fast-safe-stringify": "^2.0.7",
-    "get-bin-path": "^5.1.0",
-    "get-node": "^11.0.2",
-    "get-port": "^5.1.1",
+    "get-bin-path": "^6.0.0",
+    "get-node": "^12.0.0",
+    "get-port": "^6.0.0",
     "get-stream": "^6.0.0",
-    "has-ansi": "^4.0.0",
+    "has-ansi": "^5.0.0",
     "is-ci": "^3.0.0",
     "moize": "^6.0.0",
-    "path-key": "^3.1.1",
-    "process-exists": "^4.0.0",
-    "sinon": "^12.0.0",
+    "path-key": "^4.0.0",
+    "process-exists": "^5.0.0",
+    "sinon": "^13.0.0",
     "yarn": "^1.22.4"
   },
   "engines": {

package/src/core/constants.js

@@ -2,7 +2,7 @@ import { relative, normalize } from 'path'
 
 import { getCacheDir } from '@netlify/cache-utils'
 import mapObj from 'map-obj'
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { ROOT_PACKAGE_JSON } from '../utils/json.js'
 

package/src/core/missing_side_file.js

@@ -1,6 +1,6 @@
 import { relative } from 'path'
 
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { logMissingSideFile } from '../log/messages/core.js'
 

package/src/core/user_node_version.js

@@ -1,6 +1,6 @@
 import { version as currentVersion, execPath } from 'process'
 
-import execa from 'execa'
+import { execa } from 'execa'
 import semver from 'semver'
 
 import { addErrorInfo } from '../error/info.js'

package/src/error/handle.js

@@ -1,6 +1,6 @@
 import { cwd as getCwd } from 'process'
 
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { logBuildError } from '../log/messages/core.js'
 import { logOldCliVersionError } from '../log/old_version.js'

package/src/install/local.js

@@ -1,4 +1,4 @@
-import pkgDir from 'pkg-dir'
+import { packageDirectory } from 'pkg-dir'
 
 import { logInstallLocalPluginsDeps } from '../log/messages/install.js'
 
@@ -57,6 +57,6 @@ const hasPackageDir = function ({ packageDir }) {
 
 // We only install dependencies of local plugins that have their own `package.json`
 const removeMainRoot = async function (localPluginsOptions, buildDir) {
-  const mainPackageDir = await pkgDir(buildDir)
+  const mainPackageDir = await packageDirectory({ cwd: buildDir })
   return localPluginsOptions.filter(({ packageDir }) => packageDir !== mainPackageDir)
 }

package/src/install/main.js

@@ -1,7 +1,7 @@
 import { homedir } from 'os'
 
-import execa from 'execa'
-import pathExists from 'path-exists'
+import { execa } from 'execa'
+import { pathExists } from 'path-exists'
 
 import { addErrorInfo } from '../error/info.js'
 

package/src/install/missing.js

@@ -1,7 +1,7 @@
 import { promises as fs } from 'fs'
 import { normalize } from 'path'
 
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 import { isFile } from 'path-type'
 
 import { logInstallMissingPlugins } from '../log/messages/install.js'

package/src/log/messages/mutations.js

@@ -1,7 +1,7 @@
 import { promises as fs } from 'fs'
 import { inspect } from 'util'
 
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { log, logMessage, logSubHeader } from '../logger.js'
 

package/src/plugins/ipc.js

@@ -1,7 +1,7 @@
 import process from 'process'
 import { promisify } from 'util'
 
-import pEvent from 'p-event'
+import { pEvent } from 'p-event'
 import { v4 as uuidv4 } from 'uuid'
 
 import { jsonToError, errorToJson } from '../error/build.js'

package/src/plugins/manifest/path.js

@@ -1,4 +1,4 @@
-import locatePath from 'locate-path'
+import { locatePath } from 'locate-path'
 
 import { addErrorInfo } from '../../error/info.js'
 

package/src/plugins/spawn.js

@@ -1,6 +1,6 @@
 import { fileURLToPath } from 'url'
 
-import execa from 'execa'
+import { execaNode } from 'execa'
 
 import { addErrorInfo } from '../error/info.js'
 import { logLoadingPlugins, logOutdatedPlugins, logIncompatiblePlugins } from '../log/messages/compatibility.js'
@@ -31,7 +31,7 @@ const tStartPlugins = async function ({ pluginsOptions, buildDir, childEnv, logs
 export const startPlugins = measureDuration(tStartPlugins, 'start_plugins')
 
 const startPlugin = async function ({ pluginDir, nodePath, buildDir, childEnv }) {
-  const childProcess = execa.node(CHILD_MAIN_FILE, {
+  const childProcess = execaNode(CHILD_MAIN_FILE, {
     cwd: buildDir,
     preferLocal: true,
     localDir: pluginDir,

package/src/plugins_core/build_command.js

@@ -1,6 +1,6 @@
 import { platform } from 'process'
 
-import execa from 'execa'
+import { execa } from 'execa'
 
 import { addErrorInfo } from '../error/info.js'
 import { getBuildCommandDescription } from '../log/description.js'

package/src/plugins_core/deploy/buildbot_client.js

@@ -2,7 +2,7 @@ import net from 'net'
 import { normalize, resolve, relative } from 'path'
 import { promisify } from 'util'
 
-import pEvent from 'p-event'
+import { pEvent } from 'p-event'
 
 import { addErrorInfo } from '../../error/info.js'
 import { runsAfterDeploy } from '../../plugins/events.js'

package/src/plugins_core/functions/index.js

@@ -1,69 +1,23 @@
-import { join, resolve } from 'path'
+import { resolve } from 'path'
 
 import { zipFunctions } from '@netlify/zip-it-and-ship-it'
-import mapObject from 'map-obj'
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { log } from '../../log/logger.js'
 import { logBundleResults, logFunctionsNonExistingDir, logFunctionsToBundle } from '../../log/messages/core_steps.js'
 
 import { getZipError } from './error.js'
-import { getZisiFeatureFlags } from './feature_flags.js'
 import { getUserAndInternalFunctions, validateFunctionsSrc } from './utils.js'
+import { getZisiParameters } from './zisi.js'
 
 // Returns `true` if at least one of the functions has been configured to use
 // esbuild.
 const isUsingEsbuild = (functionsConfig = {}) =>
   Object.values(functionsConfig).some((configObject) => configObject.node_bundler === 'esbuild')
 
-// 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, functionConfig = {}, isRunningLocally }) => ({
-  externalNodeModules: functionConfig.external_node_modules,
-  includedFiles: functionConfig.included_files,
-  includedFilesBasePath: buildDir,
-  ignoredNodeModules: functionConfig.ignored_node_modules,
-  schedule: functionConfig.schedule,
-
-  // When the user selects esbuild as the Node bundler, we still want to use
-  // the legacy ZISI bundler as a fallback. Rather than asking the user to
-  // make this decision, we abstract that complexity away by injecting the
-  // fallback behavior ourselves. We do this by transforming the value
-  // `esbuild` into `esbuild_zisi`, which zip-it-and-ship-it understands.
-  nodeBundler: functionConfig.node_bundler === 'esbuild' ? 'esbuild_zisi' : functionConfig.node_bundler,
-
-  // If the build is running in buildbot, we set the Rust target directory to a
-  // path that will get cached in between builds, allowing us to speed up the
-  // build process.
-  rustTargetDirectory: isRunningLocally ? undefined : resolve(buildDir, '.netlify', 'rust-functions-cache', '[name]'),
-
-  // Go functions should be zipped only when building locally. When running in
-  // buildbot, the Go API client will handle the zipping.
-  zipGo: isRunningLocally ? true : undefined,
-})
-
-const getZisiParameters = ({
-  buildDir,
-  featureFlags,
-  functionsConfig,
-  functionsDist,
-  isRunningLocally,
-  repositoryRoot,
-}) => {
-  const isManifestEnabled = isRunningLocally || featureFlags.buildbot_create_functions_manifest === true
-  const manifest = isManifestEnabled ? join(functionsDist, 'manifest.json') : undefined
-  const config = mapObject(functionsConfig, (expression, object) => [
-    expression,
-    normalizeFunctionConfig({ buildDir, featureFlags, functionConfig: object, isRunningLocally }),
-  ])
-  const zisiFeatureFlags = getZisiFeatureFlags(featureFlags)
-
-  return { basePath: buildDir, config, manifest, featureFlags: zisiFeatureFlags, repositoryRoot }
-}
-
 const zipFunctionsAndLogResults = async ({
   buildDir,
+  childEnv,
   featureFlags,
   functionsConfig,
   functionsDist,
@@ -75,6 +29,7 @@ const zipFunctionsAndLogResults = async ({
 }) => {
   const zisiParameters = getZisiParameters({
     buildDir,
+    childEnv,
     featureFlags,
     functionsConfig,
     functionsDist,
@@ -101,6 +56,7 @@ const zipFunctionsAndLogResults = async ({
 // Plugin to package Netlify functions with @netlify/zip-it-and-ship-it
 // eslint-disable-next-line complexity
 const coreStep = async function ({
+  childEnv,
   constants: {
     INTERNAL_FUNCTIONS_SRC: relativeInternalFunctionsSrc,
     IS_LOCAL: isRunningLocally,
@@ -149,6 +105,7 @@ const coreStep = async function ({
 
   const { bundler } = await zipFunctionsAndLogResults({
     buildDir,
+    childEnv,
     featureFlags,
     functionsConfig: netlifyConfig.functions,
     functionsDist,

package/src/plugins_core/functions/zisi.js

@@ -0,0 +1,54 @@
+import { join, resolve } from 'path'
+
+import mapObject from 'map-obj'
+
+import { getZisiFeatureFlags } from './feature_flags.js'
+
+export const getZisiParameters = ({
+  buildDir,
+  childEnv,
+  featureFlags,
+  functionsConfig,
+  functionsDist,
+  isRunningLocally,
+  repositoryRoot,
+}) => {
+  const nodeVersion = childEnv.AWS_LAMBDA_JS_RUNTIME
+  const isManifestEnabled = isRunningLocally || featureFlags.buildbot_create_functions_manifest === true
+  const manifest = isManifestEnabled ? join(functionsDist, 'manifest.json') : undefined
+  const config = mapObject(functionsConfig, (expression, object) => [
+    expression,
+    normalizeFunctionConfig({ buildDir, featureFlags, functionConfig: object, isRunningLocally, nodeVersion }),
+  ])
+  const zisiFeatureFlags = getZisiFeatureFlags(featureFlags)
+
+  return { basePath: buildDir, config, manifest, featureFlags: zisiFeatureFlags, repositoryRoot }
+}
+
+// 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.
+export const normalizeFunctionConfig = ({ buildDir, functionConfig = {}, isRunningLocally, nodeVersion }) => ({
+  externalNodeModules: functionConfig.external_node_modules,
+  includedFiles: functionConfig.included_files,
+  includedFilesBasePath: buildDir,
+  ignoredNodeModules: functionConfig.ignored_node_modules,
+  nodeVersion,
+  schedule: functionConfig.schedule,
+
+  // When the user selects esbuild as the Node bundler, we still want to use
+  // the legacy ZISI bundler as a fallback. Rather than asking the user to
+  // make this decision, we abstract that complexity away by injecting the
+  // fallback behavior ourselves. We do this by transforming the value
+  // `esbuild` into `esbuild_zisi`, which zip-it-and-ship-it understands.
+  nodeBundler: functionConfig.node_bundler === 'esbuild' ? 'esbuild_zisi' : functionConfig.node_bundler,
+
+  // If the build is running in buildbot, we set the Rust target directory to a
+  // path that will get cached in between builds, allowing us to speed up the
+  // build process.
+  rustTargetDirectory: isRunningLocally ? undefined : resolve(buildDir, '.netlify', 'rust-functions-cache', '[name]'),
+
+  // Go functions should be zipped only when building locally. When running in
+  // buildbot, the Go API client will handle the zipping.
+  zipGo: isRunningLocally ? true : undefined,
+})

package/src/plugins_core/functions_install/index.js

@@ -1,4 +1,4 @@
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { installFunctionDependencies } from '../../install/functions.js'
 

package/src/steps/update_config.js

@@ -1,7 +1,7 @@
 import { isDeepStrictEqual } from 'util'
 
 import pFilter from 'p-filter'
-import pathExists from 'path-exists'
+import { pathExists } from 'path-exists'
 
 import { resolveUpdatedConfig } from '../core/config.js'
 import { addErrorInfo } from '../error/info.js'

package/src/utils/package.js

@@ -1,6 +1,6 @@
 import { dirname } from 'path'
 
-import readPkgUp from 'read-pkg-up'
+import { readPackageUp } from 'read-pkg-up'
 
 // Retrieve `package.json` from a specific directory
 export const getPackageJson = async function (cwd, { normalize } = {}) {
@@ -16,7 +16,7 @@ export const getPackageJson = async function (cwd, { normalize } = {}) {
 
 const getPackageObj = async function ({ cwd, normalize = true }) {
   try {
-    return await readPkgUp({ cwd, normalize })
+    return await readPackageUp({ cwd, normalize })
     // If the `package.json` is invalid and `normalize` is `true`, an error is
     // thrown. We return `undefined` then.
   } catch {}