feat: add confirmation prompts to unsafe cli commands

bin/run.js

@@ -3,9 +3,10 @@ import { argv } from 'process'
 
 import updateNotifier from 'update-notifier'
 
-import { createMainCommand } from '../dist/commands/index.js'
+import { runProgram } from '../dist/utils/run-program.js'
 import { error } from '../dist/utils/command-helpers.js'
 import getPackageJson from '../dist/utils/get-package-json.js'
+import { createMainCommand } from '../dist/commands/main.js'
 
 // 12 hours
 const UPDATE_CHECK_INTERVAL = 432e5
@@ -24,7 +25,8 @@ try {
 const program = createMainCommand()
 
 try {
-  await program.parseAsync(argv)
+  await runProgram(program, argv)
+
   program.onEnd()
 } catch (error_) {
   program.onEnd(error_)

docs/commands/blobs.md

@@ -57,6 +57,7 @@ netlify blobs:delete
 **Flags**
 
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `debug` (*boolean*) - Print debugging information
 
 ---
@@ -124,6 +125,7 @@ netlify blobs:set
 **Flags**
 
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `input` (*string*) - Defines the filesystem path where the blob data should be read from
 - `debug` (*boolean*) - Print debugging information
 

docs/commands/env.md

@@ -54,6 +54,7 @@ netlify env:clone
 **Flags**
 
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `from` (*string*) - Site ID (From)
 - `to` (*string*) - Site ID (To)
 - `debug` (*boolean*) - Print debugging information
@@ -167,6 +168,7 @@ netlify env:set
 
 - `context` (*string*) - Specify a deploy context or branch (contexts: "production", "deploy-preview", "branch-deploy", "dev") (default: all contexts)
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `scope` (*builds | functions | post-processing | runtime*) - Specify a scope (default: all scopes)
 - `secret` (*boolean*) - Indicate whether the environment variable value can be read again.
 - `debug` (*boolean*) - Print debugging information
@@ -202,6 +204,7 @@ netlify env:unset
 
 - `context` (*string*) - Specify a deploy context or branch (contexts: "production", "deploy-preview", "branch-deploy", "dev") (default: all contexts)
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `debug` (*boolean*) - Print debugging information
 
 **Examples**

docs/commands/init.md

@@ -17,7 +17,7 @@ netlify init
 **Flags**
 
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
-- `force` (*boolean*) - Reinitialize CI hooks if the linked site is already configured to use CI
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `git-remote-name` (*string*) - Name of Git remote to use. e.g. "origin"
 - `manual` (*boolean*) - Manually configure a git remote for CI
 - `debug` (*boolean*) - Print debugging information

docs/commands/sites.md

@@ -109,7 +109,7 @@ netlify sites:delete
 **Flags**
 
 - `filter` (*string*) - For monorepos, specify the name of the application to run the command in
-- `force` (*boolean*) - delete without prompting (useful for CI)
+- `force` (*boolean*) - Bypasses prompts & Force the command to run.
 - `debug` (*boolean*) - Print debugging information
 
 **Examples**

package.json

@@ -191,7 +191,7 @@
     "@netlify/functions": "2.8.2",
     "@sindresorhus/slugify": "2.2.1",
     "@types/fs-extra": "11.0.4",
-    "@types/inquirer": "9.0.7",
+    "@types/inquirer": "^9.0.7",
     "@types/jsonwebtoken": "9.0.7",
     "@types/lodash": "4.17.12",
     "@types/node": "20.14.8",

src/commands/addons/addons.ts

@@ -51,7 +51,7 @@ Add-ons are a way to extend the functionality of your Netlify site`,
     .description(
       `Remove an add-on extension to your site\nAdd-ons are a way to extend the functionality of your Netlify site`,
     )
-    .option('-f, --force', 'delete without prompting (useful for CI)')
+    // .option('-f, --force', 'delete without prompting (useful for CI)')
     .action(async (addonName: string, options: OptionValues, command: BaseCommand) => {
       const { addonsDelete } = await import('./addons-delete.js')
       await addonsDelete(addonName, options, command)

src/commands/base-command.ts

@@ -1,5 +1,3 @@
-import { isCI } from 'ci-info'
-
 import { existsSync } from 'fs'
 import { join, relative, resolve } from 'path'
 import process from 'process'
@@ -8,6 +6,7 @@ import { format } from 'util'
 import { DefaultLogger, Project } from '@netlify/build-info'
 import { NodeFS, NoopLogger } from '@netlify/build-info/node'
 import { resolveConfig } from '@netlify/config'
+import { isCI } from 'ci-info'
 import { Command, Help, Option } from 'commander'
 // @ts-expect-error TS(7016) FIXME: Could not find a declaration file for module 'debu... Remove this comment to see the full error message
 import debug from 'debug'
@@ -168,6 +167,13 @@ export default class BaseCommand extends Command {
   // eslint-disable-next-line workspace/no-process-cwd
   workingDir = process.cwd()
 
+  /**
+   * Determines if the command is scripted or not.
+   * If the command is scripted (SHLVL is greater than 1 or CI/CONTINUOUS_INTEGRATION is true) then some commands
+   * might behave differently.
+   */
+  scriptedCommand = Boolean(!process.stdin.isTTY || isCI || process.env.CI)
+
   /**
    * The workspace root if inside a mono repository.
    * Must not be the repository root!
@@ -187,6 +193,7 @@ export default class BaseCommand extends Command {
   createCommand(name: string): BaseCommand {
     const base = new BaseCommand(name)
       // If  --silent or --json flag passed disable logger
+      // .addOption(new Option('--force', 'Force command to run. Bypasses prompts for certain destructive commands.'))
       .addOption(new Option('--json', 'Output return values as JSON').hideHelp(true))
       .addOption(new Option('--silent', 'Silence CLI output').hideHelp(true))
       .addOption(new Option('--cwd <cwd>').hideHelp(true))

src/commands/blobs/blobs-delete.ts

@@ -1,21 +1,32 @@
 import { getStore } from '@netlify/blobs'
 
-import { chalk, error as printError } from '../../utils/command-helpers.js'
+import { chalk, error as printError, log } from '../../utils/command-helpers.js'
+import { promptBlobDelete } from '../../utils/prompts/blob-delete-prompts.js'
 
 /**
  * The blobs:delete command
  */
 export const blobsDelete = async (storeName: string, key: string, _options: Record<string, unknown>, command: any) => {
+  // Prevents prompts from blocking scripted commands
+
   const { api, siteInfo } = command.netlify
+  const { force } = _options
+
   const store = getStore({
     apiURL: `${api.scheme}://${api.host}`,
     name: storeName,
     siteID: siteInfo.id ?? '',
     token: api.accessToken ?? '',
   })
 
+  if (force === undefined) {
+    await promptBlobDelete(key, storeName)
+  }
+
   try {
     await store.delete(key)
+
+    log(`${chalk.greenBright('Success')}: Blob ${chalk.yellow(key)} deleted from store ${chalk.yellow(storeName)}`)
   } catch {
     return printError(`Could not delete blob ${chalk.yellow(key)} from store ${chalk.yellow(storeName)}`)
   }

src/commands/blobs/blobs-set.ts

@@ -4,11 +4,13 @@ import { resolve } from 'path'
 import { getStore } from '@netlify/blobs'
 import { OptionValues } from 'commander'
 
-import { chalk, error as printError, isNodeError } from '../../utils/command-helpers.js'
+import { chalk, error as printError, isNodeError, log } from '../../utils/command-helpers.js'
+import { promptBlobSetOverwrite } from '../../utils/prompts/blob-set-prompt.js'
 import BaseCommand from '../base-command.js'
 
 interface Options extends OptionValues {
   input?: string
+  force?: string | boolean
 }
 
 export const blobsSet = async (
@@ -18,20 +20,20 @@ export const blobsSet = async (
   options: Options,
   command: BaseCommand,
 ) => {
+  // Prevents prompts from blocking scripted commands
+
   const { api, siteInfo } = command.netlify
-  const { input } = options
+  const { force, input } = options
   const store = getStore({
     apiURL: `${api.scheme}://${api.host}`,
     name: storeName,
     siteID: siteInfo.id ?? '',
     token: api.accessToken ?? '',
   })
-
   let value = valueParts.join(' ')
 
   if (input) {
     const inputPath = resolve(input)
-
     try {
       value = await fs.readFile(inputPath, 'utf8')
     } catch (error) {
@@ -57,8 +59,17 @@ export const blobsSet = async (
     )
   }
 
+  if (force === undefined) {
+    const existingValue = await store.get(key)
+
+    if (existingValue) {
+      await promptBlobSetOverwrite(key, storeName)
+    }
+  }
+
   try {
     await store.set(key, value)
+    log(`${chalk.greenBright('Success')}: Blob ${chalk.yellow(key)} set in store ${chalk.yellow(storeName)}`)
   } catch {
     return printError(`Could not set blob ${chalk.yellow(key)} in store ${chalk.yellow(storeName)}`)
   }