Content
86%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, token-efficient instruction skill with concrete commands, file pointers, and a clear sequenced workflow. It would reach full marks on actionability and workflow clarity by giving concrete DOMAIN/PROFILE examples and an explicit verify->fix->retry loop.
Suggestions
Replace the templated `DOMAIN=<domain>` / `PROFILE=<profile>` placeholders with one or two concrete example invocations so the commands are copy-paste ready.
Add an explicit feedback loop: if `make check` or `make verify` fails, state that Claude should fix the issue and re-run before proceeding.
Optionally clarify what 'the affected contract' maps to in `tools/agent/docs/change-surfaces.md` so the reader knows which section to consult.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and unpadded; assumes Claude's competence and explains no known concepts, with every line an actionable directive. | 5 / 5 |
Actionability | Provides concrete executable commands (`make check`, `make verify DOMAIN=`, `make verify PROFILE=`) and file pointers, but the DOMAIN/PROFILE values are templated placeholders rather than copy-paste-ready concrete examples. | 4 / 5 |
Workflow Clarity | Clear sequence (read docs, keep consistent, update artifacts, run check, verify) with verification checkpoints present, though an explicit verify->fix->retry feedback loop is not spelled out. | 4 / 5 |
Progressive Disclosure | Short, well-organized body with a clear section header and one-level-deep, clearly signaled external references (`AGENTS.md`, `tools/agent/docs/change-surfaces.md`); no bundle files are present, so the simple-skill exception applies. | 5 / 5 |
Total | 18 / 20 Passed |