Content
81%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 strongly structured, highly actionable test-generation skill with an excellent phased workflow and reference routing. Its weaknesses are moderate redundancy (mocking and failure-mode content repeated across sections) and ten dead links to shared/base/* and shared/workflow-contract.md that do not exist in the bundle.
Suggestions
Remove or repair the dead bundle links: the nine shared/base/*.md policy links and shared/workflow-contract.md point to files that do not exist, so either ship them in the bundle or delete the Base Framework and Workflow Integration boilerplate sections.
Consolidate the triplicated mocking guidance (Phase 4 'Mocking discipline', Phase 5, and Failure Mode 1 'Over-mocking') into the single Phase 5 section that already points to references/mocks-fixtures.md.
Trim the Failure Modes section of entries that restate Core Principles (coverage-chasing = Principle 5, invented expected values = Principle 2) and drop the Routing Boundary section that duplicates the frontmatter description.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core policy content (never-invent-requirements gate, ASSUMPTION labeling, untestable-code protocol) is dense and original, but mocking is covered three times (Phase 4, Phase 5, Failure Mode 1), coverage-chasing and invented-values repeat Core Principles 5 and 2, Routing Boundary restates the frontmatter description, and the 9-link Base Framework block is boilerplate — more than the 'minor trimmable material' of anchor 4, though the padding is still a minority of the body. | 3 / 5 |
Actionability | The body gives concrete, executable guidance throughout: file-signal framework detection (package.json with jest/vitest, pyproject.toml with pytest, Cargo.toml), phase-by-phase planning tables, a quality-review checklist, and a complete, runnable Jest worked example (including toBeCloseTo for floating-point precision) that covers the common case end-to-end. | 5 / 5 |
Workflow Clarity | Six phases in strict order with an explicit gate ("Do not produce test code before completing Phases 1–3"), a Phase 6 validation checklist, and error-recovery feedback loops (ASSUMPTION labeling feeding the Risks report; untestable code feeding refactoring recommendations) match anchor 5's clear sequence, explicit validation, and checklists for complex processes. | 5 / 5 |
Progressive Disclosure | Reference routing is well-signaled and one level deep via the detection and test-type tables, and all 12 referenced references/*.md files exist; however, the 'Required packaged policies' links to shared/base/*.md and shared/workflow-contract.md are dead (no shared/ directory in the bundle), and the Base Framework boilerplate is inlined in SKILL.md, exceeding anchor 4's 'minor organization gaps' bar for a score of 5. | 4 / 5 |
Total | 17 / 20 Passed |