Content
85%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, actionable instruction skill with a clear validated workflow and tidy sectioning. The main improvement opportunity is trimming the motivational philosophy bullets for token efficiency and adding one complete executable test example.
Suggestions
Remove or compress the 'Philosophy' bullets ('Untested code is broken code', etc.) — these restate testing principles Claude already knows and add tokens without actionable guidance.
Add one complete, copy-paste-ready MockK + runTest unit test example in the Examples section so BUILD is fully executable rather than pattern-described.
Consider collapsing the emoji-laden 'Always/Ask first/Never' headers into tighter bullet groups to reduce visual/token overhead while preserving the guardrails.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly lean and well-organized, but the 'Philosophy' bullets ('Untested code is broken code', 'A failing test is a feature') are motivational padding Claude already knows and could be trimmed; not enough to drop to 3. | 4 / 5 |
Actionability | Provides concrete commands (`./gradlew ktfmtFormat`, `./gradlew testDebugUnitTest --tests ...`), specific patterns (mockk(), runTest, given-when-then test naming, `test:` PR prefix); minor gap is the absence of a full copy-paste test example. | 4 / 5 |
Workflow Clarity | SCAN/SELECT/BUILD/VERIFY/PRESENT is a clear sequence with an explicit validation checkpoint (VERIFY runs ktfmtFormat and the specific test, and applies a mutation-testing fail-then-pass check), plus Always/Ask first/Never guardrails. | 5 / 5 |
Progressive Disclosure | Under 50 lines with no external references needed; content is cleanly partitioned into Goal, Constraints, Instructions, and Examples sections, satisfying the simple-skill exception for progressive disclosure. | 5 / 5 |
Total | 18 / 20 Passed |