fix(cli): Scope DELETE confirmation prompt to `vercel api` command only by tknickman · Pull Request #14787 · vercel/vercel

tknickman · 2026-01-29T16:10:56Z

Summary

Fixes a regression from #14769 where the DELETE confirmation prompt was incorrectly applied to all DELETE operations instead of only the vercel api command.

This was breaking CI/CD pipelines that use commands like:

vercel env rm <name> --yes
vercel alias rm <alias> --yes
vercel remove <deployment> --yes

These commands have their own --yes flags for confirmation and should not be affected by the new --dangerously-skip-permissions flag which was designed specifically for the vercel api command.

Changes

packages/cli/src/util/client.ts: Made confirmMutatingOperation() public and removed the call from fetch() method
packages/cli/src/commands/api/index.ts: Added confirmMutatingOperation() calls in executeSingleRequest() and executePaginatedRequest() - scoping confirmation to only the vercel api command
packages/cli/test/mocks/client.ts: Removed workaround that set dangerouslySkipPermissions = true by default
packages/cli/test/unit/util/client-confirmation.test.ts: Updated tests to call confirmMutatingOperation() directly

Test plan

All 15 unit tests pass
TypeScript compiles without errors
vercel env rm --yes works in non-TTY mode (no longer blocked)
vercel api -X DELETE still prompts for confirmation as intended

📦 CLI Tarball Ready

The Vercel CLI tarball for this PR is now available!

Quick Test

You can test this PR's CLI directly by running:

npx https://vercel-4j27sb0ao.vercel.sh/tarballs/vercel.tgz --help

Use in vercel.json

To use this CLI version in your project builds, add to your vercel.json:

{
  "build": {
    "env": {
      "VERCEL_CLI_VERSION": "vercel@https://vercel-4j27sb0ao.vercel.sh/tarballs/vercel.tgz"
    }
  }
}

github-actions · 2026-01-29T16:12:48Z

🧪 Unit Test Strategy

Comparing: 9d1e38e → a35543f (view diff)

Strategy: Affected packages only

✅ Only testing packages that have been modified or depend on modified packages.

Affected packages - 1 (2%)

vercel

Unaffected packages - 40 (98%)

@vercel-internals/get-package-json
@vercel/backends
@vercel/build-utils
@vercel/cervel
@vercel/cli-auth
@vercel/client
@vercel/config
@vercel/detect-agent
@vercel/edge
@vercel/elysia
@vercel/error-utils
@vercel/express
@vercel/fastify
@vercel/firewall
@vercel/frameworks
@vercel/fs-detectors
@vercel/functions
@vercel/gatsby-plugin-vercel-builder
@vercel/go
@vercel/h3
@vercel/hono
@vercel/hydrogen
@vercel/introspection
@vercel/koa
@vercel/nestjs
@vercel/next
@vercel/node
@vercel/oidc
@vercel/oidc-aws-credentials-provider
@vercel/python
@vercel/python-analysis
@vercel/redwood
@vercel/related-projects
@vercel/remix-builder
@vercel/routing-utils
@vercel/ruby
@vercel/rust
@vercel/static-build
@vercel/static-config
examples

Results

Unit tests: Only affected packages will run unit tests
E2E tests: Handled separately (Version Packages PRs or run-e2e-tests label)
Type checks: Only affected packages will run type checks

This comment is automatically generated based on the affected testing strategy

vercel

Additional Suggestion:

DELETE test hangs indefinitely because it doesn't skip the confirmation prompt

The test is testing the request functionality, not the confirmation behavior. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## vercel@50.9.2 ### Patch Changes - fix(cli): Scope DELETE confirmation prompt to `vercel api` command only ([#14787](#14787)) This fixes a regression from #14769 where the DELETE confirmation prompt was incorrectly applied to all DELETE operations (e.g., `vercel env rm`, `vercel alias rm`) instead of only the `vercel api` command. Commands like `env rm` and `alias rm` have their own `--yes` flags for confirmation and should not be affected. - experimental rust runtime framework preset ([#14765](#14765)) - Add redirects command to top level help message. ([#14772](#14772)) - Updated dependencies \[]: - @vercel/static-build@2.8.28 ## @vercel/frameworks@3.16.1 ### Patch Changes - Generic node framework preset ([#14779](#14779)) - experimental rust runtime framework preset ([#14765](#14765)) ## @vercel/fs-detectors@5.7.19 ### Patch Changes - experimental rust runtime framework preset ([#14765](#14765)) - Updated dependencies \[[`9d1e38e9ff6f0431dce8e421497e1ec4fc823291`](9d1e38e), [`e800617a334377e11953df22ef40d03716daf692`](e800617)]: - @vercel/frameworks@3.16.1 Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

…ly (#14787) ## Summary Fixes a regression from #14769 where the DELETE confirmation prompt was incorrectly applied to **all** DELETE operations instead of only the `vercel api` command. This was breaking CI/CD pipelines that use commands like: - `vercel env rm <name> --yes` - `vercel alias rm <alias> --yes` - `vercel remove <deployment> --yes` These commands have their own `--yes` flags for confirmation and should not be affected by the new `--dangerously-skip-permissions` flag which was designed specifically for the `vercel api` command. ## Changes - **`packages/cli/src/util/client.ts`**: Made `confirmMutatingOperation()` public and removed the call from `fetch()` method - **`packages/cli/src/commands/api/index.ts`**: Added `confirmMutatingOperation()` calls in `executeSingleRequest()` and `executePaginatedRequest()` - scoping confirmation to only the `vercel api` command - **`packages/cli/test/mocks/client.ts`**: Removed workaround that set `dangerouslySkipPermissions = true` by default - **`packages/cli/test/unit/util/client-confirmation.test.ts`**: Updated tests to call `confirmMutatingOperation()` directly ## Test plan - [x] All 15 unit tests pass - [x] TypeScript compiles without errors - [x] `vercel env rm --yes` works in non-TTY mode (no longer blocked) - [x] `vercel api -X DELETE` still prompts for confirmation as intended ## Related - Fixes regression introduced in #14769 - Supersedes #14784 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## vercel@50.9.2 ### Patch Changes - fix(cli): Scope DELETE confirmation prompt to `vercel api` command only ([#14787](#14787)) This fixes a regression from #14769 where the DELETE confirmation prompt was incorrectly applied to all DELETE operations (e.g., `vercel env rm`, `vercel alias rm`) instead of only the `vercel api` command. Commands like `env rm` and `alias rm` have their own `--yes` flags for confirmation and should not be affected. - experimental rust runtime framework preset ([#14765](#14765)) - Add redirects command to top level help message. ([#14772](#14772)) - Updated dependencies \[]: - @vercel/static-build@2.8.28 ## @vercel/frameworks@3.16.1 ### Patch Changes - Generic node framework preset ([#14779](#14779)) - experimental rust runtime framework preset ([#14765](#14765)) ## @vercel/fs-detectors@5.7.19 ### Patch Changes - experimental rust runtime framework preset ([#14765](#14765)) - Updated dependencies \[[`9d1e38e9ff6f0431dce8e421497e1ec4fc823291`](9d1e38e), [`e800617a334377e11953df22ef40d03716daf692`](e800617)]: - @vercel/frameworks@3.16.1 Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix(cli): scope approval to api command

be67965

tknickman requested a review from a team as a code owner January 29, 2026 16:10

vercel bot deployed to Preview January 29, 2026 16:11 View deployment

vercel bot reviewed Jan 29, 2026

View reviewed changes

fix(cli): skip confirmation in DELETE request test

a35543f

The test is testing the request functionality, not the confirmation behavior. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

vercel bot deployed to Preview January 29, 2026 16:30 View deployment

jeffsee55 approved these changes Jan 29, 2026

View reviewed changes

This was referenced Jan 29, 2026

fix(cli): Make env rm --yes work in non-TTY environments #14788

Closed

vercel env rm --yes flag not working in non-interactive mode (CI/CD) #14786

Closed

tknickman merged commit dbb7d6c into main Jan 29, 2026
120 checks passed

tknickman deleted the tknickman/scope-api-flag branch January 29, 2026 16:47

vercel-release-bot mentioned this pull request Jan 29, 2026

Version Packages #14778

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(cli): Scope DELETE confirmation prompt to `vercel api` command only#14787

fix(cli): Scope DELETE confirmation prompt to `vercel api` command only#14787
tknickman merged 2 commits intomainfrom
tknickman/scope-api-flag

tknickman commented Jan 29, 2026

Uh oh!

changeset-bot bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

vercel bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

tknickman commented Jan 29, 2026

Summary

Changes

Test plan

Related

Uh oh!

changeset-bot bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

github-actions bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📦 CLI Tarball Ready

Quick Test

Use in vercel.json

Uh oh!

github-actions bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🧪 Unit Test Strategy

Results

Uh oh!

vercel bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

changeset-bot bot commented Jan 29, 2026 •

edited

Loading

github-actions bot commented Jan 29, 2026 •

edited

Loading

github-actions bot commented Jan 29, 2026 •

edited

Loading