Content
71%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 well-organized, project-specific convention reference with genuinely executable commands and a validation loop for OpenAPI drift. Its main weaknesses are inlined reference material that belongs in separate files and convention descriptions that would benefit from one compact code example.
Suggestions
Move the Key API Endpoints table and namespace/RBAC model details into references/ files, keeping SKILL.md as an overview with clearly signaled links.
Add a short controller code example showing the transport-only pattern (extract auth, bind params, wrap response) so the convention is demonstrable, not just described.
State explicitly when to run ./scripts/check-openapi-generated.sh (e.g. as a pre-PR checklist step) to turn the drift check into an explicit workflow checkpoint.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and convention-driven with no padding or explanations of concepts Claude already knows (tables for ClawHub slug mapping and endpoints, one-line RBAC role definitions). Minor trimming is possible — e.g. the 11-row Key API Endpoints table is bulk that could live in a reference file. | 4 / 5 |
Actionability | Concrete, copy-paste-ready commands are present ('make generate-api', './scripts/check-openapi-generated.sh', the openapi-typescript invocation) alongside specific class names ('DomainBadRequestException', 'ReviewTaskRequest') and exact endpoint paths. Minor gaps: conventions like 'transport only controllers' lack a code example showing the pattern. | 4 / 5 |
Workflow Clarity | The OpenAPI Contract Sync section has a clear sequence with an explicit validation step ('fails if the checked-in SDK is stale'), and the Common Pitfalls section acts as an error-avoidance checklist. Most other sections are convention references rather than workflows, so a few lack explicit checkpoints, keeping this just below the feedback-loop anchor. | 4 / 5 |
Progressive Disclosure | Sections are clearly headed and navigable, but no references/, scripts/, or assets/ bundle exists, so reference-style content (the Key API Endpoints table, RBAC/namespace model details) is inlined in the single file rather than split out. At ~135 lines it exceeds the simple-skill exception, fitting the 'content that should be separate is inline' anchor. | 3 / 5 |
Total | 15 / 20 Passed |