-
Notifications
You must be signed in to change notification settings - Fork 85
feat: Add support for skills #145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 6 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
6c60082
docs: Add a feature proposal document for the inclusion of skills sup…
patricka3125 50fd10c
slight doc restructure
patricka3125 716ddb6
update design
patricka3125 a1b45ea
implement phase 1
patricka3125 e48de90
Implement all phases
patricka3125 92c0601
trim out impl plan from docs
patricka3125 6573ebb
address PR #145 review comments: clarify skill catalog instruction, r…
patricka3125 765c3a0
refactor: replace loaded_profile passthrough with skill_prompt string
patricka3125 2c2a081
revert example profile changes, defer to follow-up PR
patricka3125 37653a1
rename skills design doc
patricka3125 a57396b
add E2E tests for skill catalog injection and skill API endpoint
patricka3125 704bdeb
remove Claude Code from skill injection E2E tests
patricka3125 6047c41
fix CI failures: black formatting and FastMCP 3.x compatibility
patricka3125 d619005
fix CI: isort ordering and FastMCP 3.x get_tools removal
patricka3125 9568df8
nit: change 'default' to 'builtin' in init skill seeding message
patricka3125 ff96cf1
fix test assertion to match 'builtin' wording change
patricka3125 deefd35
Merge branch 'main' into feat/skills
haofeif 7399a84
Merge branch 'main' into feat/skills
haofeif b518d43
Merge branch 'main' into feat/skills
haofeif d7bd380
refactor global skill catalog cleanup
patricka3125 6ac3576
polish phase 1 scope 1 followups
patricka3125 a5b1fb7
add skill injection utilities
patricka3125 1a02478
harden skill injection refresh handling
patricka3125 a6b067b
move skill catalog builder into utils
patricka3125 9a0030a
bake skill catalog into installed agent json
patricka3125 6534477
refresh installed agents after skill changes
patricka3125 5f3c2b5
add .worktrees/ to .gitignore
patricka3125 96a2e7c
clean up type ignores and remove development-cycle test artifacts
patricka3125 388c36a
move misplaced tests and hoist inline imports to module level
patricka3125 f1cab16
address PR review comments: revert out-of-scope changes, add docs
patricka3125 433d9d8
add skill catalog injection support for Copilot CLI
patricka3125 ae63496
refactor Kiro to use native skill:// resources instead of baked prompt
patricka3125 04458e6
revert mypy.ini changes: out of scope for this PR
patricka3125 3eed1d4
revert out-of-scope changes from feature branch
patricka3125 fa6977b
add .worktrees/ to .gitignore
patricka3125 cee2ca8
Merge branch 'main' into feat/skills
patricka3125 a7a0bf3
sync TerminalView.tsx with upstream/main
patricka3125 9347508
address PR nit comments: trim install comment, update Kiro references
patricka3125 d1a04ce
use ** glob for Kiro skill:// resources to match documented convention
patricka3125 a899573
add a skills guide
patricka3125 138e882
fix skills guide: skills are global, not per-profile
patricka3125 951fa19
sync api/main.py with upstream, keep only skills changes
patricka3125 41b4872
update skill guide
patricka3125 82752d7
nit: change get_skill tool to renamed "load_skill"
patricka3125 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.