Content
50%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 compact, well-sectioned skill with usable code and a clear when/not-when split, but it is held back by malformed code fences, refactor-phase content mixed into a green-phase skill, an unverifiable resource reference, and no explicit failure-feedback loop.
Suggestions
Fix the malformed code fences: open each Django block (def product_list, class ProductListView(ListView)) with its own ```python fence so the examples render and copy-paste correctly.
Move the class-based-view and generic-view refactor examples into resources/implementation-playbook.md and keep SKILL.md focused on the minimal green-phase implementation, then ensure the referenced file actually exists.
Add an explicit validation feedback loop to the Instructions, e.g. 'If tests still fail, re-read the assertion, adjust the minimal change, and re-run until green' to lift workflow clarity above 3.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean, but the class-based and generic-view refactor examples belong to the refactor phase rather than green, adding tokens that could be trimmed or moved to the playbook. | 3 / 5 |
Actionability | Concrete code is provided, but the Django code fences are malformed (missing opening fences) and the examples are illustrative rather than directly executable in context, leaving key details incomplete. | 3 / 5 |
Workflow Clarity | A four-step sequence with a 'run tests after each change' checkpoint exists, but there is no explicit validate-on-failure / fix-and-retry feedback loop, so checkpoints remain implicit. | 3 / 5 |
Progressive Disclosure | Sections are present and there is a one-level reference to resources/implementation-playbook.md, but no bundle file exists to verify it and refactor content is inlined that arguably belongs in the referenced file. | 3 / 5 |
Total | 12 / 20 Passed |