Content
86%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-structured, highly actionable skill body: copy-paste quick start, complete provider/env mapping, and genuinely non-obvious security-model guidance that is correctly kept inline while details are pushed to real, clearly-labeled reference files. The only notable gaps are mild redundancy in the security sections and the verification step not being wired into the quick-start sequence itself.
Suggestions
Add an explicit step 3 to the quick start: 'Run opensea wallet info and fix any warnings (missing per-tx caps, no mutation gate) before relying on the wallet' — wiring the existing validation into the sequence.
Merge the short 'Security' bullet section into the 'Security model' section (or the provider table's 'Best For' column) to remove the duplicated 'local dev only' warnings.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by tables, commands, and non-obvious security facts Claude cannot be assumed to know (no provider ships hardened; aggregate caps are not native; mutation gates per provider). Minor over-explanation remains: the 'Security' bullet section partially duplicates the provider table's 'Local dev/testing only' note, and the repo-placement mechanics in 'Where mutation recipes live' run longer than needed. Not a 5 because of these trimmable redundancies; well above a 3 because essentially nothing explains concepts Claude already knows. | 4 / 5 |
Actionability | The quick start gives copy-paste-ready commands (env exports plus a real 'opensea swaps execute' invocation with concrete flags), the provider table maps each provider to its exact env vars, explicit '--wallet-provider' commands cover the alternatives, and 'opensea wallet info' is given as a concrete check. The common case is fully executable end to end. | 5 / 5 |
Workflow Clarity | The quick start is clearly sequenced (pick provider and set env vars, then execute via a signing-capable command), and a validation checkpoint exists — 'Run opensea wallet info to see whether your wallet has these in place... prints loud warnings when the per-tx layer is missing'. It is not a 5 because that verification step sits deep in the security-model section rather than being integrated into the setup sequence as an explicit step with a fix-and-recheck loop. | 4 / 5 |
Progressive Disclosure | The SKILL.md is a clear overview with well-signaled, one-level-deep references that all exist in the bundle: wallet-setup.md (per-provider setup), wallet-policies.md (policy fields), wallet-funding.md (aggregate-cap pattern), each with a one-line description in the References section. The '../docs/policy-administration.md' reference is deliberately outside the skill mount and the body explains exactly why — a documented structural choice, not buried nesting. Inline content (security model, provider tables) is behavior-critical context that appropriately stays in SKILL.md. | 5 / 5 |
Total | 18 / 20 Passed |