netlify-cli 15.8.1-rc.1 → 15.9.0

package/src/utils/run-build.mjs

@@ -12,13 +12,8 @@ import { INTERNAL_FUNCTIONS_FOLDER } from './functions/index.mjs'
 
 const netlifyBuildPromise = import('@netlify/build')
 
-/**
- * Copies `netlify.toml`, if one is defined, into the `.netlify` internal
- * directory and returns the path to its new location.
- * @param {object} config
- * @param {string} config.configPath
- * @param {string} config.siteRoot
- */
+// Copies `netlify.toml`, if one is defined, into the `.netlify` internal
+// directory and returns the path to its new location.
 const copyConfig = async ({ configPath, siteRoot }) => {
   const newConfigPath = path.resolve(siteRoot, getPathInProject(['netlify.toml']))
 
@@ -31,9 +26,6 @@ const copyConfig = async ({ configPath, siteRoot }) => {
   return newConfigPath
 }
 
-/**
- * @param {string} basePath
- */
 const cleanInternalDirectory = async (basePath) => {
   const ops = [INTERNAL_FUNCTIONS_FOLDER, INTERNAL_EDGE_FUNCTIONS_FOLDER, 'netlify.toml'].map((name) => {
     const fullPath = path.resolve(basePath, getPathInProject([name]))
@@ -44,21 +36,6 @@ const cleanInternalDirectory = async (basePath) => {
   await Promise.all(ops)
 }
 
-/**
- *
- * @param {object} config
- * @param {*} config.cachedConfig
- * @param {object} config.options
- * @param {string} config.options.configPath
- * @param {*} config.options.context
- * @param {string=} config.options.cwd
- * @param {boolean} config.options.debug
- * @param {boolean} config.options.dry
- * @param {boolean} config.options.offline
- * @param {boolean} config.options.quiet
- * @param {boolean} config.options.saveConfig
- * @returns
- */
 const getBuildOptions = ({
   cachedConfig,
   options: { configPath, context, cwd = process.cwd(), debug, dry, offline, quiet, saveConfig },
@@ -79,18 +56,7 @@ const getBuildOptions = ({
   saveConfig,
 })
 
-/**
- *
- * @param {object} config
- * @param {*} config.cachedConfig
- * @param {NodeJS.ProcessEnv} config.env
- * @param {*} config.options
- * @param {import('./types.js').ServerSettings} config.settings
- * @param {*} config.site
- * @param {'build' | 'dev'} config.timeline
- * @returns
- */
-export const runNetlifyBuild = async ({ cachedConfig, env, options, settings, site, timeline = 'build' }) => {
+const runNetlifyBuild = async ({ cachedConfig, env, options, settings, site, timeline = 'build' }) => {
   const { default: buildSite, startDev } = await netlifyBuildPromise
   const sharedOptions = getBuildOptions({
     cachedConfig,
@@ -152,19 +118,13 @@ export const runNetlifyBuild = async ({ cachedConfig, env, options, settings, si
   // Run Netlify Build using the `startDev` entry point.
   const { error: startDevError, success } = await startDev(devCommand, startDevOptions)
 
-  if (!success && startDevError) {
+  if (!success) {
     error(`Could not start local development server\n\n${startDevError.message}\n\n${startDevError.stack}`)
   }
 
   return {}
 }
 
-/**
- * @param {Omit<Parameters<typeof runNetlifyBuild>[0], 'timeline'>} options
- */
 export const runDevTimeline = (options) => runNetlifyBuild({ ...options, timeline: 'dev' })
 
-/**
- * @param {Omit<Parameters<typeof runNetlifyBuild>[0], 'timeline'>} options
- */
 export const runBuildTimeline = (options) => runNetlifyBuild({ ...options, timeline: 'build' })

package/src/utils/shell.mjs

@@ -48,11 +48,7 @@ export const runCommand = (command, env = {}, spinner = null) => {
     preferLocal: true,
     // we use reject=false to avoid rejecting synchronously when the command doesn't exist
     reject: false,
-    env: {
-      // we want always colorful terminal outputs
-      FORCE_COLOR: 'true',
-      ...env,
-    },
+    env,
     // windowsHide needs to be false for child process to terminate properly on Windows
     windowsHide: false,
   })
@@ -104,13 +100,6 @@ export const runCommand = (command, env = {}, spinner = null) => {
   return commandProcess
 }
 
-/**
- *
- * @param {object} config
- * @param {string} config.command
- * @param {*} config.error
- * @returns
- */
 const isNonExistingCommandError = ({ command, error: commandError }) => {
   // `ENOENT` is only returned for non Windows systems
   // See https://github.com/sindresorhus/execa/pull/447
@@ -119,7 +108,7 @@ const isNonExistingCommandError = ({ command, error: commandError }) => {
   }
 
   // if the command is a package manager we let it report the error
-  if (['yarn', 'npm', 'pnpm'].includes(command)) {
+  if (['yarn', 'npm'].includes(command)) {
     return false
   }
 

package/src/utils/static-server.mjs

@@ -6,10 +6,6 @@ import Fastify from 'fastify'
 
 import { log, NETLIFYDEVLOG } from './command-helpers.mjs'
 
-/**
- * @param {object} config
- * @param {import('./types.js').ServerSettings} config.settings
- */
 export const startStaticServer = async ({ settings }) => {
   const server = Fastify()
   const rootPath = path.resolve(settings.dist)