Content
0%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is a verbose, abstract description of London School TDD concepts and fictional 'swarm coordination' patterns rather than actionable instructions. It explains concepts Claude already knows (mocking, behavior verification, outside-in TDD), uses fictional APIs that don't correspond to real tools or libraries, and lacks any concrete workflow or validation steps. The content reads more like a tutorial article than an operational skill.
Suggestions
Replace abstract pattern descriptions with a concrete, step-by-step workflow: e.g., '1. Read the task, 2. Write failing acceptance test, 3. Create mocks for collaborators, 4. Implement to pass, 5. Verify all mock expectations met'
Remove explanations of basic TDD/mocking concepts Claude already knows and focus only on project-specific conventions, tool configurations, or non-obvious constraints
Replace fictional APIs (swarmCoordinator, SwarmContractMonitor, createSwarmMock) with real tool invocations or remove them entirely if no actual swarm coordination tooling exists
Add explicit validation checkpoints (e.g., 'Run `npm test` after each red-green-refactor cycle and verify all mocks are satisfied before proceeding')
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Extremely verbose at ~200+ lines. Explains basic TDD concepts Claude already knows (what outside-in TDD is, what mocks are, what behavior verification means). The 'Core Responsibilities' section is pure padding. Many code examples are repetitive and illustrate the same concept (mock setup and verification) multiple times. The 'swarm coordination' concept is vaguely defined with pseudo-APIs (swarmCoordinator, SwarmContractMonitor) that don't correspond to real tools. | 1 / 3 |
Actionability | Despite having many code examples, none are truly executable or copy-paste ready. Functions like `createSwarmMock`, `extendSwarmMock`, `SwarmContractMonitor`, `swarmCoordinator.notifyTestStart` are fictional APIs with no implementation or library reference. The skill describes abstract patterns rather than providing concrete, actionable instructions for a specific task. There are no real commands, tool invocations, or specific steps Claude should follow. | 1 / 3 |
Workflow Clarity | No clear workflow or sequenced steps for Claude to follow. The content is organized as a reference document of patterns rather than a process. There are no validation checkpoints, no error recovery steps, and no clear sequence of actions. The numbered sections (1, 2, 3) describe concepts rather than workflow steps. | 1 / 3 |
Progressive Disclosure | Monolithic wall of text with no references to external files and no bundle files to support it. All content is inline with no clear hierarchy or navigation structure. The document could benefit enormously from splitting mock patterns, swarm coordination, and best practices into separate referenced files. | 1 / 3 |
Total | 4 / 12 Passed |