Content
18%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is essentially a hollow shell — it has a title and topic list but contains no actual testing knowledge, patterns, code examples, or actionable guidance. All substance is deferred to a resource file that doesn't exist in the bundle. The instructions section consists of generic boilerplate that could apply to any skill and provides nothing specific to Python testing.
Suggestions
Add concrete, executable pytest examples covering the core patterns: basic test structure, fixtures, mocking with unittest.mock/pytest-mock, and parameterization — these are the minimum for a testing skill.
Replace the generic instruction bullets with a specific TDD workflow: write test → run and see failure → implement → run and verify pass → refactor, with actual pytest commands (e.g., `pytest -x -v test_module.py`).
Either include the referenced `resources/implementation-playbook.md` in the bundle or inline the essential patterns in the SKILL.md itself, since the current content provides zero actionable guidance.
Remove the padded 'Use this skill when' list (10 items) and 'Do not use this skill when' section — these consume tokens without adding value Claude can act on.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 'Use this skill when' and 'Do not use this skill when' sections are padded with obvious items Claude doesn't need. The instructions section is generic boilerplate ('Clarify goals, constraints, and required inputs') that adds no testing-specific value. However, it's not excessively verbose overall. | 3 / 5 |
Actionability | There is zero concrete, executable guidance — no code examples, no specific commands, no pytest patterns, no fixture examples, no mocking examples. The instructions are entirely vague platitudes like 'Apply relevant best practices and validate outcomes.' Everything actionable is deferred to a resource file that doesn't exist in the bundle. | 1 / 5 |
Workflow Clarity | There is no workflow, no sequenced steps, no validation checkpoints. The four bullet points in Instructions are generic meta-instructions that could apply to any skill, not a testing workflow. No TDD cycle, no test-run-fix loop, nothing specific to testing processes. | 1 / 5 |
Progressive Disclosure | The skill references `resources/implementation-playbook.md` for detailed patterns, which is a reasonable structure in theory. However, the bundle shows no such file exists, the SKILL.md itself contains essentially no substantive content to serve as an overview, and the reference is a single generic pointer rather than well-signaled topic-specific links. | 2 / 5 |
Total | 7 / 20 Passed |