Skip to content

Conversation

@amothic
Copy link
Contributor

@amothic amothic commented Jan 7, 2026

Summary

Add Skill to the list of valid tools in the permissionRule pattern for Claude Code settings schema.

Details

The Claude Code documentation lists Skill as one of the available tools that requires permission:

Tool Description Permission Required
Skill Executes a skill within the main conversation Yes

Users can configure Skill permissions like:

{
  "permissions": {
    "allow": [
      "Skill(*)"
    ]
  }
}

However, the current schema's permissionRule pattern does not include Skill, causing validation warnings when users add Skill-related permission rules.

This PR adds Skill to the pattern to match the documented tool list.

Reference

Add Skill tool to the list of valid tools in the permissionRule pattern
to prevent validation warnings when using Skill permissions.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

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.

Copy link
Contributor

@domdomegg domdomegg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM!

@domdomegg
Copy link
Contributor

LGTM

@github-actions github-actions bot merged commit c76669f into SchemaStore:master Jan 7, 2026
5 of 6 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

Merging because @domdomegg is a code-owner of all the changes - thanks!

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.

2 participants