-
Notifications
You must be signed in to change notification settings - Fork 4k
feat: agent skills #9353
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
base: main
Are you sure you want to change the base?
feat: agent skills #9353
Conversation
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 issues found across 17 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="core/config/markdown/loadMarkdownSkills.ts">
<violation number="1" location="core/config/markdown/loadMarkdownSkills.ts:48">
P2: Magic number `slice(7)` is unclear and fragile. This appears to strip the `file://` URI scheme prefix, but should use a named constant or proper URI parsing utility for clarity and safety. Consider extracting this to a helper function or adding a comment explaining the intent.</violation>
</file>
<file name="core/llm/skills.ts">
<violation number="1" location="core/llm/skills.ts:20">
P2: Remove debug `console.log` statement before merging. The `debug1` prefix indicates this is temporary debugging code that should not be in production.</violation>
</file>
<file name="gui/src/redux/util/constructMessages.ts">
<violation number="1" location="gui/src/redux/util/constructMessages.ts:218">
P0: Critical bug: `systemMessageWithSkills` is computed but never used. `finalSystemMessage` is assigned `systemMessageWithRules` instead of `systemMessageWithSkills`, causing the skills feature to have no effect.</violation>
<violation number="2" location="gui/src/redux/util/constructMessages.ts:220">
P0: Same bug: should use `systemMessageWithSkills` instead of `systemMessageWithRules` to include skills when appending conversation summary.</violation>
</file>
Reply to cubic to teach it or ask questions. Tag @cubic-dev-ai to re-run a review.
|
I've created documentation for the agent skills feature in a separate branch: The documentation includes:
Since your PR is from a fork, I can't create a PR directly against your
Let me know which approach you prefer! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 13 files

Description
This PR introduces agent skills to the vscode/jetbrains extension.
References:
About agent skill: https://platform.claude.com/docs/en/agents-and-tools/agent-skills/overview
Integrating agent skill: https://github.com/agentskills/agentskills/blob/main/docs/integrate-skills.mdx
resolves CON-5136
AI Code Review
@continue-reviewChecklist
Screen recording or screenshot
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Adds agent skills to the VS Code/JetBrains extension so the model can discover project-specific skills and read detailed instructions via a new tool.
New Features
Migration
Written for commit e0753f5. Summary will update on new commits.