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 lean, well-structured principles document that assumes competence, defers implementation detail to well-signaled one-level-deep references, and provides concrete directives throughout. The main gaps are the absence of worked examples to ground the abstract rules (proof boundaries, oracle independence) and an implicit rather than explicit workflow sequence.
Suggestions
Add one short good-vs-bad test example to ground the abstract actionability rules, e.g., an implementation-derived oracle vs an independently derived literal expectation.
Make the implicit workflow order explicit (select scope → choose proof boundary → design test → mock boundaries → verify), tying the verification checkpoints to those steps.
Trim the three-sentence motivational intro in the opening paragraph to one sentence and split the overloaded Test Selection bullet about proof boundaries into two rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative with no basic-concept explanations ("Structure each test as Arrange, one Act, and Assert" is used without defining the pattern), assuming Claude's competence. Minor trimmable content remains: the three-sentence motivational intro ("A brittle, duplicated, or implementation-shaped test therefore becomes a false requirement…") and a few overloaded multi-clause sentences in Test Selection, so it does not quite reach "every token earns its place". | 4 / 5 |
Actionability | Gives concrete, executable directives such as "Mock direct external I/O boundaries; keep internal business logic and the boundary under test real" and "Add a regression test for every reproducible behavior bug fix". As an instruction-only skill code absence is not penalized, but no worked examples (e.g., a good-vs-bad test) are provided and the Capability Probe Postconditions section is terse, leaving minor gaps versus fully actionable coverage. | 4 / 5 |
Workflow Clarity | Sections follow a coherent implicit order — Test Selection, Design Rules, Mock/Boundary, Data Layer, Verification, Organization, Regression — and validation checkpoints are present ("Capability Probe Postconditions", schema-source verification requirements). The sequence is never made explicit as a first-then flow and checkpoints are scattered rather than tied to steps, which fits anchor 4 better than anchor 5's explicit validate-fix-retry loops; no destructive/batch cap applies. | 4 / 5 |
Progressive Disclosure | The body is a clear overview, and both referenced files (references/frontend.md, references/e2e.md) exist, are one level deep with no further nesting, and are well-signaled with explicit read-when triggers ("read when writing, changing, or reviewing component, hook, or UI integration tests"). Domain-specific implementation detail is appropriately split into the reference files, matching the anchor-5 example. | 5 / 5 |
Total | 17 / 20 Passed |