Skip to content

docs: note that unannotated MCP tools skip approval - #832

Open
bhaveshpatel640 wants to merge 4 commits into
mainfrom
AGE-2311
Open

bhaveshpatel640 wants to merge 4 commits into
mainfrom
AGE-2311

Conversation

@bhaveshpatel640

@bhaveshpatel640 bhaveshpatel640 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

docs: note that unannotated MCP tools skip approval


Note

Low Risk
Documentation-only changes to agent tool-approval defaults and MCP annotation behavior; no runtime code in this diff.

Overview
Updates Create an Agent docs so tool approval behavior matches the product: require_approval_for_tools defaults to ["@destructive"], not @write and @destructive.

Adds a Note explaining that @write / @destructive apply only to MCP tools the server annotates — unannotated or readOnlyHint: true tools are not paused by default (even if they mutate state), with guidance to use literal tool names or @all when you need approval anyway. The UI checklist and mcp_servers API table are updated accordingly, plus clearer rules for how annotation selectors resolve.

Reviewed by Cursor Bugbot for commit b678c7c. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot

changeset-bot Bot commented Sep 21, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: b678c7c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@bhaveshpatel640
bhaveshpatel640 enabled auto-merge (squash) September 21, 2026 14:47

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant