Content
68%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, appropriately concise skill body with concrete commands and a realistic test template. Its weakest point is workflow clarity: it never instructs the agent to run the generated tests and iterate on failures, leaving the core feedback loop implicit. The code template also relies on placeholders that could be more fully specified.
Suggestions
Add a validation loop to the workflow: after writing the test file, run `npm run test:run`, fix failing tests, and re-run until all tests pass before finishing.
Make the test template more complete — show how defaultProps is defined and use a real role/query (e.g., getByRole('button')) instead of '...' and '// Accessibility tests' placeholders.
Merge the duplicated accessibility guidance: fold 'Accessibility Testing' (section 5) into the 'What to Test' list to remove the repeated ARIA/keyboard/focus items.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and project-specific ('Tests go in `ComponentName.test.tsx` in the component directory', concrete npm commands) with only minor trimmable redundancy — accessibility guidance is duplicated between the 'What to Test' bullet and the 'Accessibility Testing' section — so it fits 'efficient with minor instances' rather than the every-token-earns-its-place bar of 5. | 4 / 5 |
Actionability | Gives concrete, runnable commands (npm run test:run / test:coverage) and a real import/test skeleton, but the template contains placeholders ({...defaultProps}, getByRole('...'), '// Accessibility tests') that keep it short of copy-paste-ready, matching 'mostly executable guidance with minor gaps'. | 4 / 5 |
Workflow Clarity | The numbered instructions form a clear sequence (location, setup, structure, coverage, running), but there is no validation checkpoint — no run-tests, fix-failures, re-run loop — leaving checkpoints implicit, which matches anchor 3 rather than 4. Test generation is neither destructive nor batch, so no lower cap applies. | 3 / 5 |
Progressive Disclosure | Well-organized sections with nothing inlined that belongs in a separate file, and a single one-level, clearly signaled reference to existing patterns ('packages/components/src/*/ComponentName.test.tsx'). The thin Examples section and length just over the simple-skill bar keep it at 'good structure with minor gaps' rather than 5. | 4 / 5 |
Total | 15 / 20 Passed |