refactor: restructure skill into 7 thematic sub-skills#17
Merged
Conversation
Replace flat skill/ directory (one SKILL.md + 14 sibling .md files) with skills/<theme>/SKILL.md layout where each sub-skill is a self-contained Agent Skill with its own frontmatter and routing. Pattern matches the coreyhaines31/marketingskills repo. Themes: - soroban: contract dev + testing + security + advanced patterns + pitfalls - dapp: frontend + wallets (Freighter, Wallets Kit) + signing + smart accounts - assets: Stellar Assets + trustlines + SAC bridge - data: Stellar RPC + Horizon + indexing - agentic-payments: x402 (paid APIs) + MPP (Charge + Channel) - zk-proofs: ZK verification, BLS12-381, BN254/Poseidon (status-sensitive) - standards: SEPs + CAPs + ecosystem + curated resources Cross-references between sub-skills use relative ../<name>/SKILL.md paths; in-file self-references (within merged skills like soroban) rewritten to "Part N" descriptions to avoid redundant self-links. Updated plugin.json to point at ./skills/ (plural) and refreshed README to document the new structure.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the repository from a single flat skill/ into multiple thematic, self-contained Agent Skills under skills/<theme>/SKILL.md, updates cross-skill routing links, and repoints the Claude plugin configuration to the new directory.
Changes:
- Introduces 7 theme skills (
soroban,dapp,assets,data,agentic-payments,zk-proofs,standards) each with its own frontmatter. - Rewrites internal cross-references to point to
../<theme>/SKILL.mdentrypoints. - Updates
.claude-plugin/plugin.jsonandREADME.mdto reference./skills/(plural).
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/zk-proofs/SKILL.md | Adds skill frontmatter + routing header; updates “starting points” links to new skill paths. |
| skills/standards/SKILL.md | New consolidated standards/ecosystem/resources reference skill with cross-skill routing. |
| skills/soroban/SKILL.md | New large Soroban skill consolidating contract dev, testing, security, patterns, pitfalls. |
| skills/data/SKILL.md | Adds skill frontmatter + related-skill routing; updates a cross-reference to the new dapp skill path. |
| skills/dapp/SKILL.md | Adds skill frontmatter + related-skill routing; updates a cross-reference to the new data skill path. |
| skills/assets/SKILL.md | Adds skill frontmatter + related-skill routing for assets/trustlines/SAC. |
| skills/agentic-payments/SKILL.md | New agentic payments skill covering x402 and MPP charge/channel flows. |
| skill/x402.md | Removed legacy flat-file doc in favor of skills/agentic-payments/SKILL.md. |
| skill/testing.md | Removed legacy flat-file doc in favor of skills/soroban/SKILL.md. |
| skill/standards-reference.md | Removed legacy flat-file doc in favor of skills/standards/SKILL.md. |
| skill/SKILL.md | Removed legacy monolithic skill entrypoint in favor of themed skills. |
| skill/security.md | Removed legacy flat-file doc in favor of skills/soroban/SKILL.md. |
| skill/resources.md | Removed legacy flat-file doc in favor of skills/standards/SKILL.md. |
| skill/mpp.md | Removed legacy flat-file doc in favor of skills/agentic-payments/SKILL.md. |
| skill/ecosystem.md | Removed legacy flat-file doc in favor of skills/standards/SKILL.md. |
| skill/contracts-soroban.md | Removed legacy flat-file doc in favor of skills/soroban/SKILL.md. |
| skill/common-pitfalls.md | Removed legacy flat-file doc in favor of skills/soroban/SKILL.md. |
| skill/advanced-patterns.md | Removed legacy flat-file doc in favor of skills/soroban/SKILL.md. |
| README.md | Updates documentation to describe the new skills/ structure. |
| .claude-plugin/plugin.json | Repoints plugin skill root from ./skill/ to ./skills/. |
Comments suppressed due to low confidence (2)
skills/data/SKILL.md:431
- The link target correctly points to the
dappskill, but the link text still saysfrontend-stellar-sdk.md(a file that was removed in this restructure). Rename the link text to match the new skill name/path (e.g., “dapp skill”) so the reference is self-consistent.
skills/dapp/SKILL.md:56 - This reference now links to
../data/SKILL.md, but the link text still saysapi-rpc-horizon.md(from the old flat-file layout). Update the link text to reflect the new structure (e.g., “data skill” / “RPC + Horizon skill”) to avoid pointing readers to a non-existent filename.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This was referenced May 20, 2026
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.
Replace flat skill/ directory (one SKILL.md + 14 sibling .md files) with skills//SKILL.md layout where each sub-skill is a self-contained Agent Skill with its own frontmatter and routing. Pattern matches the coreyhaines31/marketingskills repo.
Themes:
Cross-references between sub-skills use relative ..//SKILL.md paths; in-file self-references (within merged skills like soroban) rewritten to "Part N" descriptions to avoid redundant self-links.
Updated plugin.json to point at ./skills/ (plural) and refreshed README to document the new structure.