feat: move SkillsMiddleware to core deepagents package (Issues: #506) #511
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.
Summary
Closes #506
Moves
SkillsMiddlewarefromdeepagents-clito the coredeepagentspackage.This improves code reusability and allows the Skills system to be used in various environments.
Changes
deepagents/middleware/skills.pydeepagents/middleware/__init__.pySkillsMiddlewareexportdeepagents-cli/skills/*.pytests/unit_tests/test_middleware.pyUsage
Design Decisions
Export Scope
deepagents/__init__.py: No changesdeepagents.middleware: OnlySkillsMiddlewareexposeddeepagents.middleware.skills: Full API (list_skills,SkillMetadata, etc.)Backward Compatibility
All existing CLI import paths and behaviors are preserved:
~/.deepagents/{AGENT_NAME}/skills/) unchangedTesting
Unit Tests
pytest tests/unit_tests/test_middleware.py # 78 passedIntegration Tests (Actual Operation Verified)
deepagents skills list(User + Project)NO CLI (Core)
CLI
Checklist
Yo-sure/add-skills-to-core)