Content
36%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 contains genuinely useful London School TDD patterns but is padded with three overlapping coordination sections, opens with a junk duplicate frontmatter block, and leans on fictional swarm APIs that can't be executed. It also never gives the agent an actual ordered TDD workflow to follow.
Suggestions
Replace the topic-catalog layout with a sequenced workflow (e.g. 1. write failing acceptance test outside-in, 2. mock collaborator contracts, 3. drive implementation inward, 4. run the suite and refactor) with explicit validation checkpoints.
Remove or define the non-existent APIs ('createSwarmMock', 'swarmCoordinator', 'SwarmContractMonitor', 'toSatisfyContract', 'toHaveBeenCalledBefore') — either show their real implementations in a scripts/ file or drop those examples in favor of plain Jest.
Cut the duplicate YAML block at the top of the body (with its malformed '>$dev$null' shell) and merge the redundant 'Swarm Coordination Patterns', 'Swarm Integration', and 'Best Practices' sections into one, moving extended contract examples into a references/ file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~245-line body opens with a duplicate YAML frontmatter block (including malformed shell like '>$dev$null') and restates the same mock/contract/coordination material across 'Swarm Coordination Patterns', 'Swarm Integration', and 'Best Practices' — several noticeably padded sections matching anchor 2. It is not 3 because the redundancy is extensive rather than incidental, and not 1 because it does not spend space explaining concepts Claude already knows (it never defines what mocks or TDD are). | 2 / 5 |
Actionability | The core Jest examples (mockRepository definitions, toHaveBeenCalledWith assertions) are concrete, but a large share depends on undefined infrastructure — 'swarmCoordinator.notifyTestStart', 'createSwarmMock', 'extendSwarmMock', 'SwarmContractMonitor', 'toSatisfyContract', 'toHaveBeenCalledBefore' — none of which exist as shown. This matches anchor 3 (concrete guidance but incomplete / non-executable in key places): better than 2 because real executable patterns are present, worse than 4 because the missing definitions are more than minor gaps. | 3 / 5 |
Workflow Clarity | The body is organized as topic catalogs (methodology, patterns, strategies, integration) with no sequenced process — no red/green/refactor loop, no ordered outside-in steps, no validation checkpoints for when a test or contract fails. This matches anchor 2 (rough implied sequence, many gaps, poorly defined steps): the 'Outside-In Development Flow' section title gestures at a sequence without defining one, so it cannot reach 3, and it avoids 1 because the sections do imply a coherent overall direction. | 2 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are all absent), so everything is inline in one ~250-line file. Section headers are reasonable, but the methodology examples, contract-testing patterns, and swarm coordination material are all content that would belong in separate reference files, matching anchor 3 (some structure, content that should be separate is inline). Not 4 because at this length a SKILL.md overview plus one-level-deep references is clearly warranted; not 2 because the sections are clearly labeled and navigable. | 3 / 5 |
Total | 10 / 20 Passed |