Content
72%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 tight, well-organized convention doc with concrete naming and placement rules and a clear engine/product boundary. Its main gap is the lack of an explicit, validated workflow for the destructive re-snapshot operation and for adding a new fixture end-to-end.
Suggestions
Add a short numbered workflow for adding a new fixture (check for duplicates → pick grida-owned vs engine directory → name per pattern → confirm LFS/git tracking) with a verification step.
Spell out the re-snapshot step concretely — the copy/sync command and a check that staying tests still pass afterward — since 're-snapshot deliberately' leaves the destructive operation underspecified.
Consider trimming the 'What fixtures are' / 'Why we keep them' framing, which restates general fixture knowledge Claude already has.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and well-sectioned with little padding, but the 'What fixtures are' and 'Why we keep them' sections lightly explain the general concept of test fixtures — which Claude already knows — so a few tokens could be trimmed. | 4 / 5 |
Actionability | Gives concrete, executable rules — a naming pattern '<domain>-<property>[-<descriptor>].<ext>', a labeled directory tree with ownership, an explicit engine-repo URL, and a hard rule against referencing gitignored paths — with only a minor gap: the re-snapshot step is described as 're-snapshot deliberately from the engine repo' without the actual commands. | 4 / 5 |
Workflow Clarity | Guidance is presented as bulleted conventions rather than a sequenced workflow, and the destructive re-snapshot operation (overwriting frozen dirs) has no validation or checkpoint steps; per the destructive-operations cap this cannot exceed 3. | 3 / 5 |
Progressive Disclosure | A clean single-file overview with clearly signaled, one-level-deep references (a single link to fixtures/README.md and the engine repo URL), well-organized sections, and no nested or buried references; no bundle files exist to verify, but the structure as written is exemplary. | 5 / 5 |
Total | 16 / 20 Passed |