netlify-cli 16.0.2 → 16.1.0

package/npm-shrinkwrap.json

@@ -1,18 +1,18 @@
 {
   "name": "netlify-cli",
-  "version": "16.0.2",
+  "version": "16.1.0",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "netlify-cli",
-      "version": "16.0.2",
+      "version": "16.1.0",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {
         "@bugsnag/js": "7.20.2",
         "@fastify/static": "6.10.2",
-        "@netlify/build": "29.20.2",
+        "@netlify/build": "29.20.4",
         "@netlify/build-info": "7.7.3",
         "@netlify/config": "20.8.0",
         "@netlify/edge-bundler": "8.17.1",
@@ -781,9 +781,9 @@
       "integrity": "sha512-4wMPu9iN3/HL97QblBsBay3E1etIciR84izI3U+4iALY+JHCrI+a2jO0qbAZ/nxKoegypYEaiiqWXylm+/zfrw=="
     },
     "node_modules/@netlify/build": {
-      "version": "29.20.2",
-      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-29.20.2.tgz",
-      "integrity": "sha512-ivkeheWbTtyloHS7IQQk0UsyH4Ozp6zUg6v6NcLobAmuT6hwgpXzBnwaAMq1GaJSTpUBbNyTx2Oga6wuuu/Nig==",
+      "version": "29.20.4",
+      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-29.20.4.tgz",
+      "integrity": "sha512-qXht5lWE0RThGhhB5ILQv4/2Jp+p0muTDp9n+KtlqpinO1SvXicVPw+D2o3iWMsz3vQak6yX09oEvumP2Q+TVQ==",
       "dependencies": {
         "@bugsnag/js": "^7.0.0",
         "@honeycombio/opentelemetry-node": "^0.4.0",
@@ -15295,9 +15295,9 @@
       "integrity": "sha512-4wMPu9iN3/HL97QblBsBay3E1etIciR84izI3U+4iALY+JHCrI+a2jO0qbAZ/nxKoegypYEaiiqWXylm+/zfrw=="
     },
     "@netlify/build": {
-      "version": "29.20.2",
-      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-29.20.2.tgz",
-      "integrity": "sha512-ivkeheWbTtyloHS7IQQk0UsyH4Ozp6zUg6v6NcLobAmuT6hwgpXzBnwaAMq1GaJSTpUBbNyTx2Oga6wuuu/Nig==",
+      "version": "29.20.4",
+      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-29.20.4.tgz",
+      "integrity": "sha512-qXht5lWE0RThGhhB5ILQv4/2Jp+p0muTDp9n+KtlqpinO1SvXicVPw+D2o3iWMsz3vQak6yX09oEvumP2Q+TVQ==",
       "requires": {
         "@bugsnag/js": "^7.0.0",
         "@honeycombio/opentelemetry-node": "^0.4.0",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "netlify-cli",
   "description": "Netlify command line tool",
-  "version": "16.0.2",
+  "version": "16.1.0",
   "author": "Netlify Inc.",
   "type": "module",
   "engines": {
@@ -44,7 +44,7 @@
   "dependencies": {
     "@bugsnag/js": "7.20.2",
     "@fastify/static": "6.10.2",
-    "@netlify/build": "29.20.2",
+    "@netlify/build": "29.20.4",
     "@netlify/build-info": "7.7.3",
     "@netlify/config": "20.8.0",
     "@netlify/edge-bundler": "8.17.1",

package/src/commands/base-command.mjs

@@ -574,7 +574,7 @@ export default class BaseCommand extends Command {
     // configuration file and the API
     // ==================================================
     const cachedConfig = await actionCommand.getConfig({
-      cwd: this.jsWorkspaceRoot || this.workingDir,
+      cwd: flags.cwd ? this.workingDir : this.jsWorkspaceRoot || this.workingDir,
       repositoryRoot: rootDir,
       packagePath: this.workspacePackage,
       // The config flag needs to be resolved from the actual process working directory

package/src/commands/deploy/deploy.mjs

@@ -324,6 +324,7 @@ const runDeploy = async ({
   deployToProduction,
   functionsConfig,
   functionsFolder,
+  packagePath,
   silent,
   site,
   siteData,
@@ -345,13 +346,13 @@ const runDeploy = async ({
     results = await api.createSiteDeploy({ siteId, title, body: { draft, branch: alias } })
     deployId = results.id
 
-    const internalFunctionsFolder = await getInternalFunctionsDir({ base: site.root })
+    const internalFunctionsFolder = await getInternalFunctionsDir({ base: site.root, packagePath })
 
     // The order of the directories matter: zip-it-and-ship-it will prioritize
     // functions from the rightmost directories. In this case, we want user
     // functions to take precedence over internal functions.
     const functionDirectories = [internalFunctionsFolder, functionsFolder].filter(Boolean)
-    const manifestPath = skipFunctionsCache ? null : await getFunctionsManifestPath({ base: site.root })
+    const manifestPath = skipFunctionsCache ? null : await getFunctionsManifestPath({ base: site.root, packagePath })
 
     // @ts-ignore
     results = await deploySite(api, siteId, deployFolder, {
@@ -367,6 +368,7 @@ const runDeploy = async ({
       workingDir: command.workingDir,
       manifestPath,
       skipFunctionsCache,
+      siteRoot: site.root,
     })
   } catch (error_) {
     if (deployId) {
@@ -657,6 +659,7 @@ const deploy = async (options, command) => {
     functionsConfig,
     // pass undefined functionsFolder if doesn't exist
     functionsFolder: functionsFolderStat && functionsFolder,
+    packagePath: command.workspacePackage,
     silent: options.json || options.silent,
     site,
     siteData,

package/src/utils/deploy/deploy-site.mjs

@@ -40,6 +40,7 @@ export const deploySite = async (
     // API calls this the 'title'
     message: title,
     siteEnv,
+    siteRoot,
     skipFunctionsCache,
     statusCb = () => {
       /* default to noop */
@@ -78,6 +79,7 @@ export const deploySite = async (
         manifestPath,
         skipFunctionsCache,
         siteEnv,
+        rootDir: siteRoot,
       }),
     ])
   const edgeFunctionsCount = Object.keys(files).filter(isEdgeFunctionFile).length

package/src/utils/deploy/hash-fns.mjs

@@ -20,10 +20,10 @@ const getFunctionZips = async ({
   directories,
   functionsConfig,
   manifestPath,
+  rootDir,
   skipFunctionsCache,
   statusCb,
   tmpDir,
-  workingDir,
 }) => {
   statusCb({
     type: 'functions-manifest',
@@ -68,7 +68,7 @@ const getFunctionZips = async ({
   }
 
   return await zipFunctions(directories, tmpDir, {
-    basePath: workingDir,
+    basePath: rootDir,
     configFileDirectories: [getPathInProject([INTERNAL_FUNCTIONS_FOLDER])],
     config: functionsConfig,
   })

package/src/utils/functions/functions.mjs

@@ -19,15 +19,15 @@ export const SERVE_FUNCTIONS_FOLDER = 'functions-serve'
 export const getFunctionsDir = ({ config, options }, defaultValue) =>
   options.functions || config.dev?.functions || config.functionsDirectory || config.dev?.Functions || defaultValue
 
-export const getFunctionsManifestPath = async ({ base }) => {
-  const path = resolve(base, getPathInProject(['functions', 'manifest.json']))
+export const getFunctionsManifestPath = async ({ base, packagePath = '' }) => {
+  const path = resolve(base, packagePath, getPathInProject(['functions', 'manifest.json']))
   const isFile = await isFileAsync(path)
 
   return isFile ? path : null
 }
 
-export const getFunctionsDistPath = async ({ base }) => {
-  const path = resolve(base, getPathInProject(['functions']))
+export const getFunctionsDistPath = async ({ base, packagePath = '' }) => {
+  const path = resolve(base, packagePath, getPathInProject(['functions']))
   const isDirectory = await isDirectoryAsync(path)
 
   return isDirectory ? path : null
@@ -38,10 +38,11 @@ export const getFunctionsDistPath = async ({ base }) => {
  * @param {object} config
  * @param {string} config.base
  * @param {boolean=} config.ensureExists
+ * @param {string} config.packagePath
  * @returns
  */
-export const getInternalFunctionsDir = async ({ base, ensureExists }) => {
-  const path = resolve(base, getPathInProject([INTERNAL_FUNCTIONS_FOLDER]))
+export const getInternalFunctionsDir = async ({ base, ensureExists, packagePath = '' }) => {
+  const path = resolve(base, packagePath, getPathInProject([INTERNAL_FUNCTIONS_FOLDER]))
 
   if (ensureExists) {
     await fs.mkdir(path, { recursive: true })

package/src/utils/proxy.mjs

@@ -75,19 +75,35 @@ const formatEdgeFunctionError = (errorBuffer, acceptsHtml) => {
   })
 }
 
-const isInternal = function (url) {
+/**
+ * @param {string} url
+ */
+function isInternal(url) {
   return url.startsWith('/.netlify/')
 }
-const isFunction = function (functionsPort, url) {
+
+/**
+ * @param {boolean|number|undefined} functionsPort
+ * @param {string} url
+ */
+function isFunction(functionsPort, url) {
   return functionsPort && url.match(/^\/.netlify\/(functions|builders)\/.+/)
 }
 
-const getAddonUrl = function (addonsUrls, req) {
-  const matches = req.url.match(/^\/.netlify\/([^/]+)(\/.*)/)
+/**
+ * @param {Record<string, string>} addonsUrls
+ * @param {http.IncomingMessage} req
+ */
+function getAddonUrl(addonsUrls, req) {
+  const matches = req.url?.match(/^\/.netlify\/([^/]+)(\/.*)/)
   const addonUrl = matches && addonsUrls[matches[1]]
   return addonUrl ? `${addonUrl}${matches[2]}` : null
 }
 
+/**
+ * @param {string} pathname
+ * @param {string} publicFolder
+ */
 const getStatic = async function (pathname, publicFolder) {
   const alternatives = [pathname, ...alternativePathsFor(pathname)].map((filePath) =>
     path.resolve(publicFolder, filePath.slice(1)),

package/src/utils/run-build.mjs

@@ -80,7 +80,7 @@ export const runNetlifyBuild = async ({ command, env = {}, options, settings, ti
   const devCommand = async (settingsOverrides = {}) => {
     let cwd = command.workingDir
 
-    if (command.project.workspace?.packages.length) {
+    if (!options.cwd && command.project.workspace?.packages.length) {
       cwd = join(command.project.jsWorkspaceRoot, settings.baseDirectory || '')
     }
 

package/src/utils/telemetry/telemetry.mjs

@@ -13,14 +13,18 @@ import isValidEventName from './validation.mjs'
 
 const dirPath = dirname(fileURLToPath(import.meta.url))
 
-const send = function (type, payload) {
+/**
+ * @param {'track' | 'identify'} type
+ * @param {object} payload
+ */
+function send(type, payload) {
   const requestFile = join(dirPath, 'request.mjs')
   const options = JSON.stringify({
     data: payload,
     type,
   })
 
-  const args = [process.execPath, [requestFile, options]]
+  const args = /** @type {const} */ ([process.execPath, [requestFile, options]])
   if (process.env.NETLIFY_TEST_TELEMETRY_WAIT === 'true') {
     return execa(...args, {
       stdio: 'inherit',
@@ -46,7 +50,12 @@ const eventConfig = {
   ],
 }
 
-export const track = async function (eventName, payload = {}) {
+/**
+ * Tracks a custom event with the provided payload
+ * @param {string} eventName
+ * @param {{status?: string, duration?: number, [key: string]: unknown}} [payload]
+ */
+export async function track(eventName, payload = {}) {
   if (isCI) {
     return
   }
@@ -83,7 +92,14 @@ export const track = async function (eventName, payload = {}) {
   return send('track', defaultData)
 }
 
-export const identify = async function (payload) {
+/**
+ * @param {object} payload
+ * @param {string} payload.name
+ * @param {string} payload.email
+ * @param {string} payload.userId
+ * @returns
+ */
+export async function identify(payload) {
   if (isCI) {
     return
   }

package/src/utils/validation.mjs

@@ -1,6 +1,10 @@
 // @ts-check
 import { BANG, chalk } from './command-helpers.mjs'
 
+/**
+ * @param {string} exampleCommand
+ * @returns {(value:string, previous: unknown) => unknown}
+ */
 export const getGeoCountryArgParser = (exampleCommand) => (arg) => {
   // Validate that the arg passed is two letters only for country
   // See https://en.wikipedia.org/wiki/List_of_ISO_3166_country_codes