Content
78%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.
The body is concise and highly actionable, with a clear sequenced workflow and validation checkpoints. Its main weakness is progressive disclosure: it points readers to rules/*.md and a contracts/docs style guide that do not exist in the skill bundle.
Suggestions
Create the referenced rules/*.md files (gas-optimization, natspec, file-layout, naming-conventions, imports, visibility, general-rules, commands) so the 'See rules/...' links resolve, or remove the dead links.
Verify the external reference '../../../contracts/docs/contract-style-guide.md' resolves from the skill's installed location, or replace it with an in-bundle reference.
Add an explicit feedback loop to the Development Workflow (e.g., 'If lint/test fails, fix and re-run before proceeding') to lift workflow clarity toward 5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and efficient with terse bullets, a priority table, and code blocks; it does not over-explain concepts Claude already knows. It sits above the 3 anchor (no unnecessary explanation) but the inline Quick Reference code blocks duplicate detail that is also promised in the rule files, keeping it just short of the 5 anchor's 'every token earns its place'. | 4 / 5 |
Actionability | Provides copy-paste-ready correct/incorrect Solidity examples and concrete executable commands ('pnpm -F contracts run build', 'lint:fix', 'test'), covering the common cases, matching the 5 anchor for fully executable guidance. | 5 / 5 |
Workflow Clarity | The 7-step Development Workflow sequences write -> compile -> lint -> test -> NatSpec/gas review -> commit, with build/lint/test commands and a commit checklist as checkpoints. It is below 5 because there is no explicit 'if it fails, fix and retry' feedback loop, and above 3 because most checkpoints (compile, lint, test, checklist) are present and concrete. | 4 / 5 |
Progressive Disclosure | The body is well-structured as an overview with one-level-deep links to rule files ('See [rules/gas-optimization.md](rules/gas-optimization.md)'), but no rules/, references/, scripts/, or assets/ directories exist, so the referenced files are missing. Per the 'score against the actual bundle structure' guideline, the broken/absent references cap this at 3. | 3 / 5 |
Total | 16 / 20 Passed |