Content
50%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill is highly actionable with copy-paste grep/CLI commands and explicit kill-criteria checkpoints, but the large Foundry POC template contains incomplete placeholders, the end-to-end workflow is implicit rather than numbered, and the referenced deep-dive files do not exist in the bundle.
Suggestions
Complete the Foundry POC template: replace `// ... buy tokens on Uniswap`, `// Add liquidity...`, and `// Assert: received almost nothing` placeholders with actual swap/assertion calls so the code is copy-paste executable.
Add an explicit numbered end-to-end audit workflow (e.g., 1. Run pre-dive kill signals -> 2. Audit each bug class via grep + Kill-if -> 3. Run token_scanner.py -> 4. Verify on-chain authorities) to make the sequence and checkpoints unambiguous.
Either create the referenced `web3/10-meme-coin-bugs.md`, `web3/11-solana-token-audit.md`, and `web3/12-dex-lp-attacks.md` files in a `references/` bundle, or remove the 'FULL REFERENCE FILES' section so the body does not promise deep-dive files that do not exist.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and action-oriented with little concept fluff, but at ~290 lines it carries an 80-line Foundry POC template plus repeated grep blocks that could be tightened; not every token earns its place. | 2 / 3 |
Actionability | Most of the body is copy-paste ready (grep patterns, `python3 tools/token_scanner.py ...`, `solana account <MINT> --output json`), but the marquee Foundry POC has incomplete placeholders like `// ... buy tokens on Uniswap` and `// Assert: received almost nothing`, leaving key details missing. | 2 / 3 |
Workflow Clarity | Validation checkpoints exist (pre-dive kill signals, per-class 'Kill if' criteria, the numbered Solana quick checks), but the overall end-to-end audit sequence is implicit rather than a numbered procedure and there is no validate-fix-retry feedback loop. | 2 / 3 |
Progressive Disclosure | The 'FULL REFERENCE FILES' section clearly signals one-level-deep references to `web3/10-meme-coin-bugs.md`, `web3/11-solana-token-audit.md`, and `web3/12-dex-lp-attacks.md`, but none of these files exist in the bundle (no references/ dir), so the disclosure is only partially realized and much detail stays inline. | 2 / 3 |
Total | 8 / 12 Passed |