feat: skill registry + Claude Code native discovery#4
Open
rtack wants to merge 6 commits intocolmarius:mainfrom
Open
feat: skill registry + Claude Code native discovery#4rtack wants to merge 6 commits intocolmarius:mainfrom
rtack wants to merge 6 commits intocolmarius:mainfrom
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…inks Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
b987d3f to
d0d8cdb
Compare
Rename post-sync.sh to setup-claude-integration.sh to make its Claude-specific purpose explicit. De-Claude log messages in the generic sync scripts, fix check-registry.sh to use trap for safe REGISTRY.json restoration, and remove duplicate --force flag from sync-local.sh (--yes is sufficient). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Highlight Amp (original platform) and Claude Code native integration so users discover agent-specific features without digging into the skills reference. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Owner
|
Thanks Raphael for opening the PR with support for claude code. Will get to review and integrate it sometimes ~mid week. |
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
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
REGISTRY.json): machine-readable index of all skills, auto-generated from SKILL.md frontmatter viagenerate-registry.sh. Agent-system agnostic — works with Claude Code, ampcode, Cursor, or anything that reads JSON.install.shnow detects.claude/directories and creates per-skill symlinks (.claude/skills/adapt/SKILL.md→.agents/skills/adapt/SKILL.md) so skills appear in Claude Code's/slash command menu natively.triggers,keywords,invocation— used by the registry for programmatic discovery. Backward-compatible: Claude Code ignores unknown fields (verified experimentally).What changed
REGISTRY.json,generate-registry.shadapt/,ralph/,research/,tmux/triggers,keywords,invocationfrontmatterinstall.shsetup_claude_code_integration()— creates.claude/skills/symlinks, cleanup on--uninstalltest/integration/install.batsdocs/skills.md,CHANGELOG.mdHow Claude Code integration works
install.shchecks if.claude/directory exists (auto-detect).claude/skills/*/SKILL.md→../../../.agents/skills/*/SKILL.md.claude/skills/.gitignorewith explicit skill names (user-created skills stay tracked)--sync, cleaned up on--uninstallVerification
Tested in a controlled environment (
~/dev/test-skill-verify) across 4 experiments:.agents/skills/natively discovered?.claude/skills//menu?Test plan
bash -n install.sh— syntax check passesreadlink+catthrough symlink)--dry-runshows integration plan without creating files--uninstallremoves only dot-agents symlinks, preserves user skillsln -sf).claude/skills/are skipped with warning.claude/directory (should skip silently)🤖 Generated with Claude Code