docs(designs): Add skills sdk design#528
Draft
mkmeral wants to merge 5 commits intostrands-agents:mainfrom
Draft
docs(designs): Add skills sdk design#528mkmeral wants to merge 5 commits intostrands-agents:mainfrom
mkmeral wants to merge 5 commits intostrands-agents:mainfrom
Conversation
added 4 commits
February 4, 2026 23:19
This design proposes adding a skills parameter to the Agent class for loading and managing AgentSkills.io compatible skills. Key features: - Simple API: Agent(skills='./skills') - Dynamic skill management between invocations - Active skill tracking - allowed_tools enforcement via hooks - Session manager integration Addresses: strands-agents/sdk-python#1181
- Follow documentation style guide (collaborative 'we', active voice) - Teach the concept before diving into API details - Show the problem first, then the solution - Use concrete examples throughout - Remove unnecessary complexity - Clearer structure following the design template
Key changes to the Skills SDK design: - Remove BeforeToolCallEvent hook approach for tool restrictions - Add pre-filtering mechanism: filter tools BEFORE sending to model - Model only sees tools it's allowed to use (cleaner, no wasted tokens) - Add detailed implementation section showing the filtering flow - Add 'Hook-Based Tool Enforcement' to Alternatives Considered section explaining why pre-filtering is the better approach The insight: why show the model tools it can't use? Pre-filtering is more efficient and requires no error recovery logic.
Less is more - the original doc is concise and matter-of-fact. Removed verbose explanations, ASCII diagrams, and repetition.
Contributor
Documentation Deployment FailedThe documentation deployment encountered an error. Please check the deployment logs for more details. |
- Add docstrings to API signatures (properties, class methods) - Document error handling: SkillLoadError, when it's raised - Clarify active_skill detection mechanism - Add comment on allowed_tools=None meaning - Show proper import patterns in module exports - Reference DECISIONS.md re: why Skill doesn't extend HookProvider - Document behavior for non-existent tools in allowed_tools (warn, don't fail)
Contributor
Documentation Deployment FailedThe documentation deployment encountered an error. Please check the deployment logs for more details. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
WIP