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 expert-calibrated, highly actionable body: every rule is repo-specific, commands are copy-paste ready, and the procedure carries explicit validation and reconciliation checkpoints. The only weaknesses are mild repetition in the xfail/status-code prose and a single-file layout where a couple of reference tables could be split out.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with project-specific knowledge Claude cannot know (the 401-sweep rationale, scratch naming rules, xfail policy) and explains no general concepts. Minor trimmable repetition exists — e.g. the xfail section restates "never soften the assertion to match the bug" twice in slightly different forms — placing it at the 4 anchor ("minor instances of over-explanation that could be trimmed") rather than 5. | 4 / 5 |
Actionability | Fully executable throughout: a copy-paste module docstring template, an assert line with the diagnostic message format, the exact `@pytest.mark.requires_seed(...)` decorator, and exact commands (`./dev-compose.sh test-stand test --tree=...`, `uv run --project tests --frozen pytest/ruff`, `generate_schemas.py --check`, the coverage.py reconcile invocation) with flags explained. Matches the 5 anchor — copy-paste ready and covering the common cases. | 5 / 5 |
Workflow Clarity | The 9-step Procedure is clearly sequenced with explicit validation checkpoints: run and check the ledger, ruff check, the schema drift-gate `--check`, and reconciling the coverage gate, with error-handling rationale ("a failing run's ledger is the more useful one"). Feedback loops (probe 422-vs-400 rather than assume, reconcile the gate after running) match the 5 anchor. | 5 / 5 |
Progressive Disclosure | Well-organized single-file skill with clear section headers, tables, and clearly signaled cross-references to sibling skills (`insight-stand`, `stand-scenarios`, `metric-test`, `quality-vector-tests`, `file-bug-insight`) — no buried or nested references. However, the under-50-line simple-skill exception does not apply at ~280 lines, and content like the full fixture table and status-code matrix could live in a reference file, so the 4 anchor ("good structure; minor organization gaps") fits better than 5. | 4 / 5 |
Total | 18 / 20 Passed |