Add Soroban Common Mistakes to community skills - #86
Conversation
There was a problem hiding this comment.
Pull request overview
Adds the Soroban Common Mistakes security-review skill to the community directory.
Changes:
- Adds skill metadata, security topics, languages, and external source link.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
# Conflicts: # site/src/data/skills.ts
|
Thanks for this, Maria — and for following up here like I'd suggested. This is a nice addition to the catalog: I like that the checklist is Soroban-native rather than a ported EVM list (calling out storage type/TTL and reinitialization instead of reentrancy/tx.origin is exactly the right framing), and the vulnerable/fixed vault pair in One thing before I add it: the card description (and the PR description) say the skill checks "22 recurring security mistakes," but the skill itself consistently says 23 — Everything else checked out — the repo's public, the SKILL.md frontmatter is self-contained, the checklist/PR-template/Scout workflow files it references all exist, and the companion-tool links (Scout, OpenZeppelin detectors, Komet, Certora Sunbeam) resolve. Once the count's consistent I'm happy to get this listed. |
Adds the
soroban-common-mistakesskill to the community skills directory. It reviews Soroban contracts against 22 common security mistake patterns, cross-checked against the official Stellar docs.Part of a broader open security toolkit for LATAM builders, with the full guide available in both English and Spanish.
@kaankacar following up on your suggestion to submit this here = )