Content
60%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 mutation-testing section is the strongest part — concrete commands, thresholds, a decision table, and a real feedback loop — but it crowds the body with concept explanation Claude does not need, while the skill's primary job of writing tests gets only abstract bullet guidance with no example test. Disclosure to the single reference file is handled well.
Suggestions
Trim the mutation-testing primer to the decision-relevant parts (when-to-apply table, score thresholds, workflow) and cut the concept explanation and inspiration quote — Claude already knows what mutation testing is.
Add one compact example test (e.g., a Vitest/Playwright snippet showing the mock-failure and deterministic-data patterns from 'Padroes de Teste') so the core unit/component/E2E guidance is executable, not just descriptive.
Move the per-language mutation tool table and the 'Plug no kit' suggestion template into the existing reference file (or a new one) to keep SKILL.md a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body spends paragraphs explaining mutation testing from first principles ('ferramenta gera "mutacoes" no codigo (troca `>` por `>=`, deleta linha, inverte boolean)...') and coverage semantics — concepts Claude already knows — plus an inspiration-quote section and an 8-row language/tool table that could be one line per ecosystem. Tables and checklists are otherwise dense and efficient, so it is mostly efficient with unnecessary explanation rather than severely padded. | 3 / 5 |
Actionability | The mutation-testing section is concrete (`npx stryker run`, tool install commands, score thresholds, a copy-paste SQLite WAL cleanup snippet), but the skill's core task — writing the unit/component/E2E tests — has zero example tests or commands: guidance like 'Escrever testes unitarios para hooks, stores e utils' and 'usar dados de teste simples e deterministicos' stays at the abstract level. Concrete guidance exists but key executable details for the primary workflow are missing. | 3 / 5 |
Workflow Clarity | The mutation-testing section gives a numbered 8-step workflow with explicit thresholds and CI cadence, and the QA checklist has a genuine validate-fix-retry loop ('rodar a suite → item que falhar, corrigir o teste ou reportar o bug → rodar de novo. Repetir...') plus a verification-before-completion gate reference. Not a 5 because the main test-writing workflow (entradas → escrita → cobertura → handoff) is scattered across sections without one explicit sequenced process with checkpoints. | 4 / 5 |
Progressive Disclosure | The single bundle reference (references/playwright-patterns.md) exists, is clearly signaled with a one-line content summary, and is one level deep; inline content like the decision tables and anti-rationalization list is legitimately core. Minor gaps: the ~60-line mutation-testing deep-dive could itself live in a reference file, and pointers to non-bundle kit files (policies/, docs/, personas/) are numerous and unevaluated from the bundle's perspective. | 4 / 5 |
Total | 14 / 20 Passed |