Replace Claude skills generation with path-scoped rules#84
Replace Claude skills generation with path-scoped rules#84Kush Agrawal (kush-duolingo) wants to merge 1 commit into
Conversation
Skills follow the Agent Skills open standard (agentskills.io) and should be authored directly in .agents/skills/ -- no sync needed. This tool now focuses on rules (guidance/conventions) which still need per-tool sync. - Replace SkillsGenerator (.claude/skills/generated_*/SKILL.md) with ClaudeRulesGenerator (.claude/rules/generated/*.md with paths frontmatter) - Add symlink from .claude/skills/agents to .agents/skills/ so Claude Code discovers skills from the standard location - Update plugins.yaml, test fixtures, and README Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Kush Agrawal (kush-duolingo)
force-pushed
the
replace-claude-skills-with-rules
branch
from
06d7b1b to
db4fa05
Compare
Kush Agrawal (kush-duolingo)
requested review from
Art Chaidarun (artnc),
Carter L (clevin95) and
guadalupe-duolingo
Carter L (clevin95)
left a comment
Carter L (clevin95)
left a comment
There was a problem hiding this comment.
Approving under the assumption that we will convert most of our cursor rules to be agent skills before bumping the version.
We should make sure Android does the same before adopting this change.
Art Chaidarun (artnc)
left a comment
Art Chaidarun (artnc)
left a comment
There was a problem hiding this comment.
^ Speaking just as a repo maintainer without strong opinions about AI rule syncing: how backward-compatible is this PR (if that's what Carter L (@clevin95)'s comment is about)? I see that it's already doing some auto-migration, e.g. deleting old generated_*
guadalupe-duolingo
left a comment
guadalupe-duolingo
left a comment
There was a problem hiding this comment.
Thanks for this! Will leave to other reviewers to approve.
Anyone who bumps to this version will need to (as part of the version bump PR):
So it's bit of a manual version bump. Let me know if that answers the qn! |
Art Chaidarun (@artnc) looks like you'll need to hit merge |
4 participants
Description
SkillsGenerator(.claude/skills/generated_*/SKILL.md) withClaudeRulesGenerator(.claude/rules/generated/*.md) that usespathsfrontmatter for file-scoped activation.claude/skills/agentsto.agents/skills/so Claude Code discovers skills from the standard Agent Skills location.agents/skills/following the open standard (supported by Cursor, Codex, Copilot, and 30+ tools) -- no sync neededContext: https://duolingo.slack.com/archives/C08TV2ZB4SE/p1778708293248929
Verification steps
make testpassespython3 -m sync_ai_rulesin a repo with.cursor/rules/and verify.claude/rules/generated/is populated withpathsfrontmatter