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.
A well-structured skill body: an overview with executable quick commands, explicit lint feedback loops, and an exemplary one-level-deep reference layout backed by a real bundle. Weakest spots are repo-specific or nonexistent script paths in the commands, a vague final workflow step, and a few sections of trimming candidates.
Suggestions
Replace repo-specific and unverifiable commands ('sh skills/agentic-harness/skill-quality-auditor/scripts/evaluate.sh ...', 'bun run check-style ...') with portable equivalents, or document the exact preconditions for those paths.
Give workflow step 5 a concrete validation command (e.g. the actual pre-commit or CI check invocation) instead of the vague 'Validate CI/pre-commit integration for regression prevention'.
Trim the abstract 'Principles' section and move the 'Consider adding a table of contents...' paragraph out of the Anti-Patterns block into a style reference file to reduce token cost.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence (no explanation of what Markdown or linting is), but the abstract 'Principles' list, the 'Consider' paragraph awkwardly placed inside Anti-Patterns, and the self-referential 'Evaluate this skill quality' command are tokens that could be trimmed, matching anchor 4 rather than a fully lean 5. | 4 / 5 |
Actionability | Quick Commands are copy-paste ready with expected results ('bunx markdownlint-cli2 "**/*.md"') and anti-patterns include concrete BAD/GOOD pairs, but several commands reference repo-specific paths ('sh skills/agentic-harness/...', 'skills/documentation/...') and 'bun run check-style' invokes a script that does not exist in the bundle — minor gaps placing this at anchor 4. | 4 / 5 |
Workflow Clarity | The Deterministic Workflow has a clear sequence with an explicit lint-fix feedback loop ('lint fails → identify rule → fix source or add justified exception → re-lint → proceed only when clean') and a 'Do not proceed until lint is clean' checkpoint, but step 5 ('Validate CI/pre-commit integration for regression prevention') lacks any concrete check or command — a minor validation gap that fits anchor 4 better than 5. | 4 / 5 |
Progressive Disclosure | Verified against the actual bundle: all 16 reference links ('references/syntax-headings.md' through 'references/lint-api.md') point to real files exactly one level deep, grouped into clear syntax/docs/lint categories, supplemented by a working './scripts/read-reference.sh' navigation script, with no bulk detail inlined in SKILL.md — a clear match for anchor 5. | 5 / 5 |
Total | 17 / 20 Passed |