feat(skills): add plugin-style skill system for dynamic instruction l…#292
Closed
likebeans wants to merge 1 commit into
Closed
feat(skills): add plugin-style skill system for dynamic instruction l…#292likebeans wants to merge 1 commit into
likebeans wants to merge 1 commit into
Conversation
…oading Add a skill system inspired by GitHub Copilot / Cursor Rules / AGENTS.md that allows agents to dynamically discover, load, and execute structured instructions and scripts. Core components: - SkillRegistry: multi-directory scanning with hierarchical override - SkillLoader: progressive/lazy content loading with 32KB cap - SkillMatcher: keyword, path glob, and extensible semantic matching - ResourceResolver + ScriptExecutor: resource file access and safe script execution - SkillSystem facade + 6 LangChain tools for agent integration Includes: - Example "git_operations" skill with commit check script and templates - Streamlit sidebar "Skills" popover showing skill load status - 25 unit tests covering all components - Documentation in docs/Skill_System.md
JoshuaC215
pushed a commit
that referenced
this pull request
Jun 30, 2026
- #305: cut the long defense-in-depth restatement (the PR already states it) down to just the positive + negative test ask; strengthen SKILL rule 6 — never explain back a caveat the contributor already raised. - #307: reviewed the diff — connection-string format is correct (?/& handled, tls=true is right); dropped the test ask; only request is default=False. Noted minor nits. - #301/302: decided — answer on #301, close both. - #292: Joshua responding manually (reference draft only). - Add pr-299-review-prompt.md: copy-paste agent prompt to deep-review/ test/validate PR #299, plus an explanation of why the GitHub UI shows no run-CI option (no workflow_dispatch; first-time-contributor approval gate; 0 check runs). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ALYJSRixXCfLsmb2Koc1Hi
Owner
|
Hey, thanks for putting this together @likebeans , unfortunately a dynamic skill system is beyond the scope of what I want to support with this project. If you have other ideas, I would recommend to file an issue first and we can discuss so you aren't spending time (or burning tokens) on implementation that won't be used. Thanks! |
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.
Summary
Add a plugin-style skill system inspired by GitHub Copilot / Cursor Rules / AGENTS.md that allows agents to dynamically discover, load, and execute structured instructions and scripts.
Core Components
What's Included
src/skills/— Full skill system moduleTesting