netlify-cli 15.0.3 → 15.1.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "netlify-cli",
   "description": "Netlify command line tool",
-  "version": "15.0.3",
+  "version": "15.1.1",
   "author": "Netlify Inc.",
   "type": "module",
   "engines": {
@@ -44,14 +44,15 @@
   "dependencies": {
     "@bugsnag/js": "7.20.2",
     "@fastify/static": "6.10.1",
-    "@netlify/build": "29.11.4",
+    "@netlify/build": "29.11.5",
     "@netlify/build-info": "7.0.2",
-    "@netlify/config": "20.4.2",
+    "@netlify/config": "20.4.3",
     "@netlify/edge-bundler": "8.14.2",
     "@netlify/framework-info": "9.8.7",
     "@netlify/local-functions-proxy": "1.1.1",
-    "@netlify/zip-it-and-ship-it": "9.5.0",
+    "@netlify/zip-it-and-ship-it": "9.6.0",
     "@octokit/rest": "19.0.8",
+    "@skn0tt/lambda-local": "2.0.3",
     "ansi-escapes": "6.2.0",
     "ansi-styles": "6.2.1",
     "ansi-to-html": "0.7.2",
@@ -66,16 +67,16 @@
     "commander": "10.0.1",
     "comment-json": "4.2.3",
     "concordance": "5.0.4",
-    "configstore": "5.0.1",
+    "configstore": "6.0.0",
     "content-type": "1.0.5",
     "cookie": "0.5.0",
     "copy-template-dir": "1.4.0",
     "cron-parser": "4.8.1",
     "debug": "4.3.4",
     "decache": "4.6.1",
-    "dot-prop": "6.0.1",
+    "dot-prop": "7.2.0",
     "dotenv": "16.0.3",
-    "env-paths": "2.2.1",
+    "env-paths": "3.0.0",
     "envinfo": "7.8.1",
     "etag": "1.8.1",
     "execa": "5.1.1",
@@ -100,11 +101,11 @@
     "inquirer": "6.5.2",
     "inquirer-autocomplete-prompt": "1.4.0",
     "is-docker": "3.0.0",
+    "is-stream": "3.0.0",
     "is-wsl": "2.2.0",
     "isexe": "2.0.0",
     "jsonwebtoken": "9.0.0",
     "jwt-decode": "3.1.2",
-    "lambda-local": "2.0.3",
     "listr": "0.14.3",
     "locate-path": "7.2.0",
     "lodash": "4.17.21",
@@ -112,13 +113,13 @@
     "log-update": "5.0.1",
     "minimist": "1.2.8",
     "multiparty": "4.2.3",
-    "netlify": "13.1.6",
+    "netlify": "13.1.7",
     "netlify-headers-parser": "7.1.2",
     "netlify-redirect-parser": "14.1.2",
     "netlify-redirector": "0.4.0",
     "node-fetch": "2.6.11",
     "node-version-alias": "3.4.1",
-    "ora": "6.3.0",
+    "ora": "6.3.1",
     "p-filter": "3.0.0",
     "p-map": "5.5.0",
     "p-wait-for": "5.0.2",
@@ -129,7 +130,7 @@
     "prettyjson": "1.2.5",
     "pump": "3.0.0",
     "raw-body": "2.5.2",
-    "read-pkg-up": "7.0.1",
+    "read-pkg-up": "9.1.0",
     "semver": "7.5.1",
     "source-map-support": "0.5.21",
     "string-similarity": "4.0.4",
@@ -139,7 +140,7 @@
     "terminal-link": "3.0.0",
     "through2-filter": "3.0.0",
     "through2-map": "3.0.0",
-    "to-readable-stream": "2.1.0",
+    "to-readable-stream": "3.0.0",
     "toml": "3.0.0",
     "ulid": "2.3.0",
     "unixify": "1.0.0",

package/src/commands/deploy/deploy.mjs

@@ -6,7 +6,6 @@ import { cwd, env } from 'process'
 import { runCoreSteps } from '@netlify/build'
 import { restoreConfig, updateConfig } from '@netlify/config'
 import { Option } from 'commander'
-import { get } from 'dot-prop'
 import inquirer from 'inquirer'
 import isObject from 'lodash/isObject.js'
 import prettyjson from 'prettyjson'
@@ -15,7 +14,7 @@ import { cancelDeploy } from '../../lib/api.mjs'
 import { getBuildOptions, runBuild } from '../../lib/build.mjs'
 import { featureFlags as edgeFunctionsFeatureFlags } from '../../lib/edge-functions/consts.mjs'
 import { normalizeFunctionsConfig } from '../../lib/functions/config.mjs'
-import { getLogMessage } from '../../lib/log.mjs'
+import { BACKGROUND_FUNCTIONS_WARNING } from '../../lib/log.mjs'
 import { startSpinner, stopSpinner } from '../../lib/spinner.mjs'
 import {
   chalk,
@@ -74,10 +73,10 @@ const getDeployFolder = async ({ config, options, site, siteData }) => {
   let deployFolder
   if (options.dir) {
     deployFolder = resolve(cwd(), options.dir)
-  } else if (get(config, 'build.publish')) {
-    deployFolder = resolve(site.root, get(config, 'build.publish'))
-  } else if (get(siteData, 'build_settings.dir')) {
-    deployFolder = resolve(site.root, get(siteData, 'build_settings.dir'))
+  } else if (config?.build?.publish) {
+    deployFolder = resolve(site.root, config.build.publish)
+  } else if (siteData?.build_settings?.dir) {
+    deployFolder = resolve(site.root, siteData.build_settings.dir)
   }
 
   if (!deployFolder) {
@@ -138,8 +137,8 @@ const getFunctionsFolder = ({ config, options, site, siteData }) => {
     functionsFolder = resolve(cwd(), options.functions)
   } else if (funcConfig) {
     functionsFolder = resolve(site.root, funcConfig)
-  } else if (get(siteData, 'build_settings.functions_dir')) {
-    functionsFolder = resolve(site.root, get(siteData, 'build_settings.functions_dir'))
+  } else if (siteData?.build_settings?.functions_dir) {
+    functionsFolder = resolve(site.root, siteData.build_settings.functions_dir)
   }
   return functionsFolder
 }
@@ -232,14 +231,12 @@ const hasErrorMessage = (actual, expected) => {
   return false
 }
 
-const getJsonErrorMessage = (error_) => get(error_, 'json.message', '')
-
 const reportDeployError = ({ error_, failAndExit }) => {
   switch (true) {
     case error_.name === 'JSONHTTPError': {
-      const message = getJsonErrorMessage(error)
+      const message = error_?.json?.message ?? ''
       if (hasErrorMessage(message, 'Background Functions not allowed by team plan')) {
-        return failAndExit(`\n${getLogMessage('functions.backgroundNotSupported')}`)
+        return failAndExit(`\n${BACKGROUND_FUNCTIONS_WARNING}`)
       }
       warn(`JSONHTTPError: ${message} ${error_.status}`)
       warn(`\n${JSON.stringify(error_, null, '  ')}\n`)
@@ -358,8 +355,8 @@ const runDeploy = async ({
   }
 
   const siteUrl = results.deploy.ssl_url || results.deploy.url
-  const deployUrl = get(results, 'deploy.deploy_ssl_url') || get(results, 'deploy.deploy_url')
-  const logsUrl = `${get(results, 'deploy.admin_url')}/deploys/${get(results, 'deploy.id')}`
+  const deployUrl = results.deploy.deploy_ssl_url || results.deploy.deploy_url
+  const logsUrl = `${results.deploy.admin_url}/deploys/${results.deploy.id}`
 
   return {
     siteId: results.deploy.site_id,
@@ -616,7 +613,7 @@ const deploy = async (options, command) => {
         scope: 'functions',
         siteInfo: siteData,
       })
-    : get(siteData, 'build_settings.env')
+    : siteData?.build_settings?.env
 
   const functionsConfig = normalizeFunctionsConfig({
     functionsConfig: config.functions,

package/src/commands/init/init.mjs

@@ -1,6 +1,5 @@
 // @ts-check
 import { Option } from 'commander'
-import dotProp from 'dot-prop'
 import inquirer from 'inquirer'
 import isEmpty from 'lodash/isEmpty.js'
 
@@ -17,7 +16,11 @@ const persistState = ({ siteInfo, state }) => {
   state.set('siteId', siteInfo.id)
 }
 
-const getRepoUrl = ({ siteInfo }) => dotProp.get(siteInfo, 'build_settings.repo_url')
+/**
+ * @param {{} | undefined} siteInfo
+ * @returns {string | undefined}
+ */
+const getRepoUrl = (siteInfo) => siteInfo?.build_settings?.repo_url
 
 const logExistingAndExit = ({ siteInfo }) => {
   log()
@@ -187,7 +190,7 @@ export const init = async (options, command) => {
   // Add .netlify to .gitignore file
   await ensureNetlifyIgnore(repositoryRoot)
 
-  const repoUrl = getRepoUrl({ siteInfo })
+  const repoUrl = getRepoUrl(siteInfo)
   if (repoUrl && !options.force) {
     logExistingAndExit({ siteInfo })
   }
@@ -205,7 +208,7 @@ export const init = async (options, command) => {
   log()
 
   // Check for existing CI setup
-  const remoteBuildRepo = getRepoUrl({ siteInfo })
+  const remoteBuildRepo = getRepoUrl(siteInfo)
   if (remoteBuildRepo && !options.force) {
     logExistingRepoSetupAndExit({ siteName: siteInfo.name, repoUrl: remoteBuildRepo })
   }

package/src/commands/status/status-hooks.mjs

@@ -1,5 +1,4 @@
 // @ts-check
-import { get } from 'dot-prop'
 import prettyjson from 'prettyjson'
 
 import { error, log, warn } from '../../utils/command-helpers.mjs'
@@ -48,8 +47,8 @@ const statusHooks = async (options, command) => {
       id: hook.id,
       disabled: hook.disabled,
     }
-    if (get(siteData, 'build_settings.repo_url')) {
-      data.hooks[hook.id].repo_url = get(siteData, 'build_settings.repo_url')
+    if (siteData.build_settings?.repo_url) {
+      data.hooks[hook.id].repo_url = siteData.build_settings.repo_url
     }
   })
   log(`─────────────────┐

package/src/lib/functions/registry.mjs

@@ -15,7 +15,7 @@ import {
   watchDebounced,
 } from '../../utils/command-helpers.mjs'
 import { INTERNAL_FUNCTIONS_FOLDER, SERVE_FUNCTIONS_FOLDER } from '../../utils/functions/functions.mjs'
-import { getLogMessage } from '../log.mjs'
+import { BACKGROUND_FUNCTIONS_WARNING } from '../log.mjs'
 import { getPathInProject } from '../settings.mjs'
 
 import NetlifyFunction from './netlify-function.mjs'
@@ -140,7 +140,7 @@ export class FunctionsRegistry {
     }
 
     if (func.isBackground && this.isConnected && !this.capabilities.backgroundFunctions) {
-      warn(getLogMessage('functions.backgroundNotSupported'))
+      warn(BACKGROUND_FUNCTIONS_WARNING)
     }
 
     if (!func.hasValidName()) {

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

@@ -3,7 +3,7 @@ import { createRequire } from 'module'
 import path from 'path'
 
 import decache from 'decache'
-import readPkgUp from 'read-pkg-up'
+import { readPackageUp } from 'read-pkg-up'
 import sourceMapSupport from 'source-map-support'
 
 import { NETLIFYDEVERR } from '../../../../../utils/command-helpers.mjs'
@@ -112,7 +112,7 @@ const netlifyConfigToZisiConfig = ({ config, projectRoot }) =>
 export default async function handler({ config, directory, errorExit, func, projectRoot }) {
   const functionsConfig = netlifyConfigToZisiConfig({ config, projectRoot })
 
-  const packageJson = await readPkgUp(func.mainFile)
+  const packageJson = await readPackageUp(func.mainFile)
   const hasTypeModule = packageJson && packageJson.packageJson.type === 'module'
 
   // We must use esbuild for certain file extensions.

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

@@ -1,6 +1,6 @@
 import { dirname } from 'path'
 
-import lambdaLocal from 'lambda-local'
+import lambdaLocal from '@skn0tt/lambda-local'
 import winston from 'winston'
 
 import detectNetlifyLambdaBuilder from './builders/netlify-lambda.mjs'

package/src/lib/functions/server.mjs

@@ -1,5 +1,4 @@
 // @ts-check
-import { get } from 'dot-prop'
 import jwtDecode from 'jwt-decode'
 
 import { NETLIFYDEVERR, NETLIFYDEVLOG, error as errorExit, log } from '../../utils/command-helpers.mjs'
@@ -94,7 +93,7 @@ export const createHandler = function (options) {
       {},
     )
     const rawQuery = new URLSearchParams(request.query).toString()
-    const protocol = get(options, 'config.dev.https') ? 'https' : 'http'
+    const protocol = options.config?.dev?.https ? 'https' : 'http'
     const url = new URL(requestPath, `${protocol}://${request.get('host') || 'localhost'}`)
     url.search = rawQuery
     const rawUrl = url.toString()

package/src/lib/functions/synchronous.mjs

@@ -1,6 +1,8 @@
 // @ts-check
 import { Buffer } from 'buffer'
 
+import { isStream } from 'is-stream'
+
 import { chalk, log, NETLIFYDEVERR } from '../../utils/command-helpers.mjs'
 import renderErrorTemplate from '../render-error-template.mjs'
 
@@ -47,6 +49,12 @@ export const handleSynchronousFunction = function ({
   }
 
   if (result.body) {
+    if (isStream(result.body)) {
+      result.body.pipe(response)
+
+      return
+    }
+
     response.write(result.isBase64Encoded ? Buffer.from(result.body, 'base64') : result.body)
   }
   response.end()
@@ -97,8 +105,8 @@ const validateLambdaResponse = (lambdaResponse) => {
       error: `Your function response must have a numerical statusCode. You gave: $ ${lambdaResponse.statusCode}`,
     }
   }
-  if (lambdaResponse.body && typeof lambdaResponse.body !== 'string') {
-    return { error: `Your function response must have a string body. You gave: ${lambdaResponse.body}` }
+  if (lambdaResponse.body && typeof lambdaResponse.body !== 'string' && !isStream(lambdaResponse.body)) {
+    return { error: `Your function response must have a string or a stream body. You gave: ${lambdaResponse.body}` }
   }
 
   return {}

package/src/lib/functions/utils.mjs

@@ -1,12 +1,12 @@
 // @ts-check
 import { chalk, warn } from '../../utils/command-helpers.mjs'
-import { getLogMessage } from '../log.mjs'
+import { MISSING_AWS_SDK_WARNING } from '../log.mjs'
 
 export const detectAwsSdkError = ({ error }) => {
   const isAwsSdkError = error && error.errorMessage && error.errorMessage.includes("Cannot find module 'aws-sdk'")
 
   if (isAwsSdkError) {
-    warn(getLogMessage('functions.missingAwsSdk'))
+    warn(MISSING_AWS_SDK_WARNING)
   }
 }
 

package/src/lib/log.mjs

@@ -1,27 +1,18 @@
-import dotProp from 'dot-prop'
-
 import { chalk } from '../utils/command-helpers.mjs'
 
 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.
+export const BACKGROUND_FUNCTIONS_WARNING = `A serverless function ending in \`${RED_BACKGROUND}\` was detected.
 Your team’s current plan doesn’t support Background Functions, which have names ending in \`${RED_BACKGROUND}\`.
 To be able to deploy this function successfully either:
   - change the function name to remove \`${RED_BACKGROUND}\` and execute it synchronously
   - upgrade your team plan to a level that supports Background Functions (${PRO}, ${BUSINESS}, or ${ENTERPRISE})
 `
-const MISSING_AWS_SDK_WARNING = `A function has thrown an error due to a missing dependency: ${chalk.yellow('aws-sdk')}.
+export const MISSING_AWS_SDK_WARNING = `A function has thrown an error due to a missing dependency: ${chalk.yellow(
+  'aws-sdk',
+)}.
 You should add this module to the project's dependencies, using your package manager of choice:
 
 ${chalk.yellow('npm install aws-sdk --save')} or ${chalk.yellow('yarn add aws-sdk')}
 
 For more information, see https://ntl.fyi/cli-aws-sdk.`
-
-const messages = {
-  functions: {
-    backgroundNotSupported: BACKGROUND_FUNCTIONS_WARNING,
-    missingAwsSdk: MISSING_AWS_SDK_WARNING,
-  },
-}
-
-export const getLogMessage = (key) => dotProp.get(messages, key, 'Missing Log Message Key')

package/src/utils/dev.mjs

@@ -1,7 +1,6 @@
 // @ts-check
 import process from 'process'
 
-import { get } from 'dot-prop'
 import getPort from 'get-port'
 import isEmpty from 'lodash/isEmpty.js'
 
@@ -112,7 +111,7 @@ export const getSiteInformation = async ({ api, offline, site, siteInfo }) => {
         backgroundFunctions: supportsBackgroundFunctions(account),
       },
       timeouts: {
-        syncFunctions: get(siteInfo, 'functions_config.timeout', SYNCHRONOUS_FUNCTION_TIMEOUT),
+        syncFunctions: siteInfo.functions_config?.timeout ?? SYNCHRONOUS_FUNCTION_TIMEOUT,
         backgroundFunctions: BACKGROUND_FUNCTION_TIMEOUT,
       },
     }

package/src/utils/proxy.mjs

@@ -11,7 +11,7 @@ import zlib from 'zlib'
 
 import contentType from 'content-type'
 import cookie from 'cookie'
-import { get } from 'dot-prop'
+import { getProperty } from 'dot-prop'
 import generateETag from 'etag'
 import getAvailablePort from 'get-port'
 import httpProxy from 'http-proxy'
@@ -210,7 +210,7 @@ const serveRedirect = async function ({ env, match, options, proxy, req, res, si
       if ((jwtValue.exp || 0) < Math.round(Date.now() / MILLISEC_TO_SEC)) {
         console.warn(NETLIFYDEVWARN, 'Expired JWT provided in request', req.url)
       } else {
-        const presentedRoles = get(jwtValue, options.jwtRolePath) || []
+        const presentedRoles = getProperty(jwtValue, options.jwtRolePath) || []
         if (!Array.isArray(presentedRoles)) {
           console.warn(NETLIFYDEVWARN, `Invalid roles value provided in JWT ${options.jwtRolePath}`, presentedRoles)
           res.writeHead(400)

package/src/utils/state-config.mjs

@@ -2,7 +2,7 @@ import fs from 'fs'
 import path from 'path'
 import process from 'process'
 
-import dotProp from 'dot-prop'
+import { deleteProperty, getProperty, hasProperty, setProperty } from 'dot-prop'
 import { findUpSync } from 'find-up'
 import writeFileAtomic from 'write-file-atomic'
 
@@ -75,7 +75,7 @@ export default class StateConfig {
       // TODO figure out cleaner way of grabbing ENV vars
       return process.env.NETLIFY_SITE_ID
     }
-    return dotProp.get(this.all, key)
+    return getProperty(this.all, key)
   }
 
   set(...args) {
@@ -84,22 +84,22 @@ export default class StateConfig {
 
     if (args.length === 1) {
       Object.entries(key).forEach(([keyPart, value]) => {
-        dotProp.set(config, keyPart, value)
+        setProperty(config, keyPart, value)
       })
     } else {
-      dotProp.set(config, key, val)
+      setProperty(config, key, val)
     }
 
     this.all = config
   }
 
   has(key) {
-    return dotProp.has(this.all, key)
+    return hasProperty(this.all, key)
   }
 
   delete(key) {
     const config = this.all
-    dotProp.delete(config, key)
+    deleteProperty(config, key)
     this.all = config
   }