Skip to content

docs: deprecate "Skill" in allowed_tools in favor of skills option#924

Merged
qing-ant merged 1 commit intomainfrom
qing/deprecate-skill-allowed-tools
May 6, 2026
Merged

docs: deprecate "Skill" in allowed_tools in favor of skills option#924
qing-ant merged 1 commit intomainfrom
qing/deprecate-skill-allowed-tools

Conversation

@qing-ant
Copy link
Copy Markdown
Contributor

@qing-ant qing-ant commented May 6, 2026

Summary

Adds a deprecation note in src/claude_agent_sdk/types.py documenting that passing "Skill" in allowed_tools (and in AgentDefinition.tools) is deprecated. Users should use the skills option on ClaudeAgentOptions instead, which provides more granular control over which skills are available.

This is a docs-only change — no behavior changes.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

LGTM — docs-only deprecation notes, no behavior change.

Extended reasoning...

Overview

This PR adds two documentation-only changes to src/claude_agent_sdk/types.py: an inline comment on AgentDefinition.tools and a .. deprecated:: block in the ClaudeAgentOptions.allowed_tools docstring, both noting that passing "Skill" is deprecated in favor of the skills option. No code paths, type signatures, defaults, or runtime behavior are touched.

Security risks

None. The change consists entirely of comments and docstrings; there is no executable code, no input handling, and no auth/permissions logic affected.

Level of scrutiny

Low. This is a mechanical docs update that aligns with the already-documented behavior of the skills option (whose existing docstring already states that setting it auto-configures the Skill tool and that adding "Skill" to allowed_tools is unnecessary). The deprecation notes are consistent with that.

Other factors

No bugs were found by the bug hunting system, there are no outstanding reviewer comments, and the diff is small and self-contained. The reST .. deprecated:: directive is well-formed and the inline comment placement is conventional.

@qing-ant qing-ant enabled auto-merge (squash) May 6, 2026 23:32
@qing-ant qing-ant merged commit a727f82 into main May 6, 2026
11 checks passed
@qing-ant qing-ant deleted the qing/deprecate-skill-allowed-tools branch May 6, 2026 23:43
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