@netlify/build 18.13.12 → 18.15.1

package/package.json

@@ -1,14 +1,16 @@
 {
   "name": "@netlify/build",
-  "version": "18.13.12",
+  "version": "18.15.1",
   "description": "Netlify build module",
   "main": "src/core/main.js",
+  "types": "types/index.d.ts",
   "bin": {
     "netlify-build": "./src/core/bin.js"
   },
   "files": [
     "src/**/*.js",
-    "src/**/*.yml"
+    "src/**/*.yml",
+    "types/**/*.d.ts"
   ],
   "author": "Netlify Inc.",
   "contributors": [

package/src/core/config.js

@@ -123,7 +123,7 @@ const logConfigInfo = function ({ logs, configPath, buildDir, netlifyConfig, con
 // normalized.
 // We use `debug: false` to avoid any debug logs. Otherwise every configuration
 // change would create debug logs which would be too verbose.
-// Errors are propagated and assigned to the specific plugin or core command
+// Errors are propagated and assigned to the specific plugin or core step
 // which changed the configuration.
 const resolveUpdatedConfig = async function (configOpts, configMutations) {
   try {

package/src/core/constants.js

@@ -52,7 +52,7 @@ const INTERNAL_FUNCTIONS_SRC = '.netlify/functions-internal'
 
 // Retrieve constants which might change during the build if a plugin modifies
 // `netlifyConfig` or creates some default directories.
-// Unlike readonly constants, this is called again before each command.
+// Unlike readonly constants, this is called again before each build step.
 const addMutableConstants = async function ({
   constants,
   buildDir,

package/src/core/dry.js

@@ -2,27 +2,27 @@
 
 const pFilter = require('p-filter')
 
-const { logDryRunStart, logDryRunCommand, logDryRunEnd } = require('../log/messages/dry')
+const { logDryRunStart, logDryRunStep, logDryRunEnd } = require('../log/messages/dry')
 const { runsOnlyOnBuildFailure } = require('../plugins/events')
 
 // If the `dry` flag is specified, do a dry run
-const doDryRun = async function ({ buildDir, commands, netlifyConfig, constants, buildbotServerSocket, logs }) {
-  const successCommands = await pFilter(commands, ({ event, condition }) =>
-    shouldIncludeCommand({ buildDir, event, condition, netlifyConfig, constants, buildbotServerSocket }),
+const doDryRun = async function ({ buildDir, steps, netlifyConfig, constants, buildbotServerSocket, logs }) {
+  const successSteps = await pFilter(steps, ({ event, condition }) =>
+    shouldIncludeStep({ buildDir, event, condition, netlifyConfig, constants, buildbotServerSocket }),
   )
-  const eventWidth = Math.max(...successCommands.map(getEventLength))
-  const commandsCount = successCommands.length
+  const eventWidth = Math.max(...successSteps.map(getEventLength))
+  const stepsCount = successSteps.length
 
-  logDryRunStart({ logs, eventWidth, commandsCount })
+  logDryRunStart({ logs, eventWidth, stepsCount })
 
-  successCommands.forEach((command, index) => {
-    logDryRunCommand({ logs, command, index, netlifyConfig, eventWidth, commandsCount })
+  successSteps.forEach((step, index) => {
+    logDryRunStep({ logs, step, index, netlifyConfig, eventWidth, stepsCount })
   })
 
   logDryRunEnd(logs)
 }
 
-const shouldIncludeCommand = async function ({
+const shouldIncludeStep = async function ({
   buildDir,
   event,
   condition,

package/src/core/feature_flags.js

@@ -15,6 +15,7 @@ const getFeatureFlag = function (name) {
 
 // Default values for feature flags
 const DEFAULT_FEATURE_FLAGS = {
+  buildbot_build_go_functions: false,
   buildbot_es_modules_esbuild: false,
   buildbot_zisi_esbuild_parser: false,
   netlify_config_toml_backslash: false,

package/src/core/flags.js

@@ -94,12 +94,12 @@ Default: true`,
   dry: {
     alias: 'dry-run',
     boolean: true,
-    describe: `Run in dry mode, i.e. printing commands without executing them.
+    describe: `Run in dry mode, i.e. printing steps without executing them.
 Default: false`,
   },
   nodePath: {
     string: true,
-    describe: `Path to the Node.js binary to use in user commands and build plugins.
+    describe: `Path to the Node.js binary to use in the build command and plugins.
 Default: Current Node.js binary`,
   },
   functionsDistDir: {

package/src/core/main.js

@@ -3,8 +3,6 @@
 /* eslint-disable max-lines, import/max-dependencies */
 require('../utils/polyfills')
 
-const { getCommands } = require('../commands/get')
-const { runCommands } = require('../commands/run_commands')
 const { handleBuildError } = require('../error/handle')
 const { getErrorInfo } = require('../error/info')
 const { startErrorMonitor } = require('../error/monitor/start')
@@ -16,6 +14,8 @@ const { pinPlugins } = require('../plugins/pinned_version')
 const { startPlugins, stopPlugins } = require('../plugins/spawn')
 const { addCorePlugins } = require('../plugins_core/add')
 const { reportStatuses } = require('../status/report')
+const { getSteps } = require('../steps/get')
+const { runSteps } = require('../steps/run_steps')
 const { trackBuildComplete } = require('../telemetry/main')
 const { initTimers, measureDuration } = require('../time/main')
 const { reportTimers } = require('../time/report')
@@ -42,8 +42,8 @@ const { getSeverity } = require('./severity')
  * @param  {string} [flags.deployId] - Netlify Deploy ID
  * @param  {string} [flags.context] - Build context
  * @param  {string} [flags.branch] - Repository branch
- * @param  {boolean} [flags.dry=false] - Run in dry mode, i.e. printing commands without executing them
- * @param  {string} [flags.nodePath] - Path to the Node.js binary to use in user commands and build plugins
+ * @param  {boolean} [flags.dry=false] - Run in dry mode, i.e. printing steps without executing them
+ * @param  {string} [flags.nodePath] - Path to the Node.js binary to use in the build command and plugins
  * @param  {boolean} [flags.buffer=false] - Buffer output instead of printing it
  *
  * @returns {object} buildResult
@@ -75,7 +75,7 @@ const build = async function (flags = {}) {
       netlifyConfig: netlifyConfigA,
       siteInfo,
       userNodeVersion,
-      commandsCount,
+      stepsCount,
       timers,
       durationNs,
       configMutations,
@@ -104,7 +104,7 @@ const build = async function (flags = {}) {
       buildId,
       deployId,
       status,
-      commandsCount,
+      stepsCount,
       pluginsOptions,
       durationNs,
       siteInfo,
@@ -248,7 +248,7 @@ const tExecBuild = async function ({
   const {
     pluginsOptions: pluginsOptionsA,
     netlifyConfig: netlifyConfigA,
-    commandsCount,
+    stepsCount,
     timers: timersB,
     configMutations,
   } = await runAndReportBuild({
@@ -288,7 +288,7 @@ const tExecBuild = async function ({
     netlifyConfig: netlifyConfigA,
     siteInfo,
     userNodeVersion,
-    commandsCount,
+    stepsCount,
     timers: timersB,
     configMutations,
   }
@@ -331,7 +331,7 @@ const runAndReportBuild = async function ({
 }) {
   try {
     const {
-      commandsCount,
+      stepsCount,
       netlifyConfig: netlifyConfigA,
       statuses,
       pluginsOptions: pluginsOptionsA,
@@ -404,7 +404,7 @@ const runAndReportBuild = async function ({
     return {
       pluginsOptions: pluginsOptionsA,
       netlifyConfig: netlifyConfigA,
-      commandsCount,
+      stepsCount,
       timers: timersA,
       configMutations,
     }
@@ -492,7 +492,7 @@ const initAndRunBuild = async function ({
 
   try {
     const {
-      commandsCount,
+      stepsCount,
       netlifyConfig: netlifyConfigA,
       statuses,
       failedPlugins,
@@ -535,7 +535,7 @@ const initAndRunBuild = async function ({
     ])
 
     return {
-      commandsCount,
+      stepsCount,
       netlifyConfig: netlifyConfigA,
       statuses,
       pluginsOptions: pluginsOptionsA,
@@ -580,7 +580,7 @@ const runBuild = async function ({
   testOpts,
   featureFlags,
 }) {
-  const { pluginsCommands, timers: timersA } = await loadPlugins({
+  const { pluginsSteps, timers: timersA } = await loadPlugins({
     pluginsOptions,
     childProcesses,
     packageJson,
@@ -588,22 +588,22 @@ const runBuild = async function ({
     debug,
   })
 
-  const { commands, events } = getCommands(pluginsCommands)
+  const { steps, events } = getSteps(pluginsSteps)
 
   if (dry) {
-    await doDryRun({ buildDir, commands, netlifyConfig, constants, buildbotServerSocket, logs })
+    await doDryRun({ buildDir, steps, netlifyConfig, constants, buildbotServerSocket, logs })
     return { netlifyConfig }
   }
 
   const {
-    commandsCount,
+    stepsCount,
     netlifyConfig: netlifyConfigA,
     statuses,
     failedPlugins,
     timers: timersB,
     configMutations,
-  } = await runCommands({
-    commands,
+  } = await runSteps({
+    steps,
     buildbotServerSocket,
     events,
     configPath,
@@ -631,7 +631,7 @@ const runBuild = async function ({
     featureFlags,
   })
 
-  return { commandsCount, netlifyConfig: netlifyConfigA, statuses, failedPlugins, timers: timersB, configMutations }
+  return { stepsCount, netlifyConfig: netlifyConfigA, statuses, failedPlugins, timers: timersB, configMutations }
 }
 
 // Logs and reports that a build successfully ended
@@ -651,7 +651,7 @@ const telemetryReport = async function ({
   deployId,
   buildId,
   status,
-  commandsCount,
+  stepsCount,
   pluginsOptions,
   durationNs,
   siteInfo,
@@ -666,7 +666,7 @@ const telemetryReport = async function ({
       deployId,
       buildId,
       status,
-      commandsCount,
+      stepsCount,
       pluginsOptions,
       durationNs,
       siteInfo,

package/src/error/parse/location.js

@@ -29,8 +29,8 @@ const getFunctionsBundlingLocation = function ({ functionName }) {
   return `While bundling Function "${functionName}"`
 }
 
-const getCoreCommandLocation = function ({ coreCommandName }) {
-  return `During ${coreCommandName}`
+const getCoreStepLocation = function ({ coreStepName }) {
+  return `During ${coreStepName}`
 }
 
 const getBuildFailLocation = function ({ event, packageName, loadedFrom, origin }) {
@@ -54,7 +54,7 @@ const getApiLocation = function ({ endpoint, parameters }) {
 const LOCATIONS = {
   buildCommand: getBuildCommandLocation,
   functionsBundling: getFunctionsBundlingLocation,
-  coreCommand: getCoreCommandLocation,
+  coreStep: getCoreStepLocation,
   buildFail: getBuildFailLocation,
   api: getApiLocation,
 }

package/src/error/type.js

@@ -141,13 +141,13 @@ const TYPES = {
     severity: 'error',
   },
 
-  // Core command internal error
-  coreCommand: {
-    title: ({ location: { coreCommandName } }) => `Internal error during "${coreCommandName}"`,
+  // Core step internal error
+  coreStep: {
+    title: ({ location: { coreStepName } }) => `Internal error during "${coreStepName}"`,
     stackType: 'stack',
     showErrorProps: true,
     rawStack: true,
-    locationType: 'coreCommand',
+    locationType: 'coreStep',
     severity: 'error',
   },
 

package/src/log/messages/dry.js

@@ -5,8 +5,8 @@ const { arrowDown } = require('figures')
 const { logMessage, logSubHeader } = require('../logger')
 const { THEME } = require('../theme')
 
-const logDryRunStart = function ({ logs, eventWidth, commandsCount }) {
-  const columnWidth = getDryColumnWidth(eventWidth, commandsCount)
+const logDryRunStart = function ({ logs, eventWidth, stepsCount }) {
+  const columnWidth = getDryColumnWidth(eventWidth, stepsCount)
   const line = '─'.repeat(columnWidth)
   const secondLine = '─'.repeat(columnWidth)
 
@@ -23,19 +23,19 @@ ${THEME.header(`┌─${line}─┬─${secondLine}─┐
   )
 }
 
-const logDryRunCommand = function ({
+const logDryRunStep = function ({
   logs,
-  command: { event, packageName, coreCommandDescription },
+  step: { event, packageName, coreStepDescription },
   index,
   netlifyConfig,
   eventWidth,
-  commandsCount,
+  stepsCount,
 }) {
-  const columnWidth = getDryColumnWidth(eventWidth, commandsCount)
-  const fullName = getFullName(coreCommandDescription, netlifyConfig, packageName)
+  const columnWidth = getDryColumnWidth(eventWidth, stepsCount)
+  const fullName = getFullName(coreStepDescription, netlifyConfig, packageName)
   const line = '─'.repeat(columnWidth)
   const countText = `${index + 1}. `
-  const downArrow = commandsCount === index + 1 ? '  ' : ` ${arrowDown}`
+  const downArrow = stepsCount === index + 1 ? '  ' : ` ${arrowDown}`
   const eventWidthA = columnWidth - countText.length - downArrow.length
 
   logMessage(
@@ -46,14 +46,14 @@ ${THEME.header(`└─${line}─┘ `)}`,
   )
 }
 
-const getFullName = function (coreCommandDescription, netlifyConfig, packageName) {
-  return coreCommandDescription === undefined
+const getFullName = function (coreStepDescription, netlifyConfig, packageName) {
+  return coreStepDescription === undefined
     ? `Plugin ${THEME.highlightWords(packageName)}`
-    : coreCommandDescription({ netlifyConfig })
+    : coreStepDescription({ netlifyConfig })
 }
 
-const getDryColumnWidth = function (eventWidth, commandsCount) {
-  const symbolsWidth = `${commandsCount}`.length + COLUMN_EXTRA_WIDTH
+const getDryColumnWidth = function (eventWidth, stepsCount) {
+  const symbolsWidth = `${stepsCount}`.length + COLUMN_EXTRA_WIDTH
   return Math.max(eventWidth + symbolsWidth, DRY_HEADER_NAMES[1].length)
 }
 
@@ -66,6 +66,6 @@ const logDryRunEnd = function (logs) {
 
 module.exports = {
   logDryRunStart,
-  logDryRunCommand,
+  logDryRunStep,
   logDryRunEnd,
 }

package/src/log/messages/steps.js

@@ -0,0 +1,32 @@
+'use strict'
+
+const { getLogHeaderFunc } = require('../header_func')
+const { log, logMessage } = require('../logger')
+const { THEME } = require('../theme')
+
+const logStepStart = function ({ logs, event, packageName, coreStepDescription, index, error, netlifyConfig }) {
+  const description = getDescription({ coreStepDescription, netlifyConfig, packageName, event })
+  const logHeaderFunc = getLogHeaderFunc(error)
+  logHeaderFunc(logs, `${index + 1}. ${description}`)
+  logMessage(logs, '')
+}
+
+const getDescription = function ({ coreStepDescription, netlifyConfig, packageName, event }) {
+  return coreStepDescription === undefined
+    ? `${event} step from ${packageName}`
+    : coreStepDescription({ netlifyConfig })
+}
+
+const logBuildCommandStart = function (logs, buildCommand) {
+  log(logs, THEME.highlightWords(`$ ${buildCommand}`))
+}
+
+const logStepSuccess = function (logs) {
+  logMessage(logs, '')
+}
+
+module.exports = {
+  logStepStart,
+  logBuildCommandStart,
+  logStepSuccess,
+}

package/src/log/stream.js

@@ -36,7 +36,7 @@ const pushBuildCommandOutput = function (output, logsArray) {
   logsArray.push(output)
 }
 
-// Start plugin command output
+// Start plugin step output
 const pipePluginOutput = function (childProcess, logs) {
   if (logs === undefined) {
     return streamOutput(childProcess)
@@ -45,7 +45,7 @@ const pipePluginOutput = function (childProcess, logs) {
   return pushOutputToLogs(childProcess, logs)
 }
 
-// Stop streaming/buffering plugin command output
+// Stop streaming/buffering plugin step output
 const unpipePluginOutput = async function (childProcess, logs, listeners) {
   // Let `childProcess` `stdout` and `stderr` flush before stopping redirecting
   await pSetTimeout(0)

package/src/plugins/child/load.js

@@ -7,21 +7,21 @@ const { validatePlugin } = require('./validate')
 // This also requires the plugin file and fire its top-level function.
 // This also validates the plugin.
 // Do it when parent requests it using the `load` event.
-// Also figure out the list of plugin commands. This is also passed to the parent.
+// Also figure out the list of plugin steps. This is also passed to the parent.
 const load = function ({ pluginPath, inputs, packageJson }) {
   const logic = getLogic({ pluginPath, inputs })
 
   validatePlugin(logic)
 
-  const pluginCommands = getPluginCommands(logic)
+  const pluginSteps = getPluginSteps(logic)
 
   // Context passed to every event handler
-  const context = { pluginCommands, inputs, packageJson }
+  const context = { pluginSteps, inputs, packageJson }
 
-  return { pluginCommands, context }
+  return { pluginSteps, context }
 }
 
-const getPluginCommands = function (logic) {
+const getPluginSteps = function (logic) {
   return Object.entries(logic)
     .filter(isEventHandler)
     .map(([event, method]) => ({ event, method }))

package/src/plugins/child/run.js

@@ -8,9 +8,9 @@ const { getUtils } = require('./utils')
 // Run a specific plugin event handler
 const run = async function (
   { event, error, constants, envChanges, netlifyConfig },
-  { pluginCommands, inputs, packageJson },
+  { pluginSteps, inputs, packageJson },
 ) {
-  const { method } = pluginCommands.find((pluginCommand) => pluginCommand.event === event)
+  const { method } = pluginSteps.find((pluginStep) => pluginStep.event === event)
   const runState = {}
   const utils = getUtils({ event, constants, runState })
   const netlifyConfigCopy = cloneNetlifyConfig(netlifyConfig)

package/src/plugins/load.js

@@ -6,28 +6,28 @@ const { measureDuration } = require('../time/main')
 
 const { callChild } = require('./ipc')
 
-// Retrieve all plugins commands
+// Retrieve all plugins steps
 // Can use either a module name or a file path to the plugin.
 const loadPlugins = async function ({ pluginsOptions, childProcesses, packageJson, timers, debug }) {
   return pluginsOptions.length === 0
-    ? { pluginsCommands: [], timers }
+    ? { pluginsSteps: [], timers }
     : await loadAllPlugins({ pluginsOptions, childProcesses, packageJson, timers, debug })
 }
 
 const tLoadAllPlugins = async function ({ pluginsOptions, childProcesses, packageJson, debug }) {
-  const pluginsCommands = await Promise.all(
+  const pluginsSteps = await Promise.all(
     pluginsOptions.map((pluginOptions, index) =>
       loadPlugin(pluginOptions, { childProcesses, index, packageJson, debug }),
     ),
   )
-  const pluginsCommandsA = pluginsCommands.flat()
-  return { pluginsCommands: pluginsCommandsA }
+  const pluginsStepsA = pluginsSteps.flat()
+  return { pluginsSteps: pluginsStepsA }
 }
 
 // Only performed if there are some plugins
 const loadAllPlugins = measureDuration(tLoadAllPlugins, 'load_plugins')
 
-// Retrieve plugin commands for one plugin.
+// Retrieve plugin steps for one plugin.
 // Do it by executing the plugin `load` event handler.
 const loadPlugin = async function (
   { packageName, pluginPackageJson, pluginPackageJson: { version } = {}, pluginPath, inputs, loadedFrom, origin },
@@ -37,8 +37,8 @@ const loadPlugin = async function (
   const loadEvent = 'load'
 
   try {
-    const { pluginCommands } = await callChild(childProcess, 'load', { pluginPath, inputs, packageJson })
-    const pluginCommandsA = pluginCommands.map(({ event }) => ({
+    const { pluginSteps } = await callChild(childProcess, 'load', { pluginPath, inputs, packageJson })
+    const pluginStepsA = pluginSteps.map(({ event }) => ({
       event,
       packageName,
       loadedFrom,
@@ -46,7 +46,7 @@ const loadPlugin = async function (
       pluginPackageJson,
       childProcess,
     }))
-    return pluginCommandsA
+    return pluginStepsA
   } catch (error) {
     addErrorInfo(error, {
       plugin: { packageName, pluginPackageJson },

package/src/plugins_core/build_command.js

@@ -6,11 +6,11 @@ const execa = require('execa')
 
 const { addErrorInfo } = require('../error/info')
 const { getBuildCommandDescription } = require('../log/description')
-const { logBuildCommandStart } = require('../log/messages/commands')
+const { logBuildCommandStart } = require('../log/messages/steps')
 const { getBuildCommandStdio, handleBuildCommandOutput } = require('../log/stream')
 
 // Fire `build.command`
-const coreCommand = async function ({
+const coreStep = async function ({
   configPath,
   buildDir,
   nodePath,
@@ -51,7 +51,7 @@ const coreCommand = async function ({
 // We use Bash on Unix and `cmd.exe` on Windows
 const SHELL = platform === 'win32' ? true : 'bash'
 
-const coreCommandDescription = function ({
+const coreStepDescription = function ({
   netlifyConfig: {
     build: { commandOrigin: buildCommandOrigin },
   },
@@ -69,10 +69,10 @@ const hasBuildCommand = function ({
 
 const buildCommandCore = {
   event: 'onBuild',
-  coreCommand,
-  coreCommandId: 'build_command',
-  coreCommandName: 'build.command',
-  coreCommandDescription,
+  coreStep,
+  coreStepId: 'build_command',
+  coreStepName: 'build.command',
+  coreStepDescription,
   condition: hasBuildCommand,
 }
 

package/src/plugins_core/deploy/buildbot_client.js

@@ -88,9 +88,7 @@ const getDeployDir = function ({ buildDir, repositoryRoot, constants: { PUBLISH_
 const handleDeployError = function (error, errorType) {
   const errorA = new Error(`Deploy did not succeed: ${error}`)
   const errorInfo =
-    errorType === 'user'
-      ? { type: 'resolveConfig' }
-      : { type: 'coreCommand', location: { coreCommandName: 'Deploy site' } }
+    errorType === 'user' ? { type: 'resolveConfig' } : { type: 'coreStep', location: { coreStepName: 'Deploy site' } }
   addErrorInfo(errorA, errorInfo)
   throw errorA
 }

package/src/plugins_core/deploy/index.js

@@ -10,7 +10,7 @@ const {
   deploySiteWithBuildbotClient,
 } = require('./buildbot_client')
 
-const coreCommand = async function ({
+const coreStep = async function ({
   buildDir,
   configPath,
   repositoryRoot,
@@ -67,10 +67,10 @@ const shouldDeploy = function ({ buildbotServerSocket }) {
 
 const deploySite = {
   event: 'onPostBuild',
-  coreCommand,
-  coreCommandId: 'deploy_site',
-  coreCommandName: 'Deploy site',
-  coreCommandDescription: () => 'Deploy site',
+  coreStep,
+  coreStepId: 'deploy_site',
+  coreStepName: 'Deploy site',
+  coreStepDescription: () => 'Deploy site',
   condition: shouldDeploy,
 }
 

package/src/plugins_core/functions/index.js

@@ -8,11 +8,7 @@ const mapObject = require('map-obj')
 const pathExists = require('path-exists')
 
 const { log } = require('../../log/logger')
-const {
-  logBundleResults,
-  logFunctionsNonExistingDir,
-  logFunctionsToBundle,
-} = require('../../log/messages/core_commands')
+const { logBundleResults, logFunctionsNonExistingDir, logFunctionsToBundle } = require('../../log/messages/core_steps')
 
 const { getZipError } = require('./error')
 const { getUserAndInternalFunctions, validateFunctionsSrc } = require('./utils')
@@ -52,6 +48,7 @@ const getZisiParameters = ({ buildDir, featureFlags, functionsConfig, functionsD
     normalizeFunctionConfig({ buildDir, featureFlags, functionConfig: object, isRunningLocally }),
   ])
   const zisiFeatureFlags = {
+    buildGoSource: featureFlags.buildbot_build_go_functions,
     defaultEsModulesToEsbuild: featureFlags.buildbot_es_modules_esbuild,
     parseWithEsbuild: featureFlags.buildbot_zisi_esbuild_parser,
   }
@@ -89,7 +86,7 @@ const zipFunctionsAndLogResults = async ({
 
 // Plugin to package Netlify functions with @netlify/zip-it-and-ship-it
 // eslint-disable-next-line complexity
-const coreCommand = async function ({
+const coreStep = async function ({
   constants: {
     INTERNAL_FUNCTIONS_SRC: relativeInternalFunctionsSrc,
     IS_LOCAL: isRunningLocally,
@@ -152,7 +149,7 @@ const coreCommand = async function ({
   }
 }
 
-// We run this core command if at least one of the functions directories (the
+// We run this core step if at least one of the functions directories (the
 // one configured by the user or the internal one) exists. We use a dynamic
 // `condition` because the directories might be created by the build command
 // or plugins.
@@ -170,10 +167,10 @@ const hasFunctionsDirectories = async function ({ buildDir, constants: { INTERNA
 
 const bundleFunctions = {
   event: 'onBuild',
-  coreCommand,
-  coreCommandId: 'functions_bundling',
-  coreCommandName: 'Functions bundling',
-  coreCommandDescription: () => 'Functions bundling',
+  coreStep,
+  coreStepId: 'functions_bundling',
+  coreStepName: 'Functions bundling',
+  coreStepDescription: () => 'Functions bundling',
   condition: hasFunctionsDirectories,
 }