[claude-code-settings] Update Claude Code settings permissionRule#5338
[claude-code-settings] Update Claude Code settings permissionRule#5338plmrry wants to merge 10 commits intoSchemaStore:masterfrom
permissionRule#5338Conversation
Updates the `pattern` of the `permissionRule` def in `claude-code-settings.json` See `## Tools available to Claude` in this file: https://code.claude.com/docs/en/settings.md
|
Thanks for the PR! This section of the codebase is owned by @domdomegg and @bogini - if they write a comment saying "LGTM" then it will be merged. |
domdomegg
left a comment
There was a problem hiding this comment.
Hey! Thanks for this PR. I work on Claude Code at Anthropic so can help with accuracy here.
A couple of things:
-
MCPSearchshould beToolSearch— the internal tool name (what the permission system matches against) isToolSearch, notMCPSearch. Our user-facing docs call itMCPSearchwhich is confusing, but permission rules match on the actual tool.nameproperty. A rule likeMCPSearchwould never match anything. -
Missing
TaskStop— this is the tool that stops background tasks/shells.KillShellis just a backwards-compat alias forTaskStop, so both should be in the pattern. -
Merge conflict — this PR conflicts with master now (master got
LS,MultiEdit,NotebookRead,SlashCommandfrom another PR). The combined, corrected pattern should be:
^((Bash|Edit|ExitPlanMode|Glob|Grep|KillShell|LS|LSP|MultiEdit|NotebookEdit|NotebookRead|Read|Skill|SlashCommand|Task|TaskCreate|TaskGet|TaskList|TaskOutput|TaskStop|TaskUpdate|TodoWrite|ToolSearch|WebFetch|WebSearch|Write)(\\((?=.*[^)*?])[^)]+\\))?|mcp__.*)$
And in the test file, MCPSearch should be ToolSearch too.
|
@domdomegg Whoops — thanks for clarifying! (and updating!) |
Summary
Updates the
patternof thepermissionRuledef inclaude-code-settings.jsonThe current
claude-code-settings.tsis a bit outdated. Claude now supports things likeMCPSearchandLSP, andTodoWriteis no longer a tool`Schema Details
See
## Tools available to Claudein this file:https://code.claude.com/docs/en/settings.md