netlify-cli 15.1.1 → 15.3.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "netlify-cli",
   "description": "Netlify command line tool",
-  "version": "15.1.1",
+  "version": "15.3.0",
   "author": "Netlify Inc.",
   "type": "module",
   "engines": {
@@ -43,15 +43,15 @@
   },
   "dependencies": {
     "@bugsnag/js": "7.20.2",
-    "@fastify/static": "6.10.1",
-    "@netlify/build": "29.11.5",
-    "@netlify/build-info": "7.0.2",
+    "@fastify/static": "6.10.2",
+    "@netlify/build": "29.11.7",
+    "@netlify/build-info": "7.0.3",
     "@netlify/config": "20.4.3",
-    "@netlify/edge-bundler": "8.14.2",
-    "@netlify/framework-info": "9.8.7",
+    "@netlify/edge-bundler": "8.15.0",
+    "@netlify/framework-info": "9.8.8",
     "@netlify/local-functions-proxy": "1.1.1",
-    "@netlify/zip-it-and-ship-it": "9.6.0",
-    "@octokit/rest": "19.0.8",
+    "@netlify/zip-it-and-ship-it": "9.7.0",
+    "@octokit/rest": "19.0.11",
     "@skn0tt/lambda-local": "2.0.3",
     "ansi-escapes": "6.2.0",
     "ansi-styles": "6.2.1",

package/src/commands/base-command.mjs

@@ -2,6 +2,9 @@
 import process from 'process'
 import { format } from 'util'
 
+import { Project } from '@netlify/build-info'
+// eslint-disable-next-line import/extensions, n/no-missing-import
+import { NodeFS } from '@netlify/build-info/node'
 import { resolveConfig } from '@netlify/config'
 import { Command, Option } from 'commander'
 import debug from 'debug'
@@ -24,6 +27,7 @@ import {
   warn,
 } from '../utils/command-helpers.mjs'
 import getGlobalConfig from '../utils/get-global-config.mjs'
+import { getSiteByName } from '../utils/get-site.mjs'
 import openBrowser from '../utils/open-browser.mjs'
 import StateConfig from '../utils/state-config.mjs'
 import { identify, track } from '../utils/telemetry/index.mjs'
@@ -440,11 +444,40 @@ export default class BaseCommand extends Command {
       certificateFile: options.httpProxyCertificateFilename,
     })
     const apiOpts = { ...apiUrlOpts, agent }
+    const api = new NetlifyAPI(token || '', apiOpts)
+
+    // If a user passes a site name as an option instead of a site ID to options.site, the siteInfo object
+    // will only have the property siteInfo.id. Checking for one of the other properties ensures that we can do
+    // a re-call of the api.getSite() that is done in @netlify/config so we have the proper site object in all
+    // commands.
+    // options.site as a site name (and not just site id) was introduced for the deploy command, so users could
+    // deploy by name along with by id
+    let siteData = siteInfo
+    if (!siteData.url && options.site) {
+      siteData = await getSiteByName(api, options.site)
+    }
+
     const globalConfig = await getGlobalConfig()
 
+    // Get framework, add to analytics payload for every command, if a framework is set
+    const fs = new NodeFS()
+    const project = new Project(fs, buildDir)
+    const frameworks = await project.detectFrameworks()
+
+    const frameworkIDs = frameworks?.map((framework) => framework.id)
+
+    if (frameworkIDs?.length !== 0) {
+      this.setAnalyticsPayload({ frameworks: frameworkIDs })
+    }
+
+    this.setAnalyticsPayload({
+      packageManager: project.packageManager?.name,
+      buildSystem: project.buildSystems.map(({ id }) => id),
+    })
+
     actionCommand.netlify = {
       // api methods
-      api: new NetlifyAPI(token || '', apiOpts),
+      api,
       apiOpts,
       repositoryRoot,
       // current site context
@@ -458,8 +491,8 @@ export default class BaseCommand extends Command {
           state.set('siteId', id)
         },
       },
-      // Site information retrieved using the API
-      siteInfo,
+      // Site information retrieved using the API (api.getSite())
+      siteInfo: siteData,
       // Configuration from netlify.[toml/yml]
       config: normalizedConfig,
       // Used to avoid calling @netlify/config again

package/src/commands/deploy/deploy.mjs

@@ -7,6 +7,7 @@ import { runCoreSteps } from '@netlify/build'
 import { restoreConfig, updateConfig } from '@netlify/config'
 import { Option } from 'commander'
 import inquirer from 'inquirer'
+import isEmpty from 'lodash/isEmpty.js'
 import isObject from 'lodash/isObject.js'
 import prettyjson from 'prettyjson'
 
@@ -488,7 +489,7 @@ const printResults = ({ deployToProduction, json, results, runBuildCommand }) =>
  * @param {import('../base-command.mjs').default} command
  */
 const deploy = async (options, command) => {
-  const { api, site } = command.netlify
+  const { api, site, siteInfo } = command.netlify
   const alias = options.alias || options.branch
 
   command.setAnalyticsPayload({ open: options.open, prod: options.prod, json: options.json, alias: Boolean(alias) })
@@ -503,35 +504,12 @@ const deploy = async (options, command) => {
 
   await command.authenticate(options.auth)
 
-  let siteId = options.site || site.id
+  let siteId = site.id || options.site
 
   let siteData = {}
-  if (siteId) {
-    try {
-      const [{ siteError, siteFoundById }, sites] = await Promise.all([
-        api
-          .getSite({ siteId })
-          .then((data) => ({ siteFoundById: data }))
-          .catch((error_) => ({ siteError: error_ })),
-        api.listSites({ name: options.site, filter: 'all' }),
-      ])
-      const siteFoundByName = sites.find((filteredSite) => filteredSite.name === options.site)
-      if (siteFoundById) {
-        siteData = siteFoundById
-      } else if (siteFoundByName) {
-        siteData = siteFoundByName
-        siteId = siteFoundByName.id
-      } else {
-        throw siteError
-      }
-    } catch (error_) {
-      // TODO specifically handle known cases (e.g. no account access)
-      if (error_.status === 404) {
-        error('Site not found')
-      } else {
-        error(error_.message)
-      }
-    }
+  if (siteId && !isEmpty(siteInfo)) {
+    siteData = siteInfo
+    siteId = siteData.id
   } else {
     log("This folder isn't linked to a site yet")
     const NEW_SITE = '+  Create & configure a new site'

package/src/commands/dev/dev.mjs

@@ -123,7 +123,7 @@ const dev = async (options, command) => {
     exit(1)
   }
 
-  command.setAnalyticsPayload({ projectType: settings.framework || 'custom', live: options.live })
+  command.setAnalyticsPayload({ live: options.live })
 
   const liveTunnelUrl = await handleLiveTunnel({ options, site, api, settings })
   const url = liveTunnelUrl || getProxyUrl(settings)

package/src/commands/functions/functions-list.mjs

@@ -1,8 +1,9 @@
 // @ts-check
 import AsciiTable from 'ascii-table'
 
-import { error, exit, log, logJson, warn } from '../../utils/command-helpers.mjs'
+import { exit, log, logJson } from '../../utils/command-helpers.mjs'
 import { getFunctions, getFunctionsDir } from '../../utils/functions/index.mjs'
+import requiresSiteInfo from '../../utils/hooks/requires-site-info.mjs'
 
 const normalizeFunction = function (deployedFunctions, { name, urlPath: url }) {
   const isDeployed = deployedFunctions.some((deployedFunction) => deployedFunction.n === name)
@@ -15,31 +16,9 @@ const normalizeFunction = function (deployedFunctions, { name, urlPath: url }) {
  * @param {import('../base-command.mjs').default} command
  */
 const functionsList = async (options, command) => {
-  const { api, config, site } = command.netlify
+  const { config, siteInfo } = command.netlify
 
-  // get deployed site details
-  // copied from `netlify status`
-  const siteId = site.id
-  if (!siteId) {
-    warn('Did you run `netlify link` yet?')
-    error(`You don't appear to be in a folder that is linked to a site`)
-  }
-  let siteData
-  try {
-    siteData = await api.getSite({ siteId })
-  } catch (error_) {
-    // unauthorized
-    if (error_.status === 401) {
-      warn(`Log in with a different account or re-link to a site you have permission for`)
-      error(`Not authorized to view the currently linked site (${siteId})`)
-    }
-    // missing
-    if (error_.status === 404) {
-      error(`The site this folder is linked to can't be found`)
-    }
-    error(error_)
-  }
-  const deploy = siteData.published_deploy || {}
+  const deploy = siteInfo.published_deploy || {}
   const deployedFunctions = deploy.available_functions || []
 
   const functionsDir = getFunctionsDir({ options, config })
@@ -91,4 +70,5 @@ NOT the same as listing the functions that have been deployed. For that info you
     )
     .option('-f, --functions <dir>', 'Specify a functions directory to list')
     .option('--json', 'Output function data as JSON')
+    .hook('preAction', requiresSiteInfo)
     .action(functionsList)

package/src/commands/link/link.mjs

@@ -1,6 +1,7 @@
 // @ts-check
 import { Option } from 'commander'
 import inquirer from 'inquirer'
+import isEmpty from 'lodash/isEmpty.js'
 
 import { listSites } from '../../lib/api.mjs'
 import { chalk, error, exit, log } from '../../utils/command-helpers.mjs'
@@ -250,28 +251,23 @@ export const link = async (options, command) => {
     api,
     repositoryRoot,
     site: { id: siteId },
+    siteInfo,
     state,
   } = command.netlify
 
-  let siteData
-  try {
-    // @ts-ignore types from API are wrong they cannot recognize `getSite` of API
-    siteData = await api.getSite({ siteId })
-  } catch {
-    // silent api error
-  }
+  let siteData = siteInfo
 
   // Add .netlify to .gitignore file
   await ensureNetlifyIgnore(repositoryRoot)
 
   // Site id is incorrect
-  if (siteId && !siteData) {
+  if (siteId && isEmpty(siteData)) {
     log(`"${siteId}" was not found in your Netlify account.`)
     log(`Please double check your siteID and which account you are logged into via \`netlify status\`.`)
     return exit()
   }
 
-  if (siteData) {
+  if (!isEmpty(siteInfo)) {
     // If already linked to site. exit and prompt for unlink
     log(`Site already linked to "${siteData.name}"`)
     log(`Admin url: ${siteData.admin_url}`)

package/src/commands/open/open-admin.mjs

@@ -1,4 +1,5 @@
-import { error, exit, log, warn } from '../../utils/command-helpers.mjs'
+import { exit, log } from '../../utils/command-helpers.mjs'
+import requiresSiteInfo from '../../utils/hooks/requires-site-info.mjs'
 import openBrowser from '../../utils/open-browser.mjs'
 
 /**
@@ -7,42 +8,14 @@ import openBrowser from '../../utils/open-browser.mjs'
  * @param {import('../base-command.mjs').default} command
  */
 export const openAdmin = async (options, command) => {
-  const { api, site } = command.netlify
+  const { siteInfo } = command.netlify
 
   await command.authenticate()
 
-  const siteId = site.id
+  log(`Opening "${siteInfo.name}" site admin UI:`)
+  log(`> ${siteInfo.admin_url}`)
 
-  if (!siteId) {
-    warn(`No Site ID found in current directory.
-Run \`netlify link\` to connect to this folder to a site`)
-    return false
-  }
-
-  let siteData
-  try {
-    siteData = await api.getSite({ siteId })
-    log(`Opening "${siteData.name}" site admin UI:`)
-    log(`> ${siteData.admin_url}`)
-  } catch (error_) {
-    // unauthorized
-    if (error_.status === 401) {
-      warn(`Log in with a different account or re-link to a site you have permission for`)
-      error(`Not authorized to view the currently linked site (${siteId})`)
-    }
-    // site not found
-    if (error_.status === 404) {
-      log()
-      log('Please double check this ID and verify you are logged in with the correct account')
-      log()
-      log('To fix this, run `netlify unlink` then `netlify link` to reconnect to the correct site ID')
-      log()
-      error(`Site "${siteId}" not found in account`)
-    }
-    error(error_)
-  }
-
-  await openBrowser({ url: siteData.admin_url })
+  await openBrowser({ url: siteInfo.admin_url })
   exit()
 }
 
@@ -56,4 +29,5 @@ export const createOpenAdminCommand = (program) =>
     .command('open:admin')
     .description('Opens current site admin UI in Netlify')
     .addExamples(['netlify open:admin'])
+    .hook('preAction', requiresSiteInfo)
     .action(openAdmin)

package/src/commands/open/open-site.mjs

@@ -1,4 +1,5 @@
-import { error, exit, log, warn } from '../../utils/command-helpers.mjs'
+import { exit, log } from '../../utils/command-helpers.mjs'
+import requiresSiteInfo from '../../utils/hooks/requires-site-info.mjs'
 import openBrowser from '../../utils/open-browser.mjs'
 
 /**
@@ -7,33 +8,13 @@ import openBrowser from '../../utils/open-browser.mjs'
  * @param {import('../base-command.mjs').default} command
  */
 export const openSite = async (options, command) => {
-  const { api, site } = command.netlify
+  const { siteInfo } = command.netlify
 
   await command.authenticate()
 
-  const siteId = site.id
-
-  if (!siteId) {
-    warn(`No Site ID found in current directory.
-Run \`netlify link\` to connect to this folder to a site`)
-    return false
-  }
-
-  let siteData
-  let url
-  try {
-    siteData = await api.getSite({ siteId })
-    url = siteData.ssl_url || siteData.url
-    log(`Opening "${siteData.name}" site url:`)
-    log(`> ${url}`)
-  } catch (error_) {
-    // unauthorized
-    if (error_.status === 401) {
-      warn(`Log in with a different account or re-link to a site you have permission for`)
-      error(`Not authorized to view the currently linked site (${siteId})`)
-    }
-    error(error_)
-  }
+  const url = siteInfo.ssl_url || siteInfo.url
+  log(`Opening "${siteInfo.name}" site url:`)
+  log(`> ${url}`)
 
   await openBrowser({ url })
   exit()
@@ -49,4 +30,5 @@ export const createOpenSiteCommand = (program) =>
     .command('open:site')
     .description('Opens current site url in browser')
     .addExamples(['netlify open:site'])
+    .hook('preAction', requiresSiteInfo)
     .action(openSite)

package/src/commands/serve/serve.mjs

@@ -80,7 +80,7 @@ const serve = async (options, command) => {
     exit(1)
   }
 
-  command.setAnalyticsPayload({ projectType: settings.framework || 'custom', live: options.live })
+  command.setAnalyticsPayload({ live: options.live })
 
   log(`${NETLIFYDEVLOG} Building site for production`)
   log(

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

@@ -1,7 +1,8 @@
 // @ts-check
 import prettyjson from 'prettyjson'
 
-import { error, log, warn } from '../../utils/command-helpers.mjs'
+import { log } from '../../utils/command-helpers.mjs'
+import requiresSiteInfo from '../../utils/hooks/requires-site-info.mjs'
 
 /**
  * The status:hooks command
@@ -9,35 +10,13 @@ import { error, log, warn } from '../../utils/command-helpers.mjs'
  * @param {import('../base-command.mjs').default} command
  */
 const statusHooks = async (options, command) => {
-  const { api, site } = command.netlify
+  const { api, siteInfo } = command.netlify
 
   await command.authenticate()
 
-  const siteId = site.id
-  if (!siteId) {
-    warn('Did you run `netlify link` yet?')
-    error(`You don't appear to be in a folder that is linked to a site`)
-  }
-
-  let siteData
-  try {
-    siteData = await api.getSite({ siteId })
-  } catch (error_) {
-    // unauthorized
-    if (error_.status === 401) {
-      warn(`Log in with a different account or re-link to a site you have permission for`)
-      error(`Not authorized to view the currently linked site (${siteId})`)
-    }
-    // missing
-    if (error_.status === 404) {
-      error(`The site this folder is linked to can't be found`)
-    }
-    error(error_)
-  }
-
-  const ntlHooks = await api.listHooksBySiteId({ siteId: siteData.id })
+  const ntlHooks = await api.listHooksBySiteId({ siteId: siteInfo.id })
   const data = {
-    site: siteData.name,
+    site: siteInfo.name,
     hooks: {},
   }
   ntlHooks.forEach((hook) => {
@@ -47,8 +26,8 @@ const statusHooks = async (options, command) => {
       id: hook.id,
       disabled: hook.disabled,
     }
-    if (siteData.build_settings?.repo_url) {
-      data.hooks[hook.id].repo_url = siteData.build_settings.repo_url
+    if (siteInfo.build_settings?.repo_url) {
+      data.hooks[hook.id].repo_url = siteInfo.build_settings.repo_url
     }
   })
   log(`─────────────────┐
@@ -63,4 +42,8 @@ Site Hook Status │
  * @returns
  */
 export const createStatusHooksCommand = (program) =>
-  program.command('status:hooks').description('Print hook information of the linked site').action(statusHooks)
+  program
+    .command('status:hooks')
+    .description('Print hook information of the linked site')
+    .hook('preAction', requiresSiteInfo)
+    .action(statusHooks)

package/src/commands/unlink/unlink.mjs

@@ -8,7 +8,7 @@ import { track } from '../../utils/telemetry/index.mjs'
  * @param {import('../base-command.mjs').default} command
  */
 const unlink = async (options, command) => {
-  const { site, state } = command.netlify
+  const { site, siteInfo, state } = command.netlify
   const siteId = site.id
 
   if (!siteId) {
@@ -16,13 +16,7 @@ const unlink = async (options, command) => {
     return exit()
   }
 
-  let siteData = {}
-  try {
-    // @ts-ignore types from API are wrong they cannot recognize `getSite` of API
-    siteData = await command.netlify.api.getSite({ siteId })
-  } catch {
-    // ignore errors if we can't get the site
-  }
+  const siteData = siteInfo
 
   state.delete('siteId')
 

package/src/functions-templates/javascript/auth-fetch/package-lock.json

@@ -13,9 +13,9 @@
       }
     },
     "node_modules/node-fetch": {
-      "version": "2.6.9",
-      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.9.tgz",
-      "integrity": "sha512-DJm/CJkZkRjKKj4Zi4BsKVZh3ValV5IR5s7LVZnW+6YMh0W1BfNA8XSs6DLMGYlId5F3KnA70uu2qepcR08Qqg==",
+      "version": "2.6.11",
+      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.11.tgz",
+      "integrity": "sha512-4I6pdBY1EthSqDmJkiNk3JIT8cswwR9nfeW/cPdUagJYEQG7R95WRH74wpz7ma8Gh/9dI9FP+OU+0E4FvtA55w==",
       "dependencies": {
         "whatwg-url": "^5.0.0"
       },
@@ -53,9 +53,9 @@
   },
   "dependencies": {
     "node-fetch": {
-      "version": "2.6.9",
-      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.9.tgz",
-      "integrity": "sha512-DJm/CJkZkRjKKj4Zi4BsKVZh3ValV5IR5s7LVZnW+6YMh0W1BfNA8XSs6DLMGYlId5F3KnA70uu2qepcR08Qqg==",
+      "version": "2.6.11",
+      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.11.tgz",
+      "integrity": "sha512-4I6pdBY1EthSqDmJkiNk3JIT8cswwR9nfeW/cPdUagJYEQG7R95WRH74wpz7ma8Gh/9dI9FP+OU+0E4FvtA55w==",
       "requires": {
         "whatwg-url": "^5.0.0"
       }

package/src/functions-templates/javascript/google-analytics/package-lock.json

@@ -13,13 +13,13 @@
         "uuid": "^9.0.0"
       },
       "engines": {
-        "node": "^14.16.0 || >=16.0.0"
+        "node": "^14.18.0 || >=16.0.0"
       }
     },
     "node_modules/node-fetch": {
-      "version": "2.6.9",
-      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.9.tgz",
-      "integrity": "sha512-DJm/CJkZkRjKKj4Zi4BsKVZh3ValV5IR5s7LVZnW+6YMh0W1BfNA8XSs6DLMGYlId5F3KnA70uu2qepcR08Qqg==",
+      "version": "2.6.11",
+      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.11.tgz",
+      "integrity": "sha512-4I6pdBY1EthSqDmJkiNk3JIT8cswwR9nfeW/cPdUagJYEQG7R95WRH74wpz7ma8Gh/9dI9FP+OU+0E4FvtA55w==",
       "dependencies": {
         "whatwg-url": "^5.0.0"
       },
@@ -65,9 +65,9 @@
   },
   "dependencies": {
     "node-fetch": {
-      "version": "2.6.9",
-      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.9.tgz",
-      "integrity": "sha512-DJm/CJkZkRjKKj4Zi4BsKVZh3ValV5IR5s7LVZnW+6YMh0W1BfNA8XSs6DLMGYlId5F3KnA70uu2qepcR08Qqg==",
+      "version": "2.6.11",
+      "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.6.11.tgz",
+      "integrity": "sha512-4I6pdBY1EthSqDmJkiNk3JIT8cswwR9nfeW/cPdUagJYEQG7R95WRH74wpz7ma8Gh/9dI9FP+OU+0E4FvtA55w==",
       "requires": {
         "whatwg-url": "^5.0.0"
       }

package/src/utils/get-site.mjs

@@ -0,0 +1,16 @@
+import { error } from './command-helpers.mjs'
+
+export const getSiteByName = async (api, siteName) => {
+  try {
+    const sites = await api.listSites({ name: siteName, filter: 'all' })
+    const siteFoundByName = sites.find((filteredSite) => filteredSite.name === siteName)
+
+    if (!siteFoundByName) {
+      throw Error
+    }
+
+    return siteFoundByName
+  } catch {
+    error('Site not found. Please rerun "netlify link"')
+  }
+}

package/src/utils/hooks/requires-site-info.mjs

@@ -0,0 +1,29 @@
+import { error, warn } from '../command-helpers.mjs'
+/**
+ * A preAction hook that errors out if siteInfo is an empty object
+ * @param {*} command
+ */
+const requiresSiteInfo = async (command) => {
+  const { api, site } = command.netlify
+  const siteId = site.id
+  if (!siteId) {
+    warn('Did you run `netlify link` yet?')
+    return error(`You don't appear to be in a folder that is linked to a site`)
+  }
+  try {
+    await api.getSite({ siteId })
+  } catch (error_) {
+    // unauthorized
+    if (error_.status === 401) {
+      warn(`Log in with a different account or re-link to a site you have permission for`)
+      return error(`Not authorized to view the currently linked site (${siteId})`)
+    }
+    // missing
+    if (error_.status === 404) {
+      return error(`The site this folder is linked to can't be found`)
+    }
+    return error(error_)
+  }
+}
+
+export default requiresSiteInfo