Content
78%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, disciplined principles skill: every section states specific, non-obvious rules rather than restating what Claude already knows, and detail is correctly deferred to two clearly signaled, one-level-deep reference files with explicit read conditions. The main costs are density — several multi-clause bullets require re-reading — and a handful of abstract concepts that lack a short worked example to anchor their application.
Suggestions
Split the longest multi-clause bullets (the proof-boundary selection bullet in Test Selection) into 2-3 shorter rules so each directive is scannable on one line.
Add one brief worked example (a 3-4 line before/after test pair) illustrating 'one behavioral claim owned by one test' and 'proof ownership', since these are the most interpretation-heavy concepts.
Trim the motivational rationale in the opening paragraph to a single sentence; the debt-avoidance argument is inferable from the rules themselves.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and efficient with no explanation of concepts Claude already knows, and nearly every line states a non-obvious rule. A few sentences carry motivational rationale ("a brittle, duplicated, or implementation-shaped test therefore becomes a false requirement: it gets worked around") or pack multiple clauses into one bullet (the proof-boundary bullet), which could be trimmed or split, keeping it below the lean-every-token-earns-its-place level. | 4 / 5 |
Actionability | Most rules are concrete, directive, and executable as written — "Structure each test as Arrange, one Act, and Assert", "Mock direct external I/O boundaries; keep internal business logic and the boundary under test real", "Add a regression test for every reproducible behavior bug fix". A few abstract principles ("proof ownership", "keeps at most one representative case of a rule owned elsewhere") require interpretation without a worked example, which is the minor gap that keeps it from fully actionable coverage. | 4 / 5 |
Workflow Clarity | The content is clearly organized by concern in a logical order (selection, design, mocks, data layer, verification, organization, regression), and the Test Selection section embeds a usable decision order (outcome → required proof boundary → harness choice). As a principles skill with no destructive or batch operations it needs no validation checkpoints, but some procedural guidance is embedded in long dense prose rather than an unambiguous sequence, so it sits just below the top anchor. | 4 / 5 |
Progressive Disclosure | The 59-line body is a concise overview appropriately split into two real, one-level-deep bundle files, and the References section signals each with an explicit when-to-read condition: "references/frontend.md: read when writing, changing, or reviewing component, hook, or UI integration tests" and "references/e2e.md: read when writing, changing, or reviewing browser E2E tests". Both files exist, contain no nested references, and navigation is easy — a clear match for the top anchor. | 5 / 5 |
Total | 17 / 20 Passed |