Content
46%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 a well-sectioned but oversized monolith: highly concrete and executable example code, undermined by severe verbosity and zero use of progressive disclosure. Its nominal TDD workflow is nominal in name only, with the green/refactor steps left as comment stubs and no validation checkpoints anywhere.
Suggestions
Split the body into one-level-deep reference files (e.g. references/factories.md, references/api-testing.md, references/mocking.md) and keep SKILL.md as a concise overview with clearly signaled links.
Cut the body to the distinctive guidance — test settings, conftest fixture patterns, factory conventions — and drop generic test-suite examples Claude can generate itself.
Make the Red-Green-Refactor cycle an actual sequenced workflow with explicit checkpoints (write failing test → run pytest to confirm RED → implement → re-run to confirm GREEN → refactor with tests passing), including a validate-and-retry loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~700-line body reproduces standard pytest-django test-suite boilerplate for a hypothetical products app (model, view, serializer, viewset, mocking, integration sections) — patterns Claude already knows and could generate unprompted. This matches 'noticeably verbose; several unnecessary... padded sections'; it is not a 1 because there is no conceptual prose padding (no 'what is Django' explanations), only excess example volume. | 2 / 5 |
Actionability | Guidance is mostly executable: complete pytest.ini, test settings, conftest fixtures, factory definitions, and copy-paste-style test classes with concrete asserts. Minor gaps keep it from 5: examples depend on undefined fixtures ('category', 'product', 'order', 'TagFactory'), 'patch' is unimported in test_integration.py, and everything references a nonexistent 'apps.products' app. | 4 / 5 |
Workflow Clarity | The Red-Green-Refactor cycle is presented as the workflow, but the GREEN and REFACTOR steps are literal comment stubs ('# GREEN - テストを通す'), and the rest of the document is topic-organized rather than sequenced, with no validation checkpoints or run-the-tests feedback loops. This fits the 3 anchor ('sequence present but checkpoints missing'); not 4, which requires most checkpoints to be explicit. | 3 / 5 |
Progressive Disclosure | No references/, scripts/, or assets/ directories exist; the entire body is one monolithic inline document where large blocks (factory reference, full API test suites, coverage tables) clearly belong in separate one-level-deep reference files. This matches the 2 anchor ('content that clearly belongs in separate files is inlined') — not 1 because section headers do provide navigable structure. | 2 / 5 |
Total | 11 / 20 Passed |