@netlify/build 26.2.5 → 26.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/build",
-  "version": "26.2.5",
+  "version": "26.3.0",
   "description": "Netlify build module",
   "type": "module",
   "exports": "./src/core/main.js",
@@ -61,37 +61,37 @@
     "@netlify/functions-utils": "^4.0.0",
     "@netlify/git-utils": "^4.0.0",
     "@netlify/plugin-edge-handlers": "^3.0.6",
-    "@netlify/plugins-list": "^6.9.0",
+    "@netlify/plugins-list": "^6.10.1",
     "@netlify/run-utils": "^4.0.0",
-    "@netlify/zip-it-and-ship-it": "5.7.2",
-    "@sindresorhus/slugify": "^1.1.0",
+    "@netlify/zip-it-and-ship-it": "5.7.4",
+    "@sindresorhus/slugify": "^2.0.0",
     "@types/node": "^16.0.0",
-    "ansi-escapes": "^4.3.2",
+    "ansi-escapes": "^5.0.0",
     "chalk": "^5.0.0",
-    "clean-stack": "^3.0.1",
+    "clean-stack": "^4.0.0",
     "execa": "^6.0.0",
-    "figures": "^3.2.0",
+    "figures": "^4.0.0",
     "filter-obj": "^3.0.0",
     "got": "^10.0.0",
-    "indent-string": "^4.0.0",
+    "indent-string": "^5.0.0",
     "is-plain-obj": "^4.0.0",
     "js-yaml": "^4.0.0",
-    "keep-func-props": "^3.0.0",
+    "keep-func-props": "^4.0.0",
     "locate-path": "^7.0.0",
-    "log-process-errors": "^6.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": "^3.0.0",
     "p-locate": "^6.0.0",
-    "p-reduce": "^2.1.0",
+    "p-reduce": "^3.0.0",
     "path-exists": "^5.0.0",
-    "path-type": "^4.0.0",
-    "pkg-dir": "^5.0.0",
+    "path-type": "^5.0.0",
+    "pkg-dir": "^6.0.0",
     "pretty-ms": "^7.0.0",
-    "ps-list": "^7.0.0",
+    "ps-list": "^8.0.0",
     "read-pkg-up": "^9.0.0",
     "readdirp": "^3.4.0",
     "resolve": "^2.0.0-next.1",
@@ -99,9 +99,9 @@
     "safe-json-stringify": "^1.2.0",
     "semver": "^7.0.0",
     "statsd-client": "0.4.7",
-    "string-width": "^4.2.0",
+    "string-width": "^5.0.0",
     "strip-ansi": "^7.0.0",
-    "supports-color": "^8.0.0",
+    "supports-color": "^9.0.0",
     "tmp-promise": "^3.0.2",
     "ts-node": "10.4.0",
     "typescript": "^4.5.4",
@@ -117,14 +117,14 @@
     "del": "^6.0.0",
     "fast-safe-stringify": "^2.0.7",
     "get-bin-path": "^6.0.0",
-    "get-node": "^11.0.2",
-    "get-port": "^5.1.1",
+    "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",
+    "path-key": "^4.0.0",
+    "process-exists": "^5.0.0",
     "sinon": "^13.0.0",
     "yarn": "^1.22.4"
   },

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/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_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 { 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,
+})