Content
88%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.
An exceptionally actionable, well-sequenced instruction skill: concrete code, exact commands, explicit validation loops, and clear decision tables for when not to write a spec. Its only costs are repetition across sections and a monolithic single-file structure that inlines some reference material.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense and competent — it assumes Claude knows testing concepts and adds only repo-specific rules — but at ~250 lines some principles repeat (the caption-tells-the-story rule and 'read your own record' each appear twice) and a few parenthetical digressions could be trimmed. Not 5 due to this repetition; well above level 3 since nothing explains concepts Claude already knows. | 4 / 5 |
Actionability | Fully executable guidance: a complete spec example with recordAssertionEvidence calls, a world-binding snippet, exact commands ('pnpm evals:e2e <slug> --local', 'gh workflow run pr-proof.yml --ref <branch> -f pr=<number>'), and tables mapping reviewer-visible strings to their code origins. Only the first title/steps example uses '…' placeholders. | 5 / 5 |
Workflow Clarity | Clear sequence (pick lane → check do-not-write conditions → shape the proof → run locally and read your own record → design review → run on the PR) with explicit validation checkpoints and feedback loops: 'If the screenshots would not convince you, they will not convince the reviewer', fix-or-justify for design notes, and failed steps recorded and rethrown with later steps marked not-reached. | 5 / 5 |
Progressive Disclosure | No bundle files exist; the single SKILL.md is well organized with headers, decision tables, and clearly signaled cross-references to sibling skills (run-tests, open-a-pr) and repo paths (evals/journeys.ts, evals/design-review/README.md). Not 5 because the ~250-line monolith inlines reference-style material (testkit channel API, desktop nativeReason list) that could live in a separate file. | 4 / 5 |
Total | 18 / 20 Passed |